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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




>-----Original Message-----
>From: Sean Paul [mailto:sean@xxxxxxxxxx]
>Sent: Thursday, May 23, 2019 7:25 PM
>To: dri-devel@xxxxxxxxxxxxxxxxxxxxx
>Cc: Sean Paul <seanpaul@xxxxxxxxxxxx>; Shankar, Uma <uma.shankar@xxxxxxxxx>;
>Sharma, Shashank <shashank.sharma@xxxxxxxxx>; Ville Syrjälä
><ville.syrjala@xxxxxxxxxxxxxxx>; Maarten Lankhorst
><maarten.lankhorst@xxxxxxxxxxxxxxx>; Maxime Ripard
><maxime.ripard@xxxxxxxxxxx>; Sean Paul <sean@xxxxxxxxxx>; David Airlie
><airlied@xxxxxxxx>; Daniel Vetter <daniel@xxxxxxxx>; Bartlomiej Zolnierkiewicz
><b.zolnierkie@xxxxxxxxxxx>; Hans Verkuil <hansverk@xxxxxxxxx>; linux-
>fbdev@xxxxxxxxxxxxxxx
>Subject: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
>
>From: Sean Paul <seanpaul@xxxxxxxxxxxx>
>
>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@xxxxxxxxx>

>Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support")
>Cc: Uma Shankar <uma.shankar@xxxxxxxxx>
>Cc: Shashank Sharma <shashank.sharma@xxxxxxxxx>
>Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
>Cc: Maxime Ripard <maxime.ripard@xxxxxxxxxxx>
>Cc: Sean Paul <sean@xxxxxxxxxx>
>Cc: David Airlie <airlied@xxxxxxxx>
>Cc: Daniel Vetter <daniel@xxxxxxxx>
>Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx>
>Cc: "Ville Syrjälä" <ville.syrjala@xxxxxxxxxxxxxxx>
>Cc: Hans Verkuil <hansverk@xxxxxxxxx>
>Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
>Cc: linux-fbdev@xxxxxxxxxxxxxxx
>Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx>
>---
> 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





[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux