[U-Boot] [PATCH 0/9] rockchip: cosmetics, a fix and first steps on the rk3188

[Simon Glass]

Hi Heiko,

On 17 July 2016 at 09:27, Simon Glass <sjg at chromium.org> wrote:
> Hi Heiko,
>
> On 17 July 2016 at 09:20, Heiko Stübner <heiko at sntech.de> wrote:
>> Am Sonntag, 17. Juli 2016, 08:14:06 schrieb Simon Glass:
>>> Hi Heiko,
>>>
>>> On 15 July 2016 at 16:17, Heiko Stuebner <heiko at sntech.de> wrote:
>>> > I've made some nice progress on using mainline uboot on the rk3188
>>> > and would like to dump some first results.
>>> >
>>> > Right now I can use uboot on the rk3188 with the Rockchip binary ddr init,
>>> > similar to what barebox does and can even netboot a kernel image using
>>> > a usb ethernet adapter [0] .
>>> >
>>> > While working on this I found quite some cosmetic stuff that shouldn't
>>> > persist to make extending easier. So while I don't know what the policy
>>> > is for my standalone pinctrl and clock drivers (without the actual board)
>>> > at least the cosmetics + fix might get in at least.
>>>
>>> Nice work!
>>>
>>> It would be best to add the drivers with the board - otherwise they
>>> are just dead code. How far away is it?
>>
>> The big issue is the SPL. Right now I'm using Rockchip's ddr-init as spl-
>> replacement, and I'd say this second part is nearly ready - only minor
>> cleanups.
>>
>> The memory-setup is supposed to be very much similar to the rk3288 (same
>> dw_upctl and ddr-phy), but it looks like the very first steps are somewhat
>> different and I haven't been able to make the spl output anything on the
>> serial console yet ... which could stem from some difference in what the soc
>> expects or just some dumb mistake on my part :-)
>
> That's always tricky.
>
> You may already know this, but the EARLY_UART setting is used on
> rk3288 to display a character as soon as SPL starts. You might be able
> to do something similar. The main issue I had was getting the pinmux
> setting right.
>
> As a test, you can start with booting SPL from the ddr-init binary I
> suspect. I have not tried it but it should work. Then the pinmux
> should already be set up (since ddr-init outputs to serial).

I'm going to apply the earlier patches in your series. Once you have a
working board we can apply the rest (to avoid adding dead code).

Regards,
Simon