[PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()

Shankar, Uma uma.shankar at intel.com
Thu May 23 14:38:37 UTC 2019



>-----Original Message-----
>From: Sean Paul [mailto:sean at poorly.run]
>Sent: Thursday, May 23, 2019 7:25 PM
>To: dri-devel at lists.freedesktop.org
>Cc: Sean Paul <seanpaul at chromium.org>; Shankar, Uma <uma.shankar at intel.com>;
>Sharma, Shashank <shashank.sharma at intel.com>; Ville Syrjälä
><ville.syrjala at linux.intel.com>; Maarten Lankhorst
><maarten.lankhorst at linux.intel.com>; Maxime Ripard
><maxime.ripard at bootlin.com>; Sean Paul <sean at poorly.run>; David Airlie
><airlied at linux.ie>; Daniel Vetter <daniel at ffwll.ch>; Bartlomiej Zolnierkiewicz
><b.zolnierkie at samsung.com>; Hans Verkuil <hansverk at cisco.com>; linux-
>fbdev at vger.kernel.org
>Subject: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
>
>From: Sean Paul <seanpaul at chromium.org>
>
>Fixes the following warnings:
>../drivers/gpu/drm/drm_edid.c:4925: warning: Function parameter or member
>'conn_state' not described in 'drm_hdmi_infoframe_set_hdr_metadata'
>../drivers/gpu/drm/drm_edid.c:4925: warning: Excess function parameter
>'hdr_metadata' description in 'drm_hdmi_infoframe_set_hdr_metadata'

Thanks Sean Paul for fixing this.
Reviewed-by: Uma Shankar <uma.shankar at intel.com>

>Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support")
>Cc: Uma Shankar <uma.shankar at intel.com>
>Cc: Shashank Sharma <shashank.sharma at intel.com>
>Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
>Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
>Cc: Maxime Ripard <maxime.ripard at bootlin.com>
>Cc: Sean Paul <sean at poorly.run>
>Cc: David Airlie <airlied at linux.ie>
>Cc: Daniel Vetter <daniel at ffwll.ch>
>Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie at samsung.com>
>Cc: "Ville Syrjälä" <ville.syrjala at linux.intel.com>
>Cc: Hans Verkuil <hansverk at cisco.com>
>Cc: dri-devel at lists.freedesktop.org
>Cc: linux-fbdev at vger.kernel.org
>Signed-off-by: Sean Paul <seanpaul at chromium.org>
>---
> drivers/gpu/drm/drm_edid.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index
>262510c2a670..d87f574feeca 100644
>--- a/drivers/gpu/drm/drm_edid.c
>+++ b/drivers/gpu/drm/drm_edid.c
>@@ -4914,7 +4914,7 @@ static inline bool is_eotf_supported(u8 output_eotf, u8
>sink_eotf)
>  * drm_hdmi_infoframe_set_hdr_metadata() - fill an HDMI DRM infoframe with
>  *                                         HDR metadata from userspace
>  * @frame: HDMI DRM infoframe
>- * @hdr_metadata: hdr_source_metadata info from userspace
>+ * @conn_state: Connector state containing HDR metadata
>  *
>  * Return: 0 on success or a negative error code on failure.
>  */
>--
>Sean Paul, Software Engineer, Google / Chromium OS



More information about the dri-devel mailing list