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

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)
---------------------------------------------------------------- David Gibson (1): libfdt: Fix undefined behaviour in fdt_offset_ptr()
Hannes Schmelzer (1): cmd/fdt: add possibilty to have 'extrasize' on fdt resize
Simon Glass (4): libfdt: Sync up with upstream libfdt: Bring in upstream stringlist functions libfdt: Sync fdt_for_each_subnode() with upstream libfdt: Drop inlining of fdt_path_offset()
arch/arm/mach-tegra/xusb-padctl-common.c | 18 +++++----- arch/arm/mach-uniphier/board_late_init.c | 5 ++- arch/x86/cpu/irq.c | 5 +-- board/compulab/cm_fx6/cm_fx6.c | 2 +- cmd/fdt.c | 9 +++-- common/fdt_support.c | 3 +- common/image-fdt.c | 2 +- common/image-fit.c | 21 +++++------ common/image-sig.c | 8 ++--- common/image.c | 19 +++++----- drivers/clk/clk-uclass.c | 6 ++-- drivers/core/device.c | 2 +- drivers/gpio/dwapb_gpio.c | 3 +- drivers/i2c/mxc_i2c.c | 2 +- drivers/mailbox/mailbox-uclass.c | 6 ++-- drivers/net/cpsw.c | 2 +- drivers/net/keystone_net.c | 4 +-- drivers/net/mvpp2.c | 2 +- drivers/net/sun8i_emac.c | 5 +-- drivers/pci/pci_tegra.c | 2 +- drivers/phy/marvell/comphy_core.c | 2 +- drivers/pinctrl/exynos/pinctrl-exynos.c | 9 +++-- drivers/pinctrl/meson/pinctrl-meson.c | 4 +-- drivers/pinctrl/pinctrl-generic.c | 10 +++--- drivers/pinctrl/pinctrl-uclass.c | 2 +- drivers/reset/reset-uclass.c | 6 ++-- drivers/spi/fsl_qspi.c | 2 +- drivers/video/sunxi_display.c | 2 +- include/fdt_support.h | 10 +++++- include/libfdt.h | 187 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++--- ------------------------------------- lib/fdtdec.c | 4 +-- lib/libfdt/fdt.c | 13 +++---- lib/libfdt/fdt_overlay.c | 8 ++--- lib/libfdt/fdt_ro.c | 119 ++++++++++++++++++++++++++++++++++++++------------------------ lib/libfdt/fdt_rw.c | 4 ++- lib/libfdt/fdt_wip.c | 6 ++-- test/overlay/cmd_ut_overlay.c | 5 ++- 37 files changed, 298 insertions(+), 221 deletions(-)
Regards, Simon

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!

Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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.
Regards, Simon

On Wednesday 12 October 2016 08:13 PM, Simon Glass wrote:
Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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.
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@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

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@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@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.
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@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

On Thu, Oct 13, 2016 at 05:17:40PM +0530, Keerthy 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@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@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.
Ah, good catch. Simon, how do you want me to take this? I'm thinking I should take -dm and you can add these changes to -fdt and resubmit that.

Hi Tom,
On 13 October 2016 at 06:11, Tom Rini trini@konsulko.com wrote:
On Thu, Oct 13, 2016 at 05:17:40PM +0530, Keerthy 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@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@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.
Ah, good catch. Simon, how do you want me to take this? I'm thinking I should take -dm and you can add these changes to -fdt and resubmit that.
Yes that sounds good, thanks.
Regards, Simon

Hi Keerthy,
On 13 October 2016 at 05:47, Keerthy a0393675@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@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@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@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

