2014-10-23 00:06:46

by Masanari Iida

[permalink] [raw]
Subject: [PATCH] gpu: drm: Fix warning caused by a parameter description in drm_crtc.c

This patch fix following error while "make xmldocs"

Warning(.//drivers/gpu/drm/drm_crtc.c:778): Excess function parameter
'mode' description in 'drm_connector_get_cmdline_mode'

Signed-off-by: Masanari Iida <[email protected]>
---
drivers/gpu/drm/drm_crtc.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
index e79c8d3..171f9a5 100644
--- a/drivers/gpu/drm/drm_crtc.c
+++ b/drivers/gpu/drm/drm_crtc.c
@@ -766,7 +766,6 @@ static void drm_mode_remove(struct drm_connector *connector,
/**
* drm_connector_get_cmdline_mode - reads the user's cmdline mode
* @connector: connector to quwery
- * @mode: returned mode
*
* The kernel supports per-connector configration of its consoles through
* use of the video= parameter. This function parses that option and
--
2.1.2.451.g98349e5


2014-10-23 08:12:43

by Daniel Vetter

[permalink] [raw]
Subject: Re: [PATCH] gpu: drm: Fix warning caused by a parameter description in drm_crtc.c

On Thu, Oct 23, 2014 at 09:06:28AM +0900, Masanari Iida wrote:
> This patch fix following error while "make xmldocs"
>
> Warning(.//drivers/gpu/drm/drm_crtc.c:778): Excess function parameter
> 'mode' description in 'drm_connector_get_cmdline_mode'
>
> Signed-off-by: Masanari Iida <[email protected]>

Thanks, picked up for my simple drm core patches queue.
-Daniel

> ---
> drivers/gpu/drm/drm_crtc.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
> index e79c8d3..171f9a5 100644
> --- a/drivers/gpu/drm/drm_crtc.c
> +++ b/drivers/gpu/drm/drm_crtc.c
> @@ -766,7 +766,6 @@ static void drm_mode_remove(struct drm_connector *connector,
> /**
> * drm_connector_get_cmdline_mode - reads the user's cmdline mode
> * @connector: connector to quwery
> - * @mode: returned mode
> *
> * The kernel supports per-connector configration of its consoles through
> * use of the video= parameter. This function parses that option and
> --
> 2.1.2.451.g98349e5
>
> _______________________________________________
> dri-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch