Hi Tom, On Fri, 9 Sept 2022 at 13:08, Tom Rini <tr...@konsulko.com> wrote: > > On Fri, Sep 09, 2022 at 12:21:11PM -0600, Simon Glass wrote: > > Hi, > > > > On Fri, 9 Sept 2022 at 09:34, Pali Rohár <p...@kernel.org> wrote: > > > > > > During init_dram() is called also compute_lowest_common_dimm_parameters() > > > function which prints multi-line detailed output. So print also > > > re-aligning > > > filler after "Detected ?DIMM" line to have "DRAM: " output aligned. > > > > > > Signed-off-by: Pali Rohár <p...@kernel.org> > > > --- > > > drivers/ddr/fsl/lc_common_dimm_params.c | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > At some point could this move to drivers/ram ? > > Depends on if we're keeping / making drivers/ram mean uses uclass and > drivers/ddr does not, I think.
Yes, except for Altera: $ git grep UCLASS_RAM drivers/ddr drivers/ddr/altera/sdram_gen5.c: .id = UCLASS_RAM, drivers/ddr/altera/sdram_soc64.c: .id = UCLASS_RAM, Regards, Simon