From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lokesh Vutla Date: Wed, 27 Nov 2013 10:18:15 +0530 Subject: [U-Boot] [PATCH V2 12/14] ARM: AM43xx: EPOS_EVM: Add support for LPDDR2 In-Reply-To: References: <1385014699-7257-1-git-send-email-lokeshvutla@ti.com> <1385014699-7257-13-git-send-email-lokeshvutla@ti.com> <5292DC67.6030409@ti.com> Message-ID: <5295798F.7040108@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wednesday 27 November 2013 05:42 AM, Vaibhav Bedia wrote: > On Mon, Nov 25, 2013 at 12:13 AM, Lokesh Vutla wrote: >> On Friday 22 November 2013 02:16 AM, Vaibhav Bedia wrote: >>> On Thu, Nov 21, 2013 at 1:18 AM, Lokesh Vutla wrote: >>>> AM4372 EPOS EVM has 1GB LPDDR2(Part no: MT42L256M32D2LG-25 WT:A) >>>> Adding LPDDR2 init sequence and register details for the same. >>>> Below is the brief description of LPDDR2 init sequence: >>>> -> Configure VTP >>>> -> Configure DDR IO settings >>>> -> Disable initialization and refreshes until EMIF registers are programmed. >>>> -> Program Timing registers >>>> -> Program PHY control and Temp alert and ZQ config registers. >>>> -> Enable initialization and refreshes and configure SDRAM CONFIG register >>>> -> Wait till initialization is complete and the configure MR registers. >>>> >>> >>> This patch does too many things, some of which affects AM335x and needs to be >>> split up. I lost track of what you were doing as i scrolled down :\ >> It does only two things. Update IO settings and emif configuration. >> I wanted to keep these things in a single patch so that if some functionality >> breaks down I can burn down to this patch. > > That doesn't mean that you mix up everything in one humungous patch and > try to force your way through. OK, Ill separate out EMIF configuration and io settings and repost. Thanks Lokesh > >>> >>>> Signed-off-by: Lokesh Vutla >>>> --- >>>> arch/arm/cpu/armv7/am33xx/ddr.c | 147 +++++++++++++++++++- >>>> arch/arm/cpu/armv7/am33xx/emif4.c | 25 +++- >>>> arch/arm/include/asm/arch-am33xx/clocks_am33xx.h | 3 + >>>> arch/arm/include/asm/arch-am33xx/cpu.h | 5 + >>>> arch/arm/include/asm/arch-am33xx/ddr_defs.h | 33 ++++- >>>> arch/arm/include/asm/arch-am33xx/hardware_am43xx.h | 1 + >>>> arch/arm/include/asm/emif.h | 12 ++ >>>> board/isee/igep0033/board.c | 10 +- >>>> board/phytec/pcm051/board.c | 12 +- >>>> board/siemens/dxr2/board.c | 10 +- >>>> board/siemens/pxm2/board.c | 10 +- >>>> board/siemens/rut/board.c | 10 +- >>>> board/ti/am335x/board.c | 40 +++++- >>>> board/ti/am43xx/board.c | 66 +++++++++ >>>> board/ti/ti814x/evm.c | 4 +- >>>> board/ti/ti816x/evm.c | 12 +- >>>> 16 files changed, 373 insertions(+), 27 deletions(-) > > As the diffstat above shows, this patch affects a critical portion of AM335x, > TI81xx and and at the same time adds in support for a new SoC. If that's not > mixing up things i honestly don't know what is. > > And if you have some discussions off-list you should at least attempt to capture > the summary of what was agreed to do taken up as TODO for the benefit of > others. If there's a agreed upon patch to improving the code that helps is > overlooking some short term compromises. >