Hi Kever,
Thanks for new updated changes.
On Tue, Oct 22, 2019 at 1:34 PM Kever Yang kever.yang@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.