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

Hi Tom,
I dropped the translation patch until Tegra can be fixed to work with it. This is storing up problems though, since in the meantime others might rely on the current behaviour.
https://travis-ci.org/sglass68/u-boot/builds/315684225
The following changes since commit cf71338ee7a1dc16f13a4f1b89f32c624210df45:
ata: Migrate CONFIG_MVSATA_IDE to Kconfig (2017-12-12 18:16:06 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git
for you to fetch changes up to 854dfbf99b89c114ba100905e1500b8ace60e0f9:
power: pmic/regulator: Add basic support for TPS65910 (2017-12-12 19:53:45 -0700)
---------------------------------------------------------------- Felix Brack (2): power: extend prefix match to regulator-name property power: pmic/regulator: Add basic support for TPS65910
Philipp Tomsich (1): dm: reset: have the reset-command perform a COLD reset
Simon Glass (13): binman: Add a function to read ELF symbols binman: Add support for including spl/u-boot-spl.dtb binman: Add support for including spl/u-boot-spl-nodtb.bin binman: Drop a stale comment about the 'board' feature binman: Add tests binaries with binman symbols binman: Adjust size of test SPL binary binman: Support enabling debug in tests binman: Support accessing binman tables at run time binman: arm: Include the binman symbol table binman: Add binman symbol support to SPL binman: Add binman support to spl_ram.c binman: Add documentation for the symbol feature binman: tegra: Convert to use binman
Stefan Roese (1): dm: blk: Use uclass_find_first/next_device() in blk_first/next_device()
Makefile | 6 + arch/arm/config.mk | 6 +- arch/arm/cpu/u-boot-spl.lds | 7 + arch/arm/dts/tegra-u-boot.dtsi | 40 +++++ arch/arm/dts/tegra114-u-boot.dtsi | 3 + arch/arm/dts/tegra124-nyan-big-u-boot.dtsi | 2 + arch/arm/dts/tegra124-u-boot.dtsi | 3 + arch/arm/dts/tegra20-u-boot.dtsi | 11 +- arch/arm/dts/tegra210-u-boot.dtsi | 3 + arch/arm/dts/tegra30-u-boot.dtsi | 3 + arch/arm/mach-tegra/Kconfig | 1 + arch/sandbox/dts/sandbox_pmic.dtsi | 6 + common/spl/spl.c | 16 +- common/spl/spl_ram.c | 19 ++- doc/device-tree-bindings/regulator/regulator.txt | 16 +- drivers/block/blk-uclass.c | 7 +- drivers/power/pmic/Kconfig | 8 + drivers/power/pmic/Makefile | 1 + drivers/power/pmic/pmic-uclass.c | 11 +- drivers/power/pmic/pmic_tps65910_dm.c | 98 +++++++++++ drivers/power/regulator/Kconfig | 8 + drivers/power/regulator/Makefile | 1 + drivers/power/regulator/tps65910_regulator.c | 459 ++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/sysreset/sysreset-uclass.c | 2 +- include/binman_sym.h | 93 ++++++++++ include/power/sandbox_pmic.h | 5 +- include/power/tps65910_pmic.h | 130 ++++++++++++++ include/spl.h | 11 ++ test/dm/regulator.c | 2 + tools/binman/README | 32 +++- tools/binman/binman.py | 11 +- tools/binman/control.py | 3 + tools/binman/elf.py | 129 ++++++++++++++ tools/binman/elf_test.py | 122 ++++++++++++++ tools/binman/etype/entry.py | 8 + tools/binman/etype/u_boot_spl.py | 6 + tools/binman/etype/u_boot_spl_bss_pad.py | 7 +- tools/binman/etype/u_boot_spl_dtb.py | 17 ++ tools/binman/etype/u_boot_spl_nodtb.py | 17 ++ tools/binman/etype/u_boot_with_ucode_ptr.py | 9 +- tools/binman/ftest.py | 68 ++++++-- tools/binman/image.py | 79 ++++++++- tools/binman/image_test.py | 46 +++++ tools/binman/test/21_image_pad.dts | 2 +- tools/binman/test/24_sorted.dts | 4 +- tools/binman/test/28_pack_4gb_outside.dts | 4 +- tools/binman/test/29_x86-rom.dts | 6 +- tools/binman/test/51_u_boot_spl_dtb.dts | 13 ++ tools/binman/test/52_u_boot_spl_nodtb.dts | 11 ++ tools/binman/test/53_symbols.dts | 20 +++ tools/binman/test/Makefile | 18 +- tools/binman/test/bss_data.c | 2 +- tools/binman/test/u_boot_binman_syms | Bin 0 -> 4921 bytes tools/binman/test/u_boot_binman_syms.c | 14 ++ tools/binman/test/u_boot_binman_syms.lds | 30 ++++ tools/binman/test/u_boot_binman_syms_bad | Bin 0 -> 4890 bytes tools/binman/test/u_boot_binman_syms_bad.c | 1 + tools/binman/test/u_boot_binman_syms_bad.lds | 29 ++++ tools/binman/test/u_boot_binman_syms_size | Bin 0 -> 4825 bytes tools/binman/test/u_boot_binman_syms_size.c | 12 ++ tools/binman/test/u_boot_ucode_ptr.c | 2 +- 61 files changed, 1638 insertions(+), 62 deletions(-) create mode 100644 arch/arm/dts/tegra-u-boot.dtsi create mode 100644 arch/arm/dts/tegra114-u-boot.dtsi create mode 100644 arch/arm/dts/tegra124-u-boot.dtsi create mode 100644 arch/arm/dts/tegra210-u-boot.dtsi create mode 100644 arch/arm/dts/tegra30-u-boot.dtsi create mode 100644 drivers/power/pmic/pmic_tps65910_dm.c create mode 100644 drivers/power/regulator/tps65910_regulator.c create mode 100644 include/binman_sym.h create mode 100644 include/power/tps65910_pmic.h create mode 100644 tools/binman/elf.py create mode 100644 tools/binman/elf_test.py create mode 100644 tools/binman/etype/u_boot_spl_dtb.py create mode 100644 tools/binman/etype/u_boot_spl_nodtb.py create mode 100644 tools/binman/image_test.py create mode 100644 tools/binman/test/51_u_boot_spl_dtb.dts create mode 100644 tools/binman/test/52_u_boot_spl_nodtb.dts create mode 100644 tools/binman/test/53_symbols.dts create mode 100755 tools/binman/test/u_boot_binman_syms create mode 100644 tools/binman/test/u_boot_binman_syms.c create mode 100644 tools/binman/test/u_boot_binman_syms.lds create mode 100755 tools/binman/test/u_boot_binman_syms_bad create mode 120000 tools/binman/test/u_boot_binman_syms_bad.c create mode 100644 tools/binman/test/u_boot_binman_syms_bad.lds create mode 100755 tools/binman/test/u_boot_binman_syms_size create mode 100644 tools/binman/test/u_boot_binman_syms_size.c
Regards, Simon

On 12/13/2017 09:14 AM, Simon Glass wrote:
Hi Tom,
I dropped the translation patch until Tegra can be fixed to work with it. This is storing up problems though, since in the meantime others might rely on the current behaviour.
https://travis-ci.org/sglass68/u-boot/builds/315684225
The following changes since commit cf71338ee7a1dc16f13a4f1b89f32c624210df45:
ata: Migrate CONFIG_MVSATA_IDE to Kconfig (2017-12-12 18:16:06 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git
for you to fetch changes up to 854dfbf99b89c114ba100905e1500b8ace60e0f9:
power: pmic/regulator: Add basic support for TPS65910 (2017-12-12 19:53:45 -0700)
This version works fine for me.
Note that I don't believe the Tegra code needs to be fixed to work with the translation patch. The Tegra code was working perfectly with the DT parsing code that existing before the conversion to livetree, and with the livetree code without the extra translation patch. I had to submit quite a few patches to fix the translation logic in the non-livetree DT parsing code to make sure it translated in the right circumstance and not in the others. The most likely issue is that the latest translation patch does the wrong thing.

Hi Stephen,
On 13 December 2017 at 09:49, Stephen Warren swarren@wwwdotorg.org wrote:
On 12/13/2017 09:14 AM, Simon Glass wrote:
Hi Tom,
I dropped the translation patch until Tegra can be fixed to work with it. This is storing up problems though, since in the meantime others might rely on the current behaviour.
https://travis-ci.org/sglass68/u-boot/builds/315684225
The following changes since commit cf71338ee7a1dc16f13a4f1b89f32c624210df45:
ata: Migrate CONFIG_MVSATA_IDE to Kconfig (2017-12-12 18:16:06 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git
for you to fetch changes up to 854dfbf99b89c114ba100905e1500b8ace60e0f9:
power: pmic/regulator: Add basic support for TPS65910 (2017-12-12 19:53:45 -0700)
This version works fine for me.
Note that I don't believe the Tegra code needs to be fixed to work with the translation patch. The Tegra code was working perfectly with the DT parsing code that existing before the conversion to livetree, and with the livetree code without the extra translation patch. I had to submit quite a few patches to fix the translation logic in the non-livetree DT parsing code to make sure it translated in the right circumstance and not in the others. The most likely issue is that the latest translation patch does the wrong thing.
OK I have replied on the thread for that patch:
"Add translation in live tree case"
Regards, Simon

On Wed, Dec 13, 2017 at 09:14:00AM -0700, Simon Glass wrote:
Hi Tom,
I dropped the translation patch until Tegra can be fixed to work with it. This is storing up problems though, since in the meantime others might rely on the current behaviour.
Is this building binman only when the board we're building for requires it? Thanks!

Hi Tom,
On 14 December 2017 at 12:51, Tom Rini trini@konsulko.com wrote:
On Wed, Dec 13, 2017 at 09:14:00AM -0700, Simon Glass wrote:
Hi Tom,
I dropped the translation patch until Tegra can be fixed to work with it. This is storing up problems though, since in the meantime others might rely on the current behaviour.
Is this building binman only when the board we're building for requires it? Thanks!
binman itself doesn't need building (it is just Python). But it does need the libfdt Python module. That is built if CONFIG_PYLIBFDT is enabled, which it is if dtoc or binman are used. So I think it does what you want.
Regards, Simon

On Wed, Dec 13, 2017 at 09:14:00AM -0700, Simon Glass wrote:
Hi Tom,
I dropped the translation patch until Tegra can be fixed to work with it. This is storing up problems though, since in the meantime others might rely on the current behaviour.
https://travis-ci.org/sglass68/u-boot/builds/315684225
The following changes since commit cf71338ee7a1dc16f13a4f1b89f32c624210df45:
ata: Migrate CONFIG_MVSATA_IDE to Kconfig (2017-12-12 18:16:06 -0500)
are available in the Git repository at:
git://git.denx.de/u-boot-dm.git
for you to fetch changes up to 854dfbf99b89c114ba100905e1500b8ace60e0f9:
power: pmic/regulator: Add basic support for TPS65910 (2017-12-12 19:53:45 -0700)
Applied to u-boot/master, thanks!
participants (3)
-
Simon Glass
-
Stephen Warren
-
Tom Rini