linux-kernel - Re: [PATCH RESEND] regulator: bd96801: Add ERRB IRQ

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: <80071b1c-c8e1-4273-8b10-3630af8d88e8@gmail.com>
Date: Thu, 14 Nov 2024 14:04:09 +0200
From: Matti Vaittinen <mazziesaccount@...il.com>
To: Mark Brown <broonie@...nel.org>
Cc: Matti Vaittinen <matti.vaittinen@...rohmeurope.com>,
 Liam Girdwood <lgirdwood@...il.com>, Lee Jones <lee@...nel.org>,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH RESEND] regulator: bd96801: Add ERRB IRQ

On 14/11/2024 13:49, Mark Brown wrote:
> On Thu, Nov 14, 2024 at 09:19:59AM +0200, Matti Vaittinen wrote:
> 
>> Signed-off-by: Matti Vaittinen <mazziesaccount@...il.com>
>> Reviewed-by: Mark Brown <broonie@...nel.org>
>> ---
>>
>> This patch was part of the:
>> https://lore.kernel.org/all/cover.1727931468.git.mazziesaccount@gmail.com/
>> The MFD part was applied by Lee, but I didn't see the regulator part in
>> MFD or regulator repository. Perhaps it fell through the cracks? I know
>> we're late in the cycle but I re-spin this now before I forgot... We do
>> always have the next cycle, right? :)
> 
> If this is part of a MFD series I'd have expected it to go in with
> that...

AFAIR, Lee asked if the patches could be taken in separately. I replied 
"yes" - with a very small corner case of someone having the ERRB in 
device-tree already (which could've been a problem if regulator part 
went in and MFD didn't). After this Lee picked only the subset. I 
suppose this evaded your eye :) I know I should've resent this back 
then, but I just forgot to check if you picked this.

I don't see having the MFD part in without the regulators a problem 
(other than missing the ERRB functionality). So, perhaps Mark can pick 
this after v6.13-rc1 is out?

Yours,
	-- Matti