From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 30 Jul 2014 10:53:13 +0200
Subject: [U-Boot] [PATCH v1 22/25] tools/kwboot: Sync with latest
 barebox version to support Armada XP
In-Reply-To: <20140729231606.GA3029@w500.lan>
References: <1403862911-6138-1-git-send-email-sr@denx.de>
	<1403862911-6138-23-git-send-email-sr@denx.de>
	<20140703220246.GA8506@w500.lan> <53D5F1F1.9070403@denx.de>
	<20140729231606.GA3029@w500.lan>
Message-ID: <53D8B279.4040605@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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