[PATCH 5/6] gpio: Add new gpio-macsmc driver for Apple Macs

Andy Shevchenko andy.shevchenko at gmail.com
Fri Sep 2 06:36:14 PDT 2022


On Fri, Sep 2, 2022 at 4:33 PM Andy Shevchenko
<andy.shevchenko at gmail.com> wrote:
> On Fri, Sep 2, 2022 at 2:12 PM Martin Povišer <povik at cutebit.org> wrote:

...

> > in at least 9 places now, the number of which will probably grow.
> > Just to make the case for *some* printk helper.
>
> Wouldn't this be one line
>
>   printk(“%.4s = ...\n”, &cpu_to_be32(key));
>
> ?
>
> So I don't see disadvantages right now. Later on we can consider a new
> specifier _separately_, otherwise this series would be dragging for no
> sense.

Just to make my point clear. The single user or small subset of users
does not justify the new specifier. New specifier is a lot of burden
to printf() code. Find 3+ independent users and we can talk again.
That's why I suggest to either create a local (to apple whatever
stuff) helper or use existing specifiers in _this_ series.

-- 
With Best Regards,
Andy Shevchenko



More information about the linux-arm-kernel mailing list