Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt (take 4)
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
Re: [U-Boot] Please pull u-boot-fdt, take 2
Hi Tom, On 26 May 2017 at 06:49, Tom Riniwrote: > > 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
On Thu, May 25, 2017 at 09:15:33PM -0600, Simon Glass wrote: > Hi Tom, > > On 25 May 2017 at 11:42, Tom Riniwrote: > > 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
Hi Tom, On 25 May 2017 at 11:42, Tom Riniwrote: > 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
On Thu, May 25, 2017 at 11:27:10AM -0600, Simon Glass wrote: > Hi Tom, > > On 25 May 2017 at 05:19, Tom Riniwrote: > > > > 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
Hi Tom, On 25 May 2017 at 05:19, Tom Riniwrote: > > 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
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
Re: [U-Boot] Please pull u-boot-fdt
Hi Tom, On 9 May 2017 at 17:59, Tom Riniwrote: > > 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
Hi Tom, On 9 May 2017 at 17:59, Tom Riniwrote: > 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
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
+Tom On 25 November 2016 at 09:51, Simon Glasswrote: > 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
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
Re: [U-Boot] Please pull u-boot-fdt (take 2)
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
Re: [U-Boot] Please pull u-boot-fdt
On Thursday 13 October 2016 08:38 PM, Simon Glass wrote: Hi Keerthy, On 13 October 2016 at 08:59, Keerthywrote: 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
Hi Keerthy, On 13 October 2016 at 08:59, Keerthywrote: > 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
Simon, On Thursday 13 October 2016 07:54 PM, Simon Glass wrote: Hi Tom, On 12 October 2016 at 08:13, Tom Riniwrote: 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
Hi Tom, On 13 October 2016 at 08:33, Tom Riniwrote: > 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
On Thu, Oct 13, 2016 at 08:24:22AM -0600, Simon Glass wrote: > Hi Tom, > > On 12 October 2016 at 08:13, Tom Riniwrote: > > > > 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
Hi Tom, On 12 October 2016 at 08:13, Tom Riniwrote: > > 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
Hi Keerthy, On 13 October 2016 at 06:39, Keerthywrote: > > > 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
On Thursday 13 October 2016 06:08 PM, Simon Glass wrote: Hi Keerthy, On 13 October 2016 at 05:47, Keerthywrote: 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
Hi Keerthy, On 13 October 2016 at 05:47, Keerthywrote: > > > 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
Hi Tom, On 13 October 2016 at 06:11, Tom Riniwrote: > 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
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 Riniwrote: > >>>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
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 Riniwrote: 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
On Wednesday 12 October 2016 08:13 PM, Simon Glass wrote: Hi Tom, On 12 October 2016 at 08:13, Tom Riniwrote: 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
Hi Tom, On 12 October 2016 at 08:13, Tom Riniwrote: > 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
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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
Re: [U-Boot] Please pull u-boot-fdt
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