21 Feb
2019
21 Feb
'19
11:11 p.m.
Am Do., 21. Feb. 2019, 23:05 hat Marek Vasut marex@denx.de geschrieben:
On 2/21/19 11:03 PM, Simon Goldschmidt wrote:
Am Do., 21. Feb. 2019, 22:56 hat Marek Vasut <marex@denx.de mailto:marex@denx.de> geschrieben:
On 2/21/19 10:43 PM, Simon Goldschmidt wrote: > The SPL for socfpga gen5 currently takes all peripherals out ofreset
> unconditionally. To implement proper reset handling forperipherals,
> the reset node has to be provided with the SPL dts. > > In preparation to move the DDR driver to DM, the sdr node isrequired
> in SPL, too. > > This patch adds "u-boot,dm-pre-reloc" to U-Boot specific dtsi addon > files so that the reset manager and SDR driver correctly probe inSPL.
> > Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt@gmail.com <mailto:simon.k.r.goldschmidt@gmail.com>> > --- > > Changes in v2: None > > arch/arm/dts/socfpga_arria5_socdk-u-boot.dtsi | 8++++++++
> arch/arm/dts/socfpga_cyclone5_dbm_soc1.dts | 8++++++++
> arch/arm/dts/socfpga_cyclone5_de0_nano_soc-u-boot.dtsi | 8++++++++
> arch/arm/dts/socfpga_cyclone5_de10_nano.dts | 8++++++++
> arch/arm/dts/socfpga_cyclone5_de1_soc.dts | 8++++++++
> arch/arm/dts/socfpga_cyclone5_is1.dts | 8++++++++
> arch/arm/dts/socfpga_cyclone5_socdk-u-boot.dtsi | 8++++++++
> arch/arm/dts/socfpga_cyclone5_sockit-u-boot.dtsi | 8++++++++
> arch/arm/dts/socfpga_cyclone5_socrates-u-boot.dtsi | 8++++++++
> arch/arm/dts/socfpga_cyclone5_sr1500.dts | 8++++++++
> arch/arm/dts/socfpga_cyclone5_vining_fpga-u-boot.dtsi | 8++++++++
> 11 files changed, 88 insertions(+) > > diff --git a/arch/arm/dts/socfpga_arria5_socdk-u-boot.dtsi b/arch/arm/dts/socfpga_arria5_socdk-u-boot.dtsi > index c44d1ee2fa..8aaec56285 100644 > --- a/arch/arm/dts/socfpga_arria5_socdk-u-boot.dtsi > +++ b/arch/arm/dts/socfpga_arria5_socdk-u-boot.dtsi > @@ -17,6 +17,14 @@ > }; > }; > > +&rst { > + u-boot,dm-pre-reloc; > +}; > + > +&sdr { > + u-boot,dm-pre-reloc; > +}; What about some socfpga-common-u-boot.dtsi to avoid duplication ?Right. I tested with 'socfpga-u-boot.dtsi' but the hot included for gen10 as well.
hot included ? I don't think I understand.
Argh. I'm writing from my mobile...
What I meant was that file got included by the automatism so I couldn't use it.
Regards, Simon
But your suggested name should work fine I guess.
Regards, Simon