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 <[email protected]>
> Signed-off-by: Sean Paul <[email protected]>
> Signed-off-by: Rajesh Yadav <[email protected]>
> ---

/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
_______________________________________________
Freedreno mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/freedreno

Reply via email to