On Thu, May 23, 2019 at 02:38:37PM +0000, Shankar, Uma wrote: > > > >-----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> Pushed to -misc-next, thank you for the review! Sean > > >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 > -- Sean Paul, Software Engineer, Google / Chromium OS
