On Fri, Jul 11, 2025 at 04:48:58PM +0200, Clément Le Goffic wrote:
> Add 32bits LPDDR4 channel to the stm32mp257f-dk board.
>
> Signed-off-by: Clément Le Goffic <clement.legoffic@xxxxxxxxxxx>
> ---
> arch/arm64/boot/dts/st/stm32mp257f-dk.dts | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/st/stm32mp257f-dk.dts b/arch/arm64/boot/dts/st/stm32mp257f-dk.dts
> index a278a1e3ce03..a97b41f14ecc 100644
> --- a/arch/arm64/boot/dts/st/stm32mp257f-dk.dts
> +++ b/arch/arm64/boot/dts/st/stm32mp257f-dk.dts
> @@ -54,6 +54,13 @@ led-blue {
> };
> };
>
> + lpddr_channel: lpddr4-channel {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "jedec,lpddr4-channel";
Not tested because this doesn't match the binding.
> + io-width = <32>;
> + };
What would multiple channels look like? I think this needs some work.
Like it should perhaps be within the memory node. It's a lot to just say
32-bit LPDDR4 x1.
> +
> memory@80000000 {
> device_type = "memory";
> reg = <0x0 0x80000000 0x1 0x0>;
>
> --
> 2.43.0
>
