[U-Boot] [PATCH 0/2] Revert changes to __of_translate_address()

Przemyslaw Marczak p.marczak at samsung.com
Tue Jan 12 15:40:42 CET 2016


Globally enabled CONFIG_OF_TRANSLATE causes, that function dev_get_addr()
returns a different result for a node with: 'size-cells = <0>'.

The above change, breaks the Exynos boards for a long time, because of GPIO
driver, which uses the fdt in a non standard way.

After sending few approaches with adding support for that case,
the one was appllied for a temporary fix for the release:

"fdt: fix address cell count checking in fdt_translate_address()"

However, after discussion on the list, we found that this way can't be
accepted because the function __of_translate_address() as a backend
of dev_get_addr(), shouldn't be used for translating a non-MMIO addresses.

Updating some device tree files with '#size-cells = 1' and by extending
the 'reg' property by address length - fixes the problem.

This applies on u-boot-dm/master:
c5a7533 net: Fix delay in net_retry test

Przemyslaw Marczak (2):
  dts:exynos:update pinctrl size-cells and fix child regs
  Revert "fdt: fix address cell count checking in
    fdt_translate_address()"

 arch/arm/dts/exynos4210-pinctrl-uboot.dtsi |  8 ++++----
 arch/arm/dts/exynos4x12-pinctrl-uboot.dtsi | 18 +++++++++---------
 arch/arm/dts/exynos5250-pinctrl-uboot.dtsi | 16 ++++++++--------
 arch/arm/dts/exynos54xx-pinctrl-uboot.dtsi | 12 ++++++------
 arch/arm/dts/s5pc110-pinctrl.dtsi          |  4 ++--
 common/fdt_support.c                       |  7 ++++---
 6 files changed, 33 insertions(+), 32 deletions(-)

-- 
1.9.1



More information about the U-Boot mailing list