Hi Kever,

Thanks for new updated changes.

On Tue, Oct 22, 2019 at 1:34 PM Kever Yang <kever.y...@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.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to