<div dir="ltr"><div dir="ltr">Hi Thomas,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 13, 2020 at 2:00 AM Thomas Petazzoni <<a href="mailto:thomas.petazzoni@bootlin.com">thomas.petazzoni@bootlin.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun,  5 Jul 2020 01:46:45 +0530<br>
Suniel Mahesh <<a href="mailto:sunil@amarulasolutions.com" target="_blank">sunil@amarulasolutions.com</a>> wrote:<br>
<br>
> Add initial support for RK3399 based rockpi-4 targets (model A, B, C)<br>
> with below features:<br>
> <br>
>       - Custom U-Boot 2020.07-rc4<br>
>         <a href="https://github.com/amarula/u-boot-amarula.git" rel="noreferrer" target="_blank">https://github.com/amarula/u-boot-amarula.git</a><br>
>         branch rock-pi<br>
>       - Linux 5.4.46<br>
>       - GPT partition layout is being used<br>
>       - Default packages from buildroot<br>
> <br>
> Signed-off-by: Suniel Mahesh <<a href="mailto:sunil@amarulasolutions.com" target="_blank">sunil@amarulasolutions.com</a>><br>
> ---<br>
> Changes for v3:<br>
> - no changes<br>
<br>
Applied to master with similar changes than the n10 defconfig.<br>
<br>
Regarding the use of u-boot-rockchip.bin, I think I very much prefer<br>
that there is one partition for each bootloader stage. Having this huge<br>
image u-boot-rockchip.bin, with a huge gap in the middle feels odd, so<br>
I tend to prefer the solution implemented by your patch as it is.<br>
<br>
Of course, please send some updates when the ATF support and U-Boot<br>
support is fully merged, so that we can use mainline and tagged releases.<br></blockquote><div><br></div><div>I will update the target to use mainline u-boot and others once the support is fully</div><div>merged and send patches accordingly. Thanks for the review. <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks,<br>
<br>
Thomas<br>
-- <br>
Thomas Petazzoni, CTO, Bootlin<br>
Embedded Linux and Kernel engineering<br>
<a href="https://bootlin.com" rel="noreferrer" target="_blank">https://bootlin.com</a><br>
</blockquote></div></div>