On Mon, Jan 28, 2019 at 12:30 PM Marek Vasut <[email protected]> wrote: > > On 1/27/19 9:47 AM, Simon Goldschmidt wrote: > > Am 26.01.2019 um 09:58 schrieb Marek Vasut: > >> On 1/25/19 9:30 PM, Simon Goldschmidt wrote: > >>> To clean up reset handling for socfpga gen5, let's move the code snippet > >>> taking the DDR controller out of reset from SPL to the DDR driver. > >>> > >>> Signed-off-by: Simon Goldschmidt <[email protected]> > >>> --- > >>> > >>> arch/arm/mach-socfpga/spl_gen5.c | 1 - > >>> drivers/ddr/altera/sdram_gen5.c | 4 ++++ > >>> 2 files changed, 4 insertions(+), 1 deletion(-) > >>> > >>> diff --git a/arch/arm/mach-socfpga/spl_gen5.c > >>> b/arch/arm/mach-socfpga/spl_gen5.c > >>> index ccdc661d05..f9bea892b1 100644 > >>> --- a/arch/arm/mach-socfpga/spl_gen5.c > >>> +++ b/arch/arm/mach-socfpga/spl_gen5.c > >>> @@ -98,7 +98,6 @@ void board_init_f(ulong dummy) > >>> socfpga_bridges_reset(1); > >>> } > >>> - socfpga_per_reset(SOCFPGA_RESET(SDR), 0); > >>> socfpga_per_reset(SOCFPGA_RESET(UART0), 0); > >>> socfpga_per_reset(SOCFPGA_RESET(OSC1TIMER0), 0); > >>> diff --git a/drivers/ddr/altera/sdram_gen5.c > >>> b/drivers/ddr/altera/sdram_gen5.c > >>> index 821060459c..bd54c420f8 100644 > >>> --- a/drivers/ddr/altera/sdram_gen5.c > >>> +++ b/drivers/ddr/altera/sdram_gen5.c > >>> @@ -7,6 +7,7 @@ > >>> #include <div64.h> > >>> #include <watchdog.h> > >>> #include <asm/arch/fpga_manager.h> > >>> +#include <asm/arch/reset_manager.h> > >>> #include <asm/arch/sdram.h> > >>> #include <asm/arch/system_manager.h> > >>> #include <asm/io.h> > >>> @@ -434,6 +435,9 @@ int sdram_mmr_init_full(unsigned int sdr_phy_reg) > >>> SDR_CTRLGRP_DRAMADDRW_ROWBITS_LSB; > >>> int ret; > >>> + /* release DDR from reset */ > >>> + socfpga_per_reset(SOCFPGA_RESET(SDR), 0); > >>> + > >> > >> Can the reset framework do this ? > > > > Hmm, it probably could, but I see that as a diferent patch. The altera > > DDR driver currently does not work with devicetree, so using the reset > > framework here would be a bigger patch, I think. > > > > Can we do that later and clean up this by just moving the code? > > How much effort is it to switch this driver over to DM ?
I really don't know. Searching through drivers/ddr does not seem to give me an example of a DTS-enabled ddr driver. Given that, I'd rather just push this patch now. While it's true that it doesn't clean up everything, it's not as if it would make things worse. Regards, Simon _______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
