[U-Boot] [PATCH 2/3] rockchip: rk3188: Add Radxa Rock board

[Simon Glass]

Hi Heiko,

On 26 March 2017 at 13:06, Heiko Stuebner <heiko at sntech.de> wrote:
> Hi Simon,
>
> Am Samstag, 25. März 2017, 20:39:08 CEST schrieb Simon Glass:
>> On 23 March 2017 at 17:41, Heiko Stuebner <heiko at sntech.de> wrote:
>> > The Rock is a RK3188 based single board computer by Radxa.
>> > Currently it still relies on the proprietary DDR init and
>> > cannot use the generic SPL, but at least is able to boot
>> > a linux kernel and system up to a regular login prompt.
>> >
>> > Signed-off-by: Heiko Stuebner <heiko at sntech.de>
>> > Reviewed-by: Simon Glass <sjg at chromium.org>
>> > Tested-by: Kever Yang <kever.yang at rock-chips.com>
>> > ---
>> >  arch/arm/dts/Makefile                 |   1 +
>> >  arch/arm/dts/rk3188-radxarock.dts     | 382 ++++++++++++++++++++++++++++++++++
>> >  arch/arm/mach-rockchip/rk3188/Kconfig |  11 +
>> >  board/radxa/rock/Kconfig              |  15 ++
>> >  board/radxa/rock/MAINTAINERS          |   6 +
>> >  board/radxa/rock/Makefile             |   7 +
>> >  board/radxa/rock/rock.c               |   7 +
>> >  configs/rock_defconfig                |  58 ++++++
>> >  include/configs/rock.h                |  30 +++
>> >  9 files changed, 517 insertions(+)
>> >  create mode 100644 arch/arm/dts/rk3188-radxarock.dts
>> >  create mode 100644 board/radxa/rock/Kconfig
>> >  create mode 100644 board/radxa/rock/MAINTAINERS
>> >  create mode 100644 board/radxa/rock/Makefile
>> >  create mode 100644 board/radxa/rock/rock.c
>> >  create mode 100644 configs/rock_defconfig
>> >  create mode 100644 include/configs/rock.h
>>
>> I am still having trouble applying this patch. I get build errors:
>>
>>        arm:  +   rock
>> +arch/arm/Makefile:22: CONFIG_CPU_V7  -march=armv7-a
>> +make[2]: *** No rule to make target 'dts/dt.dtb', needed by
>> 'tpl/u-boot-tpl.dtb'.  Stop.
>> +make[1]: *** [tpl/u-boot-tpl.bin] Error 2
>> +make: *** [sub-make] Error 2
>>     0    0    1 /1      rock
>>
>> Also there seems to be a duplicate config:
>>
>>        arm:  +   rock
>> +In file included from include/configs/rock.h:11:0,
>> +                 from include/config.h:5,
>> +                 from include/common.h:21,
>> +                 from arch/arm/lib/asm-offsets.c:15:
>> + #define CONFIG_SYS_THUMB_BUILD
>> + ^
>> +                 from lib/asm-offsets.c:15:
>> +In file included from include/linux/kconfig.h:4:0,
>> +                 from <command-line>:0:
>> +include/generated/autoconf.h:10:0: note: this is the location of the
>> previous definition
>> + #define CONFIG_SYS_THUMB_BUILD 1
>
> looks like this got run over by another Kconfig migration on march-18.
> New patches (migration + rock board) coming up shortly.

Thanks - what toolchain are you using to test this?

Regards,
Simon