[PATCH v2 0/5] rockchip: rk3229: add sdram and sd support
Dr. Philipp Tomsich
philipp.tomsich at theobroma-systems.com
Fri Aug 18 00:36:28 PDT 2017
> On 18 Aug 2017, at 08:26, Kever Yang <kever.yang at rock-chips.com> wrote:
> On 08/17/2017 04:34 PM, Dr. Philipp Tomsich wrote:
>>> On 17 Aug 2017, at 09:17, Kever Yang <kever.yang at rock-chips.com> wrote:
>>> Add sdram driver for rk3229 and other fix like pinctrl and sd node.
>>> Changes in v2:
>>> - split this patch in two patches
>>> Kever Yang (5):
>>> rockchip: rk322x: update dram bank size
>>> rockchip: rk322x: add sdram driver
>>> rockchip: rk322x: pinctrl: using compatible name same with dts
>>> rockchip: rk322x: pinctrl: fix IO MASK error on sdcard pin
>>> rockchip: dts: rk3229: remove dram channel info
>>> arch/arm/dts/rk3229-evb.dts | 1 -
>>> arch/arm/include/asm/arch-rockchip/sdram_rk322x.h | 581 +++++++++++++++
>>> arch/arm/mach-rockchip/rk322x-board.c | 10 +-
>>> arch/arm/mach-rockchip/rk322x/Makefile | 1 +
>>> arch/arm/mach-rockchip/rk322x/sdram_rk322x.c | 855 ++++++++++++++++++++++
>> Device-model DRAM controller drivers should generally go to drivers/ram; there’s
>> already a subdirectory for the Rockchip-specific drivers created there.
> I'm sorry, I didn't see it, even with the latest mainline U-Boot,
> and both you and Simon had review the first version driver which send out
> about one month ago, I don't know why it's not applied, so I send it again with
> other patches change.
The decision to move this over to drivers/ram is only about 6 weeks old.
However, I didn’t want to add a new driver in the old location (as we’d then
have to move it in the near future ; note that for the RK3399, I’ll submit a
patch to move the driver to drivers/ram for the next release cycle).
The patch has not been applied, as there’s unaddressed review comments:
I had requested that the amount of data structures are deduplicated, as the
pctl-register seemed the same as the rk3288 and the rk3368.
I think there was a bit more code that could be shared already.
We really need to get our DRAM drivers into shape, as these are becoming
a major source of code duplication.
> - Kever
>>> drivers/pinctrl/rockchip/pinctrl_rk322x.c | 8 +-
>>> 6 files changed, 1447 insertions(+), 9 deletions(-)
>>> create mode 100644 arch/arm/include/asm/arch-rockchip/sdram_rk322x.h
>>> create mode 100644 arch/arm/mach-rockchip/rk322x/sdram_rk322x.c
>> Linux-rockchip mailing list
>> Linux-rockchip at lists.infradead.org
More information about the Linux-rockchip