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

Simon Glass sjg at chromium.org
Mon Aug 1 03:03:21 CEST 2016


Hi Michal,

On 28 July 2016 at 01:08, Michal Simek <michal.simek at xilinx.com> wrote:
> 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

Do you mean that the ordering or defaults are wrong? I have an alias
to fix that, but yes it would be good if buildman could support it.
How's your Python?

Regards,
Simon


More information about the U-Boot mailing list