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

Tabi Timur-B04825 B04825 at freescale.com
Wed Jul 18 11:57:53 EST 2012


On Tue, Jul 17, 2012 at 8:11 PM, Scott Wood <scottwood at freescale.com> 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 at 24000 {
                        reg = <0x24000 0x1000>;
                        compatible = "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".

-- 
Timur Tabi
Linux kernel developer at Freescale


More information about the Linuxppc-dev mailing list