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

2019-11-05 Thread Tom Rini
On Tue, Nov 05, 2019 at 05:23:51AM -0700, Simon Glass wrote:

> Hi Tom,
> 
> Passing run here:
> 
> https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/1210
> 
> 
> The following changes since commit 73b6e6ad254b36763419cdd3fdf406c0094517b7:
> 
>   Merge tag 'u-boot-imx-20191104' of
> https://gitlab.denx.de/u-boot/custodians/u-boot-imx (2019-11-04
> 12:57:41 -0500)
> 
> are available in the Git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git tags/fdt-pull-5nov19
> 
> for you to fetch changes up to 388560134b99dc4cc752627d3a7e9f8c8c2a89a7:
> 
>   binman: Move to use Python 3 (2019-11-04 18:15:32 -0700)
> 

Now first,
Applied to u-boot/master, thanks!

And second, which is why I've included the board maintainers list and
custodian lists, this causes a bit of size growth everywhere just about
(tbs2910 being the exception) because by default upstream now has
various input validation routines.  It doesn't catch "tell the hardware
to overvolt something" but does catch "pass garbage in the property so
we can overwrite the stack and ..." type problems.  And it's
configurable.  If you turn off all of the validation stuff, which we do
in SPL/TPL by default (size concerns!), in the main U-Boot we go from a
size growth to a size shrink.  An arbitrary PowerPC board I picked to
confirm this on grows by 985 bytes now, but if I turned everything off
it would shrink by 1100 bytes.  There's an inbetween setting that would
probably result in neutral size change.

The default is all of the input validation we can do because I believe
it's important to validate inputs when we can validate them.  I also
firmly believe board maintainers know their requirements best and can
provide a different value for their board(s).

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2019-11-05 Thread Simon Glass
Hi Tom,

Passing run here:

https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/1210


The following changes since commit 73b6e6ad254b36763419cdd3fdf406c0094517b7:

  Merge tag 'u-boot-imx-20191104' of
https://gitlab.denx.de/u-boot/custodians/u-boot-imx (2019-11-04
12:57:41 -0500)

are available in the Git repository at:

  git://git.denx.de/u-boot-fdt.git tags/fdt-pull-5nov19

for you to fetch changes up to 388560134b99dc4cc752627d3a7e9f8c8c2a89a7:

  binman: Move to use Python 3 (2019-11-04 18:15:32 -0700)


Update to latest libfdt and pylibfdt, with added size control
Update binman, dtoc, patman, buildman to Python 3
Update move_config, rkmux, microcode_tool to Python 3


