[PATCH 2/2] p3450: fix Jetson Nano SPI flash

[Svyatoslav Ryhel]

ср, 20 серп. 2025 р. о 13:09 Thierry Reding <treding at nvidia.com> пише:
>
> On Tue, Aug 19, 2025 at 04:40:14PM +0100, Peter Robinson wrote:
> > The Nano's SPI flash stopped working in U-Boot, as the
> > prior stage loaded U-Boot, the only thing it was used
> > for was save/loading env vars so update the DT so it
> > can now initialise it. It also drops enabling the old
> > TEGRA114_SPI driver, as the flash hangs off the faster
> > TEGRA210_QSPI interface, nothing on the Nano uses the
> > old interface by default so it's surplus.
> >
> > Signed-off-by: Peter Robinson <pbrobinson at gmail.com>
> > ---
> >  arch/arm/dts/tegra210-p3450-0000.dts | 9 ++++++++-
> >  arch/arm/dts/tegra210.dtsi           | 6 +++---
> >  configs/p3450-0000_defconfig         | 1 -
> >  3 files changed, 11 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/dts/tegra210-p3450-0000.dts b/arch/arm/dts/tegra210-p3450-0000.dts
> > index 9ef744ac8b0..ddeeb232de2 100644
> > --- a/arch/arm/dts/tegra210-p3450-0000.dts
> > +++ b/arch/arm/dts/tegra210-p3450-0000.dts
> > @@ -124,7 +124,14 @@
> >
> >       spi at 70410000 {
> >               status = "okay";
> > -             spi-max-frequency = <80000000>;
> > +
> > +             flash at 0 {
> > +                     compatible = "jedec,spi-nor";
> > +                     reg = <0>;
> > +                     spi-max-frequency = <104000000>;
> > +                     spi-tx-bus-width = <2>;
> > +                     spi-rx-bus-width = <2>;
> > +             };
> >       };
> >
> >       usb at 7d000000 {
> > diff --git a/arch/arm/dts/tegra210.dtsi b/arch/arm/dts/tegra210.dtsi
> > index 28ecd2b467a..6dd8fe26c38 100644
> > --- a/arch/arm/dts/tegra210.dtsi
> > +++ b/arch/arm/dts/tegra210.dtsi
> > @@ -762,10 +762,10 @@
> >               interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH>;
> >               #address-cells = <1>;
> >               #size-cells = <0>;
> > -             clocks = <&tegra_car TEGRA210_CLK_QSPI>;
> > -             clock-names = "qspi";
> > +             clocks = <&tegra_car TEGRA210_CLK_QSPI>,
> > +                     <&tegra_car TEGRA210_CLK_QSPI_PM>;
>
> Nit: We usually indent so that subsequent lines align with the first, so
> an extra space would be required here. I don't feel strongly about it,
> though, so no need to resend just for this.
>
> Acked-by: Thierry Reding <treding at nvidia.com>

Reviewed-by: Svyatoslav Ryhel <clamor95 at gmail.com>

Applied to u-boot-tegra/staging, thank you!