Re: [PATCH 1/3] lib/string_helpers: Consolidate yesno() implementation
From: Andy Shevchenko
Date: Wed Jan 19 2022 - 16:05:31 EST
On Wed, Jan 19, 2022 at 04:00:17PM -0500, Steven Rostedt wrote:
> On Wed, 19 Jan 2022 21:25:08 +0200
> Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
>
> > > I say keep it one line!
> > >
> > > Reviewed-by: Steven Rostedt (Google) <rostedt@xxxxxxxxxxx>
> >
> > I believe Sakari strongly follows the 80 rule, which means...
>
> Checkpatch says "100" I think we need to simply update the docs (the
> documentation always lags the code ;-)
The idea of checkpatch change is for old code to avoid tons of patches
to satisfy 80 rule in (mostly) staging code. Some maintainers started /
have been using relaxed approach.
--
With Best Regards,
Andy Shevchenko