Simon Glass (20):
  fdt: Add INT32_MAX to kernel.h for libfdt
  fdt: Add Kconfig options to control code size
  mx6: tbs2910: Minimise libfdt code size
  fdt: Sync up to the latest libfdt
  patman: Adjust 'command' to return strings instead of bytes
  patman: Use unicode for file I/O
  patman: Move to use Python 3
  buildman: Convert to Python 3
  test_fdt: Move to use Python 3
  test_dtoc: Move to use Python 3
  microcode_tool: Convert to Python 3
  move_config: Convert to Python 3
  rkmux: Convert to Python 3
  pylibfdt: Convert to Python 3
  pylibfdt: Sync up with upstream
  pylibfdt: Correct the type for fdt_property_stub()
  binman: Remember the pre-reset entry size
  binman: Convert a few tests to Python 3
  dtoc: Convert fdt.py to Python 3
  binman: Move to use Python 3

 configs/tbs2910_defconfig |   1 +
 include/linux/kernel.h|   2 +
 include/linux/libfdt_env.h|   1 +
 lib/Kconfig   |  33 +++
 lib/libfdt/Makefile   |   3 +-
 lib/libfdt/fdt_ro.c   | 420 +-
 scripts/dtc/libfdt/Makefile.libfdt|   7 +
 scripts/dtc/libfdt/fdt.c  | 182 ++-
 scripts/dtc/libfdt/fdt.h  |  47 +---
 scripts/dtc/libfdt/fdt_addresses.c|  94 
 scripts/dtc/libfdt/fdt_empty_tree.c   |  47 +---
 scripts/dtc/libfdt/fdt_overlay.c  |  91 +++-
 scripts/dtc/libfdt/fdt_ro.c   | 341 ++-
 scripts/dtc/libfdt/fdt_rw.c   | 119 +-
 scripts/dtc/libfdt/fdt_strerror.c |  47 +---
 scripts/dtc/libfdt/fdt_sw.c   | 241 ---
 scripts/dtc/libfdt/fdt_wip.c  |  47 +---
 scripts/dtc/libfdt/libfdt.h   | 268 +-
 scripts/dtc/libfdt/libfdt_env.h   |  48 +---
 scripts/dtc/libfdt/libfdt_internal.h  | 144 +++-
 scripts/dtc/pylibfdt/Makefile |   2 +-
 scripts/dtc/pylibfdt/libfdt.i_shipped |  51 +++--
 scripts/dtc/pylibfdt/setup.py |   2 +-
 tools/binman/binman.py|   2 +-
 tools/binman/cbfs_util_test.py|   2 +-
 tools/binman/entry.py |  25 +-
 tools/binman/entry_test.py|  15 --
 tools/binman/etype/intel_fit.py   |   2 +-
 tools/binman/ftest.py |  16 +-
 tools/buildman/board.py   |   9 +-
 tools/buildman/bsettings.py   |  20 +-
 tools/buildman/builder.py |  47 ++--
 tools/buildman/builderthread.py   |  24 +-
 tools/buildman/buildman.py|  10 +-
 tools/buildman/control.py |  44 ++--
 tools/buildman/func_test.py   |  16 +-
 tools/buildman/test.py|  22 +-
 tools/buildman/toolchain.py   |  99 
 tools/dtoc/dtoc.py|   2 +-
 tools/dtoc/fdt.py |  17 +-
 tools/dtoc/test_dtoc.py   |   1 +
 tools/dtoc/test_fdt.py|   2 +-
 tools/libfdt/fdt_rw.c |   3 +-
 tools/microcode-tool.py   |  28 +--
 tools/moveconfig.py   |  82 +++
 tools/patman/command.py   |  31 ++-
 tools/patman/func_test.py |   8 +-
 tools/patman/patchstream.py   |   4 +-
 tools/patman/patman.py|   2 +-
 tools/patman/series.py|   2 +-
 tools/patman/settings.py  |   4 +-
 tools/patman/test.py  |   4 +-
 tools/patman/tools.py |  29 ++-
 tools/rkmux.py|  16 +-
 54 files changed, 1645 insertions(+), 1181 deletions(-)
 mode change 100644 => 100755 tools/dtoc/test_dtoc.py

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-09-16 Thread Tom Rini
On Fri, Sep 15, 2017 at 01:31:29PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> Here are the overlay and dtoc series.
> 
> 
> The following changes since commit 5541543f686b43210fb92181003ff7175d4ab036:
> 
>   configs: at91: Remove CONFIG_SYS_EXTRA_OPTIONS assignment
> (2017-09-14 16:02:48 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to ea28e488f743520f7f83b341f28818c32dae1ee3:
> 
>   test: overlay: Add unit test for stacked overlay (2017-09-15 05:27:49 -0600)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-09-15 Thread Simon Glass
Hi Tom,

Here are the overlay and dtoc series.


The following changes since commit 5541543f686b43210fb92181003ff7175d4ab036:

  configs: at91: Remove CONFIG_SYS_EXTRA_OPTIONS assignment
(2017-09-14 16:02:48 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to ea28e488f743520f7f83b341f28818c32dae1ee3:

  test: overlay: Add unit test for stacked overlay (2017-09-15 05:27:49 -0600)


Franklin S Cooper Jr (1):
  doc: overlays: Tweak documentation regarding overlay

Pantelis Antoniou (12):
  fdt: Introduce helper method fdt_overlay_apply_verbose()
  fdt: Switch to using the verbose overlay application method
  fdt: Allow stacked overlays phandle references
  fit: Allow multiple images per property
  fit: Do not throw away extra configuration on fit_image_load()
  fit: Introduce methods for applying overlays on fit-load
  fit: fdt overlays doc
  doc: Document how to apply fdt overlays
  dtbo: make dtbos special
  config: sandbox: Add unit tests
  ut: fix fdt_getprop_u32() change
  test: overlay: Add unit test for stacked overlay

Simon Glass (16):
  fdt: Sync libfdt up to upstream
  dtoc: Adjust Node to record its parent
  dtoc: Add a 64-bit type and a way to convert cells into 64 bits
  dtoc: Avoid very long lines in output
  dtoc: Add support for 32 or 64-bit addresses
  dtoc: Handle 'reg' properties with unusual sizes
  dtoc: Update the Fdt class to record phandles
  dtoc: Use the Fdt's class's phandle map
  dtoc: Make is_phandle() a member function
  dtoc: Rename is_phandle() and adjust it to return more detail
  dtoc: Rename the phandle struct
  dtoc: Put each phandle on a separate line
  dtoc: Put phandle args in an array
  dtoc: Support properties containing multiple phandle values
  dtoc: Rename the auto-generated dt-structs.h file
  dtoc: Add a header to the generated files

Tien Fong Chee (1):
  libfdt: Initialize the stack variable

Tom Rini (1):
  Travis-CI: Switch back to using the top of tree dtc

 .travis.yml  |   5 +-
 cmd/fdt.c|   7 +-
 common/fdt_support.c |  31 +++
 common/image-fdt.c   |   7 +-
 common/image-fit.c   | 210 +++--
 configs/sandbox_defconfig|   2 +
 doc/README.fdt-overlays  | 114 +++
 doc/driver-model/of-plat.txt |   2 +-
 doc/uImage.FIT/command_syntax_extensions.txt |  12 +-
 doc/uImage.FIT/overlay-fdt-boot.txt  | 225 ++
 doc/uImage.FIT/source_file_format.txt|   6 +-
 drivers/clk/clk-uclass.c |   4 +-
 drivers/clk/rockchip/clk_rk3368.c|   2 +-
 drivers/clk/rockchip/clk_rk3399.c|   4 +-
 drivers/core/regmap.c|   2 +-
 include/clk.h|   4 +-
 include/dt-structs.h |  16 +-
 include/fdt_support.h|   2 +
 include/fdtdec.h |   2 +
 include/image.h  |  25 +++
 include/regmap.h |   2 +-
 include/syscon.h |   6 +-
 lib/libfdt/fdt_overlay.c | 228 +++---
 lib/libfdt/fdt_rw.c  |  20 +-
 lib/libfdt/fdt_wip.c |   2 +-
 lib/libfdt/libfdt.h  |  31 +++
 lib/libfdt/pylibfdt/libfdt.i |  58 ++
 scripts/Makefile.lib |  17 ++
 scripts/Makefile.spl |   5 +-
 test/overlay/Makefile|   1 +
 test/overlay/cmd_ut_overlay.c|  50 -
 test/overlay/test-fdt-overlay-stacked.dts|  21 +++
 tools/dtoc/dtb_platdata.py   | 204 +++-
 tools/dtoc/dtoc_test_addr32.dts  |  27 +++
 tools/dtoc/dtoc_test_addr32_64.dts   |  33 
 tools/dtoc/dtoc_test_addr64.dts  |  33 
 tools/dtoc/dtoc_test_addr64_32.dts   |  33 
 tools/dtoc/dtoc_test_phandle.dts |  16 +-
 tools/dtoc/dtoc_test_simple.dts  |  14 ++
 tools/dtoc/fdt.py|  19 +-
 tools/dtoc/fdt_util.py   |  16 ++
 tools/dtoc/test_dtoc.py  | 270 ++-
 42 files changed, 1656 insertions(+), 132 deletions(-)
 create mode 100644 doc/README.fdt-overlays
 create mode 100644 doc/uImage.FIT/overlay-fdt-boot.txt
 create mode 100644 test/overlay/test-fdt-overlay-stacked.dts
 create mode 100644 tools/dtoc/dtoc_test_addr32.dts
 create mode 100644 tools/dtoc/dtoc_test_addr32_64.dts
 create mode 100644 

Re: [U-Boot] Please pull u-boot-fdt (take 4)

2017-06-04 Thread Tom Rini
On Fri, Jun 02, 2017 at 05:59:04PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> This includes moving the error message to stderr so that buildman picks it up.
> 
> 
> 
> The following changes since commit 46bac66b20da6d50e757bdca74703153f233090b:
> 
>   sandbox: Move to use live tree (2017-06-01 07:03:17 -0600)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 99ed4a2e979150879fb70aea71898709536375d3:
> 
>   fdt: Drop fdt_select.py (2017-06-02 10:18:20 -0600)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] Please pull u-boot-fdt (take 4)

2017-06-02 Thread Simon Glass
Hi Tom,

This includes moving the error message to stderr so that buildman picks it up.



The following changes since commit 46bac66b20da6d50e757bdca74703153f233090b:

  sandbox: Move to use live tree (2017-06-01 07:03:17 -0600)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 99ed4a2e979150879fb70aea71898709536375d3:

  fdt: Drop fdt_select.py (2017-06-02 10:18:20 -0600)


Simon Glass (20):
  fdt: Add Python bindings
  pci: Correct cast for sandbox
  fdt: Correct cast for sandbox in fdtdec_setup_memory_size()
  fdt: Use SPDX format for licenses in the libfdt headers
  fdt: Move header files into lib/libfdt
  fdt: Allow swig options to be provided by Makefile
  fdt: Add all source files to the libfdt build
  fdt: Rename existing python libfdt module
  fdt: Build the new python libfdt module
  fdt: Update fdt_test to use 'dt' instead of 'fdt'
  fdt: dtoc: Add a full set of property tests
  fdt: Support use of the new python libfdt library
  fdt: Makefile: Build python libfdt library if needed
  fdt: Stop building the old python libfdt module
  fdt: Drop use of the legacy libfdt python module
  fdt: Drop fdt_fallback library
  binman: Drop a special case related to fdt_fallback
  fdt: Merge fdt_normal with its base class
  binman: Rename fdt variable to dtb
  fdt: Drop fdt_select.py

 Makefile|   17 +-
 cmd/pci.c   |3 +-
 include/fdt.h   |  112 +-
 include/libfdt.h| 2138 +-
 lib/fdtdec.c|3 +-
 lib/libfdt/fdt.h|   67 +
 lib/libfdt/libfdt.h | 2144 +++
 lib/libfdt/libfdt.swig  |  113 --
 lib/libfdt/pylibfdt/libfdt.i|  389 +
 lib/libfdt/pylibfdt/setup.py|  123 ++
 lib/libfdt/setup.py |   38 -
 scripts/Makefile.spl|   17 +-
 tools/Makefile  |   54 +-
 tools/binman/binman.py  |3 +
 tools/binman/control.py |   12 +-
 tools/binman/etype/u_boot_dtb_with_ucode.py |   24 +-
 tools/binman/fdt_test.py|   64 +-
 tools/binman/func_test.py   |   48 +-
 tools/binman/test/45_prop_test.dts  |   23 +
 tools/dtoc/dtoc.py  |3 +-
 tools/dtoc/fdt.py   |  183 ++-
 tools/dtoc/fdt_fallback.py  |  181 ---
 tools/dtoc/fdt_normal.py|  225 ---
 tools/dtoc/fdt_select.py|   36 -
 24 files changed, 3061 insertions(+), 2959 deletions(-)
 create mode 100644 lib/libfdt/fdt.h
 create mode 100644 lib/libfdt/libfdt.h
 delete mode 100644 lib/libfdt/libfdt.swig
 create mode 100644 lib/libfdt/pylibfdt/libfdt.i
 create mode 100755 lib/libfdt/pylibfdt/setup.py
 delete mode 100644 lib/libfdt/setup.py
 create mode 100644 tools/binman/test/45_prop_test.dts
 delete mode 100644 tools/dtoc/fdt_fallback.py
 delete mode 100644 tools/dtoc/fdt_normal.py
 delete mode 100644 tools/dtoc/fdt_select.py

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] Please pull u-boot-fdt (take 3)

2017-05-30 Thread Simon Glass
Hi Tom,

Here it is again!


The following changes since commit 380e86f361e4e2aef83295972863654fde157560:

  Merge git://git.denx.de/u-boot-fsl-qoriq (2017-05-26 11:19:27 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 2650c70aff2bbc39b2c4f02d84f1cf0e38993e52:

  fdt: Drop fdt_select.py (2017-05-29 15:18:36 -0600)


