[U-Boot] [PATCH v2] ddr: altera: Configuring SDRAM extra cycles timing parameters
Chin Liang See
clsee at altera.com
Wed Sep 21 03:53:08 CEST 2016
On Wed, 2016-09-21 at 03:20 +0200, Marek Vasut wrote:
> On 09/20/2016 08:05 AM, Chin Liang See wrote:
> > To enable configuration of sdr.ctrlcfg.extratime1 register which
> > enable
> > extra clocks for read to write command timing. This is critical to
> > ensure successful LPDDR2 interface
> >
> > Signed-off-by: Chin Liang See <clsee at altera.com>
> > Cc: Marek Vasut <marex at denx.de>
> > Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> > ---
> > Changes for v2
> > - Removed v1 patches #2 to #9 as no boards are using LPDDR2
> > ---
> > arch/arm/mach-socfpga/include/mach/sdram.h | 8 +++++++-
> > arch/arm/mach-socfpga/qts-filter.sh | 2 +-
> > arch/arm/mach-socfpga/wrap_sdram_config.c | 9 +++++++++
> > drivers/ddr/altera/sdram.c | 3 +++
> > 4 files changed, 20 insertions(+), 2 deletions(-)
> > [...]
>
> I'd really like to avoid the ifdef, can we do that (fix all boards to
> set the register to zero) ? Otherwise I'm fine with the patch.
>
Ok I know where you come from. ifdef will cause some test challenge in
term of coverage. In this case, let me fix all boards to zeroes.
Thanks
Chin Liang
>
More information about the U-Boot
mailing list