Page MenuHomeFreeBSD

ools/sdiodevs2h.awk: fix after removal of $FreeBSD$
ClosedPublic

Authored by bz on Aug 14 2024, 8:24 PM.
Tags
None
Referenced Files
F102913663: D46300.diff
Mon, Nov 18, 4:38 PM
Unknown Object (File)
Fri, Oct 25, 4:32 PM
Unknown Object (File)
Fri, Oct 25, 3:10 PM
Unknown Object (File)
Oct 9 2024, 11:14 PM
Unknown Object (File)
Sep 23 2024, 8:27 PM
Unknown Object (File)
Sep 22 2024, 4:22 AM
Unknown Object (File)
Sep 12 2024, 8:59 AM
Unknown Object (File)
Sep 8 2024, 7:04 AM
Subscribers
None

Details

Summary

The input file used to have a $FreeBSD$ tag on the first line and we
recorded that in the generated files to know which versions they were
based on. With the removal of all that the logic was not quite correct
anymore; fix that.

While here generate defined names which match the ones in the Linux
drivers; given we are likely to have drivers only based on those
one way or another there is absolutely no reason to fight over differently
generated names for device and vendor IDs.

Sponsored by: The FreeBSD Foundation
MFC after: 3 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 59025
Build 55912: arc lint + arc unit

Event Timeline

bz requested review of this revision.Aug 14 2024, 8:24 PM
bz created this revision.
This revision is now accepted and ready to land.Aug 14 2024, 8:27 PM

Thanks; will split it up for commits.

In D46300#1055893, @bz wrote:

Thanks; will split it up for commits.

Thanks for the cleanup. SDIO vendor / product stuff is so lightly used in FreeBSD that even the out of tree stuff I half know about won't be too hard to update.
Tweaks to names may also be needed to match Linux 100%, which isn't a terrible idea, but can also be done as we import linuxkpi sdio drivers.