Simon Glass (20):
  fdt: Add Python bindings
  pci: Correct cast for sandbox
  fdt: Correct cast for sandbox in fdtdec_setup_memory_size()
  fdt: Use SPDX format for licenses in the libfdt headers
  fdt: Move header files into lib/libfdt
  fdt: Allow swig options to be provided by Makefile
  fdt: Add all source files to the libfdt build
  fdt: Rename existing python libfdt module
  fdt: Build the new python libfdt module
  fdt: Update fdt_test to use 'dt' instead of 'fdt'
  fdt: dtoc: Add a full set of property tests
  fdt: Support use of the new python libfdt library
  fdt: Makefile: Build python libfdt library if needed
  fdt: Stop building the old python libfdt module
  fdt: Drop use of the legacy libfdt python module
  fdt: Drop fdt_fallback library
  binman: Drop a special case related to fdt_fallback
  fdt: Merge fdt_normal with its base class
  binman: Rename fdt variable to dtb
  fdt: Drop fdt_select.py

 Makefile|   16 +-
 cmd/pci.c   |3 +-
 include/fdt.h   |  112 +-
 include/libfdt.h| 2138 +-
 lib/fdtdec.c|3 +-
 lib/libfdt/fdt.h|   67 +
 lib/libfdt/libfdt.h | 2144 +++
 lib/libfdt/libfdt.swig  |  113 --
 lib/libfdt/pylibfdt/libfdt.i|  389 +
 lib/libfdt/pylibfdt/setup.py|  123 ++
 lib/libfdt/setup.py |   38 -
 scripts/Makefile.spl|   17 +-
 tools/Makefile  |   54 +-
 tools/binman/binman.py  |3 +
 tools/binman/control.py |   12 +-
 tools/binman/etype/u_boot_dtb_with_ucode.py |   24 +-
 tools/binman/fdt_test.py|   64 +-
 tools/binman/func_test.py   |   48 +-
 tools/binman/test/45_prop_test.dts  |   23 +
 tools/dtoc/dtoc.py  |3 +-
 tools/dtoc/fdt.py   |  183 ++-
 tools/dtoc/fdt_fallback.py  |  181 ---
 tools/dtoc/fdt_normal.py|  225 ---
 tools/dtoc/fdt_select.py|   36 -
 24 files changed, 3060 insertions(+), 2959 deletions(-)
 create mode 100644 lib/libfdt/fdt.h
 create mode 100644 lib/libfdt/libfdt.h
 delete mode 100644 lib/libfdt/libfdt.swig
 create mode 100644 lib/libfdt/pylibfdt/libfdt.i
 create mode 100755 lib/libfdt/pylibfdt/setup.py
 delete mode 100644 lib/libfdt/setup.py
 create mode 100644 tools/binman/test/45_prop_test.dts
 delete mode 100644 tools/dtoc/fdt_fallback.py
 delete mode 100644 tools/dtoc/fdt_normal.py
 delete mode 100644 tools/dtoc/fdt_select.py

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-fdt, take 2

2017-05-27 Thread Simon Glass
Hi Tom,

On 26 May 2017 at 06:49, Tom Rini  wrote:
>
> On Thu, May 25, 2017 at 09:15:33PM -0600, Simon Glass wrote:
> > Hi Tom,
> >
> > On 25 May 2017 at 11:42, Tom Rini  wrote:
> > > On Thu, May 25, 2017 at 11:27:10AM -0600, Simon Glass wrote:
> > >> Hi Tom,
> > >>
> > >> On 25 May 2017 at 05:19, Tom Rini  wrote:
> > >> >
> > >> > On Wed, May 24, 2017 at 06:15:25PM -0600, Simon Glass wrote:
> > >> >
> > >> > > Hi Tom,
> > >> > >
> > >> > > This incorporates the v2 patch for 'fdt: Build the new python libfdt
> > >> > > module' which should fix the problem with the original pull request.
> > >> > >
> > >> > >
> > >> > > The following changes since commit 
> > >> > > be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:
> > >> > >
> > >> > >   Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
> > >> > > 16:22:03 -0400)
> > >> > >
> > >> > > are available in the git repository at:
> > >> > >
> > >> > >   git://git.denx.de/u-boot-fdt.git
> > >> > >
> > >> > > for you to fetch changes up to 
> > >> > > da9c601049eb7c993c7f6e33ae10af7a847a483d:
> > >> > >
> > >> > >   fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)
> > >> >
> > >> > NAK.  travis-ci blows up quite badly:
> > >> > https://travis-ci.org/trini/u-boot/builds/235861889
> > >>
> > >> I'm not sure how to repeat this problem. When I try this:
> > >
> > > Your best bet is likely:
> > > https://docs.travis-ci.com/user/common-build-problems/#Troubleshooting-Locally-in-a-Docker-Image
> >
> > Sadly still no luck. I installed this one:
> >
> > travisci/ci-garnet:packer-1478744932
> >
> > It includes python-dev but not swig, so should not be able to build
> > the module. As expected I get this error:
> >
> > NO_SDL=1 ./tools/buildman/buildman -P sandbox_spl
> > boards.cfg is up to date. Nothing to do.
> > Building current source for 1 boards (1 thread, 8 jobs per thread)
> >sandbox:  +   sandbox_spl
> > +Traceback (most recent call last):
> > +  File "", line 1, in 
> > +ImportError: No module named libfdt
> > +make[2]: *** [checkdtoc] Error 1
> > +make[1]: *** [spl/u-boot-spl] Error 2
> > +make: *** [sub-make] Error 2
> >
> >
> > If I install swig then all is well. I don't see the same error.
> >
> > I'm also unsure how you get it to pass with some boards but not others...
> >
> > There is obviously something odd going on. I also cannot understand
> > why in this one:
> >
> > https://travis-ci.org/trini/u-boot/jobs/235861899#L769
> >
> > I see it trying to compile libfdt_wrap.c. That should be handled by
> > setup.py - I just cannot figure out why it would try to compile it
> > itself:
> >
> >   arm:  +  mx6sabresd_spl
> > +mv: cannot stat lib/libfdt/pylibfdt/libfdt.py: No such file or directory
> > +make[2]: *** [tools/_libfdt.so] Error 1
> > +make[1]: *** [tools] Error 2
> > +make: *** [sub-make] Error 2
> >   arm:  +  ls1021aqds_nor_lpuart
> > +x86_64-linux-gnu-gcc: error: lib/libfdt/pylibfdt/libfdt_wrap.c: No
> > such file or directory
> > +x86_64-linux-gnu-gcc: fatal error: no input files
> > +compilation terminated.
> > +error: command 'x86_64-linux-gnu-gcc' failed with exit status 4
> > +make[2]: *** [tools/_libfdt.so] Error 1
> > +make[1]: *** [tools] Error 2
> > +make: *** [sub-make] Error 2
> >   arm:  +  mx6slevk
> > +x86_64-linux-gnu-gcc: error: lib/libfdt/pylibfdt/libfdt_wrap.c: No
> > such file or directory
> > +x86_64-linux-gnu-gcc: fatal error: no input files
> > +compilation terminated.
> > +error: command 'x86_64-linux-gnu-gcc' failed with exit status 4
> > +make[2]: *** [tools/_libfdt.so] Error 1
> > +make[1]: *** [tools] Error 2
> > +make: *** [sub-make] Error 2
> >   5503 /58 mx6qsabreauto
> > boards.cfg is up to date. Nothing to do.
> > Summary of current source for 58 boards (2 threads, 1 job per thread)
> >   arm:  +  mx6sabresd_spl ls1021aqds_nor_lpuart mx6slevk
> > +mv: cannot stat lib/libfdt/pylibfdt/libfdt.py: No such file or directory
> >
> >
> > Are you able to run with V=1 to get the full make output? That might
> > help me narrow it down. Or, if you have a particular docket image,
> > please point me to it. There seems to be some init going on though
> > (e.g. /tmp/dtc).
>
> Anyone can use travis-ci :)  In the past I've had some luck debugging
> these issues (since I didn't want to mess with docker) by hacking up the
> .travis.yml file to just have a single matrix entry for whatever I was
> debugging and hack things as needed, including 'cat'ing files.

OK I found that with the prerequisites installed I can repeat this
with something like:

buildman -P sandbox

(which only builds 3 boards)

The problem is that setup.py is writing to the source directory. It
seems to be hard to stop it doing that. I was aware of it but did not
imagine the race condition it would create. I've updated the patches
and sent v3. I have a passing run on travis-ci [1]. If all looks good
I'll pull it in and try a take three early next week.

Most of my 

Re: [U-Boot] Please pull u-boot-fdt, take 2

2017-05-26 Thread Tom Rini
On Thu, May 25, 2017 at 09:15:33PM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On 25 May 2017 at 11:42, Tom Rini  wrote:
> > On Thu, May 25, 2017 at 11:27:10AM -0600, Simon Glass wrote:
> >> Hi Tom,
> >>
> >> On 25 May 2017 at 05:19, Tom Rini  wrote:
> >> >
> >> > On Wed, May 24, 2017 at 06:15:25PM -0600, Simon Glass wrote:
> >> >
> >> > > Hi Tom,
> >> > >
> >> > > This incorporates the v2 patch for 'fdt: Build the new python libfdt
> >> > > module' which should fix the problem with the original pull request.
> >> > >
> >> > >
> >> > > The following changes since commit 
> >> > > be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:
> >> > >
> >> > >   Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
> >> > > 16:22:03 -0400)
> >> > >
> >> > > are available in the git repository at:
> >> > >
> >> > >   git://git.denx.de/u-boot-fdt.git
> >> > >
> >> > > for you to fetch changes up to 
> >> > > da9c601049eb7c993c7f6e33ae10af7a847a483d:
> >> > >
> >> > >   fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)
> >> >
> >> > NAK.  travis-ci blows up quite badly:
> >> > https://travis-ci.org/trini/u-boot/builds/235861889
> >>
> >> I'm not sure how to repeat this problem. When I try this:
> >
> > Your best bet is likely:
> > https://docs.travis-ci.com/user/common-build-problems/#Troubleshooting-Locally-in-a-Docker-Image
> 
> Sadly still no luck. I installed this one:
> 
> travisci/ci-garnet:packer-1478744932
> 
> It includes python-dev but not swig, so should not be able to build
> the module. As expected I get this error:
> 
> NO_SDL=1 ./tools/buildman/buildman -P sandbox_spl
> boards.cfg is up to date. Nothing to do.
> Building current source for 1 boards (1 thread, 8 jobs per thread)
>sandbox:  +   sandbox_spl
> +Traceback (most recent call last):
> +  File "", line 1, in 
> +ImportError: No module named libfdt
> +make[2]: *** [checkdtoc] Error 1
> +make[1]: *** [spl/u-boot-spl] Error 2
> +make: *** [sub-make] Error 2
> 
> 
> If I install swig then all is well. I don't see the same error.
> 
> I'm also unsure how you get it to pass with some boards but not others...
> 
> There is obviously something odd going on. I also cannot understand
> why in this one:
> 
> https://travis-ci.org/trini/u-boot/jobs/235861899#L769
> 
> I see it trying to compile libfdt_wrap.c. That should be handled by
> setup.py - I just cannot figure out why it would try to compile it
> itself:
> 
>   arm:  +  mx6sabresd_spl
> +mv: cannot stat lib/libfdt/pylibfdt/libfdt.py: No such file or directory
> +make[2]: *** [tools/_libfdt.so] Error 1
> +make[1]: *** [tools] Error 2
> +make: *** [sub-make] Error 2
>   arm:  +  ls1021aqds_nor_lpuart
> +x86_64-linux-gnu-gcc: error: lib/libfdt/pylibfdt/libfdt_wrap.c: No
> such file or directory
> +x86_64-linux-gnu-gcc: fatal error: no input files
> +compilation terminated.
> +error: command 'x86_64-linux-gnu-gcc' failed with exit status 4
> +make[2]: *** [tools/_libfdt.so] Error 1
> +make[1]: *** [tools] Error 2
> +make: *** [sub-make] Error 2
>   arm:  +  mx6slevk
> +x86_64-linux-gnu-gcc: error: lib/libfdt/pylibfdt/libfdt_wrap.c: No
> such file or directory
> +x86_64-linux-gnu-gcc: fatal error: no input files
> +compilation terminated.
> +error: command 'x86_64-linux-gnu-gcc' failed with exit status 4
> +make[2]: *** [tools/_libfdt.so] Error 1
> +make[1]: *** [tools] Error 2
> +make: *** [sub-make] Error 2
>   5503 /58 mx6qsabreauto
> boards.cfg is up to date. Nothing to do.
> Summary of current source for 58 boards (2 threads, 1 job per thread)
>   arm:  +  mx6sabresd_spl ls1021aqds_nor_lpuart mx6slevk
> +mv: cannot stat lib/libfdt/pylibfdt/libfdt.py: No such file or directory
> 
> 
> Are you able to run with V=1 to get the full make output? That might
> help me narrow it down. Or, if you have a particular docket image,
> please point me to it. There seems to be some init going on though
> (e.g. /tmp/dtc).

