Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1552904

Re: [PATCH v3 1/5] drm/dp: add connector backpointer to drm_dp_aux

Show all headers | View raw

On Fri, Jan 6, 2017 at 9:30 AM, Tomeu Vizoso <tomeu.vizoso@collabora.com> wrote:
> This backpointer allows DP helpers to access the connector it's being
> used for.
>
> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
> ---
>
>  include/drm/drm_dp_helper.h | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
> index 55bbeb0ff594..4fa77b434594 100644
> --- a/include/drm/drm_dp_helper.h
> +++ b/include/drm/drm_dp_helper.h
> @@ -721,6 +721,7 @@ struct drm_dp_aux_msg {
>   * @name: user-visible name of this AUX channel and the I2C-over-AUX adapter
>   * @ddc: I2C adapter that can be used for I2C-over-AUX communication
>   * @dev: pointer to struct device that is the parent for this AUX channel
> + * @connector: backpointer to connector that uses this AUX channel
>   * @hw_mutex: internal mutex used for locking transfers
>   * @transfer: transfers a message representing a single AUX transaction
>   *
> @@ -757,6 +758,7 @@ struct drm_dp_aux {
>         const char *name;
>         struct i2c_adapter ddc;
>         struct device *dev;
> +       struct drm_connector *connector;

Perhaps I'm misreading the series, but it seems like you could instead
add int crc_pipe along with the other crc fields. Then when you start
the crc, set the pipe number. If you have the pipe in the crc worker,
you can wait vblank on that pipe (no pointers needed).

Reasonable?

Sean

>         struct mutex hw_mutex;
>         ssize_t (*transfer)(struct drm_dp_aux *aux,
>                             struct drm_dp_aux_msg *msg);
> --
> 2.9.3
>



-- 
Sean Paul, Software Engineer, Google / Chromium OS

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 0/5] drm/dp: Implement CRC debugfs API Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-06 15:40 +0100
  [PATCH v3 2/5] drm/bridge: analogix_dp: set connector to drm_dp_aux Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-06 15:40 +0100
  [PATCH v3 3/5] drm/dp: add helpers for capture of frame CRCs Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-06 15:40 +0100
    Re: [PATCH v3 3/5] drm/dp: add helpers for capture of frame CRCs Sean Paul <seanpaul@chromium.org> - 2017-01-06 17:00 +0100
      Re: [PATCH v3 3/5] drm/dp: add helpers for capture of frame CRCs Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-09 14:30 +0100
  [PATCH v3 1/5] drm/dp: add connector backpointer to drm_dp_aux Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-06 15:40 +0100
    Re: [PATCH v3 1/5] drm/dp: add connector backpointer to drm_dp_aux Sean Paul <seanpaul@chromium.org> - 2017-01-06 17:00 +0100
      Re: [PATCH v3 1/5] drm/dp: add connector backpointer to drm_dp_aux Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-09 10:10 +0100
        Re: [PATCH v3 1/5] drm/dp: add connector backpointer to drm_dp_aux Sean Paul <seanpaul@chromium.org> - 2017-01-09 17:50 +0100
  [PATCH v3 5/5] drm/rockchip: Implement CRC debugfs API Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-06 15:50 +0100
    Re: [PATCH v3 5/5] drm/rockchip: Implement CRC debugfs API Sean Paul <seanpaul@chromium.org> - 2017-01-06 17:00 +0100
      Re: [PATCH v3 5/5] drm/rockchip: Implement CRC debugfs API Tomeu Vizoso <tomeu.vizoso@collabora.com> - 2017-01-09 14:30 +0100

csiph-web