> -----Original Message-----
> From: dri-devel <dri-devel-boun...@lists.freedesktop.org> On Behalf Of Jani
> Nikula
> Sent: Friday, May 10, 2024 8:38 PM
> To: dri-de...@lists.freedesktop.org
> Cc: amd-...@lists.freedesktop.org; nouv...@lists.freedesktop.org; intel-
> g...@lists.freedesktop.org; Nikula, Jani <jani.nik...@intel.com>
> Subject: [RESEND 6/6] drm/connector: update edid_blob_ptr documentation
>
> Accessing the EDID via edid_blob_ptr causes chicken-and-egg problems. Keep
> edid_blob_ptr as the userspace interface that should be accessed via dedicated
> functions.
>
> Signed-off-by: Jani Nikula <jani.nik...@intel.com>
LGTM
Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.bo...@intel.com>
> ---
> include/drm/drm_connector.h | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
> index fe88d7fc6b8f..58ee9adf9091 100644
> --- a/include/drm/drm_connector.h
> +++ b/include/drm/drm_connector.h
> @@ -1636,8 +1636,12 @@ struct drm_connector {
>
> /**
> * @edid_blob_ptr: DRM property containing EDID if present.
> Protected by
> - * &drm_mode_config.mutex. This should be updated only by calling
> + * &drm_mode_config.mutex.
> + *
> + * This must be updated only by calling drm_edid_connector_update()
> or
> * drm_connector_update_edid_property().
> + *
> + * This must not be used by drivers directly.
> */
> struct drm_property_blob *edid_blob_ptr;
>
> --
> 2.39.2
