[U-Boot] [PATCH 0/9] rockchip: ram: add common code for sdram driver

Jagan Teki jagan at amarulasolutions.com
Tue Oct 22 18:38:47 UTC 2019


Hi Kever,

Thanks for new updated changes.

On Tue, Oct 22, 2019 at 1:34 PM Kever Yang <kever.yang at rock-chips.com> wrote:
>
>
> The sdram drivers for Rockchip SoCs was all separate, some of the SoCs
> has similar hardware controller and phy, so we have a change to share
> the flow and some of the functions between different SoCs.
> This patch set implement a first version common code based on PX30,
> other SoCs has similar hardware can migrate to this common code, eg.
> rk3328 and rk3399 can use this common code.
>
> This patch set also fix some bug for rk3399 by sync code from latest
> rockchip vendor code.
>
>
>
> Kever Yang (3):
>   ram: rockchip: update lpddr4 timing for rk3399
>   ram: rk3399: Sync the io setting from Rockchip vendor code
>   ram: rk3399: update calculate_stride
>
> YouMin Chen (6):
>   ram: rockchip: rename sdram driver files
>   ram: rockchip: add common sdram driver
>   ram: px30: add sdram driver
>   ram: rk3328: use common sdram driver
>   ram: rk3399: use common sdram driver
>   ram: rk3399: Sync code from rockchip vendor code

But, indeed it is difficult to review a few of the changes where one
of the patches removing existing code and replacing new code in common
sdram driver. Adding required changes one-after-another would help us
to track the proper changes in future and also more maintainable.

Jagan.


More information about the U-Boot mailing list