Message ID | 1479911104-19864-1-git-send-email-niklass@axis.com |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
Hello Niklas On 11/23/2016 3:25 PM, Niklas Cassel wrote: > From: Niklas Cassel <niklas.cassel@axis.com> > > 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@axis.com> Acked-by: Giuseppe Cavallaro <peppe.cavallaro@st.com> > --- > drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c > index b1e5f24708c9..52cd365b8e5e 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c > +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c > @@ -58,9 +58,12 @@ static int dwmac_generic_probe(struct platform_device *pdev) > > static const struct of_device_id dwmac_generic_match[] = { > { .compatible = "st,spear600-gmac"}, > + { .compatible = "snps,dwmac-3.50a"}, > { .compatible = "snps,dwmac-3.610"}, > { .compatible = "snps,dwmac-3.70a"}, > { .compatible = "snps,dwmac-3.710"}, > + { .compatible = "snps,dwmac-4.00"}, > + { .compatible = "snps,dwmac-4.10a"}, > { .compatible = "snps,dwmac"}, > { } > }; >
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c index b1e5f24708c9..52cd365b8e5e 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c @@ -58,9 +58,12 @@ static int dwmac_generic_probe(struct platform_device *pdev) static const struct of_device_id dwmac_generic_match[] = { { .compatible = "st,spear600-gmac"}, + { .compatible = "snps,dwmac-3.50a"}, { .compatible = "snps,dwmac-3.610"}, { .compatible = "snps,dwmac-3.70a"}, { .compatible = "snps,dwmac-3.710"}, + { .compatible = "snps,dwmac-4.00"}, + { .compatible = "snps,dwmac-4.10a"}, { .compatible = "snps,dwmac"}, { } };