Re: [PATCH leds v2 08/11] leds: turris-omnia: Use dev_err_probe() where appropriate

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 03, 2024 at 01:45:49PM +0300, Andy Shevchenko wrote:
> On Tue, Sep 3, 2024 at 1:20 PM Marek Behún <kabel@xxxxxxxxxx> wrote:
> >
> > Use dev_err_probe() instead of dev_err() + separate return where
> > appropriate.
> 
> ...
> 
> > +       if (ret)
> > +               return dev_err_probe(dev, ret, "Cannot set LED %pOF to software mode\n", np);
> 
> Side note: Not sure how np is being used besides the messaging. If
> it's only for the messaging, I would rather see %pfw and fwnode based
> approach.

This board has a number of Ethernet switches described in DT.  Nobody
takes ACPI seriously for any sort of complex networking. So using
fwnode is probably pointless, this board will probably never be used
with anything other than DT.

	 Andrew




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux