[U-Boot] [PATCH 5/7] rockchip: rk3368: Add initial support for RK3368 based GeekBox

[Andy Yan]

Hi Andreas:


On 2017年04月21日 10:59, Andreas Färber wrote:
> Hi Andy,
>
> Sorry for forgetting about this, and thank you for picking it up.
>
> Am 21.04.2017 um 04:32 schrieb Andy Yan:
>> From: Andreas Färber <afaerber at suse.de>
>>
>> The GeekBox is a TV box from GeekBuying, based on an MXM3 module.
>> The module can be used with base boards such as the GeekBox Landingship.
>>
>> This adds basic support to chain-load U-Boot from Rockchip's miniloader.
>>
>> $ ./lollipop_u-boot/tools/loaderimage --pack u-boot.bin u-boot.img
>>
>> Implemented is the serial console, but no boot media drivers yet.
>>
>> Note that flashing the resulting U-Boot will not allow you to enter the
>> rockusb mode any more via "Update" button. Instead, you will need to
>> short two pins on the bottom of the module to enter MaskRom mode and
>> re-flash the loader:
> ":"? See also below.
>
>> Signed-off-by: Andreas Färber <afaerber at suse.de>
>> Signed-off-by: Andy Yan <andy.yan at rock-chips.com>
>> ---
>>
>>   arch/arm/dts/Makefile                 |   2 +
>>   arch/arm/dts/rk3368-geekbox.dts       | 319 ++++++++++++++++++++++++++++++++++
>>   arch/arm/mach-rockchip/rk3368/Kconfig |  11 ++
>>   board/geekbuying/geekbox/Kconfig      |  15 ++
>>   board/geekbuying/geekbox/MAINTAINERS  |   6 +
>>   board/geekbuying/geekbox/Makefile     |   7 +
>>   board/geekbuying/geekbox/geekbox.c    |  28 +++
>>   configs/geekbox_defconfig             |  21 +++
>>   include/configs/geekbox.h             |  18 ++
>>   9 files changed, 427 insertions(+)
>>   create mode 100644 arch/arm/dts/rk3368-geekbox.dts
>>   create mode 100644 board/geekbuying/geekbox/Kconfig
>>   create mode 100644 board/geekbuying/geekbox/MAINTAINERS
>>   create mode 100644 board/geekbuying/geekbox/Makefile
>>   create mode 100644 board/geekbuying/geekbox/geekbox.c
>>   create mode 100644 configs/geekbox_defconfig
>>   create mode 100644 include/configs/geekbox.h
> One reason this series stalled is that I was asked to add a README
> instead of the text in the commit message - this is still missing in
> your submission, it seems.


     Thanks for pointing out this, I will add a README in next version.
>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
>> index ce34e3e..fbc97e8 100644
>> --- a/arch/arm/dts/Makefile
>> +++ b/arch/arm/dts/Makefile
>> @@ -41,8 +41,10 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += \
>>   	rk3288-veyron-mickey.dtb \
>>   	rk3288-veyron-minnie.dtb \
>>   	rk3328-evb.dtb \
>> +	rk3368-geekbox.dtb \
>>   	rk3399-evb.dtb \
>>   	rk3399-puma.dtb
>> +	rk3368-geekbox.dtb
> Mismerge.
>
>>   dtb-$(CONFIG_ARCH_MESON) += \
>>   	meson-gxbb-odroidc2.dtb
> [snip]
>
> Regards,
> Andreas
>