[U-Boot] Please pull u-boot-fdt

[Simon Glass]

Hi Keerthy,

On 13 October 2016 at 05:47, Keerthy <a0393675 at ti.com> wrote:
>
>
> On Thursday 13 October 2016 03:13 PM, Keerthy wrote:
>>
>>
>>
>> On Wednesday 12 October 2016 08:13 PM, Simon Glass wrote:
>>>
>>> Hi Tom,
>>>
>>> On 12 October 2016 at 08:13, Tom Rini <trini at konsulko.com> wrote:
>>>>
>>>> On Tue, Oct 11, 2016 at 02:29:01PM -0600, Simon Glass wrote:
>>>>
>>>>> Hi Tom,
>>>>>
>>>>> This is mostly syncing up with upstream. I have a few more things to do
>>>>> also (some small, some large).
>>>>>
>>>>>
>>>>> The following changes since commit
>>>>> f5fd45ff64e28a73499548358e3d1ceda0de7daf:
>>>>>
>>>>>   Merge branch 'master' of git://www.denx.de/git/u-boot-imx (2016-10-08
>>>>> 09:33:37 -0400)
>>>>>
>>>>> are available in the git repository at:
>>>>>
>>>>>   git://git.denx.de/u-boot-fdt.git
>>>>>
>>>>> for you to fetch changes up to
>>>>> 7f899e436fe1e952950a14df92600d68d55dea5c:
>>>>>
>>>>>   libfdt: Drop inlining of fdt_path_offset() (2016-10-11 14:26:46
>>>>> -0600)
>>>>>
>>>>
>>>> Something in here is breaking my TI J6 Eco platform.  I'll try and kick
>>>> a rebase and bisect of just this series later to pinpoint which one is
>>>> at fault (I'm testing all of the other stuff that piled up while my
>>>> power was out atm).  Thanks!
>>>
>>>
>>> OK, let's hold off on this one until I've had time to do more testing
>>> also.
>>
>>
>> Seems like: pulling git://git.denx.de/u-boot-dm.git or
>> git://git.denx.de/u-boot-fdt.git alone works fine on DRA72.
>>
>> When i pull both i see a boot hang at the u-boot stage.
>
>
> Tom/Simon,
>
> I found that patch:
> commit 8985d1059ab2cc9a42e827494e6030adca27145a
> Author: Simon Glass <sjg at chromium.org>
> Date:   Sun Oct 2 17:59:29 2016 -0600
>
>     libfdt: Sync fdt_for_each_subnode() with upstream
>
> Changes the signature of fdt_for_each_subnode.
>
> Hence the u-boot hang in dra72.
>
> I will send couple of patches which will take care of this signature change.

Thanks for looking at this. The signature change is intentional in
order to sync up with upstream. I suspect there is something wrong
with the patch (or one of the others).

Regards,
Simon


>
> Regards,
> Keerthy
>
>
>>
>> Log:
>>
>> U-Boot SPL 2016.11-rc1-00191-g8d9b7ac (Oct 13 2016 - 15:06:06)
>> DRA722-GP ES1.0
>> Trying to boot from MMC1
>> *** Warning - bad CRC, using default environment
>>
>> reading u-boot.img
>> reading u-boot.img
>> reading u-boot.img
>> reading u-boot.img
>>
>>
>> U-Boot 2016.11-rc1-00191-g8d9b7ac (Oct 13 2016 - 15:06:06 +0530)
>>
>> CPU  : DRA722-GP ES1.0
>> Model: TI DRA722
>> Board: DRA72x EVM REV <NULL>
>> DRAM:  1 GiB
>>
>>>
>>> Regards,
>>> Simon
>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/mailman/listinfo/u-boot
>>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> http://lists.denx.de/mailman/listinfo/u-boot