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

[Albert ARIBAUD]

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.

Amicalement,
-- 
Albert.