On Thursday 13 October 2016 06:08 PM, Simon Glass wrote:
Hi Keerthy,
On 13 October 2016 at 05:47, Keerthy a0393675@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@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@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).
I have sent couple of patches already :-). They take care of the changes for libfdt.
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@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Hi Keerthy,
On 13 October 2016 at 06:39, Keerthy a0393675@ti.com wrote:
On Thursday 13 October 2016 06:08 PM, Simon Glass wrote:
Hi Keerthy,
On 13 October 2016 at 05:47, Keerthy a0393675@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@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@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).
I have sent couple of patches already :-). They take care of the changes for libfdt.
OK great, will take a look. Thanks!
Regards, Simon

Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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!
I'm not sure how this fault could affect your platform, since Keerthy's fix applies to a driver that is not yet upstream (it's in the dm pull request).
Could there be another fault lurking?
Regards, Simon

On Thu, Oct 13, 2016 at 08:24:22AM -0600, Simon Glass wrote:
Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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!
I'm not sure how this fault could affect your platform, since Keerthy's fix applies to a driver that is not yet upstream (it's in the dm pull request).
Could there be another fault lurking?
No, it makes sense. The -dm pull adds the new driver used on J6Eco and -fdt changes the call semantics used in that new driver.

Hi Tom,
On 13 October 2016 at 08:33, Tom Rini trini@konsulko.com wrote:
On Thu, Oct 13, 2016 at 08:24:22AM -0600, Simon Glass wrote:
Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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!
I'm not sure how this fault could affect your platform, since Keerthy's fix applies to a driver that is not yet upstream (it's in the dm pull request).
Could there be another fault lurking?
No, it makes sense. The -dm pull adds the new driver used on J6Eco and -fdt changes the call semantics used in that new driver.
OK that's good. I'll sort this out once things land in mainline.
Regards, Simon

Simon,
On Thursday 13 October 2016 07:54 PM, Simon Glass wrote:
Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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!
I'm not sure how this fault could affect your platform, since Keerthy's fix applies to a driver that is not yet upstream (it's in the dm pull request).
Could there be another fault lurking?
I tried u-boot master + dm branch. That is fine. I tried u-boot master + fdt branch. That is fine.
Then i tried u-boot master + fdt branch + dm branch. That is when i saw the hang on DRA72.
Boot is fine. I am not sure if Tom saw any other issue.
Regards, Keerthy
Regards, Simon

Hi Keerthy,
On 13 October 2016 at 08:59, Keerthy a0393675@ti.com wrote:
Simon,
On Thursday 13 October 2016 07:54 PM, Simon Glass wrote:
Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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!
I'm not sure how this fault could affect your platform, since Keerthy's fix applies to a driver that is not yet upstream (it's in the dm pull request).
Could there be another fault lurking?
I tried u-boot master + dm branch. That is fine. I tried u-boot master + fdt branch. That is fine.
Then i tried u-boot master + fdt branch + dm branch. That is when i saw the hang on DRA72.
Boot is fine. I am not sure if Tom saw any other issue.
Sounds good, thanks.
BTW what's up with your hex email address?
Regards, Simon

On Thursday 13 October 2016 08:38 PM, Simon Glass wrote:
Hi Keerthy,
On 13 October 2016 at 08:59, Keerthy a0393675@ti.com wrote:
Simon,
On Thursday 13 October 2016 07:54 PM, Simon Glass wrote:
Hi Tom,
On 12 October 2016 at 08:13, Tom Rini trini@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!
I'm not sure how this fault could affect your platform, since Keerthy's fix applies to a driver that is not yet upstream (it's in the dm pull request).
Could there be another fault lurking?
I tried u-boot master + dm branch. That is fine. I tried u-boot master + fdt branch. That is fine.
Then i tried u-boot master + fdt branch + dm branch. That is when i saw the hang on DRA72.
Boot is fine. I am not sure if Tom saw any other issue.
Sounds good, thanks.
BTW what's up with your hex email address?
Oh wow! I am seeing it as j-keerthy@ti.com. Let me check what is going with my mailer. Thanks for catching it.
Regards, Simon
participants (3)
-
Keerthy
-
Simon Glass
-
Tom Rini