[U-Boot] [PATCH v2] libfdt: Introduce new ARCH_FIXUP_FDT option

Michal Simek michal.simek at xilinx.com
Thu Jul 28 09:08:50 CEST 2016


On 28.7.2016 04:24, Simon Glass wrote:
> Hi Michal,
> 
> On 24 July 2016 at 20:07, Simon Glass <sjg at chromium.org> wrote:
>> On 15 July 2016 at 01:35, Michal Simek <michal.simek at xilinx.com> wrote:
>>> Add new Kconfig option to disable arch_fixup_fdt() calls for cases where
>>> U-Boot shouldn't update memory setup in DTB file.
>>> One example of usage of this option is to boot OS with different memory
>>> setup than U-Boot use.
>>>
>>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>>> ---
>>>
>>> Changes in v2:
>>> - Change logic to be possitive - sjg
>>>
>>>  Kconfig                  | 9 +++++++++
>>>  arch/arm/lib/bootm-fdt.c | 2 ++
>>>  arch/arm/lib/bootm.c     | 2 ++
>>>  arch/mips/lib/bootm.c    | 2 ++
>>>  common/image-fdt.c       | 7 ++-----
>>>  5 files changed, 17 insertions(+), 5 deletions(-)
>>
>> Applied to u-boot-dm, thanks!
> 
> Unfortunately this breaks pic32mzdask, a MIPS board. I did not notice
> this earlier for some reason...perhaps it snuck in after my rebase.
> Can you please take a look?

Have sent v3. In v2 I didn't change that macro to follow positive
configuration. It should be good now.

BTW: Would it be possible to extend buildman to also report problems
with defconfig? I see that more and more defconfigs. I personally run
this loop time to time but will be good if this is the part of buildman.


for i in `ls configs/xilinx_zynqmp_*`; do NAME=`basename $i`; echo
$NAME; make $NAME; make savedefconfig; cp defconfig $i; done
for i in `ls configs/zynq_*`; do NAME=`basename $i`; echo $NAME; make
$NAME; make savedefconfig; cp defconfig $i; done
for i in `ls configs/microblaze*`; do NAME=`basename $i`; echo $NAME;
make $NAME; make savedefconfig; cp defconfig $i; done

Thanks,
Michal



More information about the U-Boot mailing list