On Thu, Feb 6, 2020 at 1:52 AM Harigovindan P <harigovi@xxxxxxxxxxxxxx> wrote: > > Save pll state before dsi host is powered off. Without this change > some register values gets resetted. The phy driver already does this. Why is the current implementation insufficient? > > Signed-off-by: Harigovindan P <harigovi@xxxxxxxxxxxxxx> > --- > > Changes in v1: > - Saving pll state before dsi host is powered off. > - Removed calling of save state in post_disable since everything > would be resetted and it would save only resetted values. Removed from post_disable? Thats not what I see in the change since you are adding code to dsi_mgr_bridge_post_disable()