Re: [U-Boot] Please pull u-boot-dm
On Thu, Nov 14, 2019 at 08:08:09PM -0700, Simon Glass wrote: > Hi Tom, > > Passing run here: > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/1306 > > > The following changes since commit 3ff1ff3ff76c15efe0451309af084ee6c096c583: > > Merge branch '2019-11-12-migrate-SYS_REDUNDAND_ENVIRONMENT' > (2019-11-12 13:40:58 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-14nov19 > > for you to fetch changes up to 677dac23d8ebfc0342f903aeee2a75e737fd4d7e: > > patman: fix coverletter cc list with nulls (2019-11-14 07:09:34 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, Passing run here: https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/1306 The following changes since commit 3ff1ff3ff76c15efe0451309af084ee6c096c583: Merge branch '2019-11-12-migrate-SYS_REDUNDAND_ENVIRONMENT' (2019-11-12 13:40:58 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-14nov19 for you to fetch changes up to 677dac23d8ebfc0342f903aeee2a75e737fd4d7e: patman: fix coverletter cc list with nulls (2019-11-14 07:09:34 -0600) Add OP-TEE test swuit Fix patman cc_file output Minor sandbox/pinctrl changes Anatolij Gustschin (1): patman: fix some typos in comments Heiko Stuebner (4): fdtdec: protect against another NULL phandlep in fdtdec_add_reserved_memory() fdtdec: only create phandle if caller wants it in fdtdec_add_reserved_memory() image: fdt: copy possible optee nodes to a loaded devicetree tests: add OP-TEE test suite Heinrich Schuchardt (1): sandbox: use correct type reading /proc/self/maps Patrick Delaunay (1): pinctrol: dm: remove the function pinctrl_decode_pin_config Robert Beckett (1): patman: fix coverletter cc list with nulls Makefile | 1 + arch/sandbox/cpu/os.c | 4 +- common/image-fdt.c | 8 drivers/pinctrl/pinctrl-uclass.c | 12 -- include/dm/pinctrl.h | 13 -- include/fdtdec.h | 1 + include/tee/optee.h| 9 include/test/optee.h | 14 +++ include/test/suites.h | 1 + lib/fdtdec.c | 17 lib/optee/optee.c | 140 +++ test/Kconfig | 1 + test/cmd_ut.c | 6 +++ test/optee/Kconfig | 7 test/optee/Makefile| 13 ++ test/optee/cmd_ut_optee.c | 149 +++ test/optee/test-optee-base.dts | 13 ++ test/optee/test-optee-no-optee.dts | 13 ++ test/optee/test-optee-optee.dts| 32 +++ tools/patman/cros_subprocess.py| 4 +- tools/patman/gitutil.py| 4 +- tools/patman/series.py | 2 +- tools/patman/terminal.py | 2 +- 23 files changed, 426 insertions(+), 40 deletions(-) create mode 100644 include/test/optee.h create mode 100644 test/optee/Kconfig create mode 100644 test/optee/Makefile create mode 100644 test/optee/cmd_ut_optee.c create mode 100644 test/optee/test-optee-base.dts create mode 100644 test/optee/test-optee-no-optee.dts create mode 100644 test/optee/test-optee-optee.dts 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-dm
On Tue, Oct 29, 2019 at 05:35:08PM -0600, Simon Glass wrote: > Hi Tom, > > Test results here: > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/1131 > > > The following changes since commit ffc379b42c85466e1dd4c8fee8268801f26d2ab8: > > Merge tag 'mips-pull-2019-10-25' of git://git.denx.de/u-boot-mips > (2019-10-25 20:07:24 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-29oct19 > > for you to fetch changes up to d60ae4c59df55c08dc96202ff58fed21ab3afb7d: > > fdt: Fix alignment issue when reading 64-bits properties from fdt > (2019-10-27 13:01:53 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, Test results here: https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/1131 The following changes since commit ffc379b42c85466e1dd4c8fee8268801f26d2ab8: Merge tag 'mips-pull-2019-10-25' of git://git.denx.de/u-boot-mips (2019-10-25 20:07:24 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-29oct19 for you to fetch changes up to d60ae4c59df55c08dc96202ff58fed21ab3afb7d: fdt: Fix alignment issue when reading 64-bits properties from fdt (2019-10-27 13:01:53 -0600) Fix for patman with email addresses containing commas Bootstage improvements for TPL, SPL Various sandbox and dm improvements and fixes Dmitry Torokhov (1): patman: separate emails in CC list with NULs Jean-Jacques Hiblot (2): test: regmap: check the values read from the regmap fdt: Fix alignment issue when reading 64-bits properties from fdt Kever Yang (1): dm: core: Update log method for uclass_find_device_by_seq Simon Glass (14): sandbox: test: Show hex values on failure sandbox: Drop 'const' from sandbox_write() sandbox: test: Add a prototype for sandbox_set_enable_memio() dm: regmap: Fix mask in regmap_update_bits() tiny-printf: Reduce size by removing ctype tiny-printf: Add print_grouped_ull() tiny-printf: Reorder code to support %p bloblist: Reserve an aligned base bootstage: Store the next ID in the stash bootstage: Fix counting of entries in stash bootstage: Avoid conflicts between stash/unstash bootstage: Correct relocation algorithm bootstage: Mark the start/end of TPL and SPL separately bootstage: Allow SPL to obtain bootstage info from TPL arch/sandbox/cpu/cpu.c | 3 +-- arch/sandbox/include/asm/io.h | 11 +- arch/sandbox/include/asm/test.h | 11 ++ common/board_f.c| 2 ++ common/board_r.c| 1 - common/bootstage.c | 53 ++ common/fdt_support.c| 2 +- common/spl/spl.c| 23 drivers/core/ofnode.c | 2 +- drivers/core/regmap.c | 2 +- drivers/core/uclass.c | 11 ++ include/bootstage.h | 2 ++ include/linux/libfdt_env.h | 1 + include/regmap.h| 3 ++- include/test/ut.h | 3 ++- lib/fdtdec.c| 2 +- lib/tiny-printf.c | 29 +++-- test/dm/regmap.c| 19 ++--- tools/patman/func_test.py | 4 ++-- tools/patman/patman.py | 2 +- tools/patman/series.py | 4 ++-- 21 files changed, 135 insertions(+), 55 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-dm
On Tue, Oct 15, 2019 at 08:46:46AM -0600, Simon Glass wrote: > Hi Tom, > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/950 > > > The following changes since commit 6891152a4596d38ac25d2fe1238e3b6a938554b8: > > Merge branch 'master' of git://git.denx.de/u-boot-socfpga > (2019-10-14 21:00:10 -0400) > > are available in the Git repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git tags/dm-pull-15oct19 > > for you to fetch changes up to d11ef4d54cab0e740efbceb9c6b5697a41770eea: > > sandbox: fix build error due to missing struct udevice definition > (2019-10-15 08:40:03 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/950 The following changes since commit 6891152a4596d38ac25d2fe1238e3b6a938554b8: Merge branch 'master' of git://git.denx.de/u-boot-socfpga (2019-10-14 21:00:10 -0400) are available in the Git repository at: https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git tags/dm-pull-15oct19 for you to fetch changes up to d11ef4d54cab0e740efbceb9c6b5697a41770eea: sandbox: fix build error due to missing struct udevice definition (2019-10-15 08:40:03 -0600) binman enhancements: - Dropping some test Elf files and building them from source instead - Refactoring of x86 16-bit entries - Support for SPL symbols within sections - Handle the 'notes' sections and hidden symbols in recent binutils - Improved error reporting with a tool fails libfdt and documentation fixes vboot required-key test driver model power-domain controls patman Message-Id enhancement AKASHI Takahiro (2): sandbox: fix cpu property in test.dts for pytest sandbox: fix build error due to missing struct udevice definition Anatolij Gustschin (1): dm: core: device: switch off power domain after device removal Daniele Alessandrelli (1): rsa: Return immediately if required-key verification fails Douglas Anderson (1): patman: Use the Change-Id, version, and prefix in the Message-Id Giulio Benetti (1): libfdt: fix typo on comment Jean-Jacques Hiblot (1): dm: device: Fix typo in the non-DEVRES version of devm_kmalloc_array() Kayla Theil (1): tpm2: Don't assume active low reset value Lokesh Vutla (3): dm: core: Allow for not controlling the power-domain by DM framework remoteproc: k3_arm64: Enable DM_FLAG_DEFAULT_PD_CTRL_OFF power: domain: Introduce dev_power_domain_off Matthias Brugger (4): libfdt: fdt_address_cells() and fdt_size_cells() libfdt: return correct value if #size-cells property is not present libfdt: Allow #size-cells of 0 dm: Fix default address cells return value Michael Trimarchi (1): dm: pinctrl: Skip not associated gpio phandle and rise a warning message Patrick Delaunay (3): dm: pinctrl: introduce PINCONF_RECURSIVE option dm: pinctrl: introduce PINCONF_RECURSIVE option dm: Tidy up dump output when there are many devices Peng Fan (2): power: domain: add dev_power_domain_on core: device: use dev_power_domain_on Philippe Reynes (4): pytest: vboot: add a test for required key sandbox: enable command aes sandbox64: enable command aes cmd: aes: use map_sysmem when accessing memory Ralph Siemsen (1): doc: add full path to patman README Simon Glass (33): patman: Drop binary parameter patman: Update command.Run() to handle failure better binman: Use cls instead of self for class methods binman: Allow use of help and entry-docs without libfdt binman: Drop .note section from ELF binman: Handle hidden symbols in ELF files binman: Correct use of 'replace' in IFWI tests binman: Add support for an x86 'reset' section binman: x86: Separate out 16-bit reset and init code binman: Add support for Intel FIT binman: Fix IFWI output when using an Intel FIT image binman: Use tools.Run() to run objdump binman: Use the Makefile to build ELF test files binman: Use the Makefile for u_boot_ucode_ptr binman: Use the Makefile for u_boot_no_ucode_ptr binman: Use the Makefile for u_boot_binman_syms binman: Use the Makefile for u_boot_binman_syms_size binman: Use the Makefile for u_boot_binman_syms_bad binman: Clean up unnecessary code related to ELF test files binman: Allow symbols to be resolved inside sections binman: Use underscore in test filenames binman: Rename some two-digit test files binman: Avoid needing the section size in advance binman: Increase size of TPL and SPL test data binman: Allow support for writing a size symbol to binaries binman: Add support for Intel FSP meminit binman: Fix entry comment for Intel descriptor binman: Update IFWI entry to read entries outside constructor binman: Update IFWI entry to support updates binman: Support writing symbols into entries within an IFWI binman: Write symbol info before image inclusion binman: Add logging for the number of pack passes binman: Drop comment-out code in testUpdateFdtOutput() Thomas Fitzsimmons (2): dm: device: Request next sequence number dm: spi: Do not assume first SPI bus Makefile | 10 +- arch/sandbox/dts/test.dts | 24 ++-- arch/sandbox/include/asm/u-boot-sandbox.h | 2 + arch/x86/dts/u-boot.dtsi | 9 ++ cmd/aes.c | 14 ++- configs/sandbox
Re: [U-Boot] Please pull u-boot-dm
On Mon, Jul 29, 2019 at 03:27:59PM -0600, Simon Glass wrote: > Hi Tom, > > Gitlab is here: > https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/356 > > The following changes since commit 75551c8bfc9545e31ec2ce238cac3857904007b8: > > Merge branch '2019-07-26-ti-imports' (2019-07-27 19:50:52 -0400) > > are available in the Git repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git > tags/dm-pull-29jul19 > > for you to fetch changes up to 4f4fb85ec0bfe45da11aa23ada565387ee676e80: > > Makefile: fix implementation of BINMAN_DEBUG (2019-07-29 09:38:06 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, Gitlab is here: https://gitlab.denx.de/u-boot/custodians/u-boot-dm/pipelines/356 The following changes since commit 75551c8bfc9545e31ec2ce238cac3857904007b8: Merge branch '2019-07-26-ti-imports' (2019-07-27 19:50:52 -0400) are available in the Git repository at: https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git tags/dm-pull-29jul19 for you to fetch changes up to 4f4fb85ec0bfe45da11aa23ada565387ee676e80: Makefile: fix implementation of BINMAN_DEBUG (2019-07-29 09:38:06 -0600) binman support for replacing files Simon Glass (53): dtoc: Return a non-zero exit code when tests fail binman: Move image-processing code into a function binman: Move GetFdtSet() into blob_dtb binman: Use print() to print output binman: Move image/fdt code into PrepareImagesAndDtbs() binman: Convert GetFdtSet() to use a dict binman: Rename state.GetFdts() binman: Rename state.GetFdt() binman: Adjust GetFdt() to be keyed by etype binman: Adjust state.fdt_files to be keyed by entry type binman: Simplify state.fdt_subset binman: Drop state.fdt_set as this is not needed patman: Update tout to avoid open-coding the debug levels binman: Add a bit of logging in entries when packing binman: Show a helpful error when a DT property is missing dtoc: Update Fdt.FromData() to allow a name dtoc: Update Fdt.GetNode() to handle the root node binman: Store image fdtmap when loading from a file binman: Support loading entry data from a file binman: Allow state functions to fail to return data binman: Store the entry in output_fdt_files binman: Add an image name into the fdtmap binman: Adjust Entry to read the node in a separate call binman: Add a function to obtain the image for an Entry binman: Add a constant for common entry properties binman: Allow the fdtmap to remain unchanged binman: Tidy up _SetupDtb() to use its own temporary file binman: Support updating entries in an existing image binman: Add info to allow safely repacking an image later binman: Update documentation for image creation binman: Write the original input fdtmap to a file binman: Move Image.BuildImage() into a single function binman: Add more tests for image header position binman: Allow updating entries that change size binman: Update the _testing entry to support shrinkage binman: Support shrinking a entry after packing libfdt: Copy the struct region in fdt_resize() binman: Adjust fmap to ignore CBFS files binman: Place Intel descriptor at image start binman: Add a few more features to the wishlist binman: Add a prefix before CBFS hex offsets binman: Update Entry.ReadEntry() to work through classes binman: Update Entry.WriteData() to handle special sections binman: Support replacing data in a cbfs patman: Reset the output directory when it is removed binman: Update state when replacing device-tree entries binman: Add a test function to clean up the output dir binman: Clean up all output directories in tests binman: Move control.WriteEntry further down the file binman: Update control.WriteEntry() to support writing the map binman: Split control.WriteEntryToImage() into separate functions binman: Correct the error message for invalid path binman: Add command-line support for replacing entries Stephen Warren (1): Makefile: fix implementation of BINMAN_DEBUG Makefile| 5 +- scripts/dtc/libfdt/fdt_sw.c | 2 +- tools/binman/README | 79 - tools/binman/README.entries | 11 +- tools/binman/cbfs_util.py | 14 +- tools/binman/cbfs_util_test.py | 4 +- tools/binman/cmdline.py | 17 ++ tools/binman/control.py | 408 --- tools/binman/elf.py | 9 +- tools/binman/elf_test.py| 19 +- tools/binman/entry.py | 162 ++--- tools/binman/entry_test.py | 22 ++- tools/binman/etype/_testing.py | 28 ++- tools/binman/etype/blob.py | 12 -- tools/binman/etype/blob_dtb.py | 35 +++- tools/binman/etype/cbfs.py | 24 ++- tools/binman/etype/fdtmap.py| 60 --- tools/binman/etype/fill.py | 3 + tools/binman/etype/fmap.py | 11 +- tools/binman/etype/image_header.py | 16 +- tools/binman/etype/intel_descriptor.py
Re: [U-Boot] Please pull u-boot-dm (take 3)
On Wed, Jul 24, 2019 at 01:03:37PM -0700, Simon Glass wrote: > Hi Tom, > > The following changes since commit ff8c23e784f57a7098e91a200ed7f5a48612b653: > > Merge tag 'u-boot-stm32-20190723' of > https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 > 14:16:21 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-24jul19-take3 > > for you to fetch changes up to 44e02e39a91cd91aae5a28d90259d3a6996010bf: > > dm: device: make power domain calls optional (2019-07-24 12:54:08 -0700) > Applied to u-boot/master, thanks! -- 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-dm (take 2)
On Wed, Jul 24, 2019 at 10:22 PM Tom Rini wrote: > > On Wed, Jul 24, 2019 at 10:09:29PM +0200, Simon Goldschmidt wrote: > > Am 24.07.2019 um 22:03 schrieb Simon Glass: > > >Hi Tom, > > > > > >On Wed, 24 Jul 2019 at 10:13, Tom Rini wrote: > > >> > > >>On Tue, Jul 23, 2019 at 08:36:58PM -0700, Simon Glass wrote: > > >> > > >>>Hi Tom, > > >>> > > >>>This includes the gitlab fix and also I updated ifwitool.c to drop the > > >>>unused inline functions. > > >>> > > >>> > > >>>The following changes since commit > > >>>ff8c23e784f57a7098e91a200ed7f5a48612b653: > > >>> > > >>> Merge tag 'u-boot-stm32-20190723' of > > >>>https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 > > >>>14:16:21 -0400) > > >>> > > >>>are available in the Git repository at: > > >>> > > >>> git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 > > >>> > > >>>for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: > > >>> > > >>> dm: device: make power domain calls optional (2019-07-23 20:27:58 > > >>> -0700) > > >>> > > >> > > >>We're still not there yet, sorry. Taking a look at > > >>https://gitlab.denx.de/u-boot/u-boot/-/jobs/1464 we're now running > > >>things but ifwitool is failing. Can you please take a look? You will > > >>need the patch I posted that brings us up to a newer image that has > > >>lzma-alone. Thanks! > > > > > >OK, this is another update that is needed to the gitlab config, this > > >time the --toolpath argument. Will send an update. > > > > > >I was rather hoping that pushing to gitlab u-boot-dm would start a > > >build but it does not, for me. I probably need to do some more > > >reading. > > > > > >BTW Is there an intent to shut down travis? > > > > Even if everyone seems to be switching to gitlab, can we keep the travis > > config in the repository? That makes it easy for people (like me) creating a > > github clone to ensure the world keeps building after a patch. > > > > Or is there a similar free and easy to use gitlab service comparable to > > travis? > > So that's the rub. There are free "runners" in GitLab terms for > projects but they are limited in time to I think an hour per job, so we > get back to the unmaintainable mess of splitting the build like we have > on Travis. Anyone can use their own machine as the runner, but that may > or may not be faster or even slower than Travis is. I can assure you using my own PC is not an option for me compared to Travis. I don't know how many non-custodians there are using this (and I can probably arrange with Marek to use socfpga custodian access), but having a free-to-use CI seems worth much to me. I don't know how much effort it is to keep and maintain the Travis config files though... Regards, Simon > > -- > Tom ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm (take 2)
On Wed, Jul 24, 2019 at 10:09:29PM +0200, Simon Goldschmidt wrote: > Am 24.07.2019 um 22:03 schrieb Simon Glass: > >Hi Tom, > > > >On Wed, 24 Jul 2019 at 10:13, Tom Rini wrote: > >> > >>On Tue, Jul 23, 2019 at 08:36:58PM -0700, Simon Glass wrote: > >> > >>>Hi Tom, > >>> > >>>This includes the gitlab fix and also I updated ifwitool.c to drop the > >>>unused inline functions. > >>> > >>> > >>>The following changes since commit > >>>ff8c23e784f57a7098e91a200ed7f5a48612b653: > >>> > >>> Merge tag 'u-boot-stm32-20190723' of > >>>https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 > >>>14:16:21 -0400) > >>> > >>>are available in the Git repository at: > >>> > >>> git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 > >>> > >>>for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: > >>> > >>> dm: device: make power domain calls optional (2019-07-23 20:27:58 -0700) > >>> > >> > >>We're still not there yet, sorry. Taking a look at > >>https://gitlab.denx.de/u-boot/u-boot/-/jobs/1464 we're now running > >>things but ifwitool is failing. Can you please take a look? You will > >>need the patch I posted that brings us up to a newer image that has > >>lzma-alone. Thanks! > > > >OK, this is another update that is needed to the gitlab config, this > >time the --toolpath argument. Will send an update. > > > >I was rather hoping that pushing to gitlab u-boot-dm would start a > >build but it does not, for me. I probably need to do some more > >reading. > > > >BTW Is there an intent to shut down travis? > > Even if everyone seems to be switching to gitlab, can we keep the travis > config in the repository? That makes it easy for people (like me) creating a > github clone to ensure the world keeps building after a patch. > > Or is there a similar free and easy to use gitlab service comparable to > travis? So that's the rub. There are free "runners" in GitLab terms for projects but they are limited in time to I think an hour per job, so we get back to the unmaintainable mess of splitting the build like we have on Travis. Anyone can use their own machine as the runner, but that may or may not be faster or even slower than Travis is. -- 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-dm (take 2)
On Wed, Jul 24, 2019 at 01:03:20PM -0700, Simon Glass wrote: > Hi Tom, > > On Wed, 24 Jul 2019 at 10:13, Tom Rini wrote: > > > > On Tue, Jul 23, 2019 at 08:36:58PM -0700, Simon Glass wrote: > > > > > Hi Tom, > > > > > > This includes the gitlab fix and also I updated ifwitool.c to drop the > > > unused inline functions. > > > > > > > > > The following changes since commit > > > ff8c23e784f57a7098e91a200ed7f5a48612b653: > > > > > > Merge tag 'u-boot-stm32-20190723' of > > > https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 > > > 14:16:21 -0400) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 > > > > > > for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: > > > > > > dm: device: make power domain calls optional (2019-07-23 20:27:58 -0700) > > > > > > > We're still not there yet, sorry. Taking a look at > > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1464 we're now running > > things but ifwitool is failing. Can you please take a look? You will > > need the patch I posted that brings us up to a newer image that has > > lzma-alone. Thanks! > > OK, this is another update that is needed to the gitlab config, this > time the --toolpath argument. Will send an update. > > I was rather hoping that pushing to gitlab u-boot-dm would start a > build but it does not, for me. I probably need to do some more > reading. Take a look at the email I sent to the custodians list :) You'll need to flip a switch or two on the -dm tree for it to happen, and you can provide more build power for just your tree in addition to the shared runner we have currently. > BTW Is there an intent to shut down travis? As in remove the file? Not immediately, no, there's still the question of what to do about non-custodians being able to run CI before submitting. -- 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-dm (take 2)
Am 24.07.2019 um 22:03 schrieb Simon Glass: Hi Tom, On Wed, 24 Jul 2019 at 10:13, Tom Rini wrote: On Tue, Jul 23, 2019 at 08:36:58PM -0700, Simon Glass wrote: Hi Tom, This includes the gitlab fix and also I updated ifwitool.c to drop the unused inline functions. The following changes since commit ff8c23e784f57a7098e91a200ed7f5a48612b653: Merge tag 'u-boot-stm32-20190723' of https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 14:16:21 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: dm: device: make power domain calls optional (2019-07-23 20:27:58 -0700) We're still not there yet, sorry. Taking a look at https://gitlab.denx.de/u-boot/u-boot/-/jobs/1464 we're now running things but ifwitool is failing. Can you please take a look? You will need the patch I posted that brings us up to a newer image that has lzma-alone. Thanks! OK, this is another update that is needed to the gitlab config, this time the --toolpath argument. Will send an update. I was rather hoping that pushing to gitlab u-boot-dm would start a build but it does not, for me. I probably need to do some more reading. BTW Is there an intent to shut down travis? Even if everyone seems to be switching to gitlab, can we keep the travis config in the repository? That makes it easy for people (like me) creating a github clone to ensure the world keeps building after a patch. Or is there a similar free and easy to use gitlab service comparable to travis? 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-dm (take 2)
Hi Tom, On Wed, 24 Jul 2019 at 10:13, Tom Rini wrote: > > On Tue, Jul 23, 2019 at 08:36:58PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > This includes the gitlab fix and also I updated ifwitool.c to drop the > > unused inline functions. > > > > > > The following changes since commit ff8c23e784f57a7098e91a200ed7f5a48612b653: > > > > Merge tag 'u-boot-stm32-20190723' of > > https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 > > 14:16:21 -0400) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 > > > > for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: > > > > dm: device: make power domain calls optional (2019-07-23 20:27:58 -0700) > > > > We're still not there yet, sorry. Taking a look at > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1464 we're now running > things but ifwitool is failing. Can you please take a look? You will > need the patch I posted that brings us up to a newer image that has > lzma-alone. Thanks! OK, this is another update that is needed to the gitlab config, this time the --toolpath argument. Will send an update. I was rather hoping that pushing to gitlab u-boot-dm would start a build but it does not, for me. I probably need to do some more reading. BTW Is there an intent to shut down travis? Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] Please pull u-boot-dm (take 3)
Hi Tom, The following changes since commit ff8c23e784f57a7098e91a200ed7f5a48612b653: Merge tag 'u-boot-stm32-20190723' of https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 14:16:21 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-24jul19-take3 for you to fetch changes up to 44e02e39a91cd91aae5a28d90259d3a6996010bf: dm: device: make power domain calls optional (2019-07-24 12:54:08 -0700) Minor driver-model fixes and tweaks A few device-tree fixes Binman support for extracting files from an image Anatolij Gustschin (1): dm: device: make power domain calls optional Baruch Siach (2): dm: uclass: fix comment copy/paste error buildman: fix invocation examples typos Bin Meng (4): dm: timer: Skip device that does not have a valid ofnode in pre_probe() dm: core: Call clk_set_defaults() during probe() only for a valid ofnode dm: core: Set correct "status" value for a node dm: Fix parameter description of dev_read_name() Marek Vasut (1): common: fdt_support: Add missing cpu_to_fdt32() to fdt_pci_dma_ranges() Masahiro Yamada (1): fdt: make fdt_get_base_address() return OF_BAD_ADDR when "reg" not found Sekhar Nori (1): clk: initialize clk->data when using default xlate Simon Glass (68): x86: Add ifwitool for Intel Integrated Firmware Image cbfs: Add an enum and comment for the magic number cbfs: Rename checksum to attributes_offset tools: Drop duplicate raise_on_error argument binman: Fix comment in bsection.GetEntries() binman: Correct two typos in function names in ftest binman: Add coverage tools info for Python 3 patman: Add a way to set the search path for tools binman: Add a --toolpath option to set the tool search path binman: Add missing comments to bsection binman: Add missing comments toentry binman: Tidy up help for --indir binman: Use a better error for missing Intel descriptor binman: Detect skipped tests binman: Add a function to create a sample ELF file binman: Add a function to decode an ELF file binman: Ensure that coverage has access to site packages binman: Assume Intel descriptor is at the start of the image binman: Don't assume there is an ME region binman: Update entry.SetOffsetSize to be optional binman: Allow text directly in the node patman: Add functions to compress and decompress data binman: Use the tools.Decompress method binman: Drop unnecessary debug handling binman: Use tools compression function for blob handling binman: Correct comment in u_boot_spl_elf binman: Support ELF files for TPL binman: Fix up the _DoTestFile() function -u argument binman: Allow verbosity control when running tests binman: Allow preserving test directories binman: Pass the toolpath to tests patman: Add a function to write ifwitool binman: Add a utility library for coreboot CBFS binman: Add support for CBFS entries binman: Add support for Intel IFWI entries binman: Pad empty areas of the CBFS with files binman: Add support for fixed-offset files in CBFS binman: Simplify the entry test binman: Update future features binman: Update help for new features binman: Add a convenience functions for real-DTB tests binman: Add an FDT map binman: Add an image header binman: Convert to use ArgumentParser binman: Move compression into the Entry base class binman: Drop an unused arg in Entry.Lookup() binman: Allow easy importing of entry modules binman: Fix up ProcessUpdateContents error and comments binman: Call ProcessUpdateContents() consistently binman: Add a return value to ProcessContentsUpdate() binman: Add a control for post-pack entry expansion binman: Allow entries to expand after packing binman: Allow device-tree entries to be compressed binman: Provide the actual data address for cbfs files binman: Use the cbfs memlen field only for uncompressed length binman: Support FDT update for CBFS binman: Detect bad CBFS file types binman: Allow listing the entries in an image binman: Support locating an FDT map binman: Support locating an image header binman: Support reading an image into an Image object binman: Convert Image to a subclass of Entry binman: Support listing an image binman: Allow for logging information to be displayed binman: Allow reading an entry from an image binman: Support reading from CBFS entries binman: Add an 'extract' command binman: Add a test for nested and aligned sections .gitlab-ci.yml |2 +-
Re: [U-Boot] Please pull u-boot-dm (take 2)
On Tue, Jul 23, 2019 at 08:36:58PM -0700, Simon Glass wrote: > Hi Tom, > > This includes the gitlab fix and also I updated ifwitool.c to drop the > unused inline functions. > > > The following changes since commit ff8c23e784f57a7098e91a200ed7f5a48612b653: > > Merge tag 'u-boot-stm32-20190723' of > https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 > 14:16:21 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 > > for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: > > dm: device: make power domain calls optional (2019-07-23 20:27:58 -0700) > We're still not there yet, sorry. Taking a look at https://gitlab.denx.de/u-boot/u-boot/-/jobs/1464 we're now running things but ifwitool is failing. Can you please take a look? You will need the patch I posted that brings us up to a newer image that has lzma-alone. Thanks! -- 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-dm (take 2)
Hi Tom, This includes the gitlab fix and also I updated ifwitool.c to drop the unused inline functions. The following changes since commit ff8c23e784f57a7098e91a200ed7f5a48612b653: Merge tag 'u-boot-stm32-20190723' of https://gitlab.denx.de/u-boot/custodians/u-boot-stm (2019-07-23 14:16:21 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-23jul19-take2 for you to fetch changes up to 6e38047b66e37659b937cf301e0de37a90d9eaa2: dm: device: make power domain calls optional (2019-07-23 20:27:58 -0700) Minor driver-model fixes and tweaks A few device-tree fixes Binman support for extracting files from an image Anatolij Gustschin (1): dm: device: make power domain calls optional Baruch Siach (2): dm: uclass: fix comment copy/paste error buildman: fix invocation examples typos Bin Meng (4): dm: timer: Skip device that does not have a valid ofnode in pre_probe() dm: core: Call clk_set_defaults() during probe() only for a valid ofnode dm: core: Set correct "status" value for a node dm: Fix parameter description of dev_read_name() Marek Vasut (1): common: fdt_support: Add missing cpu_to_fdt32() to fdt_pci_dma_ranges() Masahiro Yamada (1): fdt: make fdt_get_base_address() return OF_BAD_ADDR when "reg" not found Sekhar Nori (1): clk: initialize clk->data when using default xlate Simon Glass (68): x86: Add ifwitool for Intel Integrated Firmware Image cbfs: Add an enum and comment for the magic number cbfs: Rename checksum to attributes_offset tools: Drop duplicate raise_on_error argument binman: Fix comment in bsection.GetEntries() binman: Correct two typos in function names in ftest binman: Add coverage tools info for Python 3 patman: Add a way to set the search path for tools binman: Add a --toolpath option to set the tool search path binman: Add missing comments to bsection binman: Add missing comments toentry binman: Tidy up help for --indir binman: Use a better error for missing Intel descriptor binman: Detect skipped tests binman: Add a function to create a sample ELF file binman: Add a function to decode an ELF file binman: Ensure that coverage has access to site packages binman: Assume Intel descriptor is at the start of the image binman: Don't assume there is an ME region binman: Update entry.SetOffsetSize to be optional binman: Allow text directly in the node patman: Add functions to compress and decompress data binman: Use the tools.Decompress method binman: Drop unnecessary debug handling binman: Use tools compression function for blob handling binman: Correct comment in u_boot_spl_elf binman: Support ELF files for TPL binman: Fix up the _DoTestFile() function -u argument binman: Allow verbosity control when running tests binman: Allow preserving test directories binman: Pass the toolpath to tests patman: Add a function to write ifwitool binman: Add a utility library for coreboot CBFS binman: Add support for CBFS entries binman: Add support for Intel IFWI entries binman: Pad empty areas of the CBFS with files binman: Add support for fixed-offset files in CBFS binman: Simplify the entry test binman: Update future features binman: Update help for new features binman: Add a convenience functions for real-DTB tests binman: Add an FDT map binman: Add an image header binman: Convert to use ArgumentParser binman: Move compression into the Entry base class binman: Drop an unused arg in Entry.Lookup() binman: Allow easy importing of entry modules binman: Fix up ProcessUpdateContents error and comments binman: Call ProcessUpdateContents() consistently binman: Add a return value to ProcessContentsUpdate() binman: Add a control for post-pack entry expansion binman: Allow entries to expand after packing binman: Allow device-tree entries to be compressed binman: Provide the actual data address for cbfs files binman: Use the cbfs memlen field only for uncompressed length binman: Support FDT update for CBFS binman: Detect bad CBFS file types binman: Allow listing the entries in an image binman: Support locating an FDT map binman: Support locating an image header binman: Support reading an image into an Image object binman: Convert Image to a subclass of Entry binman: Support listing an image binman: Allow for logging information to be displayed binman: Allow reading an entry from an image binman: Support reading from CBFS entries binman: Add an 'extract' command binman: Add a
Re: [U-Boot] Please pull u-boot-dm
Hi Tom, On Tue, 23 Jul 2019 at 20:23, Tom Rini wrote: > > On Tue, Jul 23, 2019 at 08:08:03PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Tue, 23 Jul 2019 at 11:16, Tom Rini wrote: > > > > > > On Mon, Jul 22, 2019 at 08:48:45AM -0600, Simon Glass wrote: > > > > > > > Hi Tom, > > > > > > > > Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377 > > > > > > > > The following changes since commit > > > > 0de815356474912ef5bef9a69f0327a5a93bb2c2: > > > > > > > > Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) > > > > > > > > are available in the Git repository at: > > > > > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19 > > > > > > > > for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a: > > > > > > > > dm: device: make power domain calls optional (2019-07-20 19:50:44 > > > > -0600) > > > > > > > > > > First, in my LLVM-7 test: > > > /home/trini/u-boot/u-boot/tools/ifwitool.c:584:23: warning: unused > > > function 'read_le8' [-Wunused-function] > > > static inline uint8_t read_le8(const void *src) > > > ^ > > > /home/trini/u-boot/u-boot/tools/ifwitool.c:695:20: warning: unused > > > function 'zero_n' [-Wunused-function] > > > static inline void zero_n(void *dest, size_t n) > > > > > > Which is simple to fix. But I don't see why it's not being caught in > > > the travis test off-hand. > > > > I'm not sure either. Possible it depends on the compiler used? > > I'm concerned the travis job isn't running LLVM like we think it is. > > > > And I further see I didn't get LLVM-7 > > > included in GitLab at all yet. > > > > > > Next however I see: > > > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1222 > > > which is the binman tests failing and needs to be fixed. > > > > Oh dear, I don't seem to have any idea about gitlab yet. Have updated > > the gitlab config file. Should I stop using travis completely now? > > Long term I think GitLab CI is going to be better for custodians as > we'll have more build resources available and thus the jobs will > complete quicker. I don't know why the binman testsuite doesn't fail in > travis like it does there. Well that is because I updated the travis config but not the gitlab. Doing that. > The good news is the gitlab job runs in a > defined docker container you can pull and try locally :) OK good. Will give this all a crack at some point. 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-dm
On Tue, Jul 23, 2019 at 08:08:03PM -0700, Simon Glass wrote: > Hi Tom, > > On Tue, 23 Jul 2019 at 11:16, Tom Rini wrote: > > > > On Mon, Jul 22, 2019 at 08:48:45AM -0600, Simon Glass wrote: > > > > > Hi Tom, > > > > > > Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377 > > > > > > The following changes since commit > > > 0de815356474912ef5bef9a69f0327a5a93bb2c2: > > > > > > Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19 > > > > > > for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a: > > > > > > dm: device: make power domain calls optional (2019-07-20 19:50:44 -0600) > > > > > > > First, in my LLVM-7 test: > > /home/trini/u-boot/u-boot/tools/ifwitool.c:584:23: warning: unused function > > 'read_le8' [-Wunused-function] > > static inline uint8_t read_le8(const void *src) > > ^ > > /home/trini/u-boot/u-boot/tools/ifwitool.c:695:20: warning: unused function > > 'zero_n' [-Wunused-function] > > static inline void zero_n(void *dest, size_t n) > > > > Which is simple to fix. But I don't see why it's not being caught in > > the travis test off-hand. > > I'm not sure either. Possible it depends on the compiler used? I'm concerned the travis job isn't running LLVM like we think it is. > > And I further see I didn't get LLVM-7 > > included in GitLab at all yet. > > > > Next however I see: > > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1222 > > which is the binman tests failing and needs to be fixed. > > Oh dear, I don't seem to have any idea about gitlab yet. Have updated > the gitlab config file. Should I stop using travis completely now? Long term I think GitLab CI is going to be better for custodians as we'll have more build resources available and thus the jobs will complete quicker. I don't know why the binman testsuite doesn't fail in travis like it does there. The good news is the gitlab job runs in a defined docker container you can pull and try locally :) -- 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-dm
Hi Tom, On Tue, 23 Jul 2019 at 11:16, Tom Rini wrote: > > On Mon, Jul 22, 2019 at 08:48:45AM -0600, Simon Glass wrote: > > > Hi Tom, > > > > Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377 > > > > The following changes since commit 0de815356474912ef5bef9a69f0327a5a93bb2c2: > > > > Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19 > > > > for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a: > > > > dm: device: make power domain calls optional (2019-07-20 19:50:44 -0600) > > > > First, in my LLVM-7 test: > /home/trini/u-boot/u-boot/tools/ifwitool.c:584:23: warning: unused function > 'read_le8' [-Wunused-function] > static inline uint8_t read_le8(const void *src) > ^ > /home/trini/u-boot/u-boot/tools/ifwitool.c:695:20: warning: unused function > 'zero_n' [-Wunused-function] > static inline void zero_n(void *dest, size_t n) > > Which is simple to fix. But I don't see why it's not being caught in > the travis test off-hand. I'm not sure either. Possible it depends on the compiler used? > And I further see I didn't get LLVM-7 > included in GitLab at all yet. > > Next however I see: > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1222 > which is the binman tests failing and needs to be fixed. Oh dear, I don't seem to have any idea about gitlab yet. Have updated the gitlab config file. Should I stop using travis completely now? > > -- > 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-dm
On Mon, Jul 22, 2019 at 08:48:45AM -0600, Simon Glass wrote: > Hi Tom, > > Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377 > > The following changes since commit 0de815356474912ef5bef9a69f0327a5a93bb2c2: > > Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19 > > for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a: > > dm: device: make power domain calls optional (2019-07-20 19:50:44 -0600) > First, in my LLVM-7 test: /home/trini/u-boot/u-boot/tools/ifwitool.c:584:23: warning: unused function 'read_le8' [-Wunused-function] static inline uint8_t read_le8(const void *src) ^ /home/trini/u-boot/u-boot/tools/ifwitool.c:695:20: warning: unused function 'zero_n' [-Wunused-function] static inline void zero_n(void *dest, size_t n) Which is simple to fix. But I don't see why it's not being caught in the travis test off-hand. And I further see I didn't get LLVM-7 included in GitLab at all yet. Next however I see: https://gitlab.denx.de/u-boot/u-boot/-/jobs/1222 which is the binman tests failing and needs to be fixed. -- 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-dm
Hi Tom, Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377 The following changes since commit 0de815356474912ef5bef9a69f0327a5a93bb2c2: Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19 for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a: dm: device: make power domain calls optional (2019-07-20 19:50:44 -0600) Minor driver-model fixes and tweaks A few device-tree fixes Binman support for extracting files from an image Anatolij Gustschin (1): dm: device: make power domain calls optional Baruch Siach (2): dm: uclass: fix comment copy/paste error buildman: fix invocation examples typos Bin Meng (4): dm: timer: Skip device that does not have a valid ofnode in pre_probe() dm: core: Call clk_set_defaults() during probe() only for a valid ofnode dm: core: Set correct "status" value for a node dm: Fix parameter description of dev_read_name() Marek Vasut (1): common: fdt_support: Add missing cpu_to_fdt32() to fdt_pci_dma_ranges() Masahiro Yamada (1): fdt: make fdt_get_base_address() return OF_BAD_ADDR when "reg" not found Sekhar Nori (1): clk: initialize clk->data when using default xlate Simon Glass (68): x86: Add ifwitool for Intel Integrated Firmware Image cbfs: Add an enum and comment for the magic number cbfs: Rename checksum to attributes_offset tools: Drop duplicate raise_on_error argument binman: Fix comment in bsection.GetEntries() binman: Correct two typos in function names in ftest binman: Add coverage tools info for Python 3 patman: Add a way to set the search path for tools binman: Add a --toolpath option to set the tool search path binman: Add missing comments to bsection binman: Add missing comments toentry binman: Tidy up help for --indir binman: Use a better error for missing Intel descriptor binman: Detect skipped tests binman: Add a function to create a sample ELF file binman: Add a function to decode an ELF file binman: Ensure that coverage has access to site packages binman: Assume Intel descriptor is at the start of the image binman: Don't assume there is an ME region binman: Update entry.SetOffsetSize to be optional binman: Allow text directly in the node patman: Add functions to compress and decompress data binman: Use the tools.Decompress method binman: Drop unnecessary debug handling binman: Use tools compression function for blob handling binman: Correct comment in u_boot_spl_elf binman: Support ELF files for TPL binman: Fix up the _DoTestFile() function -u argument binman: Allow verbosity control when running tests binman: Allow preserving test directories binman: Pass the toolpath to tests patman: Add a function to write ifwitool binman: Add a utility library for coreboot CBFS binman: Add support for CBFS entries binman: Add support for Intel IFWI entries binman: Pad empty areas of the CBFS with files binman: Add support for fixed-offset files in CBFS binman: Simplify the entry test binman: Update future features binman: Update help for new features binman: Add a convenience functions for real-DTB tests binman: Add an FDT map binman: Add an image header binman: Convert to use ArgumentParser binman: Move compression into the Entry base class binman: Drop an unused arg in Entry.Lookup() binman: Allow easy importing of entry modules binman: Fix up ProcessUpdateContents error and comments binman: Call ProcessUpdateContents() consistently binman: Add a return value to ProcessContentsUpdate() binman: Add a control for post-pack entry expansion binman: Allow entries to expand after packing binman: Allow device-tree entries to be compressed binman: Provide the actual data address for cbfs files binman: Use the cbfs memlen field only for uncompressed length binman: Support FDT update for CBFS binman: Detect bad CBFS file types binman: Allow listing the entries in an image binman: Support locating an FDT map binman: Support locating an image header binman: Support reading an image into an Image object binman: Convert Image to a subclass of Entry binman: Support listing an image binman: Allow for logging information to be displayed binman: Allow reading an entry from an image binman: Support reading from CBFS entries binman: Add an 'extract' command binman: Add a test for nested and aligned sections .travis.yml
Re: [U-Boot] Please pull u-boot-dm
On Thu, Jul 11, 2019 at 11:18:43AM -0600, Simon Glass wrote: > Hi Tom, > > I picked up a different patch for the unicode handling. > > Passing build here: > https://travis-ci.org/sglass68/u-boot/builds/557041198 > > > The following changes since commit ef8b7e045ec744dce385cac4b1438c9be6e2bbc8: > > Merge https://gitlab.denx.de/u-boot/custodians/u-boot-i2c (2019-07-10 > 13:51:53 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-9jul19-take2 > > for you to fetch changes up to 7ea33579576d2bcd19df76bd8769e7ab3b4a169b: > > trace: trace buffer may exceed 2GiB (2019-07-10 16:52:58 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, I picked up a different patch for the unicode handling. Passing build here: https://travis-ci.org/sglass68/u-boot/builds/557041198 The following changes since commit ef8b7e045ec744dce385cac4b1438c9be6e2bbc8: Merge https://gitlab.denx.de/u-boot/custodians/u-boot-i2c (2019-07-10 13:51:53 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-9jul19-take2 for you to fetch changes up to 7ea33579576d2bcd19df76bd8769e7ab3b4a169b: trace: trace buffer may exceed 2GiB (2019-07-10 16:52:58 -0600) - Sandbox improvements including .dts refactor - Minor tracing and PCI improvements - Various other minor fixes - Conversion of patman, dtoc and binman to support Python 3 Alex Marginean (4): pci: fixed dm_pci_map_bar comment drivers: pci: add map_bar support for Enhanced Allocation test: dm: Add a test for PCI Enhanced Allocation drivers: pci: add API to issue FLR on a PCI function if supported Heinrich Schuchardt (5): trace: undefined reference to `trace_early_init' trace: conserve gd register trace: make call depth limit customizable trace: do not limit trace buffer to 2GiB trace: trace buffer may exceed 2GiB Patrick Delaunay (3): test: check u-boot properties in SPL device tree fdt: Allow indicating a node is for U-Boot proper only dm: doc: add documentation for pre-reloc properties in SPL and TPL Simon Glass (60): patman: Update cros_subprocess to use bytes patman: Convert print statements to Python 3 binman: Convert print statements to Python 3 binman: Don't show errors for failed tests binman: Remove use of Set() patman: Use items() instead of iteritems() binman: Use items() instead of iteritems() tools: binman: Open all binary files in binary mode tools: dtoc: Open all binary files in binary mode patman: Provide a way to get program output in binary mode binman: Use binary mode when compressing data binman: Drop an unused input file binman: Handle repeated bytes for Python 3 patman: Support use of stringIO in Python 3 patman: Move unicode helpers to tools patman: Sort series output for repeatabily patman: Avoid unicode type in settings unit tests patman: Adjust functional tests for Python 3 patman: Tidy up a few more unicode conversions patman: Don't require Python 2 dtoc: Adjust code for Python 3 dtoc: Sort platdata output from dtoc dtoc: Use GetBytes() to obtain repeating bytes dtoc: Move BytesToValue() out of the Prop class dtoc: Updates BytesToValue() for Python 3 dtoc: Use byte type instead of str in fdt dtoc: Convert the Fdt.Prop class to Python 3 dtoc: Convert the Fdt.Node class to Python 3 dtoc: Use binary mode for reading files dtoc: Test full 64-bit properties with FdtCellsToCpu() dtoc: Add a unit test for BytesToValue() dtoc: Update fdt_util for Python 3 dtoc: Update dtb_platdata to support Python 3 patman: Allow reading files in text mode binman: Avoid changing a dict during iteration binman: Convert to use bytes type binman: Update entry_test to support Python 3 patman: Update fmap code for Python 3 binman: Update 'text' entry for Python 3 binman: Fix up a format string in AssertInList() binman: Read map files as text binman: Document parallel tests binman: Update the README.entries file patman: Update cover-coverage tests for Python 3 Add a simple script to remove boards sandbox: Sync up sandbox64.dts with main DT sandbox: Create a common sandbox DT sandbox: Add an alias for SPI sandbox: Exit when SYSRESET_POWER_OFF is requested sandbox: Quit when hang() is called sandbox: spl: Lower priority of standard loader sandbox: Add a comment to board_init_f() sandbox: Allo sdl-config to be overridden sandbox: Add a memory {} node sandbox: Correct spi flash operation sandbox: Add documentation on how to run valgrind x86: Add a forward struct declaration in coreboot_tables.h bootstage: Add support for TPL record count blk: Allow control of the block cache in TPL spi: Avoid using malloc() in a critical function Thierry Reding (1): fdtdec: test: Fix memory leak Vabhav Sharma (1): drivers: serial: lpuart: Enable Little Endian Support arch/sandbox/config.mk | 5 +- arch/sandbox/cpu/spl.c | 3 +- arch/sandbox/cpu/state.c| 1 + arch/sandbox/dts/sandbox.dts| 336 +-- arch/sandbox/dts/sandbox.dtsi | 364 +++
Re: [U-Boot] Please pull u-boot-dm
Hi Tom, On Wed, 10 Jul 2019 at 08:39, Tom Rini wrote: > > On Tue, Jul 09, 2019 at 11:16:18AM -0600, Simon Glass wrote: > > > Hi Tom, > > > > Build is here: > > > > https://travis-ci.org/sglass68/u-boot/builds/556001438 > > > > (looks like there is a unicode issue I need to look at) > > > > The following changes since commit e5aee22e4be75e75a854ab64503fc80598bc2004: > > > > Prepare v2019.07 (2019-07-08 15:23:28 -0400) > > > > are available in the Git repository at: > > > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git tags/dm-pull-9jul19 > > > > for you to fetch changes up to b7eb801e0e6bdf42a592087a797deda1b0f03d69: > > > > trace: trace buffer may exceed 2GiB (2019-07-08 15:23:09 -0600) > > > > Since that unicode problem fails travis, I do need that fixed before I > can take this, thanks! Ah yes. I saw the timeout and thought it was some sort of infra problem, but of course it is buildman. I'll send a new email. 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-dm
On Tue, Jul 09, 2019 at 11:16:18AM -0600, Simon Glass wrote: > Hi Tom, > > Build is here: > > https://travis-ci.org/sglass68/u-boot/builds/556001438 > > (looks like there is a unicode issue I need to look at) > > The following changes since commit e5aee22e4be75e75a854ab64503fc80598bc2004: > > Prepare v2019.07 (2019-07-08 15:23:28 -0400) > > are available in the Git repository at: > > https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git tags/dm-pull-9jul19 > > for you to fetch changes up to b7eb801e0e6bdf42a592087a797deda1b0f03d69: > > trace: trace buffer may exceed 2GiB (2019-07-08 15:23:09 -0600) > Since that unicode problem fails travis, I do need that fixed before I can take this, thanks! -- 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-dm
Hi Tom, Build is here: https://travis-ci.org/sglass68/u-boot/builds/556001438 (looks like there is a unicode issue I need to look at) The following changes since commit e5aee22e4be75e75a854ab64503fc80598bc2004: Prepare v2019.07 (2019-07-08 15:23:28 -0400) are available in the Git repository at: https://gitlab.denx.de/u-boot/custodians/u-boot-dm.git tags/dm-pull-9jul19 for you to fetch changes up to b7eb801e0e6bdf42a592087a797deda1b0f03d69: trace: trace buffer may exceed 2GiB (2019-07-08 15:23:09 -0600) - Sandbox improvements including .dts refactor - Minor tracing and PCI improvements - Various other minor fixes - Conversion of patman, dtoc and binman to support Python 3 Alex Marginean (4): pci: fixed dm_pci_map_bar comment drivers: pci: add map_bar support for Enhanced Allocation test: dm: Add a test for PCI Enhanced Allocation drivers: pci: add API to issue FLR on a PCI function if supported Heinrich Schuchardt (5): trace: undefined reference to `trace_early_init' trace: conserve gd register trace: make call depth limit customizable trace: do not limit trace buffer to 2GiB trace: trace buffer may exceed 2GiB Patrick Delaunay (3): test: check u-boot properties in SPL device tree fdt: Allow indicating a node is for U-Boot proper only dm: doc: add documentation for pre-reloc properties in SPL and TPL Simon Glass (60): patman: Update cros_subprocess to use bytearray patman: Convert print statements to Python 3 binman: Convert print statements to Python 3 binman: Don't show errors for failed tests binman: Remove use of Set() patman: Use items() instead of iteritems() binman: Use items() instead of iteritems() tools: binman: Open all binary files in binary mode tools: dtoc: Open all binary files in binary mode patman: Provide a way to get program output in binary mode binman: Use binary mode when compressing data binman: Drop an unused input file binman: Handle repeated bytes for Python 3 patman: Support use of stringIO in Python 3 patman: Move unicode helpers to tools patman: Sort series output for repeatabily patman: Avoid unicode type in settings unit tests patman: Adjust functional tests for Python 3 patman: Tidy up a few more unicode conversions patman: Don't require Python 2 dtoc: Adjust code for Python 3 dtoc: Sort platdata output from dtoc dtoc: Use GetBytes() to obtain repeating bytes dtoc: Move BytesToValue() out of the Prop class dtoc: Updates BytesToValue() for Python 3 dtoc: Use byte type instead of str in fdt dtoc: Convert the Fdt.Prop class to Python 3 dtoc: Convert the Fdt.Node class to Python 3 dtoc: Use binary mode for reading files dtoc: Test full 64-bit properties with FdtCellsToCpu() dtoc: Add a unit test for BytesToValue() dtoc: Update fdt_util for Python 3 dtoc: Update dtb_platdata to support Python 3 patman: Allow reading files in text mode binman: Avoid changing a dict during iteration binman: Convert to use bytes type binman: Update entry_test to support Python 3 patman: Update fmap code for Python 3 binman: Update 'text' entry for Python 3 binman: Fix up a format string in AssertInList() binman: Read map files as text binman: Document parallel tests binman: Update the README.entries file patman: Update cover-coverage tests for Python 3 Add a simple script to remove boards sandbox: Sync up sandbox64.dts with main DT sandbox: Create a common sandbox DT sandbox: Add an alias for SPI sandbox: Exit when SYSRESET_POWER_OFF is requested sandbox: Quit when hang() is called sandbox: spl: Lower priority of standard loader sandbox: Add a comment to board_init_f() sandbox: Allo sdl-config to be overridden sandbox: Add a memory {} node sandbox: Correct spi flash operation sandbox: Add documentation on how to run valgrind x86: Add a forward struct declaration in coreboot_tables.h bootstage: Add support for TPL record count blk: Allow control of the block cache in TPL spi: Avoid using malloc() in a critical function Thierry Reding (1): fdtdec: test: Fix memory leak Vabhav Sharma (1): drivers: serial: lpuart: Enable Little Endian Support arch/sandbox/config.mk | 5 +- arch/sandbox/cpu/spl.c | 3 +- arch/sandbox/cpu/state.c| 1 + arch/sandbox/dts/sandbox.dts| 336 ++--- arch/sandbox/dts/sandbox.dtsi | 364 ++
Re: [U-Boot] Please pull u-boot-dm (wandboard fix)
On Sun, Jul 07, 2019 at 07:17:42AM -0600, Simon Glass wrote: > Hi Tom, > > Sorry if this is too late, but I still have this patch pending. > > > The following changes since commit 1f83431f0053f6fb20c511c391ffc687433848cf: > > board: amlogic: add mailing-list to MAINTAINERS (2019-07-04 11:36:52 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-7jul19 > > for you to fetch changes up to c2f3dade2a8efb90c20b46b3c76a81a15fafe96e: > > imx6: wandboard: allow booting from MMC 2 (2019-07-07 07:13:49 -0600) > Applied to u-boot/master, thanks! -- 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-dm (wandboard fix)
Hi Tom, Sorry if this is too late, but I still have this patch pending. The following changes since commit 1f83431f0053f6fb20c511c391ffc687433848cf: board: amlogic: add mailing-list to MAINTAINERS (2019-07-04 11:36:52 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-7jul19 for you to fetch changes up to c2f3dade2a8efb90c20b46b3c76a81a15fafe96e: imx6: wandboard: allow booting from MMC 2 (2019-07-07 07:13:49 -0600) Fix booting for wandboard Heinrich Schuchardt (1): imx6: wandboard: allow booting from MMC 2 include/configs/wandboard.h | 1 + 1 file changed, 1 insertion(+) 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-dm
On Wed, May 22, 2019 at 07:13:24AM -0600, Simon Glass wrote: > Hi Tom, > > Results here: > > https://travis-ci.org/sglass68/u-boot/builds/535552345 > > The following changes since commit e1a2ed7180adeefb6164239a18249dca5701319d: > > Merge git://git.denx.de/u-boot-mpc83xx (2019-05-21 07:13:35 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-22may19 > > for you to fetch changes up to ee730a7cd2afe445e53e92b5c37396a5b118f7af: > > dm: core: Fix dm_extended_scan_fdt() (2019-05-21 17:33:23 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, Results here: https://travis-ci.org/sglass68/u-boot/builds/535552345 The following changes since commit e1a2ed7180adeefb6164239a18249dca5701319d: Merge git://git.denx.de/u-boot-mpc83xx (2019-05-21 07:13:35 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-22may19 for you to fetch changes up to ee730a7cd2afe445e53e92b5c37396a5b118f7af: dm: core: Fix dm_extended_scan_fdt() (2019-05-21 17:33:23 -0600) Various DM fixes Addition of ofnode_get_addr_size_index() Keerthy (1): core: ofnode: Add ofnode_get_addr_size_index Patrice Chotard (1): dm: core: Fix dm_extended_scan_fdt() Simon Glass (3): spl: misc: Allow misc drivers in SPL and TPL Add an empty stdint.h file buildman: Deal more nicely with invalid build-status file Stefan Mavrodiev (1): common: fdt_support: Check mtdparts cell size Thierry Reding (1): fdtdec: Remove fdt_{addr,size}_unpack() Trent Piepho (1): core: ofnode: Have ofnode_read_u32_default return a u32 common/fdt_support.c | 31 ++-- drivers/core/ofnode.c | 15 +++--- drivers/core/root.c | 2 +- drivers/misc/Kconfig | 72 +++ drivers/misc/Makefile | 8 -- include/dm/ofnode.h | 16 ++- include/fdtdec.h | 24 include/stdint.h | 7 + lib/fdtdec.c | 8 -- lib/fdtdec_test.c | 8 -- tools/buildman/builder.py | 7 - 11 files changed, 151 insertions(+), 47 deletions(-) create mode 100644 include/stdint.h 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-dm
On Wed, Apr 24, 2019 at 09:22:43AM -0600, Simon Glass wrote: > Hi Tom, > > Build/test: > https://travis-ci.org/sglass68/u-boot/builds/523770406 > > > The following changes since commit 6c5f8dd540d7a8eff244d4c27a09451ca12c8d20: > > Merge branch 'master' of git://git.denx.de/u-boot-usb (2019-04-21 > 19:00:04 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-24apr19 > > for you to fetch changes up to 8781d04f422e110fef864dd849085054fe5b0e65: > > pci: pci.h: add missing maskbit (2019-04-23 20:26:43 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, Build/test: https://travis-ci.org/sglass68/u-boot/builds/523770406 The following changes since commit 6c5f8dd540d7a8eff244d4c27a09451ca12c8d20: Merge branch 'master' of git://git.denx.de/u-boot-usb (2019-04-21 19:00:04 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-24apr19 for you to fetch changes up to 8781d04f422e110fef864dd849085054fe5b0e65: pci: pci.h: add missing maskbit (2019-04-23 20:26:43 -0600) Various minor sandbox iumprovements Fixes for tracing with sandbox Refactoring for boot_get_fdt() Christoph Muellner (1): common: command: Add command execution tracer. Eugeniu Rosca (5): fdt: boot_get_fdt: remove redundant zeroing out fdt: boot_get_fdt: really boot w/o FDT when "goto no_fdt" fdt: boot_get_fdt: simplify no_fdt handling (non-functional) fdt: boot_get_fdt: android: compress handling (non-functional) fdt: boot_get_fdt: android: use ENV 'fdtaddr' as fallback Lukas Auer (1): simple-bus: add DM_FLAG_PRE_RELOC flag to simple-bus driver Ramon Fried (1): pci: pci.h: add missing maskbit Simon Glass (13): sandbox: Improve debugging in initcall_run_list() sandbox: Correct maths in allocation routines sandbox: Drop the printf() in setup_ram_buf() sandbox: Move pre-console buffer out of the way of tracing bootstage: Allow calling bootstage_mark() before bootstage_init() sandbox: Increase the early-trace-buffer size initcall: Drop use of header files div64: Use kernel types div64: Don't instrument the division function trace: Tidy up error returns Convert CONFIG_TRACE to Kconfig Convert CONFIG_TRACE_BUFFER_SIZE et al to Kconfig sandbox: Enable the 'trace' command when tracing is used Stefan Roese (1): dm: core: Change platform specific translation-offset handling Thierry Reding (1): fdtdec: Use fdt_setprop_u32() for fdtdec_set_phandle() arch/arm/mach-mvebu/Kconfig| 1 + arch/arm/mach-mvebu/spl.c | 12 +++--- arch/sandbox/cpu/os.c | 59 + arch/sandbox/cpu/start.c | 16 +- arch/sandbox/include/asm/global_data.h | 1 + board/sandbox/README.sandbox | 46 +++ cmd/Kconfig| 13 ++- common/board_f.c | 2 +- common/bootstage.c | 7 ++ common/command.c | 14 common/image-fdt.c | 33 ++-- configs/sandbox_defconfig | 2 +- drivers/core/Kconfig | 9 drivers/core/fdtaddr.c | 9 +--- drivers/core/root.c| 21 -- drivers/core/simple-bus.c | 1 + include/asm-generic/global_data.h | 4 include/configs/sandbox.h | 4 ++-- include/div64.h| 70 +-- include/dm/fdtaddr.h | 21 -- include/fdtdec.h | 5 - include/initcall.h | 19 +--- include/os.h | 11 ++ include/pci.h | 1 + lib/Kconfig| 57 lib/div64.c| 20 +++-- lib/fdtdec.c | 7 -- lib/trace.c| 17 ++- scripts/config_whitelist.txt | 5 - 29 files changed, 335 insertions(+), 152 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-dm
On Fri, Apr 12, 2019 at 03:51:52PM -0600, Simon Glass wrote: > Hi Tom, > > Build result at https://travis-ci.org/sglass68/u-boot/builds/519059521 > > > The following changes since commit 02f173ca156cee8526dff87603d5e446b443cde3: > > Merge branch 'master' of git://git.denx.de/u-boot-usb (2019-04-11 > 14:29:37 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-12apr19 > > for you to fetch changes up to 73c02e5e4fc1ef53d06289232edd6cc52e3d73f6: > > fdt: Fix mkimage list to try every header type (2019-04-11 20:10:50 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, Build result at https://travis-ci.org/sglass68/u-boot/builds/519059521 The following changes since commit 02f173ca156cee8526dff87603d5e446b443cde3: Merge branch 'master' of git://git.denx.de/u-boot-usb (2019-04-11 14:29:37 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-12apr19 for you to fetch changes up to 73c02e5e4fc1ef53d06289232edd6cc52e3d73f6: fdt: Fix mkimage list to try every header type (2019-04-11 20:10:50 -0600) fdtdec tests and improvements for carve-outs pinctrl race-condition fix various other fixes in sandbox, sound, mkimage, etc. Christian Gmeiner (1): dm: sound: make all functions static inline Christoph Muellner (1): dm: pinctrl: Remove obsolete function pinctrl_decode_pin_config_dm(). Eugeniu Rosca (1): core: ofnode: Fix ASAN-reported stack-buffer-overflow in of_get_address Jordan Hand (1): fdt: Fix mkimage list to try every header type Michal Simek (1): dm: Also remove interrupts property from SPL DT Patrice Chotard (1): dm: pinctrl: Avoid race condition on probe for UCLASS_PINCTRL Patrick Delaunay (7): dm: remove pre reloc properties in SPL and TPL device tree dm: pinctrl: Skip gpio-controller node in pinconfig_post_bind() syscon: update syscon_regmap_lookup_by_phandle sysreset: use syscon_regmap_lookup_by_phandle clk: at91: replace dm_fdt_pre_reloc by dm_ofnode_pre_reloc clk: socfpga: replace dm_fdt_pre_reloc by dm_ofnode_pre_reloc dm: remove unused function dm_fdt_pre_reloc Simon Glass (1): cros: Expand the Chromium OS documentation Thierry Reding (15): fdt: Remove duplicate code vsprintf: Support phys_addr_t specifier unconditionally sandbox: Use correct phys_{addr, size}_t for PHYS_64BIT=y sandbox: Properly print physical addresses libfdt: Add phandle generation helper fdtdec: Add cpu_to_fdt_{addr, size}() macros fdtdec: Add fdt_{addr, size}_unpack() helpers fdtdec: Implement fdtdec_set_phandle() fdtdec: Implement fdtdec_add_reserved_memory() fdtdec: Implement carveout support functions fdtdec: Add Kconfig symbol for tests fdtdec: test: Fix build warning fdtdec: test: Use compound statement macros fdtdec: test: Add carveout tests sandbox: Enable fdtdec tests .../devicetree/bindings/reserved-memory/reserved-memory.txt| 136 ++ arch/sandbox/dts/test.dts | 3 +- arch/sandbox/include/asm/sdl.h | 4 +- arch/sandbox/include/asm/types.h | 12 +- arch/sandbox/lib/pci_io.c | 2 +- common/fdt_support.c | 6 - configs/sandbox64_defconfig| 1 + configs/sandbox_defconfig | 1 + doc/README.chromium| 296 -- doc/README.chromium-chainload | 239 drivers/clk/altera/clk-arria10.c | 3 +- drivers/clk/at91/pmc.c | 2 +- drivers/core/ofnode.c | 21 +-- drivers/core/syscon-uclass.c | 83 ++--- drivers/core/util.c| 40 +--- drivers/pinctrl/pinctrl-uclass.c | 36 +--- drivers/sysreset/sysreset_syscon.c | 15 +- dts/Kconfig| 8 +- include/dm/pinctrl.h | 12 -- include/dm/util.h | 26 --- include/fdtdec.h | 169 + lib/Kconfig| 4 + lib/fdtdec.c | 225 +++ lib/fdtdec_test.c | 216 +- lib/libfdt/fdt_ro.c| 31 lib/vsprintf.c | 2 +- scripts/Makefile.lib | 1 + scripts/dtc/libfdt/fdt_ro.c| 31 scripts/dtc/libfdt/libfdt.h| 19 ++ scripts/dtc/libfdt/libfdt_env.h| 1 + test/dm/syscon.c
Re: [U-Boot] Please pull u-boot-dm
On Sun, Feb 10, 2019 at 03:27:56AM -0600, Simon Glass wrote: > Hi Tom, > > I thought I had already sent these but just noticed that I had not. > > The following changes since commit 97276a91db8e98f081a40ddf9dc8f81d4032a756: > > Prepare v2019.04-rc1 (2019-02-07 21:32:19 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-10feb19 > > for you to fetch changes up to afcd64579486e1745a3cb964bfe36267d90b4c3e: > > sound: Allow audio codecs to be used by other SoCs (2019-02-09 12:50:22 > -0700) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, On Sun, 10 Feb 2019 at 10:28, Simon Glass wrote: > > +Tom > > On Sun, 10 Feb 2019 at 03:27, Simon Glass wrote: > > > > Hi Tom, > > > > I thought I had already sent these but just noticed that I had not. > > > > The following changes since commit 97276a91db8e98f081a40ddf9dc8f81d4032a756: Also the build result is here: https://travis-ci.org/sglass68/u-boot/builds/491045229 [..] 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-dm
+Tom On Sun, 10 Feb 2019 at 03:27, Simon Glass wrote: > > Hi Tom, > > I thought I had already sent these but just noticed that I had not. > > The following changes since commit 97276a91db8e98f081a40ddf9dc8f81d4032a756: > > Prepare v2019.04-rc1 (2019-02-07 21:32:19 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-10feb19 > > for you to fetch changes up to afcd64579486e1745a3cb964bfe36267d90b4c3e: > > sound: Allow audio codecs to be used by other SoCs (2019-02-09 12:50:22 > -0700) > > > Samsung sound patches (applied for Samsung maintainer) > Common sound support > buildman environment support > of-platdata documentation improvements > > > Chris Packham (1): > buildman: fix typo > > Lokesh Vutla (3): > Revert "power: regulator: Return success on attempt to disable > an always-on regulator" > power: regulator: Introduce regulator_set_enable_if_allowed api > mmc: omap_hsmmc: Use regulator_set_enable_if_allowed for > enabling regulator > > Simon Glass (15): > Convert CONFIG_BOARD_TYPES to Kconfig > exynos: Drop duplicate 'model' line > exynos: Convert to use CONFIG_BLK > sound: Add a driver for max98088 > spring: Update sound to use max98088 codec > spring: Update flashmap details > samsung: Drop board_enable_audio_codec() > samsung: mmc: Drop old MMC init code > fdt: samsung: Drop unused fdt_compat_id values > fdt: tegra: Drop COMPAT_AMS_AS3722 > buildman: Write the environment out to an 'env' file > misc: Allow child devices > sound: samsung: Fix 'regiter' typo > sound: i2s: Tidy up a few comments > sound: Allow audio codecs to be used by other SoCs > > Simon Goldschmidt (1): > of-platdata: improve documentation > > arch/arm/dts/exynos5250-spring.dts | 77 ++--- > arch/arm/mach-exynos/Kconfig | 7 + > arch/arm/mach-exynos/include/mach/mmc.h| 2 - > arch/arm/mach-s5pc1xx/Kconfig | 2 + > arch/arm/mach-s5pc1xx/Makefile | 1 + > arch/arm/mach-s5pc1xx/pinmux.c | 20 +++ > board/samsung/common/board.c | 52 +-- > board/samsung/common/exynos5-dt.c | 29 > common/Kconfig | 8 + > configs/axs101_defconfig | 1 + > configs/axs103_defconfig | 1 + > configs/odroid-xu3_defconfig | 1 + > configs/odroid_defconfig | 1 + > configs/spring_defconfig | 1 + > doc/driver-model/of-plat.txt | 31 ++-- > drivers/misc/misc-uclass.c | 3 + > drivers/mmc/omap_hsmmc.c | 16 +- > drivers/mmc/s5p_sdhci.c| 50 -- > drivers/power/regulator/regulator-uclass.c | 13 +- > drivers/sound/Kconfig | 14 +- > drivers/sound/Makefile | 1 + > drivers/sound/max98088.c | 424 > ++ > drivers/sound/max98088.h | 192 +++ > drivers/sound/max98090.c | 7 - > drivers/sound/max98095.c | 7 - > drivers/sound/maxim_codec.c| 3 - > drivers/sound/samsung-i2s.c| 16 +- > drivers/sound/samsung_sound.c | 6 +- > dts/Kconfig| 6 +- > include/configs/axs10x.h | 5 - > include/configs/hsdk.h | 5 - > include/configs/odroid.h | 1 - > include/configs/odroid_xu3.h | 1 - > include/fdtdec.h | 7 - > include/i2s.h | 11 +- > include/power/regulator.h | 11 ++ > include/samsung/misc.h | 2 - > lib/fdtdec.c | 7 - > scripts/config_whitelist.txt | 1 - > test/dm/regulator.c| 21 +++ > tools/buildman/builderthread.py| 3 + > tools/buildman/control.py | 2 +- > 42 files changed, 829 insertions(+), 240 deletions(-) > create mode 100644 arch/arm/mach-s5pc1xx/pinmux.c > create mode 100644 drivers/sound/max98088.c > create mode 100644 drivers/sound/max98088.h > > Regards, > SImon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] Please pull u-boot-dm
Hi Tom, I thought I had already sent these but just noticed that I had not. The following changes since commit 97276a91db8e98f081a40ddf9dc8f81d4032a756: Prepare v2019.04-rc1 (2019-02-07 21:32:19 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-10feb19 for you to fetch changes up to afcd64579486e1745a3cb964bfe36267d90b4c3e: sound: Allow audio codecs to be used by other SoCs (2019-02-09 12:50:22 -0700) Samsung sound patches (applied for Samsung maintainer) Common sound support buildman environment support of-platdata documentation improvements Chris Packham (1): buildman: fix typo Lokesh Vutla (3): Revert "power: regulator: Return success on attempt to disable an always-on regulator" power: regulator: Introduce regulator_set_enable_if_allowed api mmc: omap_hsmmc: Use regulator_set_enable_if_allowed for enabling regulator Simon Glass (15): Convert CONFIG_BOARD_TYPES to Kconfig exynos: Drop duplicate 'model' line exynos: Convert to use CONFIG_BLK sound: Add a driver for max98088 spring: Update sound to use max98088 codec spring: Update flashmap details samsung: Drop board_enable_audio_codec() samsung: mmc: Drop old MMC init code fdt: samsung: Drop unused fdt_compat_id values fdt: tegra: Drop COMPAT_AMS_AS3722 buildman: Write the environment out to an 'env' file misc: Allow child devices sound: samsung: Fix 'regiter' typo sound: i2s: Tidy up a few comments sound: Allow audio codecs to be used by other SoCs Simon Goldschmidt (1): of-platdata: improve documentation arch/arm/dts/exynos5250-spring.dts | 77 ++--- arch/arm/mach-exynos/Kconfig | 7 + arch/arm/mach-exynos/include/mach/mmc.h| 2 - arch/arm/mach-s5pc1xx/Kconfig | 2 + arch/arm/mach-s5pc1xx/Makefile | 1 + arch/arm/mach-s5pc1xx/pinmux.c | 20 +++ board/samsung/common/board.c | 52 +-- board/samsung/common/exynos5-dt.c | 29 common/Kconfig | 8 + configs/axs101_defconfig | 1 + configs/axs103_defconfig | 1 + configs/odroid-xu3_defconfig | 1 + configs/odroid_defconfig | 1 + configs/spring_defconfig | 1 + doc/driver-model/of-plat.txt | 31 ++-- drivers/misc/misc-uclass.c | 3 + drivers/mmc/omap_hsmmc.c | 16 +- drivers/mmc/s5p_sdhci.c| 50 -- drivers/power/regulator/regulator-uclass.c | 13 +- drivers/sound/Kconfig | 14 +- drivers/sound/Makefile | 1 + drivers/sound/max98088.c | 424 ++ drivers/sound/max98088.h | 192 +++ drivers/sound/max98090.c | 7 - drivers/sound/max98095.c | 7 - drivers/sound/maxim_codec.c| 3 - drivers/sound/samsung-i2s.c| 16 +- drivers/sound/samsung_sound.c | 6 +- dts/Kconfig| 6 +- include/configs/axs10x.h | 5 - include/configs/hsdk.h | 5 - include/configs/odroid.h | 1 - include/configs/odroid_xu3.h | 1 - include/fdtdec.h | 7 - include/i2s.h | 11 +- include/power/regulator.h | 11 ++ include/samsung/misc.h | 2 - lib/fdtdec.c | 7 - scripts/config_whitelist.txt | 1 - test/dm/regulator.c| 21 +++ tools/buildman/builderthread.py| 3 + tools/buildman/control.py | 2 +- 42 files changed, 829 insertions(+), 240 deletions(-) create mode 100644 arch/arm/mach-s5pc1xx/pinmux.c create mode 100644 drivers/sound/max98088.c create mode 100644 drivers/sound/max98088.h 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-dm
On Tue, Jan 15, 2019 at 08:57:23AM -0700, Simon Glass wrote: > Hi Tom, > > https://travis-ci.org/sglass68/u-boot/builds > > The following changes since commit d3689267f92c5956e09cc7d1baa4700141662bff: > > Prepare v2019.01 (2019-01-14 17:02:36 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-15jan19 > > for you to fetch changes up to f51f6715a5013f37620c38f0430e21d4736e235a: > > lib: fdtdec: fdtdec_get_addr_size_fixed remove checks (2019-01-14 > 17:47:20 -0700) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, https://travis-ci.org/sglass68/u-boot/builds The following changes since commit d3689267f92c5956e09cc7d1baa4700141662bff: Prepare v2019.01 (2019-01-14 17:02:36 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-15jan19 for you to fetch changes up to f51f6715a5013f37620c38f0430e21d4736e235a: lib: fdtdec: fdtdec_get_addr_size_fixed remove checks (2019-01-14 17:47:20 -0700) Fix recent changes to serial API for driver model Buildman clang support and a few fixes Small fixes to 'dm tree' and regmap test Improve sandbox build compatibility A few other minor fixes Christian GMEINER (1): sandbox: add memset_io(..), memcpy_fromio(..) and memcpy_toio(..) Heinrich Schuchardt (1): sandbox: i2c_emul_find() No emulators for device 'rtc@43' Keerthy (1): lib: fdtdec: fdtdec_get_addr_size_fixed remove checks Sekhar Nori (1): common: fdt_support: print hexadecimal numbers in debug Simon Glass (16): buildman: Drop comment about Ctrl-C problem dm: Tidy up 'dm tree' output when there are many devices test: dm: regmap: Fix the long test delay buildman: Deal nicely with invalid build-status file sandbox: Correct SDL build flags efi_loader: Add a wchar_t cast in efi_file_open() log: Check printf() arguments buildman: Add support for building with clang travis: Use buildman for building with clang net: Fix error handling in sb_eth_raw_ofdata_to_platdata() buildman: Fix tabs in GetWrapper() serial: Move new functions to serial.h dm: serial: Adjust serial_getconfig() to use proper API dm: serial: Adjust serial_setconfig() to use proper API dm: serial: Adjust serial_getinfo() to use proper API dm: serial: Tidy up header file comments .travis.yml | 13 +++-- arch/sandbox/config.mk | 2 -- arch/sandbox/cpu/sdl.c | 2 +- arch/sandbox/dts/sandbox.dts| 10 -- arch/sandbox/dts/sandbox64.dts | 6 -- arch/sandbox/include/asm/io.h | 12 arch/x86/lib/acpi_table.c | 12 common/fdt_support.c| 4 ++-- drivers/core/dump.c | 4 ++-- drivers/net/sandbox-raw.c | 9 - drivers/serial/serial-uclass.c | 27 +-- include/common.h| 5 - include/log.h | 3 ++- include/regmap.h| 14 +- include/serial.h| 38 +++--- lib/efi_loader/efi_file.c | 4 ++-- lib/fdtdec.c| 10 -- test/dm/regmap.c| 7 --- test/dm/serial.c| 19 +++ test/py/tests/test_bind.py | 3 ++- tools/buildman/README | 12 ++-- tools/buildman/builderthread.py | 8 +++- tools/buildman/cmdline.py | 2 ++ tools/buildman/control.py | 2 +- tools/buildman/toolchain.py | 42 -- 25 files changed, 174 insertions(+), 96 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-dm
Hi Tom, On Fri, 4 Jan 2019 at 22:00, Tom Rini wrote: > > On Fri, Jan 04, 2019 at 01:47:08PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > https://travis-ci.org/sglass68/u-boot/builds/474968097 > > > > > > The following changes since commit f97c49d6a2f504d0f0a8aab37c67aa314e006250: > > > > Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-01-01 > > 19:55:05 -0500) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/dm-pull-4jan19 > > > > for you to fetch changes up to 1b4770dfe03b5dd1dfd2a6076c4c85ee566e360a: > > > > sandbox: Correct SDL build flags (2019-01-03 11:06:07 -0700) > > So, the clang switch doesn't catch warnings anymore. While travis is > clean, my local hand-build shows: > /home/trini/u-boot/u-boot/lib/efi_loader/efi_file.c:224:61: warning: format > specifies type 'wchar_t *' (aka 'unsigned short *') but the argument has type > 's16 *' (aka 'short *') [-Wformat] > EFI_ENTRY("%p, %p, \"%ls\", %llx, %llu", file, new_handle, file_name, > ~~~ ^ > /home/trini/u-boot/u-boot/include/efi_loader.h:40:15: note: expanded from > macro 'EFI_ENTRY' > __func__, ##__VA_ARGS__); \ > ^~~ > /home/trini/u-boot/u-boot/include/log.h:164:28: note: expanded from macro > 'debug' > debug_cond(_DEBUG, fmt, ##args) > ^~~~ > /home/trini/u-boot/u-boot/include/log.h:144:41: note: expanded from macro > 'debug_cond' > log(LOG_CATEGORY, LOGL_DEBUG, fmt, ##args); \ > ^~~~ > /home/trini/u-boot/u-boot/include/log.h:121:25: note: expanded from macro > 'log' > pr_fmt(_fmt), ##_args); \ > ^ > 1 warning generated. > > So the log changes have a problem, and the buildman integration doesn't > promote warnings right :( Actually this uncovered a problem that I had not noticed before. I sent a new series to take care of it. But given that today is release day, I am unfortunately a little late! 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-dm
On Fri, Jan 04, 2019 at 01:47:08PM -0700, Simon Glass wrote: > Hi Tom, > > https://travis-ci.org/sglass68/u-boot/builds/474968097 > > > The following changes since commit f97c49d6a2f504d0f0a8aab37c67aa314e006250: > > Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-01-01 > 19:55:05 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-4jan19 > > for you to fetch changes up to 1b4770dfe03b5dd1dfd2a6076c4c85ee566e360a: > > sandbox: Correct SDL build flags (2019-01-03 11:06:07 -0700) So, the clang switch doesn't catch warnings anymore. While travis is clean, my local hand-build shows: /home/trini/u-boot/u-boot/lib/efi_loader/efi_file.c:224:61: warning: format specifies type 'wchar_t *' (aka 'unsigned short *') but the argument has type 's16 *' (aka 'short *') [-Wformat] EFI_ENTRY("%p, %p, \"%ls\", %llx, %llu", file, new_handle, file_name, ~~~ ^ /home/trini/u-boot/u-boot/include/efi_loader.h:40:15: note: expanded from macro 'EFI_ENTRY' __func__, ##__VA_ARGS__); \ ^~~ /home/trini/u-boot/u-boot/include/log.h:164:28: note: expanded from macro 'debug' debug_cond(_DEBUG, fmt, ##args) ^~~~ /home/trini/u-boot/u-boot/include/log.h:144:41: note: expanded from macro 'debug_cond' log(LOG_CATEGORY, LOGL_DEBUG, fmt, ##args); \ ^~~~ /home/trini/u-boot/u-boot/include/log.h:121:25: note: expanded from macro 'log' pr_fmt(_fmt), ##_args); \ ^ 1 warning generated. So the log changes have a problem, and the buildman integration doesn't promote warnings right :( -- 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-dm
Hi Tom, https://travis-ci.org/sglass68/u-boot/builds/474968097 The following changes since commit f97c49d6a2f504d0f0a8aab37c67aa314e006250: Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-01-01 19:55:05 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-4jan19 for you to fetch changes up to 1b4770dfe03b5dd1dfd2a6076c4c85ee566e360a: sandbox: Correct SDL build flags (2019-01-03 11:06:07 -0700) Buildman clang support and a few fixes Small fixes to 'dm tree' and regmap test Improve sandbox build compatibility Christian GMEINER (1): sandbox: add memset_io(..), memcpy_fromio(..) and memcpy_toio(..) Sekhar Nori (1): common: fdt_support: print hexadecimal numbers in debug Simon Glass (8): log: Check printf() arguments buildman: Drop comment about Ctrl-C problem buildman: Add support for building with clang travis: Use buildman for building with clang dm: Tidy up 'dm tree' output when there are many devices test: dm: regmap: Fix the long test delay buildman: Deal nicely with invalid build-status file sandbox: Correct SDL build flags .travis.yml | 13 +++-- arch/sandbox/config.mk | 2 -- arch/sandbox/cpu/sdl.c | 2 +- arch/sandbox/include/asm/io.h | 12 common/fdt_support.c| 4 ++-- drivers/core/dump.c | 4 ++-- include/log.h | 3 ++- include/regmap.h| 14 +- test/dm/regmap.c| 7 --- test/py/tests/test_bind.py | 3 ++- tools/buildman/README | 12 ++-- tools/buildman/builderthread.py | 7 ++- tools/buildman/cmdline.py | 2 ++ tools/buildman/control.py | 2 +- tools/buildman/toolchain.py | 18 ++ 15 files changed, 74 insertions(+), 31 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-dm
On Fri, Dec 14, 2018 at 08:43:09AM -0700, Simon Glass wrote: > Hi Tom, > > See here (the one failure seems to be a network error) > https://travis-ci.org/sglass68/u-boot/builds/46333 Those are annoying and I wish it would auto-restart those jobs since it should be able to catch that as the failure most of the time. You can restart individual tasks BTW, I do that as needed. > > > The following changes since commit d117d8f19b0625f88309e47a8a32c2faa384dddc: > > Merge branch 'master' of git://git.denx.de/u-boot-i2c (2018-12-13 > 09:36:55 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-14dec18 > > for you to fetch changes up to f987177db9c988142032ed8142a093cce2378a90: > > dm: sound: Use the correct number of channels for sound (2018-12-13 > 16:37:10 -0700) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, See here (the one failure seems to be a network error) https://travis-ci.org/sglass68/u-boot/builds/46333 The following changes since commit d117d8f19b0625f88309e47a8a32c2faa384dddc: Merge branch 'master' of git://git.denx.de/u-boot-i2c (2018-12-13 09:36:55 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-14dec18 for you to fetch changes up to f987177db9c988142032ed8142a093cce2378a90: dm: sound: Use the correct number of channels for sound (2018-12-13 16:37:10 -0700) Complete conversion of sound to driver model Simon Glass (40): sandbox: Increase the pre-relocation memory snow: Expand U-Boot size dm: sound: samsung: Make local function static dm: sound: Rename en_sound_codec to sound_codec dm: sound: Drop unused pre-device-tree code dm: sound: samsung: Rename i2stx_info to samsung_i2s_priv dm: sound: Drop codec_type dm: sound: wm8994: Pass private data to internal functions dm: sound: max98095: Pass private data to internal functions dm: sound: wm8994: Create a new common init function dm: sound: wm899c: Split out interface setup code dm: sound: max98095: Split out interface setup code dm: sound: wm8994: Drop global i2c-address variable dm: sound: max98095: Drop global i2c-address variable dm: sound: wm8994: Drop g_codec_info and g_wm8994_info dm: sound: max98095: Drop g_codec_info and g_max98095_info dm: sound: wm8994: Drop wm8994_i2c_init() dm: sound: Fix up header ordering dm: sound: exynos: Correct codec bus addresses dm: sound: Create an option to use driver model for sound dm: sound: Rename samsung_i2s_priv to i2s_uc_priv dm: sound: Create a uclass for audio codecs dm: sound: Create a uclass for i2s dm: sandbox: Update sound to use two buffers dm: sound: Create a uclass for sound dm: core: Add a function to read into a unsigned int dm: sound: Start i2c IDs from 0 dm: sound: Add conversion to driver model exynos: Add proid_is_exynos542x() for common 542x exynos: Add support for exynos5420 i2s pinmux dm: sound: Move common code out of maxim98095 dm: sound: exynos: Add support for max98090 dm: exynos: sound: Convert to use driver model dm: sandbox: sound: Convert to use driver model dm: exynos: Drop CONFIG_DM_I2C_COMPAT dm: sound: Complete migration to driver model dm: sound: Fix license headers dm: sound: max98095: Tidy up error codes dm: sandbox: Allow selection of sample rate and channels dm: sound: Use the correct number of channels for sound Kconfig | 2 + arch/Kconfig | 1 + arch/arm/dts/exynos5250-smdk5250.dts | 21 - arch/arm/dts/exynos5250-snow.dts | 25 -- arch/arm/dts/exynos5250-spring.dts| 24 +- arch/arm/dts/exynos5250.dtsi | 15 ++-- arch/arm/dts/exynos5420-peach-pit.dts | 24 +- arch/arm/dts/exynos5420-smdk5420.dts | 21 - arch/arm/dts/exynos54xx.dtsi | 14 +++ arch/arm/dts/exynos5800-peach-pi.dts | 24 +- arch/arm/mach-exynos/clock.c | 31 +-- arch/arm/mach-exynos/clock_init_exynos5.c | 2 +- arch/arm/mach-exynos/common_setup.h | 4 +- arch/arm/mach-exynos/include/mach/clock.h | 3 + arch/arm/mach-exynos/include/mach/cpu.h | 4 +- arch/arm/mach-exynos/include/mach/gpio.h | 4 +- arch/arm/mach-exynos/pinmux.c | 19 - arch/arm/mach-exynos/power.c | 2 +- arch/sandbox/cpu/sdl.c| 124 ++- arch/sandbox/dts/sandbox.dts | 21 + arch/sandbox/dts/test.dts | 23 + arch/sandbox/include/asm/sdl.h| 19 +++-- arch/sandbox/include/asm/sound.h | 13 --- arch/sandbox/include/asm/test.h | 40 + cmd/sound.c | 15 +++- configs/peach-pi_defconfig| 3 +- configs/peach-pit_defconfig | 3 +- configs/sandbox64_defconfig | 1 - configs/sandbox_defconfig | 1 - configs/sandbox_flattree_defconfig| 1 - configs/sandbox_noblk_defconfig | 1 - configs/sandbox_spl_defconfig | 1 - configs/smdk5250_defconfig| 1 - configs/smdk5420_defconfig| 1 - configs/snow_defconfig| 1 - configs/spring_defconfig | 1 - drivers/core/read.c | 23 + drivers/sound/Kconfig | 8 ++ drivers/sound/Makefile| 8 +- drivers/sound/codec-uclass.c | 26 ++ drivers/soun
Re: [U-Boot] Please pull u-boot-dm
On Tuesday 11 December 2018 06:34 AM, Simon Glass wrote: > Hi, > > On Tue, 4 Dec 2018 at 21:10, Keerthy wrote: >> >> >> >> On Friday 30 November 2018 08:25 PM, Tom Rini wrote: >>> On Thu, Nov 29, 2018 at 01:55:14PM -0700, Simon Glass wrote: >>> Hi Tom, The following changes since commit e16c888fab5014b022d5781dc534f204460a073b: Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-29nov18 for you to fetch changes up to 5ca3927deff30458f5d5b384f6699f70b9509315: core: ofnode: Add ofnode_get_addr_size_index (2018-11-29 09:30:06 -0700) Results here: https://travis-ci.org/sglass68/u-boot/builds/461363284 >>> >>> NAK. I don't know _why_ but I can confirm that both within travis: >>> https://travis-ci.org/trini/u-boot/jobs/461494951 >>> and then locally when I installed clang-7.0 from https://apt.llvm.org/ I >>> see that: >>> 5ca3927deff30458f5d5b384f6699f70b9509315 is the first bad commit >>> commit 5ca3927deff30458f5d5b384f6699f70b9509315 >>> Author: Keerthy >>> Date: Mon Nov 19 11:44:48 2018 +0530 >>> >>> core: ofnode: Add ofnode_get_addr_size_index >>> >>> is totally breaking test.py on sandbox. It still starts and I don't see >>> the obvious whats wrong, but I've bisected twice to this same commit. >>> >> >> Okay i am not sure either what is going wrong there. I will take a look at >> this. > > I am worried that this might be a unicode problem. > > - build outputs error messages with backquotes, UTF-8, etc. > - pipe from builder thread to buildman breaks things into 4KB chunks > - sometimes a UTF-8 char break across a 4KB boundary > - I/O error results > > I suspect this because buildman's unicode handling is actually a bit > broken - see how quotes come through in messages on some machines. > > But it might be something else entirely. Adding Lokesh Hey Lokesh, I am yet to dig into this. Seems my patch is breaking test.py on sandbox and hence it was not pulled. Regards, Keerthy > > 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-dm
Hi, On Tue, 4 Dec 2018 at 21:10, Keerthy wrote: > > > > On Friday 30 November 2018 08:25 PM, Tom Rini wrote: > > On Thu, Nov 29, 2018 at 01:55:14PM -0700, Simon Glass wrote: > > > >> Hi Tom, > >> > >> The following changes since commit > >> e16c888fab5014b022d5781dc534f204460a073b: > >> > >> Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) > >> > >> are available in the Git repository at: > >> > >> git://git.denx.de/u-boot-dm.git tags/pull-29nov18 > >> > >> for you to fetch changes up to 5ca3927deff30458f5d5b384f6699f70b9509315: > >> > >> core: ofnode: Add ofnode_get_addr_size_index (2018-11-29 09:30:06 -0700) > >> > >> Results here: > >> > >> https://travis-ci.org/sglass68/u-boot/builds/461363284 > > > > NAK. I don't know _why_ but I can confirm that both within travis: > > https://travis-ci.org/trini/u-boot/jobs/461494951 > > and then locally when I installed clang-7.0 from https://apt.llvm.org/ I > > see that: > > 5ca3927deff30458f5d5b384f6699f70b9509315 is the first bad commit > > commit 5ca3927deff30458f5d5b384f6699f70b9509315 > > Author: Keerthy > > Date: Mon Nov 19 11:44:48 2018 +0530 > > > > core: ofnode: Add ofnode_get_addr_size_index > > > > is totally breaking test.py on sandbox. It still starts and I don't see > > the obvious whats wrong, but I've bisected twice to this same commit. > > > > Okay i am not sure either what is going wrong there. I will take a look at > this. I am worried that this might be a unicode problem. - build outputs error messages with backquotes, UTF-8, etc. - pipe from builder thread to buildman breaks things into 4KB chunks - sometimes a UTF-8 char break across a 4KB boundary - I/O error results I suspect this because buildman's unicode handling is actually a bit broken - see how quotes come through in messages on some machines. But it might be something else entirely. 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-dm
On Wed, Dec 05, 2018 at 06:30:48PM -0700, Simon Glass wrote: > Hi Tom, > > The following changes since commit 2e2a2a5d4f0c2e2642326d9000ce1f1553632e6a: > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-12-04 > 19:22:31 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-pull-5dec18 > > for you to fetch changes up to b288cd9600724ad3a0e55c8786e70741dd13deae: > > x86: acpi: Generate SPCR table (2018-12-05 07:23:16 -0700) > > > Travis here > > https://travis-ci.org/sglass68/u-boot/builds/463866708 > > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, The following changes since commit 2e2a2a5d4f0c2e2642326d9000ce1f1553632e6a: Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-12-04 19:22:31 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-pull-5dec18 for you to fetch changes up to b288cd9600724ad3a0e55c8786e70741dd13deae: x86: acpi: Generate SPCR table (2018-12-05 07:23:16 -0700) Travis here https://travis-ci.org/sglass68/u-boot/builds/463866708 Minor sandbox enhancements / fixes tpm improvements to clear up v1/v2 support buildman toolchain fixes New serial options to set/get config Andy Shevchenko (7): dm: serial: Add ->getconfig() callback dm: serial: Introduce ->getinfo() callback serial: ns16550: Group reg_* members of ns16550_platdata serial: ns16550: Read reg-io-width from device tree serial: ns16550: Provide ->getinfo() implementation x86: acpi: Add SPCR table description x86: acpi: Generate SPCR table Patrick Delaunay (1): power: regulator: denied disable on always-on regulator Philipp Tomsich (2): dm: rtc: Fix function name in comment dm: (re)sort uclass ids alphabetically Simon Glass (19): sandbox: Fix up the debug message for the image filename sandbox: Check the filename in jump_to_image_no_args() sandbox: physmem: Use mapping to support sandbox sandbox: net: Correct name copy in eth_raw_bus_post_bind() sandbox: sysreset: Update to support power-on reset sandbox: Zero the ram buffer on startup sandbox: Use 'extras' to specify 'head' files tpm: Remove use of build-time TPM versions tpm: Export tpm_clear_and_reenable() tpm: Add non-volatile index attributes needed for v2 tpm: Fix a logging warning in unpack_byte_string() cros: Correct a printf() string and comment cros_ec: Adjust to use v1 vboot context only input: i8042: Use remove() instead of exported functions video: backlight: Fix log message in enable_sequence() time: Update mdelay() to delay in one large chunk Add UINT32_MAX and UINT64_MAX Add inttypes.h binman: Add myself as maintainer Trevor Woerner (2): buildman/toolchain.py: fix toolchain directory buildman/toolchain.py: handle inconsistent tarball names Álvaro Fernández Rojas (1): dm: core: add functions to get/remap I/O addresses by name MAINTAINERS| 5 + arch/sandbox/Makefile | 4 +- arch/sandbox/config.mk | 3 +- arch/sandbox/cpu/Makefile | 5 +- arch/sandbox/cpu/os.c | 2 +- arch/sandbox/cpu/spl.c | 8 +- arch/sandbox/cpu/start.c | 7 + arch/sandbox/dts/test.dts | 1 + arch/sandbox/include/asm/state.h | 1 + arch/x86/include/asm/acpi_table.h | 51 ++ arch/x86/lib/acpi_table.c | 118 + drivers/core/fdtaddr.c | 10 ++ drivers/core/read.c| 20 +++ drivers/input/i8042.c | 35 ++-- drivers/misc/cros_ec.c | 4 +- drivers/misc/cros_ec_sandbox.c | 10 +- drivers/net/sandbox-raw-bus.c | 2 +- drivers/power/regulator/regulator-uclass.c | 5 + drivers/serial/ns16550.c | 23 ++- drivers/serial/sandbox.c | 34 drivers/serial/serial-uclass.c | 37 drivers/sysreset/sysreset_sandbox.c| 8 +- drivers/video/pwm_backlight.c | 2 +- include/common.h | 4 + include/dm/fdtaddr.h | 13 ++ include/dm/read.h | 36 include/dm/uclass-id.h | 8 +- include/i8042.h| 15 -- include/inttypes.h | 271 + include/linux/delay.h | 3 +- include/linux/kernel.h | 4 + include/ns16550.h | 4 +- include/rtc.h | 2 +- include/serial.h | 66 ++- include/tpm-common.h | 19 ++ include/tpm-v2.h | 33 lib/physmem.c | 4 +- lib/tpm-common.c | 9 +- lib/tpm-v1.c | 22 +-- test/dm/serial.c | 12 ++ test/dm/sysreset.c | 4 +- test/dm/test-fdt.c | 82 - tools/buildman/toolchain.py| 8 +- 43 files changed, 935 insertions(+), 79 deletions(-) create mode 100644 include/inttypes.h Regards, Simo
Re: [U-Boot] Please pull u-boot-dm
On Friday 30 November 2018 08:25 PM, Tom Rini wrote: > On Thu, Nov 29, 2018 at 01:55:14PM -0700, Simon Glass wrote: > >> Hi Tom, >> >> The following changes since commit e16c888fab5014b022d5781dc534f204460a073b: >> >> Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) >> >> are available in the Git repository at: >> >> git://git.denx.de/u-boot-dm.git tags/pull-29nov18 >> >> for you to fetch changes up to 5ca3927deff30458f5d5b384f6699f70b9509315: >> >> core: ofnode: Add ofnode_get_addr_size_index (2018-11-29 09:30:06 -0700) >> >> Results here: >> >> https://travis-ci.org/sglass68/u-boot/builds/461363284 > > NAK. I don't know _why_ but I can confirm that both within travis: > https://travis-ci.org/trini/u-boot/jobs/461494951 > and then locally when I installed clang-7.0 from https://apt.llvm.org/ I > see that: > 5ca3927deff30458f5d5b384f6699f70b9509315 is the first bad commit > commit 5ca3927deff30458f5d5b384f6699f70b9509315 > Author: Keerthy > Date: Mon Nov 19 11:44:48 2018 +0530 > > core: ofnode: Add ofnode_get_addr_size_index > > is totally breaking test.py on sandbox. It still starts and I don't see > the obvious whats wrong, but I've bisected twice to this same commit. > Okay i am not sure either what is going wrong there. I will take a look at this. Thanks, Keerthy ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
Hi. On Mon, 3 Dec 2018 at 00:42, Simon Goldschmidt wrote: > > On Mon, Dec 3, 2018 at 8:27 AM Kever Yang wrote: > > > > Hi Tom, > > > > With this merge, my boards can't work correctly in SPL, the console > > output is mess up. > > > > I have test on rk3229 and rk3399 boards, and finally I identify that > > below patch is the root cause: > > > > 6f57c34473 serial: ns16550: fix debug uart putc called before init > > Henri Roosen reported this to me last week. The root cause seems to be > that the ns16550 baudrate prescaler is read as 0 on his rk3399 board. > > He helped me try an additional fix but that didn't work either. > > So we can either try and fix this if it is only a problem for rockchip > boards or revert the patch and I'll think of a different solution to > fix this issue (debug uart putc called before init) for the socfpga > platform. > > I still think it would be good to fix this globally: there's nothing > keeping us from accidentally adding putc code before the debug uart is > initialized and this is something that should be detected, I think. I think it would be better to have this discussion on the patch rather than the pull request. 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-dm
On Mon, Dec 3, 2018 at 8:27 AM Kever Yang wrote: > > Hi Tom, > > With this merge, my boards can't work correctly in SPL, the console > output is mess up. > > I have test on rk3229 and rk3399 boards, and finally I identify that > below patch is the root cause: > > 6f57c34473 serial: ns16550: fix debug uart putc called before init Henri Roosen reported this to me last week. The root cause seems to be that the ns16550 baudrate prescaler is read as 0 on his rk3399 board. He helped me try an additional fix but that didn't work either. So we can either try and fix this if it is only a problem for rockchip boards or revert the patch and I'll think of a different solution to fix this issue (debug uart putc called before init) for the socfpga platform. I still think it would be good to fix this globally: there's nothing keeping us from accidentally adding putc code before the debug uart is initialized and this is something that should be detected, I think. Regards, Simon > > > Thanks, > - Kever > On 11/16/2018 09:45 PM, Tom Rini wrote: > > On Thu, Nov 15, 2018 at 06:05:05PM -0800, Simon Glass wrote: > >> Hi Tom, > >> > >> On 11 October 2018 at 07:10, Tom Rini wrote: > >>> On Tue, Oct 09, 2018 at 06:24:13PM -0600, Simon Glass wrote: > >>> > Hi Tom, > > Here is my attempt at a signed pull request. I've brought in most of the > outstanding dm patches. Please let me know if it looks OK. Do you have my > public key? > >>> Is it on the public key servers? > >> Yes, here: > >> > >> https://pgp.mit.edu/pks/lookup?search=sjg%40chromium&op=index > > So yes, it is working. On your latest PR: > > $ git show --show-signature > > commit 1d6edcbfed2af33c748f2beb399810a0441888da > > merged tag 'pull-14nov18' > > gpg: Signature made Wed 14 Nov 2018 08:07:43 PM EST using RSA key ID > > 9008ADE6 > > gpg: Good signature from "Simon Glass " > > Merge: f6206f8587fc 4c6e27f63c88 > > Author: Tom Rini > > Date: Fri Nov 16 08:37:50 2018 -0500 > > > > Merge tag 'pull-14nov18' of git://git.denx.de/u-boot-dm > > > > Thanks again! > > > > > > > > ___ > > U-Boot mailing list > > U-Boot@lists.denx.de > > https://lists.denx.de/listinfo/u-boot > > ___ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
Hi Tom, With this merge, my boards can't work correctly in SPL, the console output is mess up. I have test on rk3229 and rk3399 boards, and finally I identify that below patch is the root cause: 6f57c34473 serial: ns16550: fix debug uart putc called before init Thanks, - Kever On 11/16/2018 09:45 PM, Tom Rini wrote: > On Thu, Nov 15, 2018 at 06:05:05PM -0800, Simon Glass wrote: >> Hi Tom, >> >> On 11 October 2018 at 07:10, Tom Rini wrote: >>> On Tue, Oct 09, 2018 at 06:24:13PM -0600, Simon Glass wrote: >>> Hi Tom, Here is my attempt at a signed pull request. I've brought in most of the outstanding dm patches. Please let me know if it looks OK. Do you have my public key? >>> Is it on the public key servers? >> Yes, here: >> >> https://pgp.mit.edu/pks/lookup?search=sjg%40chromium&op=index > So yes, it is working. On your latest PR: > $ git show --show-signature > commit 1d6edcbfed2af33c748f2beb399810a0441888da > merged tag 'pull-14nov18' > gpg: Signature made Wed 14 Nov 2018 08:07:43 PM EST using RSA key ID > 9008ADE6 > gpg: Good signature from "Simon Glass " > Merge: f6206f8587fc 4c6e27f63c88 > Author: Tom Rini > Date: Fri Nov 16 08:37:50 2018 -0500 > > Merge tag 'pull-14nov18' of git://git.denx.de/u-boot-dm > > Thanks again! > > > > ___ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm (take 2)
On Fri, Nov 30, 2018 at 12:29:18PM -0700, Simon Glass wrote: > Hi Tom, > > See: > > https://travis-ci.org/sglass68/u-boot/builds/461837218?utm_medium=notification&utm_source=email > > > The following changes since commit e16c888fab5014b022d5781dc534f204460a073b: > > Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-30nov18 > > for you to fetch changes up to 1678754f5e2cbc14f9612e953b39cc08ada66866: > > core: ofnode: Fix ofnode_get_addr_index function (2018-11-29 09:30:06 > -0700) > Applied to u-boot/master, thanks! -- 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-dm (take 2)
Hi Tom, See: https://travis-ci.org/sglass68/u-boot/builds/461837218?utm_medium=notification&utm_source=email The following changes since commit e16c888fab5014b022d5781dc534f204460a073b: Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-30nov18 for you to fetch changes up to 1678754f5e2cbc14f9612e953b39cc08ada66866: core: ofnode: Fix ofnode_get_addr_index function (2018-11-29 09:30:06 -0700) Fix sound on sandbox Convert TPM fully to DM Tidy up sandbox I2C emulation Add a 'make qcheck' target for faster testing A few other misc things (dropped the final patch which breaks clang for some reason) Baruch Siach (1): ofnode: fix comment typo Heiko Schocher (1): cmd, fdt: add subcommand "get" to fdt header Keerthy (1): core: ofnode: Fix ofnode_get_addr_index function Simon Glass (17): malloc_simple: Add logging of allocations power: pmic: Correct debug/error output test: Add a 'make qcheck' target for quicker testing dm: core: Export uclass_find_device_by_phandle() dm: core: Add a few more specific child-finding functions dm: core: Put UCLASS_SIMPLE_BUS in order dm: sandbox: i2c: Add a new 'emulation parent' uclass dm: sandbox: i2c: Use new emulator parent uclass rtc: Allow child drivers sound: Correct data output in sound_create_square_wave() sound: Add sample rate as a parameter for square wave sound: sandbox: Use the correct frequency sandbox: Enable sound sandbox: Use memmove() to move overlapping regions tpm: Add a constant for the minimum supported digest size tpm: Export the open/close functions tpm: Convert to use a device parameter Makefile | 6 +- arch/sandbox/cpu/os.c | 5 +- arch/sandbox/cpu/sdl.c| 30 ++ arch/sandbox/dts/sandbox.dts | 25 +++-- arch/sandbox/dts/sandbox64.dts| 21 ++-- arch/sandbox/dts/sandbox_pmic.dtsi| 71 +++-- arch/sandbox/dts/test.dts | 28 +++-- board/gdsys/a38x/controlcenterdc.c| 25 - board/gdsys/a38x/hre.c| 66 +++- board/gdsys/a38x/hre.h| 2 +- board/gdsys/a38x/keyprogram.c | 12 ++- board/gdsys/a38x/keyprogram.h | 4 +- board/gdsys/p1022/controlcenterd-id.c | 117 + cmd/fdt.c | 40 ++- cmd/tpm-common.c | 8 +- cmd/tpm-v1.c | 140 cmd/tpm-v2.c | 78 +++--- cmd/tpm_test.c| 327 ++--- common/malloc_simple.c| 58 ++ drivers/core/device.c | 34 ++ drivers/core/ofnode.c | 17 +-- drivers/core/uclass.c | 6 +- drivers/i2c/i2c-emul-uclass.c | 77 ++ drivers/i2c/sandbox_i2c.c | 20 +--- drivers/power/pmic/act8846.c | 2 +- drivers/power/pmic/as3722.c | 14 +-- drivers/power/pmic/as3722_gpio.c | 11 +- drivers/power/pmic/i2c_pmic_emul.c| 2 +- drivers/power/pmic/lp873x.c | 6 +- drivers/power/pmic/lp87565.c | 6 +- drivers/power/pmic/max77686.c | 8 +- drivers/power/pmic/max8997.c | 4 +- drivers/power/pmic/max8998.c | 4 +- drivers/power/pmic/mc34708.c | 4 +- drivers/power/pmic/palmas.c | 8 +- drivers/power/pmic/pfuze100.c | 6 +- drivers/power/pmic/rk8xx.c| 6 +- drivers/power/pmic/rn5t567.c | 4 +- drivers/power/pmic/s2mps11.c | 8 +- drivers/power/pmic/s5m8767.c | 6 +- drivers/power/pmic/sandbox.c | 4 +- drivers/power/pmic/stpmu1.c | 2 +- drivers/power/pmic/tps65090.c | 8 +- drivers/rtc/rtc-uclass.c | 1 + drivers/sound/sound-i2s.c | 3 +- drivers/sound/sound.c | 8 +- drivers/tpm/tpm_tis_lpc.c | 47 + include/dm/device.h | 25 + include/dm/ofnode.h | 2 +- include/dm/uclass-id.h| 3 +- include/dm/uclass-internal.h | 17 +++ include/i2c.h | 21 include/log.h | 2 +- include/malloc.h | 1 + include/sound.h | 4 +- include/tpm-common.h | 36 ++- include/tpm-v1.h | 97 +++-- include/tpm-v2.h | 49 ++--- lib/tpm-common.c | 16 +-- lib/tpm-utils.h
Re: [U-Boot] Please pull u-boot-dm
On Thu, Nov 29, 2018 at 01:55:14PM -0700, Simon Glass wrote: > Hi Tom, > > The following changes since commit e16c888fab5014b022d5781dc534f204460a073b: > > Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-29nov18 > > for you to fetch changes up to 5ca3927deff30458f5d5b384f6699f70b9509315: > > core: ofnode: Add ofnode_get_addr_size_index (2018-11-29 09:30:06 -0700) > > Results here: > > https://travis-ci.org/sglass68/u-boot/builds/461363284 NAK. I don't know _why_ but I can confirm that both within travis: https://travis-ci.org/trini/u-boot/jobs/461494951 and then locally when I installed clang-7.0 from https://apt.llvm.org/ I see that: 5ca3927deff30458f5d5b384f6699f70b9509315 is the first bad commit commit 5ca3927deff30458f5d5b384f6699f70b9509315 Author: Keerthy Date: Mon Nov 19 11:44:48 2018 +0530 core: ofnode: Add ofnode_get_addr_size_index is totally breaking test.py on sandbox. It still starts and I don't see the obvious whats wrong, but I've bisected twice to this same commit. -- 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-dm
Hi Tom, The following changes since commit e16c888fab5014b022d5781dc534f204460a073b: Merge branch '2018-11-28-master-imports' (2018-11-28 23:04:58 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-29nov18 for you to fetch changes up to 5ca3927deff30458f5d5b384f6699f70b9509315: core: ofnode: Add ofnode_get_addr_size_index (2018-11-29 09:30:06 -0700) Results here: https://travis-ci.org/sglass68/u-boot/builds/461363284 Fix sound on sandbox Convert TPM fully to DM Tidy up sandbox I2C emulation Add a 'make qcheck' target for faster testing A few other misc things Baruch Siach (1): ofnode: fix comment typo Heiko Schocher (1): cmd, fdt: add subcommand "get" to fdt header Keerthy (2): core: ofnode: Fix ofnode_get_addr_index function core: ofnode: Add ofnode_get_addr_size_index Simon Glass (17): malloc_simple: Add logging of allocations power: pmic: Correct debug/error output test: Add a 'make qcheck' target for quicker testing dm: core: Export uclass_find_device_by_phandle() dm: core: Add a few more specific child-finding functions dm: core: Put UCLASS_SIMPLE_BUS in order dm: sandbox: i2c: Add a new 'emulation parent' uclass dm: sandbox: i2c: Use new emulator parent uclass rtc: Allow child drivers sound: Correct data output in sound_create_square_wave() sound: Add sample rate as a parameter for square wave sound: sandbox: Use the correct frequency sandbox: Enable sound sandbox: Use memmove() to move overlapping regions tpm: Add a constant for the minimum supported digest size tpm: Export the open/close functions tpm: Convert to use a device parameter Makefile | 6 +- arch/sandbox/cpu/os.c | 5 +- arch/sandbox/cpu/sdl.c| 30 ++ arch/sandbox/dts/sandbox.dts | 25 +++-- arch/sandbox/dts/sandbox64.dts| 21 ++-- arch/sandbox/dts/sandbox_pmic.dtsi| 71 +++-- arch/sandbox/dts/test.dts | 28 +++-- board/gdsys/a38x/controlcenterdc.c| 25 - board/gdsys/a38x/hre.c| 66 +++- board/gdsys/a38x/hre.h| 2 +- board/gdsys/a38x/keyprogram.c | 12 ++- board/gdsys/a38x/keyprogram.h | 4 +- board/gdsys/p1022/controlcenterd-id.c | 117 + cmd/fdt.c | 40 ++- cmd/tpm-common.c | 8 +- cmd/tpm-v1.c | 140 cmd/tpm-v2.c | 78 +++--- cmd/tpm_test.c| 327 ++--- common/malloc_simple.c| 58 ++ drivers/core/device.c | 34 ++ drivers/core/ofnode.c | 25 +++-- drivers/core/uclass.c | 6 +- drivers/i2c/i2c-emul-uclass.c | 77 ++ drivers/i2c/sandbox_i2c.c | 20 +--- drivers/power/pmic/act8846.c | 2 +- drivers/power/pmic/as3722.c | 14 +-- drivers/power/pmic/as3722_gpio.c | 11 +- drivers/power/pmic/i2c_pmic_emul.c| 2 +- drivers/power/pmic/lp873x.c | 6 +- drivers/power/pmic/lp87565.c | 6 +- drivers/power/pmic/max77686.c | 8 +- drivers/power/pmic/max8997.c | 4 +- drivers/power/pmic/max8998.c | 4 +- drivers/power/pmic/mc34708.c | 4 +- drivers/power/pmic/palmas.c | 8 +- drivers/power/pmic/pfuze100.c | 6 +- drivers/power/pmic/rk8xx.c| 6 +- drivers/power/pmic/rn5t567.c | 4 +- drivers/power/pmic/s2mps11.c | 8 +- drivers/power/pmic/s5m8767.c | 6 +- drivers/power/pmic/sandbox.c | 4 +- drivers/power/pmic/stpmu1.c | 2 +- drivers/power/pmic/tps65090.c | 8 +- drivers/rtc/rtc-uclass.c | 1 + drivers/sound/sound-i2s.c | 3 +- drivers/sound/sound.c | 8 +- drivers/tpm/tpm_tis_lpc.c | 47 + include/dm/device.h | 25 + include/dm/ofnode.h | 16 ++- include/dm/uclass-id.h| 3 +- include/dm/uclass-internal.h | 17 +++ include/i2c.h | 21 include/log.h | 2 +- include/malloc.h | 1 + include/sound.h | 4 +- include/tpm-common.h | 36 ++- include/tpm-v1.h | 97 +++-- include/tpm-v2.h | 49 ++--- lib/tpm-common.c | 16 +-- lib/tpm-utils.h | 21 +--- lib/tpm-v1.c
Re: [U-Boot] Please pull u-boot-dm
On Wed, Nov 21, 2018 at 05:25:59PM -0700, Simon Glass wrote: > Hi Tom, > > The following changes since commit a3e1653ddeb02f39481eba572275016171e9670c: > > Merge git://git.denx.de/u-boot-marvell (2018-11-20 12:39:16 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-tg18 > > for you to fetch changes up to a58986ca8b53d8c7a441397082f84edc7f47d19f: > > sf: Add a method to obtain the block-protect setting (2018-11-20 > 19:14:22 -0700) > > https://travis-ci.org/sglass68/u-boot/builds/457752510 > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, The following changes since commit a3e1653ddeb02f39481eba572275016171e9670c: Merge git://git.denx.de/u-boot-marvell (2018-11-20 12:39:16 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-tg18 for you to fetch changes up to a58986ca8b53d8c7a441397082f84edc7f47d19f: sf: Add a method to obtain the block-protect setting (2018-11-20 19:14:22 -0700) https://travis-ci.org/sglass68/u-boot/builds/457752510 Various minor sandbox improvements Better buildman warning handling Misc other things Baruch Siach (1): fdt: restore board_fdt_blob_setup() declaration Heinrich Schuchardt (3): sandbox: remove stray DEBUG sandbox: README: setting environment variables sandbox: README: use setenv ethrotate no Hiroyuki Yokoyama (1): cmd: fdt: Fix fdt address information after the movement Hou Zhiqiang (1): dm/pci: Change the first CFG read to Vendor ID in enumeration Liviu Dudau (1): dm: core: Widen the dump tree to show more of the driver's name. Marek Vasut (1): fdt: Fix uncompress_blob() for U-Boot proper Simon Glass (24): buildman: Only print toolchain probing with -v buildman: Detect dtc warnings buildman: Rename the good, better, worse variables buildman: Show boards with warning with w+ cros_ec: Use uint instead of u8 for parameters cros_ec: Add error logging on a few commands cros_ec: Fail if we cannot determine the flash burst size cros_ec: Align uclass data to a cache boundary cros_ec: Add new features for events and power sandbox: tpm: Allow debugging of data packages sandbox: log: Add a category for sandbox sandbox: Add a function to read a host file sandbox: cros_ec: exynos: Drop use of cros_ec_get_error() sandbox: Update some drivers to work in SPL/TPL spl: Support bootstage, log, hash and early malloc in TPL spl: lz4: Allow use of lz4 compression in SPL binman: Add a way to enable debugging from the build binman: Drop an unnecessary comma in blob handling binman: Set the pathname correctly for ELF files video: Update video_set_default_colors() to support invert string: Include the config header misc: Update read() and write() methods to return bytes xfered test: sf: Add a simple SPI flash test sf: Add a method to obtain the block-protect setting Simon Goldschmidt (1): dm: spi: prevent setting a speed of 0 Hz Makefile | 6 +- arch/arm/mach-stm32mp/cpu.c | 4 +- arch/sandbox/cpu/cpu.c| 8 +- arch/sandbox/cpu/os.c | 44 ++- arch/sandbox/dts/sandbox.dts | 15 +++ arch/sandbox/include/asm/test.h | 8 ++ board/samsung/common/board.c | 10 +- board/sandbox/README.sandbox | 20 +-- board/sandbox/sandbox.c | 9 +- cmd/fdt.c | 2 +- common/Kconfig| 35 ++ common/Makefile | 10 +- common/cros_ec.c | 12 -- drivers/clk/clk_vexpress_osc.c| 4 +- drivers/core/dump.c | 8 +- drivers/misc/altera_sysid.c | 2 +- drivers/misc/cros_ec.c| 370 --- drivers/misc/cros_ec_sandbox.c| 2 +- drivers/misc/misc_sandbox.c | 4 +- drivers/misc/rockchip-efuse.c | 2 +- drivers/misc/stm32mp_fuse.c | 12 ++ drivers/mtd/spi/sandbox.c | 10 ++ drivers/mtd/spi/sf-uclass.c | 9 ++ drivers/mtd/spi/sf_internal.h | 3 + drivers/mtd/spi/sf_probe.c| 8 ++ drivers/mtd/spi/spi_flash.c | 12 ++ drivers/pci/pci-uclass.c | 11 +- drivers/spi/spi-uclass.c | 15 ++- drivers/tpm/tpm_tis_sandbox.c | 6 + drivers/video/vidconsole-uclass.c | 2 +- drivers/video/video-uclass.c | 27 ++-- include/cros_ec.h | 89 + include/fdtdec.h | 1 + include/log.h | 1 + include/misc.h| 8 +- include/os.h | 14 +++ include/spi_flash.h | 27 include/video.h | 5 +- lib/Kconfig | 8 ++ lib/Makefile | 2 +- lib/fdtdec.c | 3 +- lib/string.c | 1 + test/dm/misc.c| 6 +- test/dm/sf.c | 55 - test/py/tests/test_bind.py| 28 ++--- tools/binman/README | 6 + tools/binman/etype/blob.py| 2 +- tools/binman/etype/u_boot_elf.py | 5 +- tools/buildman/builder.py | 41 +++--- tools/buildman/cmdline.py | 2 +- tools/buildman/control.py | 2 +- tools/buildman/test.py
Re: [U-Boot] Please pull u-boot-dm
Hi Tom, On 16 November 2018 at 05:45, Tom Rini wrote: > On Thu, Nov 15, 2018 at 06:05:05PM -0800, Simon Glass wrote: >> Hi Tom, >> >> On 11 October 2018 at 07:10, Tom Rini wrote: >> > On Tue, Oct 09, 2018 at 06:24:13PM -0600, Simon Glass wrote: >> > >> >> Hi Tom, >> >> >> >> Here is my attempt at a signed pull request. I've brought in most of the >> >> outstanding dm patches. Please let me know if it looks OK. Do you have my >> >> public key? >> > >> > Is it on the public key servers? >> >> Yes, here: >> >> https://pgp.mit.edu/pks/lookup?search=sjg%40chromium&op=index > > So yes, it is working. On your latest PR: > $ git show --show-signature > commit 1d6edcbfed2af33c748f2beb399810a0441888da > merged tag 'pull-14nov18' > gpg: Signature made Wed 14 Nov 2018 08:07:43 PM EST using RSA key ID > 9008ADE6 > gpg: Good signature from "Simon Glass " > Merge: f6206f8587fc 4c6e27f63c88 > Author: Tom Rini > Date: Fri Nov 16 08:37:50 2018 -0500 > > Merge tag 'pull-14nov18' of git://git.denx.de/u-boot-dm > > Thanks again! OK that's good to see, thanks! - SImon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On Thu, Nov 15, 2018 at 08:32:30AM -0800, Simon Glass wrote: > Hi Tom, > > The following changes since commit 208ecbad2ea8e8f3c9933213867addf16f4a: > > Merge branch 'next' (2018-11-14 11:30:07 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-14nov18 > > for you to fetch changes up to 4c6e27f63c88d065a98f438085dfc36af47d3a23: > > Revert "imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO" (2018-11-14 > 09:16:28 -0800) > Applied to u-boot/master, thanks! -- 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-dm
On Thu, Nov 15, 2018 at 06:05:05PM -0800, Simon Glass wrote: > Hi Tom, > > On 11 October 2018 at 07:10, Tom Rini wrote: > > On Tue, Oct 09, 2018 at 06:24:13PM -0600, Simon Glass wrote: > > > >> Hi Tom, > >> > >> Here is my attempt at a signed pull request. I've brought in most of the > >> outstanding dm patches. Please let me know if it looks OK. Do you have my > >> public key? > > > > Is it on the public key servers? > > Yes, here: > > https://pgp.mit.edu/pks/lookup?search=sjg%40chromium&op=index So yes, it is working. On your latest PR: $ git show --show-signature commit 1d6edcbfed2af33c748f2beb399810a0441888da merged tag 'pull-14nov18' gpg: Signature made Wed 14 Nov 2018 08:07:43 PM EST using RSA key ID 9008ADE6 gpg: Good signature from "Simon Glass " Merge: f6206f8587fc 4c6e27f63c88 Author: Tom Rini Date: Fri Nov 16 08:37:50 2018 -0500 Merge tag 'pull-14nov18' of git://git.denx.de/u-boot-dm Thanks again! -- 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-dm
Hi Tom, On 11 October 2018 at 07:10, Tom Rini wrote: > On Tue, Oct 09, 2018 at 06:24:13PM -0600, Simon Glass wrote: > >> Hi Tom, >> >> Here is my attempt at a signed pull request. I've brought in most of the >> outstanding dm patches. Please let me know if it looks OK. Do you have my >> public key? > > Is it on the public key servers? Yes, here: https://pgp.mit.edu/pks/lookup?search=sjg%40chromium&op=index > >> >> >> The following changes since commit 0a60a81ba3860946551cb79aa6486aa076e357f3: >> >> Kconfig: sandbox: enable cmd_avb and dependencies (2018-10-07 13:34:19 >> -0400) >> >> are available in the Git repository at: >> >> git://git.denx.de/u-boot-dm.git tags/dm-9oct18 >> >> for you to fetch changes up to 41b781ddf1869f5349e05ace888979f3673fe8c6: >> >> dtoc: Fix the value of SetInt() (2018-10-09 04:40:27 -0600) >> > > Applied to u-boot/master, thanks! > > -- > Tom ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On 15 November 2018 at 13:02, Tom Rini wrote: > > On Thu, Nov 15, 2018 at 12:21:15PM -0800, Simon Glass wrote: > > Hi Tom, > > > > On 15 November 2018 at 08:32, Simon Glass wrote: > > > Hi Tom, > > > > > > The following changes since commit > > > 208ecbad2ea8e8f3c9933213867addf16f4a: > > > > > > Merge branch 'next' (2018-11-14 11:30:07 -0500) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-14nov18 > > > > > > for you to fetch changes up to 4c6e27f63c88d065a98f438085dfc36af47d3a23: > > > > > > Revert "imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO" (2018-11-14 > > > 09:16:28 -0800) > > > > > > > > > - virtio implementation and supporting patches > > > - DM_FLAG_PRE_RELOC fixes > > > - regmap improvements > > > - minor buildman and sandbox things > > > > > > > > > > I am seeing strange failures with travis-ci. > > > > https://travis-ci.org/sglass68/u-boot/builds/455582198?utm_medium=notification&utm_source=email > > > > I thought it was something strange with travis-ci but the above run is > > my second attempt and it fails with the same builds. Lots of stuff > > about the filesystem tests. But they have been there for a while! > > > > Do you have any ideas? > > Yes: https://patchwork.ozlabs.org/patch/998330/ which I'm pushing > now'ish, since my WIP branch completed again with that applied. Thanks Tom, great to hear. 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-dm
On Thu, Nov 15, 2018 at 12:21:15PM -0800, Simon Glass wrote: > Hi Tom, > > On 15 November 2018 at 08:32, Simon Glass wrote: > > Hi Tom, > > > > The following changes since commit 208ecbad2ea8e8f3c9933213867addf16f4a: > > > > Merge branch 'next' (2018-11-14 11:30:07 -0500) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/pull-14nov18 > > > > for you to fetch changes up to 4c6e27f63c88d065a98f438085dfc36af47d3a23: > > > > Revert "imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO" (2018-11-14 > > 09:16:28 -0800) > > > > > > - virtio implementation and supporting patches > > - DM_FLAG_PRE_RELOC fixes > > - regmap improvements > > - minor buildman and sandbox things > > > > > > I am seeing strange failures with travis-ci. > > https://travis-ci.org/sglass68/u-boot/builds/455582198?utm_medium=notification&utm_source=email > > I thought it was something strange with travis-ci but the above run is > my second attempt and it fails with the same builds. Lots of stuff > about the filesystem tests. But they have been there for a while! > > Do you have any ideas? Yes: https://patchwork.ozlabs.org/patch/998330/ which I'm pushing now'ish, since my WIP branch completed again with that applied. -- 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-dm
Hi Tom, On 15 November 2018 at 08:32, Simon Glass wrote: > Hi Tom, > > The following changes since commit 208ecbad2ea8e8f3c9933213867addf16f4a: > > Merge branch 'next' (2018-11-14 11:30:07 -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-14nov18 > > for you to fetch changes up to 4c6e27f63c88d065a98f438085dfc36af47d3a23: > > Revert "imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO" (2018-11-14 > 09:16:28 -0800) > > > - virtio implementation and supporting patches > - DM_FLAG_PRE_RELOC fixes > - regmap improvements > - minor buildman and sandbox things > > I am seeing strange failures with travis-ci. https://travis-ci.org/sglass68/u-boot/builds/455582198?utm_medium=notification&utm_source=email I thought it was something strange with travis-ci but the above run is my second attempt and it fails with the same builds. Lots of stuff about the filesystem tests. But they have been there for a while! Do you have any ideas? Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] Please pull u-boot-dm
Hi Tom, The following changes since commit 208ecbad2ea8e8f3c9933213867addf16f4a: Merge branch 'next' (2018-11-14 11:30:07 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-14nov18 for you to fetch changes up to 4c6e27f63c88d065a98f438085dfc36af47d3a23: Revert "imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO" (2018-11-14 09:16:28 -0800) - virtio implementation and supporting patches - DM_FLAG_PRE_RELOC fixes - regmap improvements - minor buildman and sandbox things Bin Meng (50): dm: cpu: Fix print_cpuinfo() output cpu: mpc83xx: Remove unnecessary characters in the description string dm: util: Add a livetree equivalent API of dm_fdt_pre_reloc() dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt() dm: Correct pre_reloc_only parameter description in several APIs' comments dm: core: Mirror the chosen node parse logic in the livetree scanning test: dm: core: Add a test case for driver marked with DM_FLAG_PRE_RELOC flag timer: Sort Kconfig driver entries dm: core: Allow uclass to set up a device's child after it is probed test: dm: core: Add test case for uclass driver's child_post_probe() riscv: bootm: Add dm_remove_devices_flags() call to do_bootm_linux() dm: Add a new uclass driver for VirtIO transport devices virtio: Add virtio over mmio transport driver test: dm: blk: Correct blk_base test case sandbox: blk: Switch to use platdata_auto_alloc_size for the driver data efi_driver: blk: Switch to use platdata_auto_alloc_size for the driver data blk: Call part_init() in the post_probe() method blk: Drop blk_prepare_device() blk: Make blk_next_free_devnum() public riscv: qemu: Enumerate virtio bus during early boot riscv: qemu: Include some useful commands kconfig: Introduce HAVE_ARCH_IOMAP x86: Implement arch-specific io accessor routines virtio: Add virtio over pci transport driver arm: qemu: Add a Kconfig in the board directory arm: qemu: Enumerate virtio bus during early boot x86: qemu: Imply virtio PCI transport and device drivers dm: pci: Add APIs to find next capability and extended capability test: dm: pci: Add cases for finding next PCI capability APIs virtio: pci: Support non-legacy PCI transport device virtio: net: Support non-legacy device virtio: Add a Sandbox transport driver test: dm: virtio: Add test cases for virtio uclass doc: Document virtio support arm: stm32mp: Remove DM_FLAG_PRE_RELOC flag clk: Remove DM_FLAG_PRE_RELOC flag in various drivers gpio: Remove DM_FLAG_PRE_RELOC flag in various drivers i2c: omap24xx: Surround DM_FLAG_PRE_RELOC flag with OF_CONTROL check mmc: omap: Surround DM_FLAG_PRE_RELOC flag with OF_CONTROL check pinctrl: Remove DM_FLAG_PRE_RELOC flag in various drivers ram: bmips: Remove DM_FLAG_PRE_RELOC flag timer: Remove DM_FLAG_PRE_RELOC flag in various drivers serial: Remove DM_FLAG_PRE_RELOC flag in various drivers sysreset: Remove DM_FLAG_PRE_RELOC flag in various drivers video: simplefb: Remove DM_FLAG_PRE_RELOC flag watchdog: Remove DM_FLAG_PRE_RELOC flag in various drivers dm: doc: Update description of pre-relocation support cpu: Add DM_FLAG_PRE_RELOC flag to various cpu drivers cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes Revert "imx8qxp_mek: Disable CONFIG_DISPLAY_CPUINFO" Liviu Dudau (1): uclass: Use uclass_foreach_dev() macro instead of open coding Mario Six (18): board_f: Use static print_cpuinfo if CONFIG_CPU is active test: regmap: Increase size of syscon0 memory regmap: Fix documentation regmap: Add documentation regmap: Improve error handling misc: Sort Makefile entries misc: Add gdsys_soc driver misc: Add IHS FPGA driver core: ofnode: Fix mem leak in error path regmap: Introduce init_range regmap: Add error output mips: Implement {in, out}_{le, be}_{16, 32, 64} and {in, out}_8 regmap: Add raw read/write functions regmap: Support reading from specific range regmap: Define regmap_{get,set} test: regmap: Add test for regmap_{set, get} regmap: Add endianness support regmap: Add overview documentation Simon Glass (2): sandbox: Try to start the RAM buffer at a particular address buildman: Add a --boards option to specify particular boards to build Simon Goldschmidt (1): serial: ns16550: fix debug uart putc called before init Tuomas Tynkkynen (5): virtio: Add codes for virtual queue/ring management virtio: Add net driver support blk: Introduce IF_TYPE_VIRTIO virtio: Add block driver support
Re: [U-Boot] Please pull u-boot-dm
On Tue, Nov 06, 2018 at 07:42:39PM -0700, Simon Glass wrote: > Hi Bin, > > On 6 November 2018 at 19:29, Bin Meng wrote: > > Hi Simon, > > > > On Wed, Nov 7, 2018 at 10:25 AM Simon Glass wrote: > >> > >> Hi Bin, > >> > >> On 6 November 2018 at 19:22, Bin Meng wrote: > >> > > >> > Hi Simon, > >> > > >> > On Wed, Nov 7, 2018 at 10:07 AM Simon Glass wrote: > >> > > > >> > > Hi Bin, > >> > > > >> > > On 6 November 2018 at 18:04, Bin Meng wrote: > >> > > > > >> > > > Hi Simon, > >> > > > > >> > > > On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: > >> > > > > > >> > > > > Hi Tom, > >> > > > > > >> > > > > The following changes since commit > >> > > > > 5ef76e59c12c79d106ebda70b710468aa6bd8b75: > >> > > > > > >> > > > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 > >> > > > > 08:12:21 -0500) > >> > > > > > >> > > > > are available in the Git repository at: > >> > > > > > >> > > > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 > >> > > > > > >> > > > > for you to fetch changes up to > >> > > > > 9413033c3d5d2bc44eefd1919c60522598cc1bd6: > >> > > > > > >> > > > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes > >> > > > > (2018-11-06 13:56:18 -0700) > >> > > > > > >> > > > > > >> > > > > dm: DM_FLAG_PRE_RELOC fixes for release > >> > > > > sandbox CPU fixes for release > >> > > > > > >> > > > > >> > > > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did > >> > > > not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean > >> > > > up series should be a follow-up to that tag. > >> > > > >> > > I abandoned that as I think Tom felt that with the flag problem it was > >> > > a bit to late to fix everything. > >> > > > >> > > If your series relies on that then I think we have a problem... > >> > > >> > Actually this DM_FLAG_PRE_RELOC clean up series is to make previous > >> > series complementary, IOW fix potential issues. Previously Tom > >> > rejected that was because that broken Tegra which Stephen reported, > >> > and with the clean up series this should be fine. > >> > >> We are just a week from the release. What is broken if we apply none > >> of these patches and pull them in after the release? > > > > I suspect we need revert the patch that is causing all these issues > > ... If that's the way we prefer to do, I will take a look ... > > I am happy to make it work - will await Tom's thoughts also. I suppose at this point in time, a revert for release is probably best. -- 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-dm
Hi Bin, On 6 November 2018 at 19:29, Bin Meng wrote: > Hi Simon, > > On Wed, Nov 7, 2018 at 10:25 AM Simon Glass wrote: >> >> Hi Bin, >> >> On 6 November 2018 at 19:22, Bin Meng wrote: >> > >> > Hi Simon, >> > >> > On Wed, Nov 7, 2018 at 10:07 AM Simon Glass wrote: >> > > >> > > Hi Bin, >> > > >> > > On 6 November 2018 at 18:04, Bin Meng wrote: >> > > > >> > > > Hi Simon, >> > > > >> > > > On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: >> > > > > >> > > > > Hi Tom, >> > > > > >> > > > > The following changes since commit >> > > > > 5ef76e59c12c79d106ebda70b710468aa6bd8b75: >> > > > > >> > > > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 >> > > > > 08:12:21 -0500) >> > > > > >> > > > > are available in the Git repository at: >> > > > > >> > > > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 >> > > > > >> > > > > for you to fetch changes up to >> > > > > 9413033c3d5d2bc44eefd1919c60522598cc1bd6: >> > > > > >> > > > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes >> > > > > (2018-11-06 13:56:18 -0700) >> > > > > >> > > > > >> > > > > dm: DM_FLAG_PRE_RELOC fixes for release >> > > > > sandbox CPU fixes for release >> > > > > >> > > > >> > > > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did >> > > > not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean >> > > > up series should be a follow-up to that tag. >> > > >> > > I abandoned that as I think Tom felt that with the flag problem it was >> > > a bit to late to fix everything. >> > > >> > > If your series relies on that then I think we have a problem... >> > >> > Actually this DM_FLAG_PRE_RELOC clean up series is to make previous >> > series complementary, IOW fix potential issues. Previously Tom >> > rejected that was because that broken Tegra which Stephen reported, >> > and with the clean up series this should be fine. >> >> We are just a week from the release. What is broken if we apply none >> of these patches and pull them in after the release? > > I suspect we need revert the patch that is causing all these issues > ... If that's the way we prefer to do, I will take a look ... > I am happy to make it work - will await Tom's thoughts also. 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-dm
Hi Simon, On Wed, Nov 7, 2018 at 10:25 AM Simon Glass wrote: > > Hi Bin, > > On 6 November 2018 at 19:22, Bin Meng wrote: > > > > Hi Simon, > > > > On Wed, Nov 7, 2018 at 10:07 AM Simon Glass wrote: > > > > > > Hi Bin, > > > > > > On 6 November 2018 at 18:04, Bin Meng wrote: > > > > > > > > Hi Simon, > > > > > > > > On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: > > > > > > > > > > Hi Tom, > > > > > > > > > > The following changes since commit > > > > > 5ef76e59c12c79d106ebda70b710468aa6bd8b75: > > > > > > > > > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 > > > > > 08:12:21 -0500) > > > > > > > > > > are available in the Git repository at: > > > > > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 > > > > > > > > > > for you to fetch changes up to > > > > > 9413033c3d5d2bc44eefd1919c60522598cc1bd6: > > > > > > > > > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes > > > > > (2018-11-06 13:56:18 -0700) > > > > > > > > > > > > > > > dm: DM_FLAG_PRE_RELOC fixes for release > > > > > sandbox CPU fixes for release > > > > > > > > > > > > > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did > > > > not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean > > > > up series should be a follow-up to that tag. > > > > > > I abandoned that as I think Tom felt that with the flag problem it was > > > a bit to late to fix everything. > > > > > > If your series relies on that then I think we have a problem... > > > > Actually this DM_FLAG_PRE_RELOC clean up series is to make previous > > series complementary, IOW fix potential issues. Previously Tom > > rejected that was because that broken Tegra which Stephen reported, > > and with the clean up series this should be fine. > > We are just a week from the release. What is broken if we apply none > of these patches and pull them in after the release? I suspect we need revert the patch that is causing all these issues ... If that's the way we prefer to do, I will take a look ... Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
Hi Bin, On 6 November 2018 at 19:22, Bin Meng wrote: > > Hi Simon, > > On Wed, Nov 7, 2018 at 10:07 AM Simon Glass wrote: > > > > Hi Bin, > > > > On 6 November 2018 at 18:04, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: > > > > > > > > Hi Tom, > > > > > > > > The following changes since commit > > > > 5ef76e59c12c79d106ebda70b710468aa6bd8b75: > > > > > > > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 > > > > 08:12:21 -0500) > > > > > > > > are available in the Git repository at: > > > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 > > > > > > > > for you to fetch changes up to 9413033c3d5d2bc44eefd1919c60522598cc1bd6: > > > > > > > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes > > > > (2018-11-06 13:56:18 -0700) > > > > > > > > > > > > dm: DM_FLAG_PRE_RELOC fixes for release > > > > sandbox CPU fixes for release > > > > > > > > > > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did > > > not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean > > > up series should be a follow-up to that tag. > > > > I abandoned that as I think Tom felt that with the flag problem it was > > a bit to late to fix everything. > > > > If your series relies on that then I think we have a problem... > > Actually this DM_FLAG_PRE_RELOC clean up series is to make previous > series complementary, IOW fix potential issues. Previously Tom > rejected that was because that broken Tegra which Stephen reported, > and with the clean up series this should be fine. We are just a week from the release. What is broken if we apply none of these patches and pull them in after the release? 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-dm
Hi Simon, On Wed, Nov 7, 2018 at 10:07 AM Simon Glass wrote: > > Hi Bin, > > On 6 November 2018 at 18:04, Bin Meng wrote: > > > > Hi Simon, > > > > On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: > > > > > > Hi Tom, > > > > > > The following changes since commit > > > 5ef76e59c12c79d106ebda70b710468aa6bd8b75: > > > > > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 > > > 08:12:21 -0500) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 > > > > > > for you to fetch changes up to 9413033c3d5d2bc44eefd1919c60522598cc1bd6: > > > > > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes (2018-11-06 > > > 13:56:18 -0700) > > > > > > > > > dm: DM_FLAG_PRE_RELOC fixes for release > > > sandbox CPU fixes for release > > > > > > > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did > > not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean > > up series should be a follow-up to that tag. > > I abandoned that as I think Tom felt that with the flag problem it was > a bit to late to fix everything. > > If your series relies on that then I think we have a problem... Actually this DM_FLAG_PRE_RELOC clean up series is to make previous series complementary, IOW fix potential issues. Previously Tom rejected that was because that broken Tegra which Stephen reported, and with the clean up series this should be fine. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
Hi Bin, On 6 November 2018 at 18:04, Bin Meng wrote: > > Hi Simon, > > On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: > > > > Hi Tom, > > > > The following changes since commit 5ef76e59c12c79d106ebda70b710468aa6bd8b75: > > > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 08:12:21 > > -0500) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 > > > > for you to fetch changes up to 9413033c3d5d2bc44eefd1919c60522598cc1bd6: > > > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes (2018-11-06 > > 13:56:18 -0700) > > > > > > dm: DM_FLAG_PRE_RELOC fixes for release > > sandbox CPU fixes for release > > > > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did > not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean > up series should be a follow-up to that tag. I abandoned that as I think Tom felt that with the flag problem it was a bit to late to fix everything. If your series relies on that then I think we have a problem... 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-dm
Hi Simon, On Wed, Nov 7, 2018 at 5:51 AM Simon Glass wrote: > > Hi Tom, > > The following changes since commit 5ef76e59c12c79d106ebda70b710468aa6bd8b75: > > Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 08:12:21 > -0500) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 > > for you to fetch changes up to 9413033c3d5d2bc44eefd1919c60522598cc1bd6: > > cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes (2018-11-06 > 13:56:18 -0700) > > > dm: DM_FLAG_PRE_RELOC fixes for release > sandbox CPU fixes for release > What happened to the last PR from u-boot-dm, tag pull-15oct-18? I did not see that was merged to u-boot/master. The DM_FLAG_PRE_RELOC clean up series should be a follow-up to that tag. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] Please pull u-boot-dm
Hi Tom, The following changes since commit 5ef76e59c12c79d106ebda70b710468aa6bd8b75: Merge branch 'master' of git://git.denx.de/u-boot-sh (2018-11-04 08:12:21 -0500) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-6-nov-18 for you to fetch changes up to 9413033c3d5d2bc44eefd1919c60522598cc1bd6: cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes (2018-11-06 13:56:18 -0700) dm: DM_FLAG_PRE_RELOC fixes for release sandbox CPU fixes for release Bin Meng (15): arm: stm32mp: Remove DM_FLAG_PRE_RELOC flag clk: Remove DM_FLAG_PRE_RELOC flag in various drivers gpio: Remove DM_FLAG_PRE_RELOC flag in various drivers i2c: omap24xx: Surround DM_FLAG_PRE_RELOC flag with OF_CONTROL check mmc: omap: Surround DM_FLAG_PRE_RELOC flag with OF_CONTROL check pinctrl: Remove DM_FLAG_PRE_RELOC flag in various drivers ram: bmips: Remove DM_FLAG_PRE_RELOC flag timer: Remove DM_FLAG_PRE_RELOC flag in various drivers serial: Remove DM_FLAG_PRE_RELOC flag in various drivers sysreset: Remove DM_FLAG_PRE_RELOC flag in various drivers video: simplefb: Remove DM_FLAG_PRE_RELOC flag watchdog: Remove DM_FLAG_PRE_RELOC flag in various drivers dm: doc: Update description of pre-relocation support cpu: Add DM_FLAG_PRE_RELOC flag to various cpu drivers cpu: sandbox: Add "u-boot, dm-pre-reloc" for all cpu nodes arch/arm/mach-stm32mp/bsec.c | 1 - arch/sandbox/dts/test.dts| 3 +++ arch/x86/cpu/baytrail/cpu.c | 1 + arch/x86/cpu/broadwell/cpu.c | 1 + arch/x86/cpu/cpu_x86.c | 1 + arch/x86/cpu/ivybridge/model_206ax.c | 1 + arch/x86/cpu/tangier/sysreset.c | 1 - doc/driver-model/README.txt | 16 drivers/clk/altera/clk-arria10.c | 1 - drivers/clk/clk_pic32.c | 1 - drivers/clk/clk_zynq.c | 1 - drivers/clk/exynos/clk-exynos7420.c | 3 --- drivers/clk/owl/clk_s900.c | 1 - drivers/gpio/omap_gpio.c | 2 ++ drivers/gpio/stm32f7_gpio.c | 2 +- drivers/gpio/tegra186_gpio.c | 1 - drivers/gpio/tegra_gpio.c| 1 - drivers/i2c/omap24xx_i2c.c | 2 ++ drivers/mmc/omap_hsmmc.c | 2 ++ drivers/pinctrl/broadcom/pinctrl-bcm283x.c | 2 ++ drivers/pinctrl/exynos/pinctrl-exynos7420.c | 1 - drivers/pinctrl/nxp/pinctrl-imx5.c | 2 ++ drivers/pinctrl/nxp/pinctrl-imx6.c | 2 ++ drivers/pinctrl/nxp/pinctrl-imx7.c | 2 ++ drivers/pinctrl/nxp/pinctrl-imx7ulp.c| 2 ++ drivers/pinctrl/pinctrl-single.c | 1 - drivers/pinctrl/uniphier/pinctrl-uniphier-pro4.c | 2 ++ drivers/pinctrl/uniphier/pinctrl-uniphier-pro5.c | 2 ++ drivers/ram/bmips_ram.c | 1 - drivers/serial/altera_jtag_uart.c| 1 - drivers/serial/altera_uart.c | 1 - drivers/serial/arm_dcc.c | 1 - drivers/serial/atmel_usart.c | 2 ++ drivers/serial/ns16550.c | 2 ++ drivers/serial/serial_ar933x.c | 1 - drivers/serial/serial_arc.c | 1 - drivers/serial/serial_bcm283x_mu.c | 2 ++ drivers/serial/serial_bcm283x_pl011.c| 2 ++ drivers/serial/serial_bcm6345.c | 1 - drivers/serial/serial_efi.c | 1 - drivers/serial/serial_intel_mid.c| 1 - drivers/serial/serial_lpuart.c | 1 - drivers/serial/serial_meson.c| 1 - drivers/serial/serial_mvebu_a3700.c | 1 - drivers/serial/serial_mxc.c | 2 ++ drivers/serial/serial_omap.c | 2 ++ drivers/serial/serial_owl.c | 1 - drivers/serial/serial_pic32.c| 1 - drivers/serial/serial_pl01x.c| 2 ++ drivers/serial/serial_s5p.c | 1 - drivers/serial/serial_sh.c | 2 ++ drivers/serial/serial_sti_asc.c | 1 - drivers/serial/serial_stm32.c| 2 ++ drivers/serial/serial_xuartlite.c| 1 - drivers/serial/serial_zynq.c | 1 - drivers/sysreset/sysreset_x86.c | 1 - drivers/timer/ag101p_timer.c | 1 - drivers/timer/altera_timer.c | 1 - drivers/timer/arc_timer.c| 1 - drivers/timer/ast_timer.c
Re: [U-Boot] Please pull u-boot-dm
On Wed, Oct 24, 2018 at 08:33:01AM -0600, Simon Glass wrote: > Hi Tom, > > On 24 October 2018 at 08:28, Bin Meng wrote: > > Hi Simon, > > > > On Wed, Oct 24, 2018 at 10:14 PM Simon Glass wrote: > >> > >> Hi Tom, > >> > >> On 19 October 2018 at 06:10, Tom Rini wrote: > >> > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: > >> >> Hi, > >> >> > >> >> On 17 October 2018 at 11:14, Stephen Warren > >> >> wrote: > >> >> > On 10/16/18 8:33 PM, Bin Meng wrote: > >> >> >> > >> >> >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > >> >> >>> > >> >> >>> > >> >> >>> Hi Stephen, > >> >> >>> > >> >> >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > >> >> >>> > >> >> >>> wrote: > >> >> > >> >> > >> >> On 10/15/18 5:43 PM, Bin Meng wrote: > >> >> > > >> >> > Hi Stephen, > >> >> > > >> >> > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > >> >> > > >> >> > wrote: > >> >> >> > >> >> >> > >> >> >> On 10/15/18 3:53 PM, Stephen Warren wrote: > >> >> >>> > >> >> >>> On 10/15/18 2:17 PM, Simon Glass wrote: > >> >> > >> >> Hi Tom, > >> >> > >> >> The following changes since commit > >> >> 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > >> >> > >> >> Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > >> >> git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > >> >> > >> >> are available in the Git repository at: > >> >> > >> >> git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > >> >> > >> >> for you to fetch changes up to > >> >> 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > >> >> > >> >> buildman: Add a --boards option to specify particular > >> >> boards to > >> >> build (2018-10-15 08:20:43 -0600) > >> >> >>> > >> >> >>> > >> >> >>> There's something wrong with these patches; both Jetson TX1 and > >> >> >>> Jetson > >> >> >>> TX2 (both 64-bit ARM without SPL) fail to boot with this > >> >> >>> branch; the > >> >> >>> system hangs with no output at all from U-Boot. Jetson TK1 > >> >> >>> (32-bit > >> >> >>> ARM > >> >> >>> with SPL) and sandbox boot fine. > >> >> >> > >> >> >> > >> >> >> Reverting the following solves the issue: > >> >> >> > >> >> >> 1) dm: core: Mirror the chosen node parse logic in the livetree > >> >> >> scanning > >> >> >> > >> >> >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > >> >> >> lists_bind_fdt() > >> >> >> > >> >> >> Reverting just (1) or just (2) does not. Can you please take a > >> >> >> look? > >> >> >> Thanks. > >> >> > > >> >> > > >> >> > The failure is probably due to some drivers that Tegra uses are > >> >> > not > >> >> > properly written, ie: these drivers are declared via U_BOOT_DRIVER > >> >> > with flags DM_FLAG_PRE_RELOC yet it was never working (bound > >> >> > before > >> >> > relocation) until patch (1) & (2). Now these drivers get bound > >> >> > before > >> >> > relocation but they were never bound before (assume they were > >> >> > never > >> >> > required before relocation), so we should identify which driver > >> >> > were > >> >> > wrongly written, but as a quick solution, can you please enlarge > >> >> > the > >> >> > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > >> >> > >> >> > >> >> Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves > >> >> the > >> >> issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > >> >> tegra186_gpio.c (depending on board/SoC) also solves this. > >> >> > >> >> I suppose the correct solution is to remove the DM flag from the > >> >> driver, > >> >> since if it never used to work before your patches, it's clear > >> >> that the > >> >> flag/feature is not actually required. > >> >> >>> > >> >> >>> > >> >> >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio > >> >> >>> driver. > >> >> > >> >> Thanks Bin, > >> >> > >> >> >> > >> >> >> > >> >> >> Ah, it turns out so many drivers/dts are mis-written ... > >> >> >> > >> >> >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > >> >> >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > >> >> >> have been added to workaround the DM bug (fixed in patch 1 & 2 > >> >> >> mentioned in this thread). But now since the DM bug has been fixed, > >> >> >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I > >> >> >> am > >> >> >> afraid I don't have the knowledge to clean up all of these. > >> >> >> > >> >> >> Of course, I've identified some additional drivers that have > >> >> >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have > >> >> >> "u-boot,dm-pre-reloc". These drivers should be
Re: [U-Boot] Please pull u-boot-dm
Hi Tom, On 24 October 2018 at 08:28, Bin Meng wrote: > Hi Simon, > > On Wed, Oct 24, 2018 at 10:14 PM Simon Glass wrote: >> >> Hi Tom, >> >> On 19 October 2018 at 06:10, Tom Rini wrote: >> > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: >> >> Hi, >> >> >> >> On 17 October 2018 at 11:14, Stephen Warren wrote: >> >> > On 10/16/18 8:33 PM, Bin Meng wrote: >> >> >> >> >> >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: >> >> >>> >> >> >>> >> >> >>> Hi Stephen, >> >> >>> >> >> >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren >> >> >>> >> >> >>> wrote: >> >> >> >> >> >> On 10/15/18 5:43 PM, Bin Meng wrote: >> >> > >> >> > Hi Stephen, >> >> > >> >> > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren >> >> > >> >> > wrote: >> >> >> >> >> >> >> >> >> On 10/15/18 3:53 PM, Stephen Warren wrote: >> >> >>> >> >> >>> On 10/15/18 2:17 PM, Simon Glass wrote: >> >> >> >> Hi Tom, >> >> >> >> The following changes since commit >> >> 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: >> >> >> >> Merge tag 'arc-more-updates-for-2018.11-rc2-2' of >> >> git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) >> >> >> >> are available in the Git repository at: >> >> >> >> git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 >> >> >> >> for you to fetch changes up to >> >> 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: >> >> >> >> buildman: Add a --boards option to specify particular >> >> boards to >> >> build (2018-10-15 08:20:43 -0600) >> >> >>> >> >> >>> >> >> >>> There's something wrong with these patches; both Jetson TX1 and >> >> >>> Jetson >> >> >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; >> >> >>> the >> >> >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit >> >> >>> ARM >> >> >>> with SPL) and sandbox boot fine. >> >> >> >> >> >> >> >> >> Reverting the following solves the issue: >> >> >> >> >> >> 1) dm: core: Mirror the chosen node parse logic in the livetree >> >> >> scanning >> >> >> >> >> >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in >> >> >> lists_bind_fdt() >> >> >> >> >> >> Reverting just (1) or just (2) does not. Can you please take a >> >> >> look? >> >> >> Thanks. >> >> > >> >> > >> >> > The failure is probably due to some drivers that Tegra uses are not >> >> > properly written, ie: these drivers are declared via U_BOOT_DRIVER >> >> > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before >> >> > relocation) until patch (1) & (2). Now these drivers get bound >> >> > before >> >> > relocation but they were never bound before (assume they were never >> >> > required before relocation), so we should identify which driver were >> >> > wrongly written, but as a quick solution, can you please enlarge the >> >> > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? >> >> >> >> >> >> Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the >> >> issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or >> >> tegra186_gpio.c (depending on board/SoC) also solves this. >> >> >> >> I suppose the correct solution is to remove the DM flag from the >> >> driver, >> >> since if it never used to work before your patches, it's clear that >> >> the >> >> flag/feature is not actually required. >> >> >>> >> >> >>> >> >> >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio >> >> >>> driver. >> >> >> >> Thanks Bin, >> >> >> >> >> >> >> >> >> >> >> Ah, it turns out so many drivers/dts are mis-written ... >> >> >> >> >> >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts >> >> >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must >> >> >> have been added to workaround the DM bug (fixed in patch 1 & 2 >> >> >> mentioned in this thread). But now since the DM bug has been fixed, >> >> >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am >> >> >> afraid I don't have the knowledge to clean up all of these. >> >> >> >> >> >> Of course, I've identified some additional drivers that have >> >> >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have >> >> >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the >> >> >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. >> >> > >> >> > >> >> > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the >> >> > Tegra >> >> > GPIO driver specifically, so I think a simple patch removing the >> >> > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of >> >> > course, I didn't look at any non-Tegra DTs, or at drivers other than >> >> >
Re: [U-Boot] Please pull u-boot-dm
Hi Simon, On Wed, Oct 24, 2018 at 10:14 PM Simon Glass wrote: > > Hi Tom, > > On 19 October 2018 at 06:10, Tom Rini wrote: > > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: > >> Hi, > >> > >> On 17 October 2018 at 11:14, Stephen Warren wrote: > >> > On 10/16/18 8:33 PM, Bin Meng wrote: > >> >> > >> >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > >> >>> > >> >>> > >> >>> Hi Stephen, > >> >>> > >> >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > >> >>> wrote: > >> > >> > >> On 10/15/18 5:43 PM, Bin Meng wrote: > >> > > >> > Hi Stephen, > >> > > >> > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > >> > > >> > wrote: > >> >> > >> >> > >> >> On 10/15/18 3:53 PM, Stephen Warren wrote: > >> >>> > >> >>> On 10/15/18 2:17 PM, Simon Glass wrote: > >> > >> Hi Tom, > >> > >> The following changes since commit > >> 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > >> > >> Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > >> git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > >> > >> are available in the Git repository at: > >> > >> git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > >> > >> for you to fetch changes up to > >> 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > >> > >> buildman: Add a --boards option to specify particular boards > >> to > >> build (2018-10-15 08:20:43 -0600) > >> >>> > >> >>> > >> >>> There's something wrong with these patches; both Jetson TX1 and > >> >>> Jetson > >> >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; > >> >>> the > >> >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit > >> >>> ARM > >> >>> with SPL) and sandbox boot fine. > >> >> > >> >> > >> >> Reverting the following solves the issue: > >> >> > >> >> 1) dm: core: Mirror the chosen node parse logic in the livetree > >> >> scanning > >> >> > >> >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > >> >> lists_bind_fdt() > >> >> > >> >> Reverting just (1) or just (2) does not. Can you please take a look? > >> >> Thanks. > >> > > >> > > >> > The failure is probably due to some drivers that Tegra uses are not > >> > properly written, ie: these drivers are declared via U_BOOT_DRIVER > >> > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > >> > relocation) until patch (1) & (2). Now these drivers get bound before > >> > relocation but they were never bound before (assume they were never > >> > required before relocation), so we should identify which driver were > >> > wrongly written, but as a quick solution, can you please enlarge the > >> > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > >> > >> > >> Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > >> issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > >> tegra186_gpio.c (depending on board/SoC) also solves this. > >> > >> I suppose the correct solution is to remove the DM flag from the > >> driver, > >> since if it never used to work before your patches, it's clear that > >> the > >> flag/feature is not actually required. > >> >>> > >> >>> > >> >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio > >> >>> driver. > >> > >> Thanks Bin, > >> > >> >> > >> >> > >> >> Ah, it turns out so many drivers/dts are mis-written ... > >> >> > >> >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > >> >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > >> >> have been added to workaround the DM bug (fixed in patch 1 & 2 > >> >> mentioned in this thread). But now since the DM bug has been fixed, > >> >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am > >> >> afraid I don't have the knowledge to clean up all of these. > >> >> > >> >> Of course, I've identified some additional drivers that have > >> >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have > >> >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the > >> >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. > >> > > >> > > >> > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the > >> > Tegra > >> > GPIO driver specifically, so I think a simple patch removing the > >> > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of > >> > course, I didn't look at any non-Tegra DTs, or at drivers other than > >> > Tegra > >> > GPIO. > >> > >> OK. > >> > >> I tend to agree that it is getting a bit late. I am considering > >> delaying this pull request until the next merge window. I still have > >> regmap and virtio to bring in. Tom
Re: [U-Boot] Please pull u-boot-dm
Hi BIn, On 24 October 2018 at 07:40, Bin Meng wrote: > Hi Simon, > > On Mon, Oct 22, 2018 at 7:11 PM Tom Rini wrote: >> >> On Mon, Oct 22, 2018 at 05:46:20PM +0800, Bin Meng wrote: >> > Hi Simon, >> > >> > On Fri, Oct 19, 2018 at 8:10 PM Tom Rini wrote: >> > > >> > > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: >> > > > Hi, >> > > > >> > > > On 17 October 2018 at 11:14, Stephen Warren >> > > > wrote: >> > > > > On 10/16/18 8:33 PM, Bin Meng wrote: >> > > > >> >> > > > >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: >> > > > >>> >> > > > >>> >> > > > >>> Hi Stephen, >> > > > >>> >> > > > >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren >> > > > >>> >> > > > >>> wrote: >> > > > >> > > > >> > > > On 10/15/18 5:43 PM, Bin Meng wrote: >> > > > > >> > > > > Hi Stephen, >> > > > > >> > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren >> > > > > >> > > > > wrote: >> > > > >> >> > > > >> >> > > > >> On 10/15/18 3:53 PM, Stephen Warren wrote: >> > > > >>> >> > > > >>> On 10/15/18 2:17 PM, Simon Glass wrote: >> > > > >> > > > Hi Tom, >> > > > >> > > > The following changes since commit >> > > > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: >> > > > >> > > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of >> > > > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) >> > > > >> > > > are available in the Git repository at: >> > > > >> > > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 >> > > > >> > > > for you to fetch changes up to >> > > > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: >> > > > >> > > > buildman: Add a --boards option to specify particular >> > > > boards to >> > > > build (2018-10-15 08:20:43 -0600) >> > > > >>> >> > > > >>> >> > > > >>> There's something wrong with these patches; both Jetson TX1 and >> > > > >>> Jetson >> > > > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this >> > > > >>> branch; the >> > > > >>> system hangs with no output at all from U-Boot. Jetson TK1 >> > > > >>> (32-bit >> > > > >>> ARM >> > > > >>> with SPL) and sandbox boot fine. >> > > > >> >> > > > >> >> > > > >> Reverting the following solves the issue: >> > > > >> >> > > > >> 1) dm: core: Mirror the chosen node parse logic in the livetree >> > > > >> scanning >> > > > >> >> > > > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in >> > > > >> lists_bind_fdt() >> > > > >> >> > > > >> Reverting just (1) or just (2) does not. Can you please take a >> > > > >> look? >> > > > >> Thanks. >> > > > > >> > > > > >> > > > > The failure is probably due to some drivers that Tegra uses are >> > > > > not >> > > > > properly written, ie: these drivers are declared via >> > > > > U_BOOT_DRIVER >> > > > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound >> > > > > before >> > > > > relocation) until patch (1) & (2). Now these drivers get bound >> > > > > before >> > > > > relocation but they were never bound before (assume they were >> > > > > never >> > > > > required before relocation), so we should identify which driver >> > > > > were >> > > > > wrongly written, but as a quick solution, can you please enlarge >> > > > > the >> > > > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? >> > > > >> > > > >> > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves >> > > > the >> > > > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c >> > > > or >> > > > tegra186_gpio.c (depending on board/SoC) also solves this. >> > > > >> > > > I suppose the correct solution is to remove the DM flag from the >> > > > driver, >> > > > since if it never used to work before your patches, it's clear >> > > > that the >> > > > flag/feature is not actually required. >> > > > >>> >> > > > >>> >> > > > >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio >> > > > >>> driver. >> > > > >> > > > Thanks Bin, >> > > > >> > > > >> >> > > > >> >> > > > >> Ah, it turns out so many drivers/dts are mis-written ... >> > > > >> >> > > > >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts >> > > > >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must >> > > > >> have been added to workaround the DM bug (fixed in patch 1 & 2 >> > > > >> mentioned in this thread). But now since the DM bug has been fixed, >> > > > >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I >> > > > >> am >> > > > >> afraid I don't have the knowledge to clean up all of these. >> > > > >> >> > > > >> Of course, I've identified some additio
Re: [U-Boot] Please pull u-boot-dm
Hi Tom, On 19 October 2018 at 06:10, Tom Rini wrote: > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: >> Hi, >> >> On 17 October 2018 at 11:14, Stephen Warren wrote: >> > On 10/16/18 8:33 PM, Bin Meng wrote: >> >> >> >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: >> >>> >> >>> >> >>> Hi Stephen, >> >>> >> >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren >> >>> wrote: >> >> >> On 10/15/18 5:43 PM, Bin Meng wrote: >> > >> > Hi Stephen, >> > >> > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren >> > wrote: >> >> >> >> >> >> On 10/15/18 3:53 PM, Stephen Warren wrote: >> >>> >> >>> On 10/15/18 2:17 PM, Simon Glass wrote: >> >> Hi Tom, >> >> The following changes since commit >> 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: >> >> Merge tag 'arc-more-updates-for-2018.11-rc2-2' of >> git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) >> >> are available in the Git repository at: >> >> git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 >> >> for you to fetch changes up to >> 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: >> >> buildman: Add a --boards option to specify particular boards to >> build (2018-10-15 08:20:43 -0600) >> >>> >> >>> >> >>> There's something wrong with these patches; both Jetson TX1 and >> >>> Jetson >> >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the >> >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit >> >>> ARM >> >>> with SPL) and sandbox boot fine. >> >> >> >> >> >> Reverting the following solves the issue: >> >> >> >> 1) dm: core: Mirror the chosen node parse logic in the livetree >> >> scanning >> >> >> >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in >> >> lists_bind_fdt() >> >> >> >> Reverting just (1) or just (2) does not. Can you please take a look? >> >> Thanks. >> > >> > >> > The failure is probably due to some drivers that Tegra uses are not >> > properly written, ie: these drivers are declared via U_BOOT_DRIVER >> > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before >> > relocation) until patch (1) & (2). Now these drivers get bound before >> > relocation but they were never bound before (assume they were never >> > required before relocation), so we should identify which driver were >> > wrongly written, but as a quick solution, can you please enlarge the >> > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? >> >> >> Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the >> issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or >> tegra186_gpio.c (depending on board/SoC) also solves this. >> >> I suppose the correct solution is to remove the DM flag from the driver, >> since if it never used to work before your patches, it's clear that the >> flag/feature is not actually required. >> >>> >> >>> >> >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio >> >>> driver. >> >> Thanks Bin, >> >> >> >> >> >> >> Ah, it turns out so many drivers/dts are mis-written ... >> >> >> >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts >> >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must >> >> have been added to workaround the DM bug (fixed in patch 1 & 2 >> >> mentioned in this thread). But now since the DM bug has been fixed, >> >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am >> >> afraid I don't have the knowledge to clean up all of these. >> >> >> >> Of course, I've identified some additional drivers that have >> >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have >> >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the >> >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. >> > >> > >> > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the Tegra >> > GPIO driver specifically, so I think a simple patch removing the >> > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of >> > course, I didn't look at any non-Tegra DTs, or at drivers other than Tegra >> > GPIO. >> >> OK. >> >> I tend to agree that it is getting a bit late. I am considering >> delaying this pull request until the next merge window. I still have >> regmap and virtio to bring in. Tom, what do you think? >> > > Start the PR over and look for things it really makes sense to grab at > -rc2 (bugfixes, Kconfig migrations, etc) is what I think we need to do > at this point, thanks! Just to follow up, I really don't see anything that fits that category. Bin, are any of your patches necessary for this release to actually fix boards? Regards, Simon ___
Re: [U-Boot] Please pull u-boot-dm
Hi Simon, On Mon, Oct 22, 2018 at 7:11 PM Tom Rini wrote: > > On Mon, Oct 22, 2018 at 05:46:20PM +0800, Bin Meng wrote: > > Hi Simon, > > > > On Fri, Oct 19, 2018 at 8:10 PM Tom Rini wrote: > > > > > > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: > > > > Hi, > > > > > > > > On 17 October 2018 at 11:14, Stephen Warren > > > > wrote: > > > > > On 10/16/18 8:33 PM, Bin Meng wrote: > > > > >> > > > > >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > > > > >>> > > > > >>> > > > > >>> Hi Stephen, > > > > >>> > > > > >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > > > > >>> > > > > >>> wrote: > > > > > > > > > > > > On 10/15/18 5:43 PM, Bin Meng wrote: > > > > > > > > > > Hi Stephen, > > > > > > > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > > > > > > > > > wrote: > > > > >> > > > > >> > > > > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > > > > >>> > > > > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > > > > > > > > Hi Tom, > > > > > > > > The following changes since commit > > > > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > > > > > > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > > > > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > > > > > > > are available in the Git repository at: > > > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > > > > > > > for you to fetch changes up to > > > > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > > > > > > > buildman: Add a --boards option to specify particular > > > > boards to > > > > build (2018-10-15 08:20:43 -0600) > > > > >>> > > > > >>> > > > > >>> There's something wrong with these patches; both Jetson TX1 and > > > > >>> Jetson > > > > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this > > > > >>> branch; the > > > > >>> system hangs with no output at all from U-Boot. Jetson TK1 > > > > >>> (32-bit > > > > >>> ARM > > > > >>> with SPL) and sandbox boot fine. > > > > >> > > > > >> > > > > >> Reverting the following solves the issue: > > > > >> > > > > >> 1) dm: core: Mirror the chosen node parse logic in the livetree > > > > >> scanning > > > > >> > > > > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > > > > >> lists_bind_fdt() > > > > >> > > > > >> Reverting just (1) or just (2) does not. Can you please take a > > > > >> look? > > > > >> Thanks. > > > > > > > > > > > > > > > The failure is probably due to some drivers that Tegra uses are > > > > > not > > > > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > > > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound > > > > > before > > > > > relocation) until patch (1) & (2). Now these drivers get bound > > > > > before > > > > > relocation but they were never bound before (assume they were > > > > > never > > > > > required before relocation), so we should identify which driver > > > > > were > > > > > wrongly written, but as a quick solution, can you please enlarge > > > > > the > > > > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > > > > > > > > > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves > > > > the > > > > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > > > > tegra186_gpio.c (depending on board/SoC) also solves this. > > > > > > > > I suppose the correct solution is to remove the DM flag from the > > > > driver, > > > > since if it never used to work before your patches, it's clear > > > > that the > > > > flag/feature is not actually required. > > > > >>> > > > > >>> > > > > >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio > > > > >>> driver. > > > > > > > > Thanks Bin, > > > > > > > > >> > > > > >> > > > > >> Ah, it turns out so many drivers/dts are mis-written ... > > > > >> > > > > >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > > > > >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > > > > >> have been added to workaround the DM bug (fixed in patch 1 & 2 > > > > >> mentioned in this thread). But now since the DM bug has been fixed, > > > > >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I > > > > >> am > > > > >> afraid I don't have the knowledge to clean up all of these. > > > > >> > > > > >> Of course, I've identified some additional drivers that have > > > > >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have > > > > >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the > > > > >> DM_FLAG_PRE_RELOC flag like the Tegra G
Re: [U-Boot] Please pull u-boot-dm
On Mon, Oct 22, 2018 at 05:46:20PM +0800, Bin Meng wrote: > Hi Simon, > > On Fri, Oct 19, 2018 at 8:10 PM Tom Rini wrote: > > > > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: > > > Hi, > > > > > > On 17 October 2018 at 11:14, Stephen Warren wrote: > > > > On 10/16/18 8:33 PM, Bin Meng wrote: > > > >> > > > >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > > > >>> > > > >>> > > > >>> Hi Stephen, > > > >>> > > > >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > > > >>> > > > >>> wrote: > > > > > > > > > On 10/15/18 5:43 PM, Bin Meng wrote: > > > > > > > > Hi Stephen, > > > > > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > > > > > > > wrote: > > > >> > > > >> > > > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > > > >>> > > > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > > > > > > Hi Tom, > > > > > > The following changes since commit > > > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > > > > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > > > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > > > > > for you to fetch changes up to > > > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > > > > > buildman: Add a --boards option to specify particular > > > boards to > > > build (2018-10-15 08:20:43 -0600) > > > >>> > > > >>> > > > >>> There's something wrong with these patches; both Jetson TX1 and > > > >>> Jetson > > > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; > > > >>> the > > > >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit > > > >>> ARM > > > >>> with SPL) and sandbox boot fine. > > > >> > > > >> > > > >> Reverting the following solves the issue: > > > >> > > > >> 1) dm: core: Mirror the chosen node parse logic in the livetree > > > >> scanning > > > >> > > > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > > > >> lists_bind_fdt() > > > >> > > > >> Reverting just (1) or just (2) does not. Can you please take a > > > >> look? > > > >> Thanks. > > > > > > > > > > > > The failure is probably due to some drivers that Tegra uses are not > > > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > > > > relocation) until patch (1) & (2). Now these drivers get bound > > > > before > > > > relocation but they were never bound before (assume they were never > > > > required before relocation), so we should identify which driver were > > > > wrongly written, but as a quick solution, can you please enlarge the > > > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > > > > > > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > > > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > > > tegra186_gpio.c (depending on board/SoC) also solves this. > > > > > > I suppose the correct solution is to remove the DM flag from the > > > driver, > > > since if it never used to work before your patches, it's clear that > > > the > > > flag/feature is not actually required. > > > >>> > > > >>> > > > >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio > > > >>> driver. > > > > > > Thanks Bin, > > > > > > >> > > > >> > > > >> Ah, it turns out so many drivers/dts are mis-written ... > > > >> > > > >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > > > >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > > > >> have been added to workaround the DM bug (fixed in patch 1 & 2 > > > >> mentioned in this thread). But now since the DM bug has been fixed, > > > >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am > > > >> afraid I don't have the knowledge to clean up all of these. > > > >> > > > >> Of course, I've identified some additional drivers that have > > > >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have > > > >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the > > > >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. > > > > > > > > > > > > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the > > > > Tegra > > > > GPIO driver specifically, so I think a simple patch removing the > > > > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of > > > > course, I didn't look at any non-Tegra DTs, or at drivers other than > > > > Tegra > > > > GPIO. > > > > > > OK. > > > > > > I tend to agree
Re: [U-Boot] Please pull u-boot-dm
Hi Simon, On Fri, Oct 19, 2018 at 8:10 PM Tom Rini wrote: > > On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: > > Hi, > > > > On 17 October 2018 at 11:14, Stephen Warren wrote: > > > On 10/16/18 8:33 PM, Bin Meng wrote: > > >> > > >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > > >>> > > >>> > > >>> Hi Stephen, > > >>> > > >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > > >>> wrote: > > > > > > On 10/15/18 5:43 PM, Bin Meng wrote: > > > > > > Hi Stephen, > > > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > > wrote: > > >> > > >> > > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > > >>> > > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > > > > Hi Tom, > > > > The following changes since commit > > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > > > for you to fetch changes up to > > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > > > buildman: Add a --boards option to specify particular boards > > to > > build (2018-10-15 08:20:43 -0600) > > >>> > > >>> > > >>> There's something wrong with these patches; both Jetson TX1 and > > >>> Jetson > > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > > >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit > > >>> ARM > > >>> with SPL) and sandbox boot fine. > > >> > > >> > > >> Reverting the following solves the issue: > > >> > > >> 1) dm: core: Mirror the chosen node parse logic in the livetree > > >> scanning > > >> > > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > > >> lists_bind_fdt() > > >> > > >> Reverting just (1) or just (2) does not. Can you please take a look? > > >> Thanks. > > > > > > > > > The failure is probably due to some drivers that Tegra uses are not > > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > > > relocation) until patch (1) & (2). Now these drivers get bound before > > > relocation but they were never bound before (assume they were never > > > required before relocation), so we should identify which driver were > > > wrongly written, but as a quick solution, can you please enlarge the > > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > > > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > > tegra186_gpio.c (depending on board/SoC) also solves this. > > > > I suppose the correct solution is to remove the DM flag from the > > driver, > > since if it never used to work before your patches, it's clear that the > > flag/feature is not actually required. > > >>> > > >>> > > >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio > > >>> driver. > > > > Thanks Bin, > > > > >> > > >> > > >> Ah, it turns out so many drivers/dts are mis-written ... > > >> > > >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > > >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > > >> have been added to workaround the DM bug (fixed in patch 1 & 2 > > >> mentioned in this thread). But now since the DM bug has been fixed, > > >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am > > >> afraid I don't have the knowledge to clean up all of these. > > >> > > >> Of course, I've identified some additional drivers that have > > >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have > > >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the > > >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. > > > > > > > > > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the Tegra > > > GPIO driver specifically, so I think a simple patch removing the > > > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of > > > course, I didn't look at any non-Tegra DTs, or at drivers other than Tegra > > > GPIO. > > > > OK. > > > > I tend to agree that it is getting a bit late. I am considering > > delaying this pull request until the next merge window. I still have > > regmap and virtio to bring in. Tom, what do you think? > > > > Start the PR over and look for things it really makes sense to grab at > -rc2 (bugfixes, Kconfig migrations, etc) is what I think we need to do > at this point, thanks! I just realized that without my fix in
Re: [U-Boot] Please pull u-boot-dm
On Thu, Oct 18, 2018 at 09:27:49PM -0600, Simon Glass wrote: > Hi, > > On 17 October 2018 at 11:14, Stephen Warren wrote: > > On 10/16/18 8:33 PM, Bin Meng wrote: > >> > >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > >>> > >>> > >>> Hi Stephen, > >>> > >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > >>> wrote: > > > On 10/15/18 5:43 PM, Bin Meng wrote: > > > > Hi Stephen, > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > wrote: > >> > >> > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > >>> > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > > Hi Tom, > > The following changes since commit > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > for you to fetch changes up to > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > buildman: Add a --boards option to specify particular boards to > build (2018-10-15 08:20:43 -0600) > >>> > >>> > >>> There's something wrong with these patches; both Jetson TX1 and > >>> Jetson > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit > >>> ARM > >>> with SPL) and sandbox boot fine. > >> > >> > >> Reverting the following solves the issue: > >> > >> 1) dm: core: Mirror the chosen node parse logic in the livetree > >> scanning > >> > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > >> lists_bind_fdt() > >> > >> Reverting just (1) or just (2) does not. Can you please take a look? > >> Thanks. > > > > > > The failure is probably due to some drivers that Tegra uses are not > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > > relocation) until patch (1) & (2). Now these drivers get bound before > > relocation but they were never bound before (assume they were never > > required before relocation), so we should identify which driver were > > wrongly written, but as a quick solution, can you please enlarge the > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > tegra186_gpio.c (depending on board/SoC) also solves this. > > I suppose the correct solution is to remove the DM flag from the driver, > since if it never used to work before your patches, it's clear that the > flag/feature is not actually required. > >>> > >>> > >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio > >>> driver. > > Thanks Bin, > > >> > >> > >> Ah, it turns out so many drivers/dts are mis-written ... > >> > >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > >> have been added to workaround the DM bug (fixed in patch 1 & 2 > >> mentioned in this thread). But now since the DM bug has been fixed, > >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am > >> afraid I don't have the knowledge to clean up all of these. > >> > >> Of course, I've identified some additional drivers that have > >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have > >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the > >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. > > > > > > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the Tegra > > GPIO driver specifically, so I think a simple patch removing the > > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of > > course, I didn't look at any non-Tegra DTs, or at drivers other than Tegra > > GPIO. > > OK. > > I tend to agree that it is getting a bit late. I am considering > delaying this pull request until the next merge window. I still have > regmap and virtio to bring in. Tom, what do you think? > Start the PR over and look for things it really makes sense to grab at -rc2 (bugfixes, Kconfig migrations, etc) is what I think we need to do at this point, thanks! -- 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-dm
Hi, On 17 October 2018 at 11:14, Stephen Warren wrote: > On 10/16/18 8:33 PM, Bin Meng wrote: >> >> On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: >>> >>> >>> Hi Stephen, >>> >>> On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren >>> wrote: On 10/15/18 5:43 PM, Bin Meng wrote: > > Hi Stephen, > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > wrote: >> >> >> On 10/15/18 3:53 PM, Stephen Warren wrote: >>> >>> On 10/15/18 2:17 PM, Simon Glass wrote: Hi Tom, The following changes since commit 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: Merge tag 'arc-more-updates-for-2018.11-rc2-2' of git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: buildman: Add a --boards option to specify particular boards to build (2018-10-15 08:20:43 -0600) >>> >>> >>> There's something wrong with these patches; both Jetson TX1 and >>> Jetson >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit >>> ARM >>> with SPL) and sandbox boot fine. >> >> >> Reverting the following solves the issue: >> >> 1) dm: core: Mirror the chosen node parse logic in the livetree >> scanning >> >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in >> lists_bind_fdt() >> >> Reverting just (1) or just (2) does not. Can you please take a look? >> Thanks. > > > The failure is probably due to some drivers that Tegra uses are not > properly written, ie: these drivers are declared via U_BOOT_DRIVER > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > relocation) until patch (1) & (2). Now these drivers get bound before > relocation but they were never bound before (assume they were never > required before relocation), so we should identify which driver were > wrongly written, but as a quick solution, can you please enlarge the > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or tegra186_gpio.c (depending on board/SoC) also solves this. I suppose the correct solution is to remove the DM flag from the driver, since if it never used to work before your patches, it's clear that the flag/feature is not actually required. >>> >>> >>> Thank you Stephen. I will prepare a patch to update the tegra*_gpio >>> driver. Thanks Bin, >> >> >> Ah, it turns out so many drivers/dts are mis-written ... >> >> For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts >> file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must >> have been added to workaround the DM bug (fixed in patch 1 & 2 >> mentioned in this thread). But now since the DM bug has been fixed, >> these "u-boot,dm-pre-reloc" properties are really unnecessary but I am >> afraid I don't have the knowledge to clean up all of these. >> >> Of course, I've identified some additional drivers that have >> DM_FLAG_PRE_RELOC flag set but dts file doesn't have >> "u-boot,dm-pre-reloc". These drivers should be updated to remove the >> DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. > > > I don't see any cases where u-boot,dm-pre-reloc exists in DT for the Tegra > GPIO driver specifically, so I think a simple patch removing the > DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of > course, I didn't look at any non-Tegra DTs, or at drivers other than Tegra > GPIO. OK. I tend to agree that it is getting a bit late. I am considering delaying this pull request until the next merge window. I still have regmap and virtio to bring in. Tom, what do you think? 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-dm
On 10/16/18 8:33 PM, Bin Meng wrote: On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: Hi Stephen, On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren wrote: On 10/15/18 5:43 PM, Bin Meng wrote: Hi Stephen, On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren wrote: On 10/15/18 3:53 PM, Stephen Warren wrote: On 10/15/18 2:17 PM, Simon Glass wrote: Hi Tom, The following changes since commit 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: Merge tag 'arc-more-updates-for-2018.11-rc2-2' of git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: buildman: Add a --boards option to specify particular boards to build (2018-10-15 08:20:43 -0600) There's something wrong with these patches; both Jetson TX1 and Jetson TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM with SPL) and sandbox boot fine. Reverting the following solves the issue: 1) dm: core: Mirror the chosen node parse logic in the livetree scanning 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt() Reverting just (1) or just (2) does not. Can you please take a look? Thanks. The failure is probably due to some drivers that Tegra uses are not properly written, ie: these drivers are declared via U_BOOT_DRIVER with flags DM_FLAG_PRE_RELOC yet it was never working (bound before relocation) until patch (1) & (2). Now these drivers get bound before relocation but they were never bound before (assume they were never required before relocation), so we should identify which driver were wrongly written, but as a quick solution, can you please enlarge the CONFIG_SYS_MALLOC_F_LEN of your board and have a try? Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or tegra186_gpio.c (depending on board/SoC) also solves this. I suppose the correct solution is to remove the DM flag from the driver, since if it never used to work before your patches, it's clear that the flag/feature is not actually required. Thank you Stephen. I will prepare a patch to update the tegra*_gpio driver. Ah, it turns out so many drivers/dts are mis-written ... For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must have been added to workaround the DM bug (fixed in patch 1 & 2 mentioned in this thread). But now since the DM bug has been fixed, these "u-boot,dm-pre-reloc" properties are really unnecessary but I am afraid I don't have the knowledge to clean up all of these. Of course, I've identified some additional drivers that have DM_FLAG_PRE_RELOC flag set but dts file doesn't have "u-boot,dm-pre-reloc". These drivers should be updated to remove the DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. I don't see any cases where u-boot,dm-pre-reloc exists in DT for the Tegra GPIO driver specifically, so I think a simple patch removing the DM_FLAG_PRE_RELOC flag from the two Tegra GPIO drivers would be fine. Of course, I didn't look at any non-Tegra DTs, or at drivers other than Tegra GPIO. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On Wed, Oct 17, 2018 at 10:33 AM Bin Meng wrote: > > On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > > > > Hi Stephen, > > > > On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren > > wrote: > > > > > > On 10/15/18 5:43 PM, Bin Meng wrote: > > > > Hi Stephen, > > > > > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > > > wrote: > > > >> > > > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > > > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > > > Hi Tom, > > > > > > The following changes since commit > > > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > > > > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > > > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > > > > > are available in the Git repository at: > > > > > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > > > > > for you to fetch changes up to > > > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > > > > > buildman: Add a --boards option to specify particular boards to > > > build (2018-10-15 08:20:43 -0600) > > > >>> > > > >>> There's something wrong with these patches; both Jetson TX1 and Jetson > > > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > > > >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM > > > >>> with SPL) and sandbox boot fine. > > > >> > > > >> Reverting the following solves the issue: > > > >> > > > >> 1) dm: core: Mirror the chosen node parse logic in the livetree > > > >> scanning > > > >> > > > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > > > >> lists_bind_fdt() > > > >> > > > >> Reverting just (1) or just (2) does not. Can you please take a look? > > > >> Thanks. > > > > > > > > The failure is probably due to some drivers that Tegra uses are not > > > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > > > > relocation) until patch (1) & (2). Now these drivers get bound before > > > > relocation but they were never bound before (assume they were never > > > > required before relocation), so we should identify which driver were > > > > wrongly written, but as a quick solution, can you please enlarge the > > > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > > > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > > > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > > > tegra186_gpio.c (depending on board/SoC) also solves this. > > > > > > I suppose the correct solution is to remove the DM flag from the driver, > > > since if it never used to work before your patches, it's clear that the > > > flag/feature is not actually required. > > > > Thank you Stephen. I will prepare a patch to update the tegra*_gpio driver. > > Ah, it turns out so many drivers/dts are mis-written ... > > For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts > file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must > have been added to workaround the DM bug (fixed in patch 1 & 2 > mentioned in this thread). But now since the DM bug has been fixed, > these "u-boot,dm-pre-reloc" properties are really unnecessary but I am > afraid I don't have the knowledge to clean up all of these. > I don't think we have enough time before release to address all of these issues. Let's go with the following: Simon, can you please just drop the 3 patches below: dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt() dm: core: Mirror the chosen node parse logic in the livetree scanning test: dm: core: Add a test case for driver marked with DM_FLAG_PRE_RELOC flag > Of course, I've identified some additional drivers that have > DM_FLAG_PRE_RELOC flag set but dts file doesn't have > "u-boot,dm-pre-reloc". These drivers should be updated to remove the > DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. > I will work with a complete patchset targeting next release to clean up these. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On Wed, Oct 17, 2018 at 7:01 AM Bin Meng wrote: > > Hi Stephen, > > On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren wrote: > > > > On 10/15/18 5:43 PM, Bin Meng wrote: > > > Hi Stephen, > > > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > > wrote: > > >> > > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > > Hi Tom, > > > > The following changes since commit > > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > > > are available in the Git repository at: > > > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > > > for you to fetch changes up to > > 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > > > buildman: Add a --boards option to specify particular boards to > > build (2018-10-15 08:20:43 -0600) > > >>> > > >>> There's something wrong with these patches; both Jetson TX1 and Jetson > > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > > >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM > > >>> with SPL) and sandbox boot fine. > > >> > > >> Reverting the following solves the issue: > > >> > > >> 1) dm: core: Mirror the chosen node parse logic in the livetree scanning > > >> > > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > > >> lists_bind_fdt() > > >> > > >> Reverting just (1) or just (2) does not. Can you please take a look? > > >> Thanks. > > > > > > The failure is probably due to some drivers that Tegra uses are not > > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > > > relocation) until patch (1) & (2). Now these drivers get bound before > > > relocation but they were never bound before (assume they were never > > > required before relocation), so we should identify which driver were > > > wrongly written, but as a quick solution, can you please enlarge the > > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > > tegra186_gpio.c (depending on board/SoC) also solves this. > > > > I suppose the correct solution is to remove the DM flag from the driver, > > since if it never used to work before your patches, it's clear that the > > flag/feature is not actually required. > > Thank you Stephen. I will prepare a patch to update the tegra*_gpio driver. Ah, it turns out so many drivers/dts are mis-written ... For example, the driver has the DM_FLAG_PRE_RELOC flag, and the dts file also has "u-boot,dm-pre-reloc". The "u-boot,dm-pre-reloc" must have been added to workaround the DM bug (fixed in patch 1 & 2 mentioned in this thread). But now since the DM bug has been fixed, these "u-boot,dm-pre-reloc" properties are really unnecessary but I am afraid I don't have the knowledge to clean up all of these. Of course, I've identified some additional drivers that have DM_FLAG_PRE_RELOC flag set but dts file doesn't have "u-boot,dm-pre-reloc". These drivers should be updated to remove the DM_FLAG_PRE_RELOC flag like the Tegra GPIO drivers. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
Hi Stephen, On Wed, Oct 17, 2018 at 12:27 AM Stephen Warren wrote: > > On 10/15/18 5:43 PM, Bin Meng wrote: > > Hi Stephen, > > > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren > > wrote: > >> > >> On 10/15/18 3:53 PM, Stephen Warren wrote: > >>> On 10/15/18 2:17 PM, Simon Glass wrote: > Hi Tom, > > The following changes since commit > 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > buildman: Add a --boards option to specify particular boards to > build (2018-10-15 08:20:43 -0600) > >>> > >>> There's something wrong with these patches; both Jetson TX1 and Jetson > >>> TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > >>> system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM > >>> with SPL) and sandbox boot fine. > >> > >> Reverting the following solves the issue: > >> > >> 1) dm: core: Mirror the chosen node parse logic in the livetree scanning > >> > >> 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > >> lists_bind_fdt() > >> > >> Reverting just (1) or just (2) does not. Can you please take a look? > >> Thanks. > > > > The failure is probably due to some drivers that Tegra uses are not > > properly written, ie: these drivers are declared via U_BOOT_DRIVER > > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > > relocation) until patch (1) & (2). Now these drivers get bound before > > relocation but they were never bound before (assume they were never > > required before relocation), so we should identify which driver were > > wrongly written, but as a quick solution, can you please enlarge the > > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? > > Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the > issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or > tegra186_gpio.c (depending on board/SoC) also solves this. > > I suppose the correct solution is to remove the DM flag from the driver, > since if it never used to work before your patches, it's clear that the > flag/feature is not actually required. Thank you Stephen. I will prepare a patch to update the tegra*_gpio driver. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On 10/15/18 5:43 PM, Bin Meng wrote: Hi Stephen, On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren wrote: On 10/15/18 3:53 PM, Stephen Warren wrote: On 10/15/18 2:17 PM, Simon Glass wrote: Hi Tom, The following changes since commit 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: Merge tag 'arc-more-updates-for-2018.11-rc2-2' of git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: buildman: Add a --boards option to specify particular boards to build (2018-10-15 08:20:43 -0600) There's something wrong with these patches; both Jetson TX1 and Jetson TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM with SPL) and sandbox boot fine. Reverting the following solves the issue: 1) dm: core: Mirror the chosen node parse logic in the livetree scanning 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt() Reverting just (1) or just (2) does not. Can you please take a look? Thanks. The failure is probably due to some drivers that Tegra uses are not properly written, ie: these drivers are declared via U_BOOT_DRIVER with flags DM_FLAG_PRE_RELOC yet it was never working (bound before relocation) until patch (1) & (2). Now these drivers get bound before relocation but they were never bound before (assume they were never required before relocation), so we should identify which driver were wrongly written, but as a quick solution, can you please enlarge the CONFIG_SYS_MALLOC_F_LEN of your board and have a try? Yes, bumping CONFIG_SYS_MALLOC_F_LEN from 0x1800 to 0x1a90 solves the issue. Alternatively, removing DM_FLAG_PRE_RELOC in tegra_gpio.c or tegra186_gpio.c (depending on board/SoC) also solves this. I suppose the correct solution is to remove the DM flag from the driver, since if it never used to work before your patches, it's clear that the flag/feature is not actually required. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On Tue, Oct 16, 2018 at 07:43:21AM +0800, Bin Meng wrote: > Hi Stephen, > > On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren wrote: > > > > On 10/15/18 3:53 PM, Stephen Warren wrote: > > > On 10/15/18 2:17 PM, Simon Glass wrote: > > >> Hi Tom, > > >> > > >> The following changes since commit > > >> 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > > >> > > >>Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > > >> git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > > >> > > >> are available in the Git repository at: > > >> > > >>git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > > >> > > >> for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > > >> > > >>buildman: Add a --boards option to specify particular boards to > > >> build (2018-10-15 08:20:43 -0600) > > > > > > There's something wrong with these patches; both Jetson TX1 and Jetson > > > TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > > > system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM > > > with SPL) and sandbox boot fine. > > > > Reverting the following solves the issue: > > > > 1) dm: core: Mirror the chosen node parse logic in the livetree scanning > > > > 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > > lists_bind_fdt() > > > > Reverting just (1) or just (2) does not. Can you please take a look? Thanks. > > The failure is probably due to some drivers that Tegra uses are not > properly written, ie: these drivers are declared via U_BOOT_DRIVER > with flags DM_FLAG_PRE_RELOC yet it was never working (bound before > relocation) until patch (1) & (2). Now these drivers get bound before > relocation but they were never bound before (assume they were never > required before relocation), so we should identify which driver were > wrongly written, but as a quick solution, can you please enlarge the > CONFIG_SYS_MALLOC_F_LEN of your board and have a try? and I'm going to hold off on this PR until we have this sorted out. Thanks all! -- 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-dm
Hi Stephen, On Tue, Oct 16, 2018 at 6:43 AM Stephen Warren wrote: > > On 10/15/18 3:53 PM, Stephen Warren wrote: > > On 10/15/18 2:17 PM, Simon Glass wrote: > >> Hi Tom, > >> > >> The following changes since commit > >> 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: > >> > >>Merge tag 'arc-more-updates-for-2018.11-rc2-2' of > >> git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) > >> > >> are available in the Git repository at: > >> > >>git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 > >> > >> for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: > >> > >>buildman: Add a --boards option to specify particular boards to > >> build (2018-10-15 08:20:43 -0600) > > > > There's something wrong with these patches; both Jetson TX1 and Jetson > > TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the > > system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM > > with SPL) and sandbox boot fine. > > Reverting the following solves the issue: > > 1) dm: core: Mirror the chosen node parse logic in the livetree scanning > > 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in > lists_bind_fdt() > > Reverting just (1) or just (2) does not. Can you please take a look? Thanks. The failure is probably due to some drivers that Tegra uses are not properly written, ie: these drivers are declared via U_BOOT_DRIVER with flags DM_FLAG_PRE_RELOC yet it was never working (bound before relocation) until patch (1) & (2). Now these drivers get bound before relocation but they were never bound before (assume they were never required before relocation), so we should identify which driver were wrongly written, but as a quick solution, can you please enlarge the CONFIG_SYS_MALLOC_F_LEN of your board and have a try? Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On 10/15/18 3:53 PM, Stephen Warren wrote: On 10/15/18 2:17 PM, Simon Glass wrote: Hi Tom, The following changes since commit 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: Merge tag 'arc-more-updates-for-2018.11-rc2-2' of git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: buildman: Add a --boards option to specify particular boards to build (2018-10-15 08:20:43 -0600) There's something wrong with these patches; both Jetson TX1 and Jetson TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM with SPL) and sandbox boot fine. Reverting the following solves the issue: 1) dm: core: Mirror the chosen node parse logic in the livetree scanning 2) dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt() Reverting just (1) or just (2) does not. Can you please take a look? Thanks. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-dm
On 10/15/18 2:17 PM, Simon Glass wrote: Hi Tom, The following changes since commit 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: Merge tag 'arc-more-updates-for-2018.11-rc2-2' of git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: buildman: Add a --boards option to specify particular boards to build (2018-10-15 08:20:43 -0600) There's something wrong with these patches; both Jetson TX1 and Jetson TX2 (both 64-bit ARM without SPL) fail to boot with this branch; the system hangs with no output at all from U-Boot. Jetson TK1 (32-bit ARM with SPL) and sandbox boot fine. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] Please pull u-boot-dm
Hi Tom, The following changes since commit 6e7a186dc5d50f563e224e9ae7be70defff7ee0d: Merge tag 'arc-more-updates-for-2018.11-rc2-2' of git://git.denx.de/u-boot-arc (2018-10-15 07:20:07 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/pull-15oct-18 for you to fetch changes up to 02f2d266c75106a2fefd1f4e8e6f703fe00ed21d: buildman: Add a --boards option to specify particular boards to build (2018-10-15 08:20:43 -0600) Bring in part of the regmap series (the part that builds) Minor fixes in core driver model Bin Meng (8): dm: cpu: Fix print_cpuinfo() output cpu: mpc83xx: Remove unnecessary characters in the description string dm: util: Add a livetree equivalent API of dm_fdt_pre_reloc() dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt() dm: Correct pre_reloc_only parameter description in several APIs' comments dm: core: Mirror the chosen node parse logic in the livetree scanning test: dm: core: Add a test case for driver marked with DM_FLAG_PRE_RELOC flag timer: Sort Kconfig driver entries Mario Six (8): test: regmap: Increase size of syscon0 memory regmap: Fix documentation regmap: Add documentation regmap: Improve error handling misc: Sort Makefile entries misc: Add gdsys_soc driver misc: Add IHS FPGA driver core: ofnode: Fix mem leak in error path Simon Glass (2): sandbox: Try to start the RAM buffer at a particular address buildman: Add a --boards option to specify particular boards to build Documentation/devicetree/bindings/misc/gdsys,iocon_fpga.txt | 19 + Documentation/devicetree/bindings/misc/gdsys,iocpu_fpga.txt | 19 + Documentation/devicetree/bindings/misc/gdsys,soc.txt| 16 + arch/sandbox/cpu/os.c | 7 +- arch/sandbox/dts/test.dts | 6 +- common/board_f.c| 2 +- drivers/core/device.c | 2 +- drivers/core/lists.c| 9 +- drivers/core/ofnode.c | 4 +- drivers/core/regmap.c | 23 + drivers/core/root.c | 20 +- drivers/core/util.c | 25 + drivers/cpu/mpc83xx_cpu.c | 2 +- drivers/misc/Kconfig| 17 + drivers/misc/Makefile | 60 +-- drivers/misc/gdsys_soc.c| 74 +++ drivers/misc/gdsys_soc.h| 23 + drivers/misc/ihs_fpga.c | 867 ++ drivers/misc/ihs_fpga.h | 49 ++ drivers/serial/serial-uclass.c | 2 +- drivers/timer/Kconfig | 110 ++--- drivers/timer/timer-uclass.c| 2 +- include/dm/device-internal.h| 4 +- include/dm/lists.h | 9 +- include/dm/root.h | 17 +- include/dm/util.h | 27 +- include/regmap.h| 48 +- test/dm/bus.c | 2 +- test/dm/regmap.c| 2 +- test/dm/test-fdt.c | 29 +- tools/buildman/README | 12 +- tools/buildman/board.py | 28 +- tools/buildman/cmdline.py | 4 +- tools/buildman/control.py | 20 +- tools/buildman/test.py | 31 +- 35 files changed, 1440 insertions(+), 151 deletions(-) create mode 100644 Documentation/devicetree/bindings/misc/gdsys,iocon_fpga.txt create mode 100644 Documentation/devicetree/bindings/misc/gdsys,iocpu_fpga.txt create mode 100644 Documentation/devicetree/bindings/misc/gdsys,soc.txt create mode 100644 drivers/misc/gdsys_soc.c create mode 100644 drivers/misc/gdsys_soc.h create mode 100644 drivers/misc/ihs_fpga.c create mode 100644 drivers/misc/ihs_fpga.h 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-dm
On Tue, Oct 09, 2018 at 06:24:13PM -0600, Simon Glass wrote: > Hi Tom, > > Here is my attempt at a signed pull request. I've brought in most of the > outstanding dm patches. Please let me know if it looks OK. Do you have my > public key? Is it on the public key servers? > > > The following changes since commit 0a60a81ba3860946551cb79aa6486aa076e357f3: > > Kconfig: sandbox: enable cmd_avb and dependencies (2018-10-07 13:34:19 > -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git tags/dm-9oct18 > > for you to fetch changes up to 41b781ddf1869f5349e05ace888979f3673fe8c6: > > dtoc: Fix the value of SetInt() (2018-10-09 04:40:27 -0600) > Applied to u-boot/master, thanks! -- 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-dm
Hi Tom, On 9 October 2018 at 18:24, Simon Glass wrote: > > Hi Tom, > > Here is my attempt at a signed pull request. I've brought in most of the > outstanding dm patches. Please let me know if it looks OK. Do you have my > public key? > Build result here: https://travis-ci.org/sglass68/u-boot/builds/439071186 Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] Please pull u-boot-dm
Hi Tom, Here is my attempt at a signed pull request. I've brought in most of the outstanding dm patches. Please let me know if it looks OK. Do you have my public key? The following changes since commit 0a60a81ba3860946551cb79aa6486aa076e357f3: Kconfig: sandbox: enable cmd_avb and dependencies (2018-10-07 13:34:19 -0400) are available in the Git repository at: git://git.denx.de/u-boot-dm.git tags/dm-9oct18 for you to fetch changes up to 41b781ddf1869f5349e05ace888979f3673fe8c6: dtoc: Fix the value of SetInt() (2018-10-09 04:40:27 -0600) Test improvements to tidy up output and drop duplicate tests Sandbox SPL/TPL support Various dm-related improvements Marek Vasut (2): ofnode: Replace of_n_addr_cells with of_n_size_cells ofnode: Add missing address translation into ofnode_get_addr_size() Simon Glass (74): sandbox: Unprotect DATA regions in bus tests patman: Handle unicode in _ProjectConfigParser tests test/py: Fix unicode handling for log filtering buildman: Make the toolchain test more forgiving Makefile: Add a 'check' target for make test: Simplify the PATH setup test: Print the name of each test before running it test: Tidy up comments and variable name binman: Add a default path to libfdt.py binman: Fix up removal of temporary directories binman: Separate out testSplBssPad() buildman: dtoc: Suppress unwanted output from test tools: Set an initial value for indir patman: Don't clear progress in tout unless it was used test: Reduce the number of tests run with sandbox_flattree binman: Run tests concurrently log: Correct definition of log_msg_ret() log: Add helpers for common log levels sandbox: Support file truncation with os_open() sandbox: Add a way to write data to the host filesystem sandbox: spi: Drop command-line SPI option sandbox: Support booting from TPL to SPL sandbox: Add a flag to set the default log level sandbox: Remove the old memory file later sandbox: spi: Add more logging sandbox: video: Speed up video output sandbox: Add a debug UART serial: sandbox: Allow serial output without device tree sandbox: tpm: Tidy up enums and return values sandbox: tpm: Enhance to support the latest Chromium OS dm: spi: Clean up detection of sandbox SPI emulator sandbox: Restore blocking I/O on exit dm: core: Alloc uclass-private data to be cache-aligned dm: core: Update some functions to use const dm: core: Add a function to find the first inactive child dm: core: Update ofnode to read binman-style flash entry sf: Avoid allocating memory on every read operation spl: input: Allow input in SPL and TPL Makefile: Add a warning if SPL/TPL cannot be built blk: Support block drivers in TPL Kconfig: Convert CONFIG_RTC_MC146818 to Kconfig rtc: Allow use of RTC in SPL and TPL fdt: Document the fact that dtc is now built doc: Update docs for device tree in SPL, TPL fdt: Allow indicating a node is for U-Boot proper only tpm: Add support for SPL and TPL serial: Allow serial to be absent in TPL fdt: Allow libfdt in TPL cros: Update cros_ec code to use struct udevice cros: Adjust board_get_cros_ec_dev() to return a udevice dm: spi: Add logging of some return values fdt: Remove fdtdec_decode_region() function video: Adjust video_clear() to return an error tpm: Use livetree and allow children tpm: Tidy up logging in tpm-common.c tpm: Add a few new commands for v1 binman: Move to three-digit test-file numbers log: Add comments to the rest of the log categories Add a header file for strings Rename GPT_HEADER_SIGNATURE to avoid conflict cros: Update ec_commands to latest version x86: Update mtrr functions to allow leaving cache alone cros_ec: Update cros_ec_read_hash() to specify the image cros_ec: Add support for v3 messages on LPC test: panel: Add a test for the panel uclass panel: Expand the backlight support ctags: Minor changes to fix ctags output fdt: Allow C++ comments in link scripts and DT files pci: Add a little more debugging to pci_rom sysreset: Tidy up a few comments and logging sysreset: Add a way to find the last reset video: at91: Adjust vidconsole_position_cursor() to use char pos video: Tidy up a few comments in video.o dtoc: Fix the value of SetInt() .travis.yml |1 + Makefile |8 +- arch/sandbox/cpu/os.c | 85 +- arch/sandbox/cpu/start.c | 19 +- arch/sandbox/cpu/state.c |6 +- arch/sandbox/dts/sandbox.dts | 20 +- arch/sandbox/dts/sandbox64.dts | 20 +- arch/sandbox/dts/sandbox_pmic.dtsi |
Re: [U-Boot] Please pull u-boot-dm
Hi Jens and Simon, ne 30. 9. 2018 v 22:07 odesílatel Simon Glass napsal: > Hi Tom, > > Here are some binman and core driver-model enhancements. > > https://travis-ci.org/sglass68/u-boot/builds/435045761 > > Re signed tags, I did read this link: > > http://web.mit.edu/git/www/howto/using-signed-tag-in-pull-request.html > > but I get: > > error: gpg failed to sign the data > error: unable to sign the tag > The tag message has been left in .git/TAG_EDITMSG > > I have tried for a while to get gpg going. I used to use it a long > time again. Do you know any good instructions for that? > > Regards, > Simon > > > The following changes since commit > bbef20d479441b01d62252cf127498c58078b2c3: > > Merge tag 'xilinx-for-v2018.11' of > git://git.denx.de/u-boot-microblaze (2018-09-27 08:29:10 -0400) > > are available in the Git repository at: > > git://git.denx.de/u-boot-dm.git > > for you to fetch changes up to 31b8217e83a63d1c8c70edcdcdf5aff3b1791640: > > dm: test: Add "/firmware" node scan test (2018-09-29 11:49:35 -0600) > > > Jens Wiklander (1): > fdt: fdtdec_setup_memory_banksize() use livetree > > This patch is breaking SPL at least on zynq board. I didn't test zynqmp but I think situation will be very similar. I am ending in this while loop and never come out of that. 1205 while (ofnode_read_resource(mem, reg++, &res)) { 1206 reg = 0; 1207 mem = get_next_memory_node(mem); 1208 if (!ofnode_valid(mem)) { 1209 if (bank) 1210 return 0; 1211 goto missing_node; 1212 } 1213 } Thanks, Michal -- Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/ Maintainer of Linux kernel - Xilinx Zynq ARM architecture Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot