u-boot-brain/arch/arm/mach-rockchip/rk3399
eric.gao@rock-chips.com bc8e8fe40b rockchip: rk3399: Add missing sentinel in syscon
when enable PMIC rk808,the system will halt at very
 early stage,log is shown as bellow.

INFO:    plat_rockchip_pmu_init(1211): pd status 3e
INFO:    BL31: Initializing runtime services
INFO:    BL31: Preparing for EL3 exit to normal world
INFO:    Entry point address = 0x200000
INFO:    SPSR = 0x3c9
time 44561b, 0 (<<----Just stop here)

It's caused by the absence of "{ }" in syscon_rk3399.c
,which will lead to memory overflow like below.According
 to Sysmap file ,we can find the function buck_get_value
of rk808 is just follow the compatible struct,the pointer
"of_match" point to "buck_get_value",but it is not a
struct and don't have member of compatible, In this case,
system crash. So,on the face, it looks like that rk808 is
guilty.but he is really innocent.

while (of_match->compatible) { <<----------
    if (!strcmp(of_match->compatible, compat)) {
    *of_idp = of_match;
    return 0;
    }
    of_match++;
}

Signed-off-by: Eric Gao <eric.gao@rock-chips.com>
Reviewed-by: Kever Yang <kever.yang@rock-chips.com>
Tested-by: Kever Yang <kever.yang@rock-chips.com>
2017-04-15 10:13:17 -06:00
..
clk_rk3399.c rockchip: clk: rk3399: update driver for spl 2017-03-16 16:03:43 -06:00
Kconfig rockchip: ARM64: split RK3399-Q7 board off the RK3399-EVB board 2017-04-15 10:13:17 -06:00
Makefile rockchip: arm64: rk3399: add ddr controller driver 2017-03-16 16:03:45 -06:00
rk3399.c rockchip: arm64: rk3399: remove unconditional debug message 2017-04-04 20:01:57 -06:00
sdram_rk3399.c rockchip: arm64: rk3399: add ddr controller driver 2017-03-16 16:03:45 -06:00
syscon_rk3399.c rockchip: rk3399: Add missing sentinel in syscon 2017-04-15 10:13:17 -06:00