Hi Simon,
On 2016年08月01日 10:21, Simon Glass wrote:
Hi Ziyuan,
On 31 July 2016 at 20:13, Ziyuan Xu <xzy...@rock-chips.com> wrote:
Hi Simon,
On 2016年08月01日 09:51, Simon Glass wrote:
Hi Sandy,
On 28 July 2016 at 07:49, Sandy Patterson <apatter...@sightlogix.com>
wrote:
Add an extra byte so that this data is not byteswapped.
Signed-off-by: Sandy Patterson <apatter...@sightlogix.com>
---
arch/arm/dts/rk3288-rock2-square.dts | 2 +-
arch/arm/dts/rk3288-veyron.dtsi | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
Acked-by: Simon Glass <s...@chromium.org>
Do these board use OF_PLATDATA? I thought not.
Yes, only firefly-rk3288 board use OF_PLATDATA. But driver get
rk3288_sdram_channel via fdtdec_get_byte_array with the size which is
sizeof(struct rk3288_sdram_channel).
In commit 9ca7e67 rockchip: Update the sdram-channel property to support
of-platdata, you add dummy element in struct rk3288_sdram_channel and size
was changed to 9.
Without this fix, driver get rk3288_sdram_channel failed.
Maybe add CONFIG_IS_ENABLED(OF_PLATDATA) for distinction is better, how
about?
struct rk3288_sdram_channel {
u8 rank;
u8 col;
u8 bk;
u8 bw;
u8 dbw;
u8 row_3_4;
u8 cs0_row;
u8 cs1_row;
#if CONFIG_IS_ENABLED(OF_PLATDATA)
/*
* For of-platdata, which would otherwise convert this into two
* byte-swapped integers. With a size of 9 bytes, this struct will
* appear in of-platdata as a byte array.
*/
u8 dummy;
#endif
};
Yes, but I'm happy with either solution. Your one may be a little
easier to understand, but if someone switches a board over to
OF_PLATDATA then it will be confusing... Please let me know which you
prefer.
OF_PLATDATA is used to reduce the size of the SPL, right? In most cases,
some rk3288 boards use OF_LIBFDT. If OF_PLATDATA is really required, I
think your comment is very clear.
I perfer my above opinion? :-)
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot