On Tue, Feb 18, 2025 at 05:38:14PM +0000, Caleb Connolly wrote:
>
>
> On 2/18/25 08:54, Varadarajan Narayanan wrote:
> > Add initial support for the IPQ9574 MMC based RDP platforms.
> > Define memory layout statically.
> >
> > Signed-off-by: Varadarajan Narayanan <quic_var...@quicinc.com>
> > ---
> > v3: Fix comments and give additional details
> > No code change
> >
> > v2: Use upstream and override DT instead of own DT
> > ---
> > arch/arm/dts/ipq9574-rdp433-u-boot.dtsi | 25 +++++++++++++++++++++++++
> > 1 file changed, 25 insertions(+)
> > create mode 100644 arch/arm/dts/ipq9574-rdp433-u-boot.dtsi
> >
> > diff --git a/arch/arm/dts/ipq9574-rdp433-u-boot.dtsi
> > b/arch/arm/dts/ipq9574-rdp433-u-boot.dtsi
> > new file mode 100644
> > index 0000000000..390e2338d6
> > --- /dev/null
> > +++ b/arch/arm/dts/ipq9574-rdp433-u-boot.dtsi
> > @@ -0,0 +1,25 @@
> > +// SPDX-License-Identifier: BSD-3-Clause
> > +/*
> > + * Copyright (c) 2025, Qualcomm Innovation Center, Inc. All rights
> > reserved.
> > + */
> > +
> > +/ {
> > + /* Will be removed when SMEM parsing is updated */
> > + memory@40000000 {
> > + device_type = "memory";
> > + reg = <0x0 0x40000000 0x0 0x40000000>,
> > + <0x0 0x4a500000 0x0 0x00100000>;
> > + };
> > +};
> > +
> > +&sdhc_1 {
> > + sdhci-caps-mask = <0x0 0x04000000>;
> > + sdhci-caps = <0x0 0x04000000>; /* SDHCI_CAN_VDD_180 */
> > +
> > + /*
> > + * This reset is needed to clear out the settings done by
> > + * previous boot loader. Without this the SDHCI_RESET_ALL
> > + * reset done sdhci_init() times out.
> > + */
>
> Thanks, this should be accompanied by a patch upstream adding this property
> so it can be removed from here later.
The driver in Linux and U-Boot are different and it doesn't need it.
-Varada
> Kind regards,
> > + resets = <&gcc GCC_SDCC_BCR>;
> > +};
>
> --
> Caleb (they/them)
