linux-kernel - Re: [PATCH 0/3] staging: pi433: Fix includes & macros.

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]

Message-ID: <ZgHMkIX6RvP5dqyp@smile.fi.intel.com>
Date: Mon, 25 Mar 2024 21:12:16 +0200
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: Shahar Avidar <ikobh7@...il.com>, elder@...aro.org, robh@...nel.org,
	parthiban.veerasooran@...rochip.com, linux-staging@...ts.linux.dev,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] staging: pi433: Fix includes & macros.

On Mon, Mar 25, 2024 at 07:51:16PM +0100, Greg KH wrote:
> On Mon, Mar 25, 2024 at 08:41:50PM +0200, Andy Shevchenko wrote:
> > On Mon, Mar 25, 2024 at 07:05:46PM +0100, Greg KH wrote:
> > > On Mon, Mar 25, 2024 at 11:18:42AM +0200, Andy Shevchenko wrote:
> > > > On Sun, Mar 24, 2024 at 11:31:58AM +0200, Shahar Avidar wrote:
> > > > > 1. Untangle include hierarchy. 
> > > > > 2. Update #minors the driver can accept.
> > > > > 3. Make use of general macro instead of magic number.
> > > > 
> > > > > Shahar Avidar (3):
> > > > >   staging: pi433: Use headers in appropriate files.
> > > > >   staging: pi433: Reduce N_PI433_MINORS to conform with N_SPI_MINORS.
> > > > >   staging: pi433: Make use of spi mode macro instead of magic number.
> > > > 
> > > > For patches 1 and 3
> > > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
> > > 
> > > That's impossible for b4 to parse, it would have applied this to all of
> > > the commits if I had taken them :(
> > 
> > You can apply only patches 1 and 3 as long as they are independent to patch 2.
> > 
> > 	b4 am -st -P 1,3 ...
> 
> I don't do that, it would not scale at all :)

Yeah, this requires too much maintainer's involvement.

Alternative option, in case you are fine with patch 2, is to drop my tags and
apply.

-- 
With Best Regards,
Andy Shevchenko