[U-Boot] [PATCH V2 1/2] [NEXT] orion5x: fix relocation-incompatible code

[Prafulla Wadaskar]

> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.aribaud at free.fr] 
> Sent: Monday, September 27, 2010 12:05 PM
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik
> Subject: Re: [U-Boot] [PATCH V2 1/2] [NEXT] orion5x: fix 
> relocation-incompatible code
> 
> Le 27/09/2010 07:52, Prafulla Wadaskar a écrit :
> 
> >> + * NOTES:
> 
> >> + * 2) in order to avoid losing access to BOOTCS when
> >> disabling window 7,
> >> + *    first configure window 6 for BOOTCS, then configure
> >> window 7 for BOOTCS,
> >> + *    then configure windows 6 for its own target.
> 
> >> +/* Disable window 6, configure it for FLASH, enable it. */
> >> +	writel(0,&winregs[6].ctrl);
> >> +	writel(ORION5X_ADR_BOOTROM,&winregs[6].base);
> >> +	writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_SZ_BOOTROM,
> >> +		ORION5X_TARGET_DEVICE, ORION5X_ATTR_BOOTROM,
> >>   		ORION5X_WIN_ENABLE),&winregs[6].ctrl);
> >> -	writel(ORION5X_ADR_DEV_CS2,&winregs[6].base);
> >> -
> >> -	/* Window 7: BOOT Memory address space */
> >> +/* Disable window 7, configure it for FLASH, enable it. */
> >> +	writel(0,&winregs[7].ctrl);
> >> +	writel(ORION5X_ADR_BOOTROM,&winregs[7].base);
> >>   	writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_SZ_BOOTROM,
> >>   		ORION5X_TARGET_DEVICE, ORION5X_ATTR_BOOTROM,
> >>   		ORION5X_WIN_ENABLE),&winregs[7].ctrl);
> >> -	writel(ORION5X_ADR_BOOTROM,&winregs[7].base);
> >> +/* Disable window 6, configure it for its intended target,
> >> enable it. */
> >> +	writel(0,&winregs[6].ctrl);
> >
> > This conflicts with above win6 configuration.
> > If done purposely then pls provide comments.
> >
> > Regards..
> > Prafulla . .
> 
> Er... The comment *is* there, right before the sequence.

Why do you need to configure win6 twice?

Regards..
Prafulla . .