Differential D35006

psm: Swap the unit member in the softc for a device_t.
ClosedPublic
Actions

Authored by jhb on Apr 20 2022, 8:15 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Jan 26, 4:23 AM

	Unknown Object (File)
	Fri, Jan 24, 7:25 AM

	Unknown Object (File)
	Jan 3 2025, 11:29 AM

	Unknown Object (File)
	Dec 13 2024, 10:38 PM

	Unknown Object (File)
	Nov 1 2024, 1:16 AM

	Unknown Object (File)
	Oct 31 2024, 10:24 AM

	Unknown Object (File)
	Oct 26 2024, 4:47 PM

	Unknown Object (File)
	Oct 26 2024, 4:47 PM

View All 58 Files

Subscribers

None

Details

Reviewers

Commits

rGdb3f075d8761: psm: Swap the unit member in the softc for a device_t.
rGa6c09f20e8e0: psm: Swap the unit member in the softc for a device_t.

Summary

This entails various changes to make this driver more "modern"
(new-bus vs pre-new-bus) using device_log() and device_printf() rather
than psm%d. It also fixes the device_busy/unbusy calls to use sc->dev
directly rather than looking the device_t up via the devclass and
unit.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb requested review of this revision.Apr 20 2022, 8:15 PM

jhb created this revision.

Harbormaster completed remote builds in B45278: Diff 105236.Apr 20 2022, 8:15 PM

imp accepted this revision.Apr 20 2022, 9:24 PM

imp added inline comments.

sys/dev/atkbdc/psm.c
1053	device_log() might not be a bad idea... But this is fine here.

This revision is now accepted and ready to land.Apr 20 2022, 9:24 PM

jhb added inline comments.Apr 21 2022, 4:14 PM

sys/dev/atkbdc/psm.c
1053	Oh, we have that now, yes. Ok, I'd rather use that.

Use device_log.

This revision now requires review to proceed.Apr 21 2022, 4:34 PM

Harbormaster completed remote builds in B45300: Diff 105269.Apr 21 2022, 4:34 PM

jhb edited the summary of this revision. (Show Details)Apr 21 2022, 4:35 PM

imp accepted this revision.Apr 21 2022, 6:01 PM

This revision is now accepted and ready to land.Apr 21 2022, 6:01 PM

Closed by commit rGa6c09f20e8e0: psm: Swap the unit member in the softc for a device_t. (authored by jhb). · Explain WhyApr 21 2022, 8:55 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGa6c09f20e8e0: psm: Swap the unit member in the softc for a device_t..

jhb added a commit: rGdb3f075d8761: psm: Swap the unit member in the softc for a device_t..Jan 24 2023, 4:48 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

atkbdc/

199 lines

Diff 105318

sys/dev/atkbdc/psm.c

Loading...