On Wed, Nov 30, 2022 at 01:06:38PM +0200, Andy Shevchenko wrote:
> On Wed, Nov 30, 2022 at 11:39:42AM +0100, Hans de Goede wrote:
> > On 11/30/22 11:01, Andy Shevchenko wrote:
> > > On Wed, Nov 30, 2022 at 1:12 AM Hans de Goede <hdegoede@xxxxxxxxxx> wrote:
...
> > >> + (polarity == GPIO_ACTIVE_HIGH) ? "high" : "low");
> > >
> > > Perhaps
> > >
> > > static inline str_high_low(bool v)
> > > {
> > > return v ? "high" : "low";
> > > }
> > >
> > > In the string_helpers.h? If you are okay with the idea, you may use my
> > > tag ahead for that patch.
> >
> > That is an interesting idea. but IMHO best done in a follow-up series
> > after checking for similar code in the rest of the kernel.
> >
> > This is based on the assumption that "selling" this new helper to
> > the string_helpers.h maintainer will be easier if there are multiple
> > consumers of it right away.
>
> strings_helper.h maintainer is any known kernel subsystem maintainer + me
> (as a reviewer / main contributor to that library). That's why I proposed
> the solution and my tag would be enough to route it via your tree.
>
> So, offer still stays.
(You may check last few commits against the header and see the tags)
--
With Best Regards,
Andy Shevchenko
