[PATCH v2 2/2] ARM: mvebu: Use system controller to get the soc id when possible
Andrew Lunn
andrew at lunn.ch
Fri Jun 20 12:12:04 PDT 2014
> > +int mvebu_system_controller_get_soc_id(u32 *dev, u32 *rev)
> > +{
> > + if (of_machine_is_compatible("marvell,armada38x") &&
>
> As Sergei commented on your documentation patch, we try to avoid
> wildcards in compatible strings. Perhaps a match list would be more
> appropriate?
I'm being pedantic, but in this invocation, it is not a wildcard. It will
match one and only one compatible string.
However, we have the issue that 38x is not introduced in this patch,
it has been there for a while. So would you like another patch series
removing it from the DT files? A quick grep suggests nothing is using
"marvell,armada38x" except this patch. So it should not be a backwards
compatible issue.
Andrew
More information about the linux-arm-kernel
mailing list