[U-Boot] [PATCH V3 0/4] Fix fdt 'reg' parsing and unbreak few Exynos4 boards

Simon Glass sjg at chromium.org
Sat Oct 3 15:36:53 CEST 2015


Hi,

On 30 September 2015 at 19:30, Simon Glass <sjg at chromium.org> wrote:
> Hi Przemyslaw,
>
> On 30 September 2015 at 06:25, Przemyslaw Marczak <p.marczak at samsung.com> wrote:
>> Hello Tom, Simon,
>>
>>
>> On 09/30/2015 03:13 PM, Tom Rini wrote:
>>>
>>> On Wed, Sep 30, 2015 at 01:14:49PM +0200, Przemyslaw Marczak wrote:
>>>
>>>> Booting of Odroid U3/X2 with SD card, ends with error:
>>>>
>>>> MMC:   EXYNOS DWMMC: 0
>>>> Card did not respond to voltage select!
>>>> *** Warning - MMC init failed, using default environment
>>>>
>>>> Generally this was broken, because of wrong addresses,
>>>> assigned to GPIOs.
>>>> The source of the problem was in rework of lib/fdtdec.c, after which
>>>> function fdtdec_get_addr() doesn't work as previous and function
>>>> dev_get_addr() doesn't works as expected.
>>>>
>>>> The code after rework in lib/fdtdec.c assumed, that #size-cells property,
>>>> should be always greater or equal to 1, this was wrong, because it can be
>>>> 0.
>>>>
>>>> In case of debugging the issue, I found, that mmc clock was computed
>>>> wrong,
>>>> for Exynos4, because of function get_mmc_clk(), which always returns -1
>>>> for
>>>> this SoC.
>>>>
>>>> The patchset should fix booting on all Exynos4 boards, however it was
>>>> tested
>>>> on: Odroid X2 / U3 / XU3 and Trats / Trats2.
>>>>
>>>> Przemyslaw Marczak (4):
>>>>    fdtdec: fix parsing 'reg' property with zero value in '#size-cells'
>>>>    gpio: s5p: call: dev_get_addr() instead of fdtdec_get_addr()
>>>>    mach-exynos: clock: restore calling dead exynos4_get_mmc_clk()
>>>>    trats: fdt: disable unused DW MMC
>>>>
>>>>   arch/arm/dts/exynos4210-trats.dts |  4 ++++
>>>>   arch/arm/mach-exynos/clock.c      | 10 ++++------
>>>>   drivers/gpio/s5p_gpio.c           | 18 +++++++++++-------
>>>>   lib/fdtdec.c                      |  2 +-
>>>>   4 files changed, 20 insertions(+), 14 deletions(-)
>>>
>>>
>>> Should I grab this directly or expect a PR from the DT or Samsung tree?
>>> Thanks!
>>>
>>
>> If this is not a problem for you, then it will be nice :)
>>
>> Simon,
>> Is that good to you?
>
> Yes, thank you both.

Hmm I'm going to pick this up for the DT tree as there's another patch
needed from Stephen also. Tom if you already have this locally let me
know.

Regards,
Simon


More information about the U-Boot mailing list