Re: [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA

2016-08-27 Thread Simon Glass 
On 12 August 2016 at 11:20, Simon Glass  wrote:
> On 12 August 2016 at 01:43, Ziyuan Xu  wrote:
>> Add a condition to determine the rk3288_sdram_channel size.
>>
>> This patch fixes read sdram_channel property failed from DT on rk3288
>> boards, which not enable OF_PLATDATA.
>>
>> Signed-off-by: Ziyuan Xu 
>> ---
>>
>>  arch/arm/include/asm/arch-rockchip/sdram.h | 4 
>>  1 file changed, 4 insertions(+)
>
> Acked-by: Simon Glass 

Applied to u-boot-rockchip/next, thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA

2016-08-12 Thread Ziyuan Xu 
Add a condition to determine the rk3288_sdram_channel size.

This patch fixes read sdram_channel property failed from DT on rk3288
boards, which not enable OF_PLATDATA.

Signed-off-by: Ziyuan Xu 
---

 arch/arm/include/asm/arch-rockchip/sdram.h | 4 
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/include/asm/arch-rockchip/sdram.h 
b/arch/arm/include/asm/arch-rockchip/sdram.h
index e08e28f..82c3d07 100644
--- a/arch/arm/include/asm/arch-rockchip/sdram.h
+++ b/arch/arm/include/asm/arch-rockchip/sdram.h
@@ -24,12 +24,16 @@ struct rk3288_sdram_channel {
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.
+*
+* If OF_PLATDATA enabled, need to add a dummy byte in dts.(i.e 0xff)
 */
u8 dummy;
+#endif
 };
 
 struct rk3288_sdram_pctl_timing {
-- 
2.9.2


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA

2016-08-12 Thread Simon Glass 
On 12 August 2016 at 01:43, Ziyuan Xu  wrote:
> Add a condition to determine the rk3288_sdram_channel size.
>
> This patch fixes read sdram_channel property failed from DT on rk3288
> boards, which not enable OF_PLATDATA.
>
> Signed-off-by: Ziyuan Xu 
> ---
>
>  arch/arm/include/asm/arch-rockchip/sdram.h | 4 
>  1 file changed, 4 insertions(+)

Acked-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot