[U-Boot] [PATCH][v3] powerpc/CoreNet: add tool to support pbl image build.

[Xie Shaohui-B21989]

>-----Original Message-----
>From: Andy Fleming [mailto:afleming at gmail.com]
>Sent: Thursday, August 09, 2012 5:10 AM
>To: Xie Shaohui-B21989
>Cc: u-boot at lists.denx.de
>Subject: Re: [U-Boot] [PATCH][v3] powerpc/CoreNet: add tool to support pbl
>image build.
>
>On Tue, Jun 12, 2012 at 3:11 AM, Shaohui Xie <Shaohui.Xie at freescale.com>
>wrote:
>> Provides a tool to build boot Image for PBL(Pre boot loader) which is
>> used on Freescale CoreNet SoCs, PBL can be used to load some
>> instructions and/or data for pre-initialization. The default output
>> image is u-boot.pbl, for more details please refer to
>doc/README.pblimage.
>>
>> Signed-off-by: Shaohui Xie <Shaohui.Xie at freescale.com>
>> ---
>> changes for v3:
>> 1. dump config.mk, aligned to comment of Wolfgang Denk; 2. split one
>> configuration file to two, one is for RCW, one is for PBI commands,
>> add second entry in mkimage for second configuration file, aligned to
>> comment of Wood Scott; 3. refined the README.pblimage;
>>
>>  Makefile                           |    6 +
>>  board/freescale/corenet_ds/pbi.cfg |   51 ++++++
>>  board/freescale/corenet_ds/rcw.cfg |   32 ++++
>
>
>If I apply this patch, and then run:
>
>make P5020DS_SDCARD_config
>make u-boot.pbl
>
>And then follow the instructions from the README:
>
>2). Boot from SD/MMC
>Write u-boot.pbl to SD/MMC from offset 0x1000.
>=>tftp 100000 u-boot.pbl
>=>mmcinfo
>=>mmc write 100000 8 441
>Change SW1[1:5] = off off on on off.
>
>Will it work?
>
>If I replace the word P5020DS with P3041DS, will it still work?
>
>What I'm trying to figure out is whether rcw.cfg and pbi.cfg are truly
>generic for all 3 systems this patch claims support for, or if I have to
>replace them with different versions for those P* systems. My guess is
>that the rcw.cfg needs to be changed, but I wanted to check. If it needs
>to be changed, then what we need is something like rcw_p5020ds.cfg,
>rcw_p4080ds.cfg, rcw_p3041.cfg.
[Xie Shaohui] You are right. The pbl.cfg is general, but rcw.cfg is different on different boards, it must be replaced according to boards when building boot Image for different boards, but the RCW is really based on user's choice, there are so many different settings user can do to change RCW. What should this situation be handled? Or we just provide a default RCW for each board respectively, but user will still need to replace it according to his case.


Best Regards, 
Shaohui Xie