[PATCH 34/41] drm/bridge: analogix_dp: Allow master driver to cleanup in unbind

Tomasz Figa tfiga at chromium.org
Fri Mar 10 07:09:57 UTC 2017


Hi Sean,

On Fri, Mar 10, 2017 at 1:32 PM, Sean Paul <seanpaul at chromium.org> wrote:
>
> From: Tomasz Figa <tfiga at chromium.org>
>
> Since we take the ownership of drvdata, the master driver does not have
> any means of accessing its own data from unbind callback and all it can
> do is calling the analogix unbind callback. However it might be
> necessary to do some clean up in the master driver before we really
> unbind the analogix part (such as PSR unregister in Rockchip case), so
> this patch provides a plat_data->cleanup() callback which is called at
> the beginning of analogix_dp_unbind().
>
> Cc: Kristian H. Kristensen <hoegsberg at chromium.org>
> Cc: Brian Norris <briannorris at chromium.org>
> Signed-off-by: Tomasz Figa <tfiga at chromium.org>
> Signed-off-by: Sean Paul <seanpaul at chromium.org>
> ---
>  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 3 +++
>  include/drm/bridge/analogix_dp.h                   | 1 +
>  2 files changed, 4 insertions(+)

FYI, we ended up reverting this and subsequent patches from this
series up to 37, as we found a simpler and cleaner solution. Please
see following 2 CLs for reference:
https://chromium-review.googlesource.com/442390
https://chromium-review.googlesource.com/442392
https://chromium-review.googlesource.com/442391

Anyway, thanks for sending the patches upstream, it's really helpful
given my time constraints.

Best regards,
Tomasz


More information about the dri-devel mailing list