RE: [PATCH] arm64: dts: renesas: rzg2ul-smarc-som: Enable serial NOR flash

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert,

Thanks for the feedback.

> Subject: Re: [PATCH] arm64: dts: renesas: rzg2ul-smarc-som: Enable serial
> NOR flash
> 
> Hi Biju,
> 
> On Thu, Feb 9, 2023 at 5:40 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote:
> > Enable Renesas at25ql128a flash connected to QSPI0. Also disable the
> > node from rzfive-smarc-som as it is untested.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > --- a/arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/rzg2ul-smarc-som.dtsi
> > @@ -179,6 +179,18 @@ eth1_pins: eth1 {
> >                          <RZG2L_PORT_PINMUX(18, 5, 1)>; /* IRQ7 */
> >         };
> >
> > +       qspi0_pins: qspi0 {
> > +               qspi0-data {
> > +                       pins = "QSPI0_IO0", "QSPI0_IO1", "QSPI0_IO2",
> "QSPI0_IO3";
> > +                       power-source = <1800>;
> > +               };
> > +
> > +               qspi0-ctrl {
> > +                       pins = "QSPI0_SPCLK", "QSPI0_SSL";
> > +                       power-source = <1800>;
> > +               };
> 
> I guess there is no need for the subnodes, as all pins use the same power-
> source value?

OK, will remove subnode.

> 
> > +       };
> > +
> >         sdhi0_emmc_pins: sd0emmc {
> >                 sd0_emmc_data {
> >                         pins = "SD0_DATA0", "SD0_DATA1", "SD0_DATA2",
> > "SD0_DATA3", @@ -230,6 +242,38 @@ sd0_mux_uhs {
> >         };
> >  };
> >
> > +&sbc {
> > +       pinctrl-0 = <&qspi0_pins>;
> > +       pinctrl-names = "default";
> > +       status = "okay";
> > +
> > +       flash@0 {
> > +               compatible = "jedec,spi-nor";
> > +               reg = <0>;
> > +               spi-max-frequency = <50000000>;
> > +               spi-tx-bus-width = <1>;
> 
> Why not <4>? According to the datasheet, AT25QL128A supports quad read and
> write.

With a value of <4>, currently flash write is failing. Need to debug this
Issue later.

root@smarc-rzg2ul:~# modprobe spi-rpc-if
[   93.604723] spi-nor spi1.0: spi-nor-generic (16384 Kbytes)
[   93.668403] 2 fixed-partitions partitions found on MTD device spi1.0
[   93.676226] Creating 2 MTD partitions on "spi1.0":
[   93.682982] 0x000000000000-0x000000200000 : "boot"
[   93.692657] 0x000000200000-0x000001000000 : "user"
root@smarc-rzg2ul:~# cd /cip-test-scripts/
root@smarc-rzg2ul:/cip-test-scripts# ./rpcif_t_001.sh

EXIT|FAIL|rpcif_t_001.sh|[00:00:53] Writing to flash failed||

Cheers,
Biju





> 
> > +               spi-rx-bus-width = <4>;
> 
> The rest LGTM.
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like
> that.
>                                 -- Linus Torvalds




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux