linux-kernel - Re: [PATCH v1 1/4] seq_file: Introduce DEFINE_SHOW

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 24 Jan 2018 09:11:38 +0200
From:   Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To:     Darren Hart <dvhart@...radead.org>,
        Lee Jones <lee.jones@...aro.org>
Cc:     platform-driver-x86@...r.kernel.org,
        Linus Walleij <linus.walleij@...aro.org>,
        Marcel Holtmann <marcel@...tmann.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 1/4] seq_file: Introduce DEFINE_SHOW_ATTRIBUTE()
 helper macro

On Tue, 2018-01-23 at 16:05 -0800, Darren Hart wrote:
> On Tue, Jan 23, 2018 at 09:15:08AM +0000, Lee Jones wrote:
> > On Mon, 22 Jan 2018, Andy Shevchenko wrote:
> > 
> > > The DEFINE_SHOW_ATTRIBUTE() helper macro would be useful for
> > > current
> > > users, which are many of them, and for new comers to decrease code
> > > duplication.
> > > 
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
> > > ---
> > >  drivers/mfd/ab8500-debugfs.c    | 14 --------------
> > 
> > Acked-by: Lee Jones <lee.jones@...aro.org>
> 
> Thanks Andy,
> 
> Acked-by: Darren Hart (VMware) <dvhart@...radead.org>

Just to clarify a bit.

Since the patch series is based on linux-next, the idea is to push it
and send as second PR to Linus, after others will be ready in mainline.

The rationale is to allow many other current users and perhaps new
comers, e.g. [1], to utilize it already in the next cycle.

[1]: https://patchwork.kernel.org/patch/10172393/

-- 
Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Intel Finland Oy