On Fri, May 25, 2018 at 02:26:06PM -0700, Jeykumar Sankaran wrote: > Switch DPU from dsi-staging to upstream dsi driver. To make > the switch atomic, this change includes: > - remove dpu connector layers > - clean up dpu connector dependencies in encoder/crtc > - compile out writeback and display port drivers > - compile out dsi-staging driver (separate patch submitted to > remove the driver) > - adapt upstream device hierarchy > > changes in v2: > - remove files not applicable upstream (Sean Paul) > - remove compiled out non-dsi display init (Sean Paul) > - split unrelated changes into separate patch set (Sean Paul) > changes in v3: > - fix compilation warning > - compile out dsi staging > > Signed-off-by: Jeykumar Sankaran <jsanka@xxxxxxxxxxxxxx> > Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx> > Signed-off-by: Rajesh Yadav <ryadav@xxxxxxxxxxxxxx> > --- /snip > @@ -1089,7 +1095,7 @@ void dpu_rm_release(struct dpu_rm *rm, struct drm_encoder *enc) > { > struct dpu_rm_rsvp *rsvp; > struct drm_connector *conn; > - uint64_t top_ctrl; > + uint64_t top_ctrl = 0; This will always just force the else path. Just remove it all. Sean /snip -- Sean Paul, Software Engineer, Google / Chromium OS -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html