[U-Boot] [PATCH 1/2] configs: am43xx_evm: Merge in usbhost defconfig

[Andrew F. Davis]

On 03/12/2017 10:16 PM, Lokesh Vutla wrote:
> 
> 
> On Saturday 11 March 2017 03:32 AM, Andrew F. Davis wrote:
>> I was once again going to try to sync the mode specific defconfigs
>> with the main AM43xx defconfig when I decided it may make more
>> sense to merge in USB host boot support in to the main defconfig.
>>
>> The reason we had separate defconfigs before was that SPL needs
>> to be placed in OCMC SRAM by the ROM when booting from USB as
>> it uses DMA and ARM internal SRAM space is not externally
>> accessible. All this means is we need to have our SPL load address
>> be greater than 0x4030_0000 when USB booting. For other cases
>> we use 0x402F_0000, this gives us an extra 64kB of space in addition
>> to the 256kB of OCMC RAM, as our SPL is not even half this size
>> currently this is not strictly needed.
>>
>> Make all boot modes load to 0x4030_0000+, add SPL USB host
>> boot support to the main am43xx defconfig, and then remove
> 
> This breaks all the supported peripheral boot modes. Have you tested
> UART/ethernet boot with this patch?
> 

Ahh, I tested memory modes only.

I must have misunderstood the datasheet. It claims when using peripheral
boot modes on a GP device the image header (containing the desired
relocation address) is "not required", but if this patch breaks
peripheral boot then it really means it is ignored, even if it is
included in the image.

> I specifically wanted a separate defconfig for usb_host as it is a
> special case and rest of the boot modes can be supported using a single
> defconfig. If you don't like this approach, please create separate
> defconfigs for peripheral and memory boot modes.
> 

I don't have any strong preference for how the non-HS devices are
handled, so if you don't like this approach then this patch can be
safely ignored. (2/2 is still valid)

Thanks,
Andrew

> Thanks and regards,
> Lokesh
>