All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA
@ 2016-08-12  7:43 Ziyuan Xu
  2016-08-12 17:20 ` Simon Glass
  0 siblings, 1 reply; 3+ messages in thread
From: Ziyuan Xu @ 2016-08-12  7:43 UTC (permalink / raw)
  To: u-boot

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 <xzy.xu@rock-chips.com>
---

 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

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA
  2016-08-12  7:43 [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA Ziyuan Xu
@ 2016-08-12 17:20 ` Simon Glass
  2016-08-27 16:06   ` Simon Glass
  0 siblings, 1 reply; 3+ messages in thread
From: Simon Glass @ 2016-08-12 17:20 UTC (permalink / raw)
  To: u-boot

On 12 August 2016 at 01:43, Ziyuan Xu <xzy.xu@rock-chips.com> 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 <xzy.xu@rock-chips.com>
> ---
>
>  arch/arm/include/asm/arch-rockchip/sdram.h | 4 ++++
>  1 file changed, 4 insertions(+)

Acked-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA
  2016-08-12 17:20 ` Simon Glass
@ 2016-08-27 16:06   ` Simon Glass
  0 siblings, 0 replies; 3+ messages in thread
From: Simon Glass @ 2016-08-27 16:06 UTC (permalink / raw)
  To: u-boot

On 12 August 2016 at 11:20, Simon Glass <sjg@chromium.org> wrote:
> On 12 August 2016 at 01:43, Ziyuan Xu <xzy.xu@rock-chips.com> 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 <xzy.xu@rock-chips.com>
>> ---
>>
>>  arch/arm/include/asm/arch-rockchip/sdram.h | 4 ++++
>>  1 file changed, 4 insertions(+)
>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-rockchip/next, thanks!

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-08-27 16:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-12  7:43 [U-Boot] [PATCH] rockchip: use dummy byte only enable OF_PLATDATA Ziyuan Xu
2016-08-12 17:20 ` Simon Glass
2016-08-27 16:06   ` Simon Glass

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.