Anyone can use travis-ci :)  In the past I've had some luck debugging
these issues (since I didn't want to mess with docker) by hacking up the
.travis.yml file to just have a single matrix entry for whatever I was
debugging and hack things as needed, including 'cat'ing files.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-fdt, take 2

2017-05-25 Thread Simon Glass
Hi Tom,

On 25 May 2017 at 11:42, Tom Rini  wrote:
> On Thu, May 25, 2017 at 11:27:10AM -0600, Simon Glass wrote:
>> Hi Tom,
>>
>> On 25 May 2017 at 05:19, Tom Rini  wrote:
>> >
>> > On Wed, May 24, 2017 at 06:15:25PM -0600, Simon Glass wrote:
>> >
>> > > Hi Tom,
>> > >
>> > > This incorporates the v2 patch for 'fdt: Build the new python libfdt
>> > > module' which should fix the problem with the original pull request.
>> > >
>> > >
>> > > The following changes since commit 
>> > > be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:
>> > >
>> > >   Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
>> > > 16:22:03 -0400)
>> > >
>> > > are available in the git repository at:
>> > >
>> > >   git://git.denx.de/u-boot-fdt.git
>> > >
>> > > for you to fetch changes up to da9c601049eb7c993c7f6e33ae10af7a847a483d:
>> > >
>> > >   fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)
>> >
>> > NAK.  travis-ci blows up quite badly:
>> > https://travis-ci.org/trini/u-boot/builds/235861889
>>
>> I'm not sure how to repeat this problem. When I try this:
>
> Your best bet is likely:
> https://docs.travis-ci.com/user/common-build-problems/#Troubleshooting-Locally-in-a-Docker-Image

Sadly still no luck. I installed this one:

travisci/ci-garnet:packer-1478744932

It includes python-dev but not swig, so should not be able to build
the module. As expected I get this error:

NO_SDL=1 ./tools/buildman/buildman -P sandbox_spl
boards.cfg is up to date. Nothing to do.
Building current source for 1 boards (1 thread, 8 jobs per thread)
   sandbox:  +   sandbox_spl
+Traceback (most recent call last):
+  File "", line 1, in 
+ImportError: No module named libfdt
+make[2]: *** [checkdtoc] Error 1
+make[1]: *** [spl/u-boot-spl] Error 2
+make: *** [sub-make] Error 2


If I install swig then all is well. I don't see the same error.

I'm also unsure how you get it to pass with some boards but not others...

There is obviously something odd going on. I also cannot understand
why in this one:

https://travis-ci.org/trini/u-boot/jobs/235861899#L769

I see it trying to compile libfdt_wrap.c. That should be handled by
setup.py - I just cannot figure out why it would try to compile it
itself:

  arm:  +  mx6sabresd_spl
+mv: cannot stat lib/libfdt/pylibfdt/libfdt.py: No such file or directory
+make[2]: *** [tools/_libfdt.so] Error 1
+make[1]: *** [tools] Error 2
+make: *** [sub-make] Error 2
  arm:  +  ls1021aqds_nor_lpuart
+x86_64-linux-gnu-gcc: error: lib/libfdt/pylibfdt/libfdt_wrap.c: No
such file or directory
+x86_64-linux-gnu-gcc: fatal error: no input files
+compilation terminated.
+error: command 'x86_64-linux-gnu-gcc' failed with exit status 4
+make[2]: *** [tools/_libfdt.so] Error 1
+make[1]: *** [tools] Error 2
+make: *** [sub-make] Error 2
  arm:  +  mx6slevk
+x86_64-linux-gnu-gcc: error: lib/libfdt/pylibfdt/libfdt_wrap.c: No
such file or directory
+x86_64-linux-gnu-gcc: fatal error: no input files
+compilation terminated.
+error: command 'x86_64-linux-gnu-gcc' failed with exit status 4
+make[2]: *** [tools/_libfdt.so] Error 1
+make[1]: *** [tools] Error 2
+make: *** [sub-make] Error 2
  5503 /58 mx6qsabreauto
boards.cfg is up to date. Nothing to do.
Summary of current source for 58 boards (2 threads, 1 job per thread)
  arm:  +  mx6sabresd_spl ls1021aqds_nor_lpuart mx6slevk
+mv: cannot stat lib/libfdt/pylibfdt/libfdt.py: No such file or directory


Are you able to run with V=1 to get the full make output? That might
help me narrow it down. Or, if you have a particular docket image,
please point me to it. There seems to be some init going on though
(e.g. /tmp/dtc).

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-fdt, take 2

