On Thu, May 23, 2019 at 02:38:37PM +0000, Shankar, Uma wrote: > > > >-----Original Message----- > >From: Sean Paul [mailto:s...@poorly.run] > >Sent: Thursday, May 23, 2019 7:25 PM > >To: dri-devel@lists.freedesktop.org > >Cc: Sean Paul <seanp...@chromium.org>; Shankar, Uma <uma.shan...@intel.com>; > >Sharma, Shashank <shashank.sha...@intel.com>; Ville Syrjälä > ><ville.syrj...@linux.intel.com>; Maarten Lankhorst > ><maarten.lankho...@linux.intel.com>; Maxime Ripard > ><maxime.rip...@bootlin.com>; Sean Paul <s...@poorly.run>; David Airlie > ><airl...@linux.ie>; Daniel Vetter <dan...@ffwll.ch>; Bartlomiej > >Zolnierkiewicz > ><b.zolnier...@samsung.com>; Hans Verkuil <hansv...@cisco.com>; linux- > >fb...@vger.kernel.org > >Subject: [PATCH] drm/edid: Fix docbook in > >drm_hdmi_infoframe_set_hdr_metadata() > > > >From: Sean Paul <seanp...@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.shan...@intel.com>
Pushed to -misc-next, thank you for the review! Sean > > >Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support") > >Cc: Uma Shankar <uma.shan...@intel.com> > >Cc: Shashank Sharma <shashank.sha...@intel.com> > >Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> > >Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com> > >Cc: Maxime Ripard <maxime.rip...@bootlin.com> > >Cc: Sean Paul <s...@poorly.run> > >Cc: David Airlie <airl...@linux.ie> > >Cc: Daniel Vetter <dan...@ffwll.ch> > >Cc: Bartlomiej Zolnierkiewicz <b.zolnier...@samsung.com> > >Cc: "Ville Syrjälä" <ville.syrj...@linux.intel.com> > >Cc: Hans Verkuil <hansv...@cisco.com> > >Cc: dri-devel@lists.freedesktop.org > >Cc: linux-fb...@vger.kernel.org > >Signed-off-by: Sean Paul <seanp...@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 > -- Sean Paul, Software Engineer, Google / Chromium OS _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel