Re: [RESEND PATCH 2/3] net: stmmac: dwmac-generic: add missing compatible strings
From: David Miller
Date: Thu Dec 08 2016 - 11:37:11 EST
From: Niklas Cassel <niklas.cassel@xxxxxxxx>
Date: Wed, 7 Dec 2016 13:41:07 +0100
> From: Niklas Cassel <niklas.cassel@xxxxxxxx>
>
> devicetree binding for stmmac states:
> - compatible: Should be "snps,dwmac-<ip_version>", "snps,dwmac"
> For backwards compatibility: "st,spear600-gmac" is also supported.
>
> Since dwmac-generic.c calls stmmac_probe_config_dt explicitly,
> another alternative would have been to remove all compatible strings
> other than "snps,dwmac" and "st,spear600-gmac" from dwmac-generic.c.
>
> However, that would probably do more good than harm, since when trying
> to figure out what hardware a certain driver supports, you usually look
> at the compatible strings in the struct of_device_id, and not in some
> function defined in a completely different file.
>
> No functional change intended.
>
> Signed-off-by: Niklas Cassel <niklas.cassel@xxxxxxxx>
> Acked-by: Giuseppe Cavallaro <peppe.cavallaro@xxxxxx>
Applied.