linux-kernel - Re: [PATCH v2 6/6] arm64: defconfig: Set CONFIG_MFD_TPS6594

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] [day] [month] [year] [list]

Message-ID: <ecf3f42b-9758-4ee4-b1e4-a003933ea225@ti.com>
Date: Mon, 22 Apr 2024 17:06:48 -0500
From: Judith Mendez <jm@...com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Rob Herring
	<robh@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Catalin Marinas <catalin.marinas@....com>,
        Will Deacon <will@...nel.org>
CC: Vignesh Raghavendra <vigneshr@...com>, Conor Dooley <conor+dt@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, Bhavya Kapoor <b-kapoor@...com>,
        Dasnavis
 Sabiya <sabiya.d@...com>
Subject: Re: [PATCH v2 6/6] arm64: defconfig: Set CONFIG_MFD_TPS6594_I2C=y

Hi,

On 4/19/24 8:59 AM, Krzysztof Kozlowski wrote:
> On 17/04/2024 00:16, Judith Mendez wrote:
>> SK-AM62A-LP uses TPS6593x PMIC (interfaced over I2C) to power the SoC
>> and various other peripherals on the board [1].
>>
>> Booting SD with UHS modes have a dependency on TPS6593x PMIC driver so
>> change to built in order to boot using SD boot at the higher speed modes.
>>
>> Fixes: f9010eb938be ("arm64: defconfig: Enable TPS6593 PMIC for SK-AM62A")
> 
> No, that's not a fix. Fix is using proper initramfs, because bug is lack
> of initramfs, not having it as module.
> 
> I don't agree, because I prefer to have this as module.

ok, will drop this patch, thanks.

~ Judith

> 
>> Signed-off-by: Judith Mendez <jm@...com>
> 
> 
> Best regards,
> Krzysztof
>