[U-Boot] [PATCH v2 15/22] omap4: add sdram init support

Wolfgang Denk
2011-05-15T20:01:49Z

Dear Aneesh V,

In message <1305472900-4004-16-git-send-email-aneesh@ti.com> you wrote:
> Add support for the SDRAM controller (EMIF).
>
> Signed-off-by: Aneesh V <aneesh@ti.com>
> V2:
> * Changes for makefile changes
> * Minor corrections in do_lpddr2_init()
> * Minor corrections to read_idle interval calculation
> * Sanity test of memory after doing the initialization
> * Fixed warnings reported with with latest GCC compilers
> ---
> arch/arm/cpu/armv7/omap4/Makefile | 3 +
> arch/arm/cpu/armv7/omap4/board.c | 2 +-
> arch/arm/cpu/armv7/omap4/emif.c | 298 +++++++++++
> arch/arm/cpu/armv7/omap4/sdram_elpida.c | 118 +++++
> arch/arm/include/asm/arch-omap4/emif.h | 719 +++++++++++++++++++++++++++
> arch/arm/include/asm/arch-omap4/omap4.h | 11 +
> arch/arm/include/asm/arch-omap4/sys_proto.h | 1 +
> include/configs/omap4_sdp4430.h | 5 -
> spl/board/ti/omap4.mk | 9 +-
> 9 files changed, 1159 insertions(+), 7 deletions(-)
> create mode 100644 arch/arm/cpu/armv7/omap4/emif.c
> create mode 100644 arch/arm/cpu/armv7/omap4/sdram_elpida.c
> create mode 100644 arch/arm/include/asm/arch-omap4/emif.h

I'm really surprised to see this patch at this position in the middle
of this patch series.

Can you please explain why you think this is the right place for it?

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
2000 pounds of chinese soup = 1 Won Ton