2017-05-25 Thread Tom Rini
On Thu, May 25, 2017 at 11:27:10AM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On 25 May 2017 at 05:19, Tom Rini  wrote:
> >
> > On Wed, May 24, 2017 at 06:15:25PM -0600, Simon Glass wrote:
> >
> > > Hi Tom,
> > >
> > > This incorporates the v2 patch for 'fdt: Build the new python libfdt
> > > module' which should fix the problem with the original pull request.
> > >
> > >
> > > The following changes since commit 
> > > be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:
> > >
> > >   Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
> > > 16:22:03 -0400)
> > >
> > > are available in the git repository at:
> > >
> > >   git://git.denx.de/u-boot-fdt.git
> > >
> > > for you to fetch changes up to da9c601049eb7c993c7f6e33ae10af7a847a483d:
> > >
> > >   fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)
> >
> > NAK.  travis-ci blows up quite badly:
> > https://travis-ci.org/trini/u-boot/builds/235861889
> 
> I'm not sure how to repeat this problem. When I try this:

Your best bet is likely:
https://docs.travis-ci.com/user/common-build-problems/#Troubleshooting-Locally-in-a-Docker-Image

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-fdt, take 2

2017-05-25 Thread Simon Glass
Hi Tom,

On 25 May 2017 at 05:19, Tom Rini  wrote:
>
> On Wed, May 24, 2017 at 06:15:25PM -0600, Simon Glass wrote:
>
> > Hi Tom,
> >
> > This incorporates the v2 patch for 'fdt: Build the new python libfdt
> > module' which should fix the problem with the original pull request.
> >
> >
> > The following changes since commit be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:
> >
> >   Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
> > 16:22:03 -0400)
> >
> > are available in the git repository at:
> >
> >   git://git.denx.de/u-boot-fdt.git
> >
> > for you to fetch changes up to da9c601049eb7c993c7f6e33ae10af7a847a483d:
> >
> >   fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)
>
> NAK.  travis-ci blows up quite badly:
> https://travis-ci.org/trini/u-boot/builds/235861889

I'm not sure how to repeat this problem. When I try this:

buildman -P boston32r2el

I don't see any errors.

This looks like what the test is running. Any ideas what I am missing?

I wonder if the environment is missing something, or has something
extra, with respect to python-dev, etc.?

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-fdt, take 2

2017-05-25 Thread Tom Rini
On Wed, May 24, 2017 at 06:15:25PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> This incorporates the v2 patch for 'fdt: Build the new python libfdt
> module' which should fix the problem with the original pull request.
> 
> 
> The following changes since commit be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:
> 
>   Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
> 16:22:03 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to da9c601049eb7c993c7f6e33ae10af7a847a483d:
> 
>   fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)

NAK.  travis-ci blows up quite badly:
https://travis-ci.org/trini/u-boot/builds/235861889

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] Please pull u-boot-fdt, take 2

2017-05-24 Thread Simon Glass
Hi Tom,

This incorporates the v2 patch for 'fdt: Build the new python libfdt
module' which should fix the problem with the original pull request.


The following changes since commit be62fbf376261ab3a4ed5db3bf54d5df9e216d9f:

  Merge branch 'rmobile' of git://git.denx.de/u-boot-sh (2017-05-23
16:22:03 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to da9c601049eb7c993c7f6e33ae10af7a847a483d:

  fdt: Drop fdt_select.py (2017-05-24 18:12:31 -0600)


Simon Glass (20):
  fdt: Add Python bindings
  pci: Correct cast for sandbox
  fdt: Correct cast for sandbox in fdtdec_setup_memory_size()
  fdt: Use SPDX format for licenses in the libfdt headers
  fdt: Move header files into lib/libfdt
  fdt: Allow swig options to be provided by Makefile
  fdt: Add all source files to the libfdt build
  fdt: Rename existing python libfdt module
  fdt: Build the new python libfdt module
  fdt: Update fdt_test to use 'dt' instead of 'fdt'
  fdt: dtoc: Add a full set of property tests
  fdt: Support use of the new python libfdt library
  fdt: Makefile: Build python libfdt library if needed
  fdt: Stop building the old python libfdt module
  fdt: Drop use of the legacy libfdt python module
  fdt: Drop fdt_fallback library
  binman: Drop a special case related to fdt_fallback
  fdt: Merge fdt_normal with its base class
  binman: Rename fdt variable to dtb
  fdt: Drop fdt_select.py

 Makefile|   16 +-
 cmd/pci.c   |3 +-
 include/fdt.h   |  112 +-
 include/libfdt.h| 2138 +-
 lib/fdtdec.c|3 +-
 lib/libfdt/fdt.h|   67 +
 lib/libfdt/libfdt.h | 2144 +++
 lib/libfdt/libfdt.swig  |  113 --
 lib/libfdt/pylibfdt/libfdt.i|  389 +
 lib/libfdt/pylibfdt/setup.py|  123 ++
 lib/libfdt/setup.py |   38 -
 scripts/Makefile.spl|   17 +-
 tools/Makefile  |   48 +-
 tools/binman/binman.py  |3 +
 tools/binman/control.py |   12 +-
 tools/binman/etype/u_boot_dtb_with_ucode.py |   24 +-
 tools/binman/fdt_test.py|   64 +-
 tools/binman/func_test.py   |   48 +-
 tools/binman/test/45_prop_test.dts  |   23 +
 tools/dtoc/dtoc.py  |3 +-
 tools/dtoc/fdt.py   |  183 ++-
 tools/dtoc/fdt_fallback.py  |  181 ---
 tools/dtoc/fdt_normal.py|  225 ---
 tools/dtoc/fdt_select.py|   36 -
 24 files changed, 3054 insertions(+), 2959 deletions(-)
 create mode 100644 lib/libfdt/fdt.h
 create mode 100644 lib/libfdt/libfdt.h
 delete mode 100644 lib/libfdt/libfdt.swig
 create mode 100644 lib/libfdt/pylibfdt/libfdt.i
 create mode 100755 lib/libfdt/pylibfdt/setup.py
 delete mode 100644 lib/libfdt/setup.py
 create mode 100644 tools/binman/test/45_prop_test.dts
 delete mode 100644 tools/dtoc/fdt_fallback.py
 delete mode 100644 tools/dtoc/fdt_normal.py
 delete mode 100644 tools/dtoc/fdt_select.py

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-05-18 Thread Simon Glass
Hi Tom,

On 9 May 2017 at 17:59, Tom Rini  wrote:
>
> On Tue, May 09, 2017 at 04:53:36PM -0600, Simon Glass wrote:
>
> > Hi Tom,
> >
> > Here's the conversion to use the upstream pylibfdt implementation.
> >
> >
> > The following changes since commit 85ea850976daea57c8045f3569566fad5ce9fe0f:
> >
> >   p1_p2_rdb: Fix unused variable warning (2017-05-09 09:11:04 -0400)
> >
> > are available in the git repository at:
> >
> >   git://git.denx.de/u-boot-fdt.git
> >
> > for you to fetch changes up to 8fbd55f0f8730e3a6994798f4e533138d15be6aa:
> >
> >   fdt: Drop fdt_select.py (2017-05-09 14:56:19 -0600)
> >
>
> NAK.  I get:
> $ ./test/py/test.py --bd sandbox --build
> make O=/home/trini/work/u-boot/u-boot/build-sandbox -s sandbox_defconfig
> make O=/home/trini/work/u-boot/u-boot/build-sandbox -s -j8
> Assembler messages:
> Fatal error: can't create
> build/temp.linux-x86_64-2.7/../lib/libfdt/fdt.o: No such file or
> directory
> error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
> make[2]: *** [tools/_libfdt.so] Error 1
> make[2]: *** Waiting for unfinished jobs
> make[1]: *** [tools] Error 2
> make: *** [sub-make] Error 2
> Exit code: 2

This is due to some weirdness with setup.py not allowing source files
to come from a patch starting with "..". I'll send an updated patch,
pull it in and then resend the pull request.

>
> --
> Tom

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-05-10 Thread Simon Glass
Hi Tom,

