From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 11 Feb 2009 11:22:31 +0100
Subject: [U-Boot] [PATCH] ppc4xx: Fix initialization of the SDRAM_CODT
	register
In-Reply-To: <2BA6BDB4C941C94BBD8375D55E05CCF91252772225@us-bv-m12.global.tektronix.net>
References: <2BA6BDB4C941C94BBD8375D55E05CCF91252772225@us-bv-m12.global.tektronix.net>
Message-ID: <200902111122.31783.sr@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Tuesday 10 February 2009, carolyn.j.smith at tektronix.com wrote:
> This fixes the initialization of the SDRAM_CODT register in the ppc4xx DDR2
> initialization code. It also removes use of the
> SDRAM_CODT_FEEDBACK_RCV_SINGLE_END and SDRAM_CODT_FEEDBACK_DRV_SINGLE_END
> #define's since they are reserved bits.
>
> Signed-off-by: Carolyn Smith <carolyn.smith@tektronix.com>

Patch doesn't apply:

Applying: ppc4xx: Fix initialization of the SDRAM_CODT register
error: patch failed: cpu/ppc4xx/44x_spd_ddr2.c:1101
error: cpu/ppc4xx/44x_spd_ddr2.c: patch does not apply
Patch failed at 0001.

How did you generate this patch? I suggest using "git format-patch".

Please find some other comments below.

> ---
> diff -uprN old/cpu/ppc4xx/44x_spd_ddr2.c new/cpu/ppc4xx/44x_spd_ddr2.c
> --- old/cpu/ppc4xx/44x_spd_ddr2.c       2009-02-10 09:12:56.000000000 -0800
> +++ new/cpu/ppc4xx/44x_spd_ddr2.c       2009-02-10 09:13:34.000000000 -0800
> @@ -1101,11 +1101,11 @@ static void program_codt(unsigned long *
>          * Set the SDRAM Controller On Die Termination Register
>         
> *-----------------------------------------------------------------*/
> mfsdram(SDRAM_CODT, codt);
> -       codt |= (SDRAM_CODT_IO_NMODE
> -                & (~SDRAM_CODT_DQS_SINGLE_END
> -                   & ~SDRAM_CODT_CKSE_SINGLE_END
> -                   & ~SDRAM_CODT_FEEBBACK_RCV_SINGLE_END
> -                   & ~SDRAM_CODT_FEEBBACK_DRV_SINGLE_END));
> +
> +       codt |= SDRAM_CODT_IO_NMODE;
> +
> +       codt &= ~(SDRAM_CODT_DQS_SINGLE_END | SDRAM_CODT_CKSE_SINGLE_END);

Please remove the empty line between those two codt assignments.

> +

And don't add another empty line here.

>
>         for (dimm_num = 0; dimm_num < num_dimm_banks; dimm_num++) {
>                 if (dimm_populated[dimm_num] != SDRAM_NONE) {
>

Please fix and resubmit.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================