Re: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
From: Sean Paul <sean@poorly.run>
Date: 2019-05-23 17:12:39
Also in:
dri-devel
On Thu, May 23, 2019 at 02:38:37PM +0000, Shankar, Uma wrote:
quoted
-----Original Message----- From: Sean Paul [mailto:sean@poorly.run] Sent: Thursday, May 23, 2019 7:25 PM To: dri-devel@lists.freedesktop.org Cc: Sean Paul <redacted>; Shankar, Uma <redacted>; Sharma, Shashank [off-list ref]; Ville Syrjälä [off-list ref]; Maarten Lankhorst [off-list ref]; Maxime Ripard [off-list ref]; Sean Paul [off-list ref]; David Airlie [off-list ref]; Daniel Vetter [off-list ref]; Bartlomiej Zolnierkiewicz [off-list ref]; Hans Verkuil [off-list ref]; linux- fbdev@vger.kernel.org Subject: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata() From: Sean Paul <redacted> 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 <redacted>
Pushed to -misc-next, thank you for the review! Sean
quoted
Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support") Cc: Uma Shankar <redacted> Cc: Shashank Sharma <redacted> Cc: Ville Syrjälä <redacted> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Maxime Ripard <redacted> Cc: Sean Paul <sean@poorly.run> Cc: David Airlie <redacted> Cc: Daniel Vetter <redacted> Cc: Bartlomiej Zolnierkiewicz <redacted> Cc: "Ville Syrjälä" <redacted> Cc: Hans Verkuil <redacted> Cc: dri-devel@lists.freedesktop.org Cc: linux-fbdev@vger.kernel.org Signed-off-by: Sean Paul <redacted> --- 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 index262510c2a670..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, u8sink_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
-- Sean Paul, Software Engineer, Google / Chromium OS