On 9 May 2017 at 17:59, Tom Rini  wrote:
> On Tue, May 09, 2017 at 04:53:36PM -0600, Simon Glass wrote:
>
>> Hi Tom,
>>
>> Here's the conversion to use the upstream pylibfdt implementation.
>>
>>
>> The following changes since commit 85ea850976daea57c8045f3569566fad5ce9fe0f:
>>
>>   p1_p2_rdb: Fix unused variable warning (2017-05-09 09:11:04 -0400)
>>
>> are available in the git repository at:
>>
>>   git://git.denx.de/u-boot-fdt.git
>>
>> for you to fetch changes up to 8fbd55f0f8730e3a6994798f4e533138d15be6aa:
>>
>>   fdt: Drop fdt_select.py (2017-05-09 14:56:19 -0600)
>>
>
> NAK.  I get:
> $ ./test/py/test.py --bd sandbox --build
> make O=/home/trini/work/u-boot/u-boot/build-sandbox -s sandbox_defconfig
> make O=/home/trini/work/u-boot/u-boot/build-sandbox -s -j8
> Assembler messages:
> Fatal error: can't create
> build/temp.linux-x86_64-2.7/../lib/libfdt/fdt.o: No such file or
> directory
> error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
> make[2]: *** [tools/_libfdt.so] Error 1
> make[2]: *** Waiting for unfinished jobs
> make[1]: *** [tools] Error 2
> make: *** [sub-make] Error 2
> Exit code: 2

