linux-kernel - Re: [PATCH] mfd: syscon: Restore device_node_to

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: <20250204092152.iyjqcz25jp6uelci@lcpd911>
Date: Tue, 4 Feb 2025 14:51:52 +0530
From: Dhruva Gole <d-gole@...com>
To: "Rob Herring (Arm)" <robh@...nel.org>
CC: Lee Jones <lee@...nel.org>, Arnd Bergmann <arnd@...db.de>,
        Krzysztof
 Kozlowski <krzysztof.kozlowski@...aro.org>,
        Pankaj Dubey
	<pankaj.dubey@...sung.com>,
        Will McVicker <willmcvicker@...gle.com>,
        John
 Madieu <john.madieu.xa@...renesas.com>,
        Nishanth Menon <nm@...com>, Vaishnav
 Achath <vaishnav.a@...com>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] mfd: syscon: Restore device_node_to_regmap() for
 non-syscon nodes

On Jan 24, 2025 at 13:16:44 -0600, Rob Herring (Arm) wrote:
> Commit ba5095ebbc7a ("mfd: syscon: Allow syscon nodes without a
> "syscon" compatible") broke drivers which call device_node_to_regmap()
> on nodes without a "syscon" compatible. Restore the prior behavior for
> device_node_to_regmap().
> 
> This also makes using device_node_to_regmap() incompatible with
> of_syscon_register_regmap() again, so add kerneldoc for
> device_node_to_regmap() and syscon_node_to_regmap() to make it clear
> how and when each one should be used.
> 
> Fixes: ba5095ebbc7a ("mfd: syscon: Allow syscon nodes without a "syscon" compatible")
> Cc: Will McVicker <willmcvicker@...gle.com>
> Cc: John Madieu <john.madieu.xa@...renesas.com>
> Cc: Nishanth Menon <nm@...com>
> Reported-by: Vaishnav Achath <vaishnav.a@...com>
> Signed-off-by: Rob Herring (Arm) <robh@...nel.org>
> ---
>  drivers/mfd/syscon.c | 29 ++++++++++++++++++++++++++---
>  1 file changed, 26 insertions(+), 3 deletions(-)

Tested this for basic boot on TI K3 AM625-SK with next-20250204.
Tested-by: Dhruva Gole <d-gole@...com>

-- 
Best regards,
Dhruva Gole
Texas Instruments Incorporated