linux-kernel - Re: [PATCH] arm64: dts: ti: k3-j721e-common-proc-board: Add bootph-all to HBMC nodes

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: <20250411112651.4kt4qgiz4ikylbwi@onboard>
Date: Fri, 11 Apr 2025 06:26:51 -0500
From: Nishanth Menon <nm@...com>
To: Anurag Dutta <a-dutta@...com>
CC: <vigneshr@...com>, <kristo@...nel.org>, <u-kumar1@...com>,
        <vaishnav.a@...com>, <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: ti: k3-j721e-common-proc-board: Add
 bootph-all to HBMC nodes

On 13:56-20250411, Anurag Dutta wrote:
> Add bootph-all to HBMC controller and hyperflash nodes for successful
> hyperflash boot on j721e-evm.
> 
> Signed-off-by: Anurag Dutta <a-dutta@...com>
> ---
> 
> Test logs : https://gist.github.com/anuragdutta731/90a492589557c2ec2881e1de50a62006
> 
>  arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> index 4421852161dd..9ada749f16ba 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> @@ -570,6 +570,13 @@ &usb1 {
>  	maximum-speed = "high-speed";
>  };
>  
> +&hbmc {
> +	bootph-all;

Documentation/devicetree/bindings/dts-coding-style.rst
Please add an EoL

> +	flash@0,0 {
> +		bootph-all;
> +	};
> +};

Why &hbmc, why not in the SoM.dtsi?

> +
>  &ospi1 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&mcu_fss0_ospi1_pins_default>;
> -- 
> 2.34.1
> 

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D