I did see that once on a buildman run some weeks ago and could not
repeat it after various tries. I'll try harder.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-05-09 Thread Tom Rini
On Tue, May 09, 2017 at 04:53:36PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> Here's the conversion to use the upstream pylibfdt implementation.
> 
> 
> The following changes since commit 85ea850976daea57c8045f3569566fad5ce9fe0f:
> 
>   p1_p2_rdb: Fix unused variable warning (2017-05-09 09:11:04 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 8fbd55f0f8730e3a6994798f4e533138d15be6aa:
> 
>   fdt: Drop fdt_select.py (2017-05-09 14:56:19 -0600)
> 

NAK.  I get:
$ ./test/py/test.py --bd sandbox --build
make O=/home/trini/work/u-boot/u-boot/build-sandbox -s sandbox_defconfig
make O=/home/trini/work/u-boot/u-boot/build-sandbox -s -j8
Assembler messages:
Fatal error: can't create
build/temp.linux-x86_64-2.7/../lib/libfdt/fdt.o: No such file or
directory
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
make[2]: *** [tools/_libfdt.so] Error 1
make[2]: *** Waiting for unfinished jobs
make[1]: *** [tools] Error 2
make: *** [sub-make] Error 2
Exit code: 2

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-05-09 Thread Simon Glass
Hi Tom,

Here's the conversion to use the upstream pylibfdt implementation.


The following changes since commit 85ea850976daea57c8045f3569566fad5ce9fe0f:

  p1_p2_rdb: Fix unused variable warning (2017-05-09 09:11:04 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 8fbd55f0f8730e3a6994798f4e533138d15be6aa:

  fdt: Drop fdt_select.py (2017-05-09 14:56:19 -0600)


Simon Glass (20):
  fdt: Add Python bindings
  pci: Correct cast for sandbox
  fdt: Correct cast for sandbox in fdtdec_setup_memory_size()
  fdt: Use SPDX format for licenses in the libfdt headers
  fdt: Move header files into lib/libfdt
  fdt: Allow swig options to be provided by Makefile
  fdt: Add all source files to the libfdt build
  fdt: Rename existing python libfdt module
  fdt: Build the new python libfdt module
  fdt: Update fdt_test to use 'dt' instead of 'fdt'
  fdt: dtoc: Add a full set of property tests
  fdt: Support use of the new python libfdt library
  fdt: Makefile: Build python libfdt library if needed
  fdt: Stop building the old python libfdt module
  fdt: Drop use of the legacy libfdt python module
  fdt: Drop fdt_fallback library
  binman: Drop a special case related to fdt_fallback
  fdt: Merge fdt_normal with its base class
  binman: Rename fdt variable to dtb
  fdt: Drop fdt_select.py

 Makefile|   16 +-
 cmd/pci.c   |3 +-
 include/fdt.h   |  112 +---
 include/libfdt.h| 2138
+-
 lib/fdtdec.c|3 +-
 lib/libfdt/fdt.h|   67 +++
 lib/libfdt/libfdt.h | 2144
+++
 lib/libfdt/libfdt.swig  |  113 
 lib/libfdt/pylibfdt/libfdt.i|  389 +
 lib/libfdt/pylibfdt/setup.py|  123 +
 lib/libfdt/setup.py |   38 --
 scripts/Makefile.spl|   17 +-
 tools/Makefile  |   43 +-
 tools/binman/binman.py  |3 +
 tools/binman/control.py |   12 +-
 tools/binman/etype/u_boot_dtb_with_ucode.py |   24 +-
 tools/binman/fdt_test.py|   64 ++-
 tools/binman/func_test.py   |   48 +-
 tools/binman/test/45_prop_test.dts  |   23 +
 tools/dtoc/dtoc.py  |3 +-
 tools/dtoc/fdt.py   |  183 +-
 tools/dtoc/fdt_fallback.py  |  181 --
 tools/dtoc/fdt_normal.py|  225 
 tools/dtoc/fdt_select.py|   36 --
 24 files changed, 3049 insertions(+), 2959 deletions(-)
 create mode 100644 lib/libfdt/fdt.h
 create mode 100644 lib/libfdt/libfdt.h
 delete mode 100644 lib/libfdt/libfdt.swig
 create mode 100644 lib/libfdt/pylibfdt/libfdt.i
 create mode 100755 lib/libfdt/pylibfdt/setup.py
 delete mode 100644 lib/libfdt/setup.py
 create mode 100644 tools/binman/test/45_prop_test.dts
 delete mode 100644 tools/dtoc/fdt_fallback.py
 delete mode 100644 tools/dtoc/fdt_normal.py
 delete mode 100644 tools/dtoc/fdt_select.py

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-04-11 Thread Tom Rini
On Mon, Apr 10, 2017 at 02:52:44PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> Just a few minor fixes.
> 
> 
> The following changes since commit 01cce5fdd098add2b8aa570468cb35fca5d778fe:
> 
>   Merge git://git.denx.de/u-boot-x86 (2017-04-10 08:07:29 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 9963890b8be1d208035945abc5ba9f77637b6542:
> 
>   libfdt: fix build with Python 3 (2017-04-10 13:25:19 -0600)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-04-10 Thread Simon Glass
Hi Tom,

Just a few minor fixes.


The following changes since commit 01cce5fdd098add2b8aa570468cb35fca5d778fe:

  Merge git://git.denx.de/u-boot-x86 (2017-04-10 08:07:29 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 9963890b8be1d208035945abc5ba9f77637b6542:

  libfdt: fix build with Python 3 (2017-04-10 13:25:19 -0600)


Simon Glass (1):
  fdt: Bring in changes from v1.4.4

Stefan Agner (1):
  libfdt: fix build with Python 3

 lib/libfdt/fdt_overlay.c  | 23 ---
 lib/libfdt/fdt_rw.c   |  3 ++-
 lib/libfdt/fdt_strerror.c |  3 +++
 lib/libfdt/setup.py   |  2 +-
 4 files changed, 18 insertions(+), 13 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


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

2017-01-14 Thread Tom Rini
On Sat, Jan 14, 2017 at 10:14:44AM -0700, Simon Glass wrote:

> Hi Tom,
> 
> Just a few fixes.
> 
> The following changes since commit 70c1e0474a9df2c4493b4e2330cc41d3132b4e90:
> 
>   Merge git://git.denx.de/u-boot-rockchip (2017-01-12 21:20:51 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to b05bf6c75d03c925737e228472b694cbeaa503c2:
> 
>   cmd/fdt: Make fdt get value endian-safe for single-cell properties
> (2017-01-14 10:09:46 -0700)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2017-01-14 Thread Simon Glass
Hi Tom,

Just a few fixes.

The following changes since commit 70c1e0474a9df2c4493b4e2330cc41d3132b4e90:

  Merge git://git.denx.de/u-boot-rockchip (2017-01-12 21:20:51 -0500)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to b05bf6c75d03c925737e228472b694cbeaa503c2:

  cmd/fdt: Make fdt get value endian-safe for single-cell properties
(2017-01-14 10:09:46 -0700)


Andreas Färber (1):
  cmd/fdt: Make fdt get value endian-safe for single-cell properties

David Gibson (1):
  libfdt: Correct fdt handling of overlays without fixups and base
trees without symbols

Stefan Agner (1):
  cmd: fdt: Print error message when fdt application fails

 cmd/fdt.c| 10 +++---
 lib/libfdt/fdt_overlay.c |  7 ++-
 2 files changed, 13 insertions(+), 4 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-11-26 Thread Tom Rini
On Fri, Nov 25, 2016 at 09:51:02AM -0700, Simon Glass wrote:

> Hi Tom.
> 
> The following changes since commit 543bd27353d2c5679057fe09aa2d02259687ff32:
> 
>   MAINTAINERS: SUNXI: Update maintainership (2016-11-22 09:07:26 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 643f8d4c07788f28ad283e3fcb29d88a4b16a4be:
> 
>   MAINTAINERS: Fix syntax and update filename for FDT (2016-11-25
> 09:48:09 -0700)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-11-25 Thread Simon Glass
+Tom

On 25 November 2016 at 09:51, Simon Glass  wrote:
> Hi Tom.
>
> The following changes since commit 543bd27353d2c5679057fe09aa2d02259687ff32:
>
>   MAINTAINERS: SUNXI: Update maintainership (2016-11-22 09:07:26 -0500)
>
> are available in the git repository at:
>
>   git://git.denx.de/u-boot-fdt.git
>
> for you to fetch changes up to 643f8d4c07788f28ad283e3fcb29d88a4b16a4be:
>
>   MAINTAINERS: Fix syntax and update filename for FDT (2016-11-25
> 09:48:09 -0700)
>
> 
> Andreas Färber (1):
>   MAINTAINERS: Fix syntax and update filename for FDT
>
>  MAINTAINERS | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Regards,
> Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-11-25 Thread Simon Glass
Hi Tom.

The following changes since commit 543bd27353d2c5679057fe09aa2d02259687ff32:

  MAINTAINERS: SUNXI: Update maintainership (2016-11-22 09:07:26 -0500)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 643f8d4c07788f28ad283e3fcb29d88a4b16a4be:

  MAINTAINERS: Fix syntax and update filename for FDT (2016-11-25
09:48:09 -0700)


Andreas Färber (1):
  MAINTAINERS: Fix syntax and update filename for FDT

 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-24 Thread Tom Rini
On Sun, Oct 23, 2016 at 09:01:17PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> Here's another patch syncing up with libfdt upstream. Maxime suggest
> that this should go in now.
> 
> 
> The following changes since commit 3431b392ad50ff37fa3d6e7715c6a99c74d692dc:
> 
>   Merge tag 'signed-efi-next' of git://github.com/agraf/u-boot
> (2016-10-19 07:48:16 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 610db7058f1d9ed668aef7692f8f90f05e0923f8:
> 
>   libfdt: Sync overlay with upstream (2016-10-23 12:08:48 -0700)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-23 Thread Simon Glass
Hi Tom,

Here's another patch syncing up with libfdt upstream. Maxime suggest
that this should go in now.


The following changes since commit 3431b392ad50ff37fa3d6e7715c6a99c74d692dc:

  Merge tag 'signed-efi-next' of git://github.com/agraf/u-boot
(2016-10-19 07:48:16 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 610db7058f1d9ed668aef7692f8f90f05e0923f8:

  libfdt: Sync overlay with upstream (2016-10-23 12:08:48 -0700)


Maxime Ripard (1):
  libfdt: Sync overlay with upstream

 include/libfdt.h  |  24 ++---
 lib/libfdt/fdt_overlay.c  | 199
+++-
 lib/libfdt/fdt_strerror.c |   3 ++
 3 files changed, 147 insertions(+), 79 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull u-boot-fdt (take 2)

2016-10-14 Thread Tom Rini
On Thu, Oct 13, 2016 at 04:53:40PM -0600, Simon Glass wrote:

> Hi Tom,
> 
> Here is a second attempt with this patch pulled in:
> 
> http://patchwork.ozlabs.org/patch/681814/
> 
> 
> The following changes since commit c69f6d04ec66433f2360490a5cd0263c83aab18f:
> 
>   Merge branch 'master' of http://git.denx.de/u-boot-mmc (2016-10-13
> 08:13:56 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 42b7600d62ae288a8c12431d232b89b26ec61721:
> 
>   libfdt: Drop inlining of fdt_path_offset() (2016-10-13 14:12:40 -0600)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Please pull u-boot-fdt (take 2)

2016-10-13 Thread Simon Glass
Hi Tom,

Here is a second attempt with this patch pulled in:

http://patchwork.ozlabs.org/patch/681814/


The following changes since commit c69f6d04ec66433f2360490a5cd0263c83aab18f:

  Merge branch 'master' of http://git.denx.de/u-boot-mmc (2016-10-13
08:13:56 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 42b7600d62ae288a8c12431d232b89b26ec61721:

  libfdt: Drop inlining of fdt_path_offset() (2016-10-13 14:12:40 -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/power/pmic/palmas.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 ++-
 38 files changed, 299 insertions(+), 222 deletions(-)


Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Keerthy



On Thursday 13 October 2016 08:38 PM, Simon Glass wrote:

Hi Keerthy,

On 13 October 2016 at 08:59, Keerthy  wrote:

Simon,

On Thursday 13 October 2016 07:54 PM, Simon Glass wrote:


Hi Tom,

On 12 October 2016 at 08:13, Tom Rini  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-keer...@ti.com. Let me check what is going 
with my mailer. Thanks for catching it.




Regards,
Simon


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Simon Glass
Hi Keerthy,

On 13 October 2016 at 08:59, Keerthy  wrote:
> Simon,
>
> On Thursday 13 October 2016 07:54 PM, Simon Glass wrote:
>>
>> Hi Tom,
>>
>> On 12 October 2016 at 08:13, Tom Rini  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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Keerthy

Simon,

On Thursday 13 October 2016 07:54 PM, Simon Glass wrote:

Hi Tom,

On 12 October 2016 at 08:13, Tom Rini  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


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Simon Glass
Hi Tom,

On 13 October 2016 at 08:33, Tom Rini  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  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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Tom Rini
On Thu, Oct 13, 2016 at 08:24:22AM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On 12 October 2016 at 08:13, Tom Rini  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.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Simon Glass
Hi Tom,

On 12 October 2016 at 08:13, Tom Rini  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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Simon Glass
Hi Keerthy,

On 13 October 2016 at 06:39, Keerthy  wrote:
>
>
> On Thursday 13 October 2016 06:08 PM, Simon Glass wrote:
>>
>> Hi Keerthy,
>>
>> On 13 October 2016 at 05:47, 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  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 
>>> 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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Keerthy



On Thursday 13 October 2016 06:08 PM, Simon Glass wrote:

Hi Keerthy,

On 13 October 2016 at 05:47, 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  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 
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 
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

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Simon Glass
Hi Keerthy,

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


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

2016-10-13 Thread Simon Glass
Hi Tom,

On 13 October 2016 at 06:11, Tom Rini  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  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 
>> 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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Tom Rini
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  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 
> 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.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Keerthy



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  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 
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 
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

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-13 Thread Keerthy



On Wednesday 12 October 2016 08:13 PM, Simon Glass wrote:

Hi Tom,

On 12 October 2016 at 08:13, Tom Rini  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 
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


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

2016-10-12 Thread Simon Glass
Hi Tom,

On 12 October 2016 at 08:13, Tom Rini  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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-12 Thread Tom Rini
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!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-10-11 Thread Simon Glass
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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-01-22 Thread Simon Glass
Hi Tom,

I've been sitting on this but I don't see much else so we may as well it get in.


The following changes since commit 6905f4d3c7be46fed4859f51f0a8f9a1107c22e7:

  Merge git://git.denx.de/u-boot-dm (2016-01-21 11:49:49 -0500)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 67871a595873930a0a536b1685e5caac53766701:

  devicetree: use wildcard to clean arch subdir (2016-01-22 14:41:14 -0700)


Thomas Chou (1):
  devicetree: use wildcard to clean arch subdir

 dts/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2016-01-22 Thread Tom Rini
On Fri, Jan 22, 2016 at 02:42:47PM -0700, Simon Glass wrote:

> Hi Tom,
> 
> I've been sitting on this but I don't see much else so we may as well it get 
> in.
> 
> 
> The following changes since commit 6905f4d3c7be46fed4859f51f0a8f9a1107c22e7:
> 
>   Merge git://git.denx.de/u-boot-dm (2016-01-21 11:49:49 -0500)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to 67871a595873930a0a536b1685e5caac53766701:
> 
>   devicetree: use wildcard to clean arch subdir (2016-01-22 14:41:14 -0700)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-10-03 Thread Simon Glass
Hi Tom,

Here are a few last-minute bug fixes mostly stemming from the
fdtdec_get_addr() change earlier.


The following changes since commit fbb0c7bd92255bfcb13826a8ac81be6e2f94ba48:

  Merge branch 'master' of git://git.denx.de/u-boot-tegra (2015-10-02
20:35:49 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to cce573e8d806fd430e7584b36bce6d62ae0430e8:

  trats: fdt: disable unused DW MMC (2015-10-03 14:39:19 +0100)


Przemyslaw Marczak (4):
  fdtdec: fix parsing 'reg' property with zero value in '#size-cells'
  gpio: s5p: call: dev_get_addr() instead of fdtdec_get_addr()
  mach-exynos: clock: restore calling dead exynos4_get_mmc_clk()
  trats: fdt: disable unused DW MMC

Simon Glass (1):
  dts: Add a comment about CONFIG_OF_EMBED being for local use

Stephen Warren (1):
  fdt: fix fdtdec_get_addr_size not to require any size cells

 arch/arm/dts/exynos4210-trats.dts |  4 
 arch/arm/mach-exynos/clock.c  | 10 --
 drivers/gpio/s5p_gpio.c   | 18 +++---
 dts/Kconfig   |  4 +++-
 lib/fdtdec.c  |  7 ---
 5 files changed, 26 insertions(+), 17 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-10-03 Thread Tom Rini
On Sat, Oct 03, 2015 at 03:45:21PM +0100, Simon Glass wrote:

> Hi Tom,
> 
> Here are a few last-minute bug fixes mostly stemming from the
> fdtdec_get_addr() change earlier.
> 
> 
> The following changes since commit fbb0c7bd92255bfcb13826a8ac81be6e2f94ba48:
> 
>   Merge branch 'master' of git://git.denx.de/u-boot-tegra (2015-10-02
> 20:35:49 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-fdt.git
> 
> for you to fetch changes up to cce573e8d806fd430e7584b36bce6d62ae0430e8:
> 
>   trats: fdt: disable unused DW MMC (2015-10-03 14:39:19 +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-10-02 Thread Simon Glass
The following changes since commit 1f8836396de8215b7f460616926052b32597bb29:

  Prepare v2015.10-rc4 (2015-09-28 16:57:42 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 8809141907528550750420e14c24fa66e089b3f8:

  dts: Add a comment about CONFIG_OF_EMBED being for local use
(2015-09-28 22:30:31 -0600)


Simon Glass (1):
  dts: Add a comment about CONFIG_OF_EMBED being for local use

 dts/Kconfig | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-07-21 Thread Tom Rini
On Mon, Jul 20, 2015 at 08:04:26AM -0600, Simon Glass wrote:

 Hi Tom,
 
 A few things in my queue. I would like to do a sync with upstream
 libfdt too but have not got to that yet.
 
 
 The following changes since commit 605e15db2b54302364a2528d3c6604fbc57be846:
 
   Merge git://git.denx.de/u-boot-x86 (2015-07-15 10:41:20 -0400)
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-fdt.git
 
 for you to fetch changes up to 73e1e7952a2a629dc071d894594df4852acc11ad:
 
   libfdt: fix error code of fdt_count_strings() (2015-07-20 07:21:47 -0600)
 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-07-20 Thread Simon Glass
Hi Tom,

A few things in my queue. I would like to do a sync with upstream
libfdt too but have not got to that yet.


The following changes since commit 605e15db2b54302364a2528d3c6604fbc57be846:

  Merge git://git.denx.de/u-boot-x86 (2015-07-15 10:41:20 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 73e1e7952a2a629dc071d894594df4852acc11ad:

  libfdt: fix error code of fdt_count_strings() (2015-07-20 07:21:47 -0600)


Andre Przywara (1):
  fdt: prevent clearing memory node if there are no banks

Haikun Wang (1):
  fdt: armv8: Fix build warnings on armv8

Masahiro Yamada (3):
  libfdt: fix description of fdt_get_string()
  libfdt: fix error code of fdt_get_string_index()
  libfdt: fix error code of fdt_count_strings()

Sudeep Holla (1):
  cmd_fdt: save fdtaddr in hex format

 common/cmd_fdt.c   |  2 +-
 common/fdt_support.c   |  3 +++
 drivers/spi/fsl_dspi.c |  4 ++--
 include/fdt_support.h  | 26 ++
 include/libfdt.h   |  2 +-
 lib/fdtdec.c   |  4 ++--
 lib/libfdt/fdt_ro.c|  4 ++--
 7 files changed, 37 insertions(+), 8 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-06-06 Thread Tom Rini
On Fri, Jun 05, 2015 at 10:08:53AM -0600, Simon Glass wrote:

 Hi Tom,
 
 A few things I found in patchwork.
 
 
 The following changes since commit d8abb46b37fadff0349adb376df6d3ecd09ee7d1:
 
   board/BuR/common: fix netconsole (2015-06-02 12:58:51 -0400)
 
 are available in the git repository at:
 
   git://git.denx.de/u-boot-fdt.git
 
 for you to fetch changes up to 3c4c142e5deed2a9362e65e45372044652ac57b5:
 
   fdt: Documentation for a few support functions aside their
 prototypes (2015-06-05 08:32:58 -0600)
 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-06-05 Thread Simon Glass
Hi Tom,

A few things I found in patchwork.


The following changes since commit d8abb46b37fadff0349adb376df6d3ecd09ee7d1:

  board/BuR/common: fix netconsole (2015-06-02 12:58:51 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 3c4c142e5deed2a9362e65e45372044652ac57b5:

  fdt: Documentation for a few support functions aside their
prototypes (2015-06-05 08:32:58 -0600)


Paul Kocialkowski (2):
  fdt: Pass the device serial number through devicetree
  fdt: Documentation for a few support functions aside their prototypes

 common/fdt_support.c  | 25 +
 common/image-fdt.c|  4 
 doc/device-tree-bindings/root.txt |  4 
 include/fdt_support.h | 27 +++
 4 files changed, 60 insertions(+)
 create mode 100644 doc/device-tree-bindings/root.txt

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-04-28 Thread Tom Rini
On Fri, Apr 24, 2015 at 09:37:02AM -0600, Simon Glass wrote:

 Hi Tom,
 
 The following changes since commit d8c1d5d5fb6eafbc532982125f006e49f2c40e71:
 
   Merge branch 'buildman' of git://git.denx.de/u-boot-x86 (2015-04-23
 14:56:47 -0400)
 
 are available in the git repository at:
 
   http://git.denx.de/u-boot-fdt.git
 
 for you to fetch changes up to 77d7fff8cec2652be8c2494b6b66d14a398ec860:
 
   fdt: Fix handling of paths with options in them (2015-04-23 22:54:32 -0600)
 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-04-24 Thread Simon Glass
Hi Tom,

The following changes since commit d8c1d5d5fb6eafbc532982125f006e49f2c40e71:

  Merge branch 'buildman' of git://git.denx.de/u-boot-x86 (2015-04-23
14:56:47 -0400)

are available in the git repository at:

  http://git.denx.de/u-boot-fdt.git

for you to fetch changes up to 77d7fff8cec2652be8c2494b6b66d14a398ec860:

  fdt: Fix handling of paths with options in them (2015-04-23 22:54:32 -0600)


Hans de Goede (1):
  fdt: Fix handling of paths with options in them

 lib/libfdt/fdt_ro.c | 25 ++---
 1 file changed, 22 insertions(+), 3 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-02-18 Thread Tom Rini
On Wed, Feb 18, 2015 at 06:09:28AM -0700, Simon Glass wrote:

 Hi Tom,
 
 The following changes since commit 5745f8c4fd5807becf7f246625e153388293aedc:
 
   Merge git://git.denx.de/u-boot-marvell (2015-02-16 08:44:03 -0500)
 
 are available in the git repository at:
 
   http://git.denx.de/u-boot-fdt.git
 
 for you to fetch changes up to c71a0164d9b23e624552fb614bcb426a9b57:
 
   cmd_fdt: Print the control fdt in terms of virtual memory
 (2015-02-17 20:19:16 -0700)
 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


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

2015-02-18 Thread Simon Glass
Hi Tom,

The following changes since commit 5745f8c4fd5807becf7f246625e153388293aedc:

  Merge git://git.denx.de/u-boot-marvell (2015-02-16 08:44:03 -0500)

are available in the git repository at:

  http://git.denx.de/u-boot-fdt.git

for you to fetch changes up to c71a0164d9b23e624552fb614bcb426a9b57:

  cmd_fdt: Print the control fdt in terms of virtual memory
(2015-02-17 20:19:16 -0700)


Joe Hershberger (2):
  cmd_fdt: Actually fix fdt command in sandbox
  cmd_fdt: Print the control fdt in terms of virtual memory

 common/bootm.c|  2 +-
 common/cmd_fdt.c  | 10 +-
 common/image-fdt.c|  2 +-
 include/fdt_support.h |  2 +-
 4 files changed, 8 insertions(+), 8 deletions(-)

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot