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

["Andreas Färber" <afaerber@suse.de>]

Hi Andy,

Am 21.04.2017 um 08:17 schrieb Andy Yan:
> On 2017年04月21日 10:59, Andreas Färber wrote:
>> Am 21.04.2017 um 04:32 schrieb Andy Yan:
>>> 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.

With that line dropped, I have tested the series on top of
u-boot-rockchip.git next on the GeekBox.
(New tweezers worked after three tries instead of dozens - progress!)

> U-Boot 2017.05-rc2-00088-gfdac36918d-dirty (Apr 30 2017 - 10:19:48 +0200)
> 
> Model: GeekBox
> DRAM:  2 GiB
> rkclk_init apllb(816000000) apll(800000000) dpll(1200000000) cpll(400000000) gpll(576000000)

Can we limit this output to debug builds please?

> MMC:   
> Using default environment
> 
> In:    serial at ff690000
> Out:   serial at ff690000
> Err:   serial at ff690000
> Net:   Net Initialization Skipped
> No ethernet found.
> Hit any key to stop autoboot:  0 

I notice that we're still lacking environment variables for distro boot:

fdtfile
fdt_addr_r
kernel_addr_r
ramdisk_addr_r
scriptaddr

I'll look into fdtfile and would appreciate if you could include the
addresses in v2.

It seems that despite your added drivers, we're still not able to boot?
What will be the quickest driver to enable? Net? MMC?

For rockusb support (Eddie's series) we probably need USB drivers first?

Regards,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)