[U-Boot] [U-Boot, 2/3] rockchip: doc: update U-Boot location info

Jagan Teki jagan at amarulasolutions.com
Fri Nov 10 11:12:33 UTC 2017


On Fri, Nov 10, 2017 at 3:53 PM, Dr. Philipp Tomsich
<philipp.tomsich at theobroma-systems.com> wrote:
> Kever,
>
>> On 10 Nov 2017, at 08:24, Goldschmidt Simon <sgoldschmidt at de.pepperl-fuchs.com> wrote:
>>
>>>> Update rockchip U-Boot location to 0x4000/16384.
>
> At least the location given in the commit message is consistent:
>         (gdb) p 0x4000
>         $1 = 16384
>
>>>> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
>>>> Acked-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
>>>> Reviewed-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
>>>> ---
>>>>
>>>> doc/README.rockchip | 6 +++---
>>>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>>>
>>>
>>> Applied to u-boot-rockchip, thanks!
>>
>> I just found this commit has calculated the size wrong. 16384 blocks should be 8MB, not 4MB.
>
> Could you take a look at this and let us know if this should be changed?
> And if it needs a change (and you want it in Monday’s release): please submit a
> patch and I’ll try to generate a PR to Tom over the weekend…

This even require some README.rockchip and falcon offset changes,
Sorry I really don't understand why the new features been added for
release tree.

thanks!
-- 
Jagan Teki
Senior Linux Kernel Engineer | Amarula Solutions
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.


More information about the U-Boot mailing list