[PATCH v2 15/15] iio: inkern: fix coding style warnings
Nuno Sá
noname.nuno at gmail.com
Mon Jul 11 07:05:31 PDT 2022
On Mon, 2022-07-11 at 15:15 +0200, Andy Shevchenko wrote:
> On Mon, Jul 11, 2022 at 2:40 PM Nuno Sá <nuno.sa at analog.com> wrote:
> >
> > Just cosmetics. No functional change intended...
>
> ...
>
> > - if (name == NULL && channel_name == NULL)
> > + if (!name && !channel_name)
> > return ERR_PTR(-ENODEV);
>
> After this change in place, I think it's better to convert it to
>
> if (!(name || channel_name))
>
> which shows intentions clearer.
>
No strong feelings from my side so I can do that if a v3 is needed
(which, most likely, will be needed).
- Nuno Sá
More information about the Linux-mediatek
mailing list