Message ID | 20211126125007.1319946-1-alvin@pqrs.dk (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [net,1/3] net: dsa: realtek-smi: don't log an error on EPROBE_DEFER | expand |
On 26.11.2021 13:50, Alvin Šipraga wrote: > From: Alvin Šipraga <alsi@bang-olufsen.dk> > > Probe deferral is not an error, so don't log this as an error: > > [0.590156] realtek-smi ethernet-switch: unable to register switch ret = -517 > > Signed-off-by: Alvin Šipraga <alsi@bang-olufsen.dk> > --- > drivers/net/dsa/realtek-smi-core.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/dsa/realtek-smi-core.c b/drivers/net/dsa/realtek-smi-core.c > index c66ebd0ee217..9415dd81ce5a 100644 > --- a/drivers/net/dsa/realtek-smi-core.c > +++ b/drivers/net/dsa/realtek-smi-core.c > @@ -456,7 +456,9 @@ static int realtek_smi_probe(struct platform_device *pdev) > smi->ds->ops = var->ds_ops; > ret = dsa_register_switch(smi->ds); > if (ret) { > - dev_err(dev, "unable to register switch ret = %d\n", ret); > + if (ret != -EPROBE_DEFER) Better use dev_err_probe(). > + dev_err(dev, "unable to register switch ret = %d\n", > + ret); > return ret; > } > return 0; >
On 11/29/21 08:13, Heiner Kallweit wrote: > On 26.11.2021 13:50, Alvin Šipraga wrote: >> From: Alvin Šipraga <alsi@bang-olufsen.dk> >> >> Probe deferral is not an error, so don't log this as an error: >> >> [0.590156] realtek-smi ethernet-switch: unable to register switch ret = -517 >> >> Signed-off-by: Alvin Šipraga <alsi@bang-olufsen.dk> >> --- >> drivers/net/dsa/realtek-smi-core.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/net/dsa/realtek-smi-core.c b/drivers/net/dsa/realtek-smi-core.c >> index c66ebd0ee217..9415dd81ce5a 100644 >> --- a/drivers/net/dsa/realtek-smi-core.c >> +++ b/drivers/net/dsa/realtek-smi-core.c >> @@ -456,7 +456,9 @@ static int realtek_smi_probe(struct platform_device *pdev) >> smi->ds->ops = var->ds_ops; >> ret = dsa_register_switch(smi->ds); >> if (ret) { >> - dev_err(dev, "unable to register switch ret = %d\n", ret); >> + if (ret != -EPROBE_DEFER) > > Better use dev_err_probe(). Didn't know about that - thanks. I'll send a v2. > >> + dev_err(dev, "unable to register switch ret = %d\n", >> + ret); >> return ret; >> } >> return 0; >> >
diff --git a/drivers/net/dsa/realtek-smi-core.c b/drivers/net/dsa/realtek-smi-core.c index c66ebd0ee217..9415dd81ce5a 100644 --- a/drivers/net/dsa/realtek-smi-core.c +++ b/drivers/net/dsa/realtek-smi-core.c @@ -456,7 +456,9 @@ static int realtek_smi_probe(struct platform_device *pdev) smi->ds->ops = var->ds_ops; ret = dsa_register_switch(smi->ds); if (ret) { - dev_err(dev, "unable to register switch ret = %d\n", ret); + if (ret != -EPROBE_DEFER) + dev_err(dev, "unable to register switch ret = %d\n", + ret); return ret; } return 0;