[yocto] [meta-chip][RFC PATCH 3/3] chip: Add chip-u-boot-scr recipe and flash script

[Drew Moseley]

> On Jun 22, 2017, at 12:22 AM, Petter Mabäcker <petter at technux.se> wrote:
> 
> 2017-06-19 16:40 skrev Trevor Woerner:
> 
>> On Mon, Jun 19, 2017 at 7:22 AM, Drew Moseley <drew.moseley at mender.io <mailto:drew.moseley at mender.io>> wrote:
>>>> On Jun 19, 2017, at 1:48 AM, Trevor Woerner <twoerner at gmail.com <mailto:twoerner at gmail.com>> wrote:
>>>>> 
>>>>> If you have a serial console connected to +your board, you will see the progress and a message on the console will indicate when +the flashing is completed.
>>>> hmmm, never saw this. I have a console connected, the moment the script started, on the console I got: U-Boot SPL 2016.01 (Jun 19 2017 - 00:32:19) DRAM: 512 MiB CPU: 1008000000Hz, AXI/AHB/APB: 3/2/2 Trying to boot from which looks right, the date+time are good.
>>> Did you see the proper serial output after using a smaller image?
>> Yes. With a smaller image (~90MB) the flash update worked. Maybe
>> another improvement for a v2 would be for the script to look at the
>> size of /full/path/to/UBI_IMAGE and issue a warning/error if it's too
>> big?
>> 
>> I think the wording of what to expect on the host and over the serial
>> console could use some updating. Not to mention you say: "the status
>> LED on the CHIP board will flash 30 times per second..." which doesn't
>> seem to be the case, it looks more like the LED is toggled every 2
>> seconds (??), in any case, it's a lot slower than 30/sec.
> I also had the same experience as Trevor, that the led seems to flash every 2 seconds. Also the code indicate that 2 x 1 sec sleep is used in this area, so please go through the above description when it comes to led experience during flashing. Another minor comment is that I believe you should update the UBI_IMAGE example in README to use the "default" meta-chip hwup image (chip-hwup-image-chip.ubi).
> 
> BR Petter
> 
> Petter Mabäcker
> 
> Technux <petter at technux.se>
> www.technux.se

Indeed.  The comment was intended to say that it flashes 30/min.  Obviously flashing 30/sec would not be a very easy thing for people to spot.

I’ll work through the comments and submit V2.  It will take me a few days.

Many thanks for the feedback.

Drew

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.yoctoproject.org/pipermail/yocto/attachments/20170622/694cb4dc/attachment.html>