Page MenuHomeFreeBSD

LinuxKPI: firmware name compat
Changes PlannedPublic

Authored by bz on Jun 4 2021, 5:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 9:31 AM
Unknown Object (File)
Dec 23 2023, 6:14 AM
Unknown Object (File)
Dec 23 2023, 12:26 AM
Unknown Object (File)
Nov 30 2023, 7:06 AM
Unknown Object (File)
Nov 6 2023, 7:16 PM
Unknown Object (File)
Oct 3 2023, 11:08 PM
Unknown Object (File)
May 29 2023, 12:58 AM
Unknown Object (File)
Dec 27 2022, 4:25 AM
Subscribers

Details

Reviewers
hselasky
Group Reviewers
linuxkpi
Summary

"Flatten" more latters found in places where names are derived from
device tree files. This is needed by some wireless drivers.

MFC after: 10 days

Diff Detail

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

Event Timeline

bz requested review of this revision.Jun 4 2021, 5:17 PM

Out of curiosity do we need to check after each iteration of flattening, or can we just flatten all first (via e.g. strpbrk if in libkern?)

The nesting for additional characters ("/", ",", ".", ...) seems odd, perhaps this could benefit from a bit broader rework

sys/compat/linuxkpi/common/src/linux_firmware.c
90–91

Also need to change this?

This revision is now accepted and ready to land.Jun 4 2021, 9:04 PM

Out of curiosity do we need to check after each iteration of flattening, or can we just flatten all first (via e.g. strpbrk if in libkern?)

Yeah, I was wondering about that too; I need to do a full analysis of how we create various firmware images again to see what needs to be checked individually and what we can make a "policy" you do it all.

Let's keep this here and I'll do the due diligens when I have a few minutes and write things up as well into something like a man page or wiki or just a README... with samples how to build the module Makefiles.
And then I'll probably find that between the 4 drivers I tried I can even make some of Makefile.inc more generalized into sys/conf/

sys/compat/linuxkpi/common/src/linux_firmware.c
90–91

Probably. I assume I didn't notice given they all have / as well in first place.

Can someone please request changes on the review for @emaste 's comments and the newly added '@' (in the base system). Seems I cannot do that myself in Phab :(

bz planned changes to this revision.Jul 3 2022, 10:26 PM

Going to look into this next week to clear out more open reviews of mine...