[U-Boot] [PATCH 3/4] arm: mvebu: Add runtime boot-device detection
Kevin Smith
kevin.smith at elecsyscorp.com
Wed Jan 13 17:34:03 CET 2016
On 01/13/2016 10:18 AM, Stefan Roese wrote:
> On 13.01.2016 17:12, Kevin Smith wrote:
>> On 01/09/2016 03:07 AM, Stefan Roese wrote:
>>> @@ -100,14 +100,24 @@
>>> #if defined(CONFIG_ARMADA_38X)
>>> /* SAR values for Armada 38x */
>>> #define CONFIG_SAR_REG (MVEBU_REGISTER(0x18600))
>>> +
>>> #define SAR_CPU_FREQ_OFFS 10
>>> #define SAR_CPU_FREQ_MASK (0x1f << SAR_CPU_FREQ_OFFS)
>>> #define SAR_BOOT_DEVICE_OFFS 4
>>> #define SAR_BOOT_DEVICE_MASK (0x1f << SAR_BOOT_DEVICE_OFFS)
>>> +
>>> +#define BOOT_DEV_SEL_OFFS 4
>>> +#define BOOT_DEV_SEL_MASK (0x1f << BOOT_DEV_SEL_OFFS)
>>> +
>> It is a little confusing that this is added incorrectly here, then
>> corrected in the next patch ... could this just be added as 0x3f here?
>
> Ah, right. I've fixed this define in the next patch:
>
> arm: mvebu: Add runtime detection of UART (xmodem) boot-mode
>
> Which is already waiting for upstreaming as well (once Luka picks it
> up). I hope this is okay. Otherwise I would need to re-do those
> patches again.
>
> Thanks,
> Stefan
>
Oh, sure, no problem. Not worth redoing that work.
Thanks,
Kevin
More information about the U-Boot
mailing list