Page MenuHomeFreeBSD
Differential D19827

aac(4): Add subvendor / subdevice to MODULE_PNP_INFO()'s descriptor_string
Needs ReviewPublic
Actions

Authored by rpokala on Apr 5 2019, 4:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 6:00 AM
Unknown Object (File)
Apr 25 2023, 6:49 PM
Unknown Object (File)
Apr 8 2023, 12:22 AM
Unknown Object (File)
Dec 27 2022, 6:38 AM
Subscribers
None

Details

Reviewers
imp
Summary

I noticed that devmatch reported many apparently duplicated entries
in the list for aac.ko. This is because the driver's probe() actually
matches on the full (vendor, device, subvendor, subdevice) tuple, but
only (vendor, device) were passed to MODULE_PNP_INFO(). With updated
string, there are no more purported duplicates.

Test Plan

Run devmatch -ud with and without the change, and compare.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 23526
Build 22530: arc lint + arc unit

Event Timeline

rpokala created this revision.Apr 5 2019, 4:50 AM
Harbormaster completed remote builds in B23526: Diff 55825.Apr 5 2019, 4:50 AM
imp added a comment.Apr 5 2019, 5:20 AM

I specifically allowed those duplicates when I was originally reviewing this change....

rpokala added a comment.Apr 5 2019, 5:31 AM
In D19827#425235, @imp wrote:

I specifically allowed those duplicates when I was originally reviewing this change....

Interesting. Why? If the reasons still stand, then I'll change this to just add a comment instead.

imp added a comment.EditedApr 5 2019, 5:35 AM

I'm trying to recall that. I know that the different types of subdevices don't change what's probed (as there are catch-all entries). I had a specific reason, but I can't recall it now.

I'll check my notes in the morning.

Revision Contents
Changeset List

PathSize
sys/
dev/
aac/
2 lines

Diff 55825

View Options

sys/dev/aac/aac_pci.c

Loading...