Should of_device_is_compatible() use strcmp() rather than strncasecmp()?
scottwood at freescale.com
Wed Jul 4 08:15:11 EST 2007
Segher Boessenkool wrote:
> Huh? Are you saying things are matching on substrings?
> That's even more wrong!
Yes, unfortunately. I tried getting the BRG frequency from the fsl,cpm
node and I got the fsl,cpm-enet node instead. :-P
ucc_geth requires device_type == network to avoid matching ucc_geth phy.
>> Does anything actually rely on this behavior?
> Yeah I'm pretty sure some things do. Those are bugs that
> need fixing. My plan was to print a big fat warning whenever
> a case-insensitive match wouldn't match when following the
> rules; I never got around to actually implementing that though.
> Maybe some day ;-)
How about a CONFIG_NOT_BROKEN_DEVICE_TREE that disables this and other
legacy stuff? If unset, warnings will be printed whenever broken things
are detected. Boards which need proper device tree parsing can select
the config option.
I'd prefer the other way around (CONFIG_BROKEN_DEVICE_TREE), but it'd
need to default y at first, and kconfig has no unselect command that I
More information about the Linuxppc-dev