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

[Michal Simek]

On 1.8.2016 03:03, Simon Glass wrote:
> 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.

Ordering and when config options go to Kconfig then some options are
becoming default.

> How's your Python?

I do write some pytest stuff but definitely not any close to any expert
level.

Thanks,
Michal