Page MenuHomeFreeBSD

LinuxKPI: mod_devicetable, add comment about USB
ClosedPublic

Authored by bz on Sat, Jan 31, 12:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 4, 2:54 AM
Unknown Object (File)
Sun, Feb 1, 1:12 PM
Unknown Object (File)
Sun, Feb 1, 12:08 PM
Unknown Object (File)
Sun, Feb 1, 10:37 AM
Unknown Object (File)
Sun, Feb 1, 1:40 AM
Unknown Object (File)
Sun, Feb 1, 1:05 AM
Unknown Object (File)
Sat, Jan 31, 8:27 PM
Unknown Object (File)
Sat, Jan 31, 6:26 PM

Details

Summary

USB structures are defined with the native USB implementation.
Just leave a comment why they are not here.

Sponsored by: The FreeBSD Foundation
MFC after: 3 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bz requested review of this revision.Sat, Jan 31, 12:48 AM

This is a simple comment. Anyone any comments on it? Otherwise it'll go in within 48 hours.

dumbbell added inline comments.
sys/compat/linuxkpi/common/include/linux/mod_devicetable.h
87

I don’t understand what you mean here.

bz marked an inline comment as done.Mon, Feb 9, 7:06 PM
bz added inline comments.
sys/compat/linuxkpi/common/include/linux/mod_devicetable.h
87

Normally we would define struct usb_device_id here but the USB stack is tight into the native USB stack.
MODULE_DEVICE_TABLE_BUS_usb is (well will be, see somewhere in the stack) defined in usb.h.

This is similar to what we have in PCI (where all this is/was just in pci.h):
https://cgit.freebsd.org/src/blame/sys/compat/linuxkpi/common/include/linux/pci.h#n65 and following.

It is just now time to break this up so that further bus attachments (USB, SDIO) can be added.

dumbbell added inline comments.
sys/compat/linuxkpi/common/include/linux/mod_devicetable.h
87

Thanks for the clarification!

This revision is now accepted and ready to land.Mon, Feb 9, 7:10 PM
This revision was automatically updated to reflect the committed changes.
bz marked an inline comment as done.