Add MODULE_PNP_INFO() to the driver to make it autoload if not linked statically into the kernel
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 45402 Build 42290: arc lint + arc unit
Event Timeline
Comment Actions
Nice to see this. Perhaps a follow-up change removing rtsx from amd64/i386 GENERIC configs would be appropriate?
| sys/dev/rtsx/rtsx.c | ||
|---|---|---|
| 3928 | It would be better to reuse the list from main probe function above without duplication. I haven't looked closer into the format string of the macro, but I think it should be possible. | |
| sys/i386/conf/GENERIC | ||
| 322 | I wonder whether anybody use rtsx as boot device. I know that some systems use MMC cards as one, that is why we have sdhci in GENERIC. | |
| sys/i386/conf/GENERIC | ||
|---|---|---|
| 322 | I don't think this is frequent. I try it on my ACER to no avail. | |
| sys/dev/rtsx/rtsx.c | ||
|---|---|---|
| 3914 | Does this work? I didn't think that you could specify vendor= like this. I can't seem to find the code that parses it like that, can you help me find what I'm overlooking? | |
| sys/dev/rtsx/rtsx.c | ||
|---|---|---|
| 3914 | I test it at every boot on my laptop. I follow | |
| sys/dev/rtsx/rtsx.c | ||
|---|---|---|
| 3914 | Ah, indeed. I'd forgotten about T. | |
| sys/amd64/conf/GENERIC | ||
|---|---|---|
| 366 | I think you should pick between keeping the device in GENERIC or removing the line completely, rather than commenting it out. | |
| sys/amd64/conf/GENERIC | ||
|---|---|---|
| 366 | Thanks! | |
Comment Actions
Hello,
Now that 13.2-RELEASE is almost done I think it would be nice to have this update committed for 14.0-RELEASE.
With the PNP, the kernel is smaller for the configuration without Realtek.