dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
@ 2019-05-23 13:54 Sean Paul
  2019-05-23 14:38 ` Shankar, Uma
  0 siblings, 1 reply; 3+ messages in thread
From: Sean Paul @ 2019-05-23 13:54 UTC (permalink / raw)
  To: dri-devel
  Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Maxime Ripard,
	Hans Verkuil, David Airlie, Uma Shankar, Sean Paul, Sean Paul

From: Sean Paul <seanpaul@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'

Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support")
Cc: Uma Shankar <uma.shankar@intel.com>
Cc: Shashank Sharma <shashank.sharma@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Maxime Ripard <maxime.ripard@bootlin.com>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
Cc: Hans Verkuil <hansverk@cisco.com>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-fbdev@vger.kernel.org
Signed-off-by: Sean Paul <seanpaul@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

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* RE: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
  2019-05-23 13:54 [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata() Sean Paul
@ 2019-05-23 14:38 ` Shankar, Uma
  2019-05-23 17:12   ` Sean Paul
  0 siblings, 1 reply; 3+ messages in thread
From: Shankar, Uma @ 2019-05-23 14:38 UTC (permalink / raw)
  To: Sean Paul, dri-devel@lists.freedesktop.org
  Cc: linux-fbdev@vger.kernel.org, Bartlomiej Zolnierkiewicz,
	Maxime Ripard, Hans Verkuil, David Airlie, Sean Paul



>-----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 <seanpaul@chromium.org>; Shankar, Uma <uma.shankar@intel.com>;
>Sharma, Shashank <shashank.sharma@intel.com>; Ville Syrjälä
><ville.syrjala@linux.intel.com>; Maarten Lankhorst
><maarten.lankhorst@linux.intel.com>; Maxime Ripard
><maxime.ripard@bootlin.com>; Sean Paul <sean@poorly.run>; David Airlie
><airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Bartlomiej Zolnierkiewicz
><b.zolnierkie@samsung.com>; Hans Verkuil <hansverk@cisco.com>; linux-
>fbdev@vger.kernel.org
>Subject: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
>
>From: Sean Paul <seanpaul@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@intel.com>

>Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support")
>Cc: Uma Shankar <uma.shankar@intel.com>
>Cc: Shashank Sharma <shashank.sharma@intel.com>
>Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>Cc: Maxime Ripard <maxime.ripard@bootlin.com>
>Cc: Sean Paul <sean@poorly.run>
>Cc: David Airlie <airlied@linux.ie>
>Cc: Daniel Vetter <daniel@ffwll.ch>
>Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
>Cc: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
>Cc: Hans Verkuil <hansverk@cisco.com>
>Cc: dri-devel@lists.freedesktop.org
>Cc: linux-fbdev@vger.kernel.org
>Signed-off-by: Sean Paul <seanpaul@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

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
  2019-05-23 14:38 ` Shankar, Uma
@ 2019-05-23 17:12   ` Sean Paul
  0 siblings, 0 replies; 3+ messages in thread
From: Sean Paul @ 2019-05-23 17:12 UTC (permalink / raw)
  To: Shankar, Uma
  Cc: linux-fbdev@vger.kernel.org, Bartlomiej Zolnierkiewicz,
	Maxime Ripard, dri-devel@lists.freedesktop.org, Hans Verkuil,
	David Airlie, Sean Paul, Sean Paul

On Thu, May 23, 2019 at 02:38:37PM +0000, Shankar, Uma wrote:
> 
> 
> >-----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 <seanpaul@chromium.org>; Shankar, Uma <uma.shankar@intel.com>;
> >Sharma, Shashank <shashank.sharma@intel.com>; Ville Syrjälä
> ><ville.syrjala@linux.intel.com>; Maarten Lankhorst
> ><maarten.lankhorst@linux.intel.com>; Maxime Ripard
> ><maxime.ripard@bootlin.com>; Sean Paul <sean@poorly.run>; David Airlie
> ><airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Bartlomiej Zolnierkiewicz
> ><b.zolnierkie@samsung.com>; Hans Verkuil <hansverk@cisco.com>; linux-
> >fbdev@vger.kernel.org
> >Subject: [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata()
> >
> >From: Sean Paul <seanpaul@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@intel.com>

Pushed to -misc-next, thank you for the review!

Sean

> 
> >Fixes: 2cdbfd66a829 ("drm: Enable HDR infoframe support")
> >Cc: Uma Shankar <uma.shankar@intel.com>
> >Cc: Shashank Sharma <shashank.sharma@intel.com>
> >Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> >Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> >Cc: Maxime Ripard <maxime.ripard@bootlin.com>
> >Cc: Sean Paul <sean@poorly.run>
> >Cc: David Airlie <airlied@linux.ie>
> >Cc: Daniel Vetter <daniel@ffwll.ch>
> >Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> >Cc: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
> >Cc: Hans Verkuil <hansverk@cisco.com>
> >Cc: dri-devel@lists.freedesktop.org
> >Cc: linux-fbdev@vger.kernel.org
> >Signed-off-by: Sean Paul <seanpaul@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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-05-23 17:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-23 13:54 [PATCH] drm/edid: Fix docbook in drm_hdmi_infoframe_set_hdr_metadata() Sean Paul
2019-05-23 14:38 ` Shankar, Uma
2019-05-23 17:12   ` Sean Paul

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox