Message ID | 20210817093621.3.If40996e1d9e99b57f73247bf6cde58c84a9eb6c4@changeid |
---|---|
State | Superseded |
Delegated to: | Kever Yang |
Headers | show |
Series | [1/3] spi: rockchip_sfc: Impletment set_speed logic | expand |
On Tue, Aug 17, 2021 at 09:41:01AM +0800, Jon Lin wrote: > TX single line is more compatible for corresponding board > > Signed-off-by: Jon Lin <jon.lin@rock-chips.com> > --- > There are still some bugs with this. Note that if you want for now you can just abandon this patch (or fix it, either way is fine). If you abandon it I'll make sure once the Linux driver is upstream the nodes match between it and U-Boot. Otherwise, you'll need to make the changes below to fix this. > arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi | 2 +- > arch/arm/dts/rk3326-odroid-go2.dts | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi b/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi > index 741e8dd935..0990005a15 100644 > --- a/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi > +++ b/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi > @@ -70,7 +70,7 @@ > u-boot,dm-pre-reloc; > }; > > -&spi_flash { > +&{/sfc@ff3a0000/flash@0} { > u-boot,dm-pre-reloc; > }; > This causes the devicetree to fail to compile, since the node is not getting changed from spi_flash in the main dts file. > diff --git a/arch/arm/dts/rk3326-odroid-go2.dts b/arch/arm/dts/rk3326-odroid-go2.dts > index 6f91f5040b..57e7f409d8 100644 > --- a/arch/arm/dts/rk3326-odroid-go2.dts > +++ b/arch/arm/dts/rk3326-odroid-go2.dts > @@ -629,7 +629,7 @@ The node should change here to just "flash@0" instead of "spi_flash: xt25f128b@0" to match upstream (proposed). Also note that on the upstream (proposed) the pinctrl values are in a different order, and the sfc_cs pinctrl is called the sfc_cs0. > reg = <0>; > spi-max-frequency = <108000000>; > spi-rx-bus-width = <2>; > - spi-tx-bus-width = <2>; > + spi-tx-bus-width = <1>; > }; > }; > > -- > 2.17.1 > > >
On 2021/8/19 1:13, Chris Morgan wrote: > On Tue, Aug 17, 2021 at 09:41:01AM +0800, Jon Lin wrote: >> TX single line is more compatible for corresponding board >> >> Signed-off-by: Jon Lin <jon.lin@rock-chips.com> >> --- >> > There are still some bugs with this. Note that if you want for now > you can just abandon this patch (or fix it, either way is fine). > If you abandon it I'll make sure once the Linux driver is upstream > the nodes match between it and U-Boot. Otherwise, you'll need to make > the changes below to fix this. I had abandon it because I think it would be more accurate for you to deal with it >> arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi | 2 +- >> arch/arm/dts/rk3326-odroid-go2.dts | 2 +- >> 2 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi b/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi >> index 741e8dd935..0990005a15 100644 >> --- a/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi >> +++ b/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi >> @@ -70,7 +70,7 @@ >> u-boot,dm-pre-reloc; >> }; >> >> -&spi_flash { >> +&{/sfc@ff3a0000/flash@0} { >> u-boot,dm-pre-reloc; >> }; >> > This causes the devicetree to fail to compile, since the node is not > getting changed from spi_flash in the main dts file. > >> diff --git a/arch/arm/dts/rk3326-odroid-go2.dts b/arch/arm/dts/rk3326-odroid-go2.dts >> index 6f91f5040b..57e7f409d8 100644 >> --- a/arch/arm/dts/rk3326-odroid-go2.dts >> +++ b/arch/arm/dts/rk3326-odroid-go2.dts >> @@ -629,7 +629,7 @@ > The node should change here to just "flash@0" instead of > "spi_flash: xt25f128b@0" to match upstream (proposed). > > Also note that on the upstream (proposed) the pinctrl values are in a > different order, and the sfc_cs pinctrl is called the sfc_cs0. > >> reg = <0>; >> spi-max-frequency = <108000000>; >> spi-rx-bus-width = <2>; >> - spi-tx-bus-width = <2>; >> + spi-tx-bus-width = <1>; >> }; >> }; >> >> -- >> 2.17.1 >> >> >> > >
diff --git a/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi b/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi index 741e8dd935..0990005a15 100644 --- a/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi +++ b/arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi @@ -70,7 +70,7 @@ u-boot,dm-pre-reloc; }; -&spi_flash { +&{/sfc@ff3a0000/flash@0} { u-boot,dm-pre-reloc; }; diff --git a/arch/arm/dts/rk3326-odroid-go2.dts b/arch/arm/dts/rk3326-odroid-go2.dts index 6f91f5040b..57e7f409d8 100644 --- a/arch/arm/dts/rk3326-odroid-go2.dts +++ b/arch/arm/dts/rk3326-odroid-go2.dts @@ -629,7 +629,7 @@ reg = <0>; spi-max-frequency = <108000000>; spi-rx-bus-width = <2>; - spi-tx-bus-width = <2>; + spi-tx-bus-width = <1>; }; };
TX single line is more compatible for corresponding board Signed-off-by: Jon Lin <jon.lin@rock-chips.com> --- arch/arm/dts/rk3326-odroid-go2-u-boot.dtsi | 2 +- arch/arm/dts/rk3326-odroid-go2.dts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)