[U-Boot] [PATCH v1 22/25] tools/kwboot: Sync with latest barebox version to support Armada XP

Stefan Roese
2014-07-30T08:53:13Z

Hi Luka,

On 30.07.2014 01:16, Luka Perkov wrote:

<snip>

>>> -#define KWBOOT_MSG_REQ_DELAY 1000 /* ms */
>>> -#define KWBOOT_MSG_RSP_TIMEO 1000 /* ms */
>>> +#define KWBOOT_MSG_REQ_DELAY 10 /* ms */
>>> +#define KWBOOT_MSG_RSP_TIMEO 50 /* ms */
>>
>> Unfortunately these settings do not work with Armada XP. I tested a bit and
>> it seems that e.g. this setting works on Armada XP:
>>
>> #define KWBOOT_MSG_REQ_DELAY 10 /* ms */
>> #define KWBOOT_MSG_RSP_TIMEO 1000 /* ms */
>>
>> Could you please check this on your Kirkwood platform? And perhaps find a
>> configuration that works for the SoC's? That would be great!
>
> I just tried this and it does not work on kirkwood. I've stumbled upon
> this thread though, it might be useful:
>
> http://lists.infradead.org/pipermail/barebox/2014-June/019618.html

Yes, thanks.

> Can we just add a flag to kwboot which will use appropriate values for
> kirkwood and armada? We can keep kirkwood by default, that way we won't
> break anything.

If nobody comes up with a working solution (timeout values that work for
both SoC's), then such a parameter is of course an option.

Thanks,
Stefan