Re: [PATCH] of: require a match on all fields of of_device_id

On Tue, Jul 17, 2012 at 8:11 PM, Scott Wood [off-list ref] wrote=
:

Commit 107a84e61cdd3406c842a0e4be7efffd3a05dba6 ("of: match by compatible
property first") breaks the gianfar ethernet driver found on various
Freescale PPC chips.

There are, for unfortunate historical reasons, two nodes with a
compatible of "gianfar".

Would it be worth updating the binding for the two nodes to make the
compatible property different?  We could do something like this:

ethernet@24000 {
                        reg =3D <0x24000 0x1000>;
                        compatible =3D "fsl,gianfar-eth", "gianfar";
                        ...
};

(and something similar for MDIO nodes)

and update all the drivers to look for both strings.  After a few
years, we can delete "gianfar".

--=20
Timur Tabi
Linux kernel developer at Freescale=