All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Cédric Le Goater" <clg@kaod.org>
To: Arnout Vandecappelle <arnout@mind.be>, <buildroot@buildroot.org>
Cc: chin-ting_kuo@aspeedtech.com, troy_lee@aspeedtech.com,
	Joel Stanley <joel@jms.id.au>
Subject: Re: [Buildroot] [PATCH v2 2/2] configs/aspeed_ast2500evb: new defconfig
Date: Tue, 21 Jun 2022 11:17:00 +0200	[thread overview]
Message-ID: <278ae830-93b2-e0be-9f1c-164b88248497@kaod.org> (raw)
In-Reply-To: <1ae45d2d-5619-c740-58b3-4623e9587b30@mind.be>

On 6/20/22 22:04, Arnout Vandecappelle wrote:
> 
> 
> On 06/06/2022 15:57, Cédric Le Goater wrote:
>> The kernel is from upstream with a custom config because the default
>> 'aspeed_g5' defconfig builds a zImage file image too big for the flash
>> layout.
> 
>   Still, we prefer to use the in-tree defconfig with a fragment to remove the stuff that is not needed.

I had that before. I will refresh the patch.

>> U-Boot is from the OpenBMC branch where most of development is
>> done. Latest mainline should work.
> 
>   In that case, we generally prefer the mainline then.

ok. I will give it try. Anyhow, we should improve our upstream support.
  
>> The main resulting file from the build is a flash image. The partition
>> layout matches the OpenBMC one for 32M chips. It makes it easier to
>> update the different partitions from Linux. Intermediate files can be
>> used to boot from U-boot over the network or to boot QEMU using
>> -kernel/-initrd/-dtb.
> 
>   Ooh, very nice that the same config can be booted in qemu as well!

Yes. The same flash image can be used for QEMU also. I didn't know
how to add a QEMU test for a non-QEMU board for it but that'd be
nice.

> 
>   Also, same comments as for the first patch.

Thanks,

C.


  
> 
>   Regards,
>   Arnout
> 
>>
>> Reviewed-by: Joel Stanley <joel@jms.id.au>
>> Signed-off-by: Cédric Le Goater <clg@kaod.org>
>> ---
>>
>>   Changes in v2:
>>
>>   - minor cleanups in readme.txt
>>   board/aspeed/ast2500-evb/genimage.cfg   |  40 ++++
>>   board/aspeed/ast2500-evb/linux.config   | 262 ++++++++++++++++++++++++
>>   board/aspeed/ast2500-evb/readme.txt     |  80 ++++++++
>>   board/aspeed/ast2500-evb/uboot.fragment |   1 +
>>   configs/aspeed_ast2500evb_defconfig     |  58 ++++++
>>   5 files changed, 441 insertions(+)
>>   create mode 100644 board/aspeed/ast2500-evb/genimage.cfg
>>   create mode 100644 board/aspeed/ast2500-evb/linux.config
>>   create mode 100644 board/aspeed/ast2500-evb/readme.txt
>>   create mode 100644 board/aspeed/ast2500-evb/uboot.fragment
>>   create mode 100644 configs/aspeed_ast2500evb_defconfig
> 
> [snip]
> 

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2022-06-21  9:33 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-06 13:57 [Buildroot] [PATCH v2 1/2] configs/aspeed_ast2600evb: new defconfig Cédric Le Goater
2022-06-06 13:57 ` [Buildroot] [PATCH v2 2/2] configs/aspeed_ast2500evb: " Cédric Le Goater
2022-06-20 20:04   ` Arnout Vandecappelle
2022-06-21  9:17     ` Cédric Le Goater [this message]
2022-06-06 22:57 ` [Buildroot] [PATCH v2 1/2] configs/aspeed_ast2600evb: " Joel Stanley
2022-06-07  7:05   ` Cédric Le Goater
2022-06-20 20:00 ` Arnout Vandecappelle
2022-06-21 11:23   ` Cédric Le Goater

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=278ae830-93b2-e0be-9f1c-164b88248497@kaod.org \
    --to=clg@kaod.org \
    --cc=arnout@mind.be \
    --cc=buildroot@buildroot.org \
    --cc=chin-ting_kuo@aspeedtech.com \
    --cc=joel@jms.id.au \
    --cc=troy_lee@aspeedtech.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.