diff mbox series

fbdev: hgafb: fix kernel-doc comments

Message ID 20231205231408.1234-1-rdunlap@infradead.org (mailing list archive)
State Accepted, archived
Headers show
Series fbdev: hgafb: fix kernel-doc comments | expand

Commit Message

Randy Dunlap Dec. 5, 2023, 11:14 p.m. UTC
Fix kernel-doc warnings found when using "W=1".

hgafb.c:370: warning: No description found for return value of 'hgafb_open'
hgafb.c:384: warning: No description found for return value of 'hgafb_release'
hgafb.c:406: warning: No description found for return value of 'hgafb_setcolreg'
hgafb.c:425: warning: No description found for return value of 'hgafb_pan_display'
hgafb.c:425: warning: expecting prototype for hga_pan_display(). Prototype was for hgafb_pan_display() instead
hgafb.c:455: warning: No description found for return value of 'hgafb_blank'

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Ferenc Bakonyi <fero@drama.obuda.kando.hu>
Cc: Helge Deller <deller@gmx.de>
Cc: linux-nvidia@lists.surfsouth.com
Cc: linux-fbdev@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
---
 drivers/video/fbdev/hgafb.c |   13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

Comments

Helge Deller Dec. 12, 2023, 1:18 p.m. UTC | #1
On 12/6/23 00:14, Randy Dunlap wrote:
> Fix kernel-doc warnings found when using "W=1".
>
> hgafb.c:370: warning: No description found for return value of 'hgafb_open'
> hgafb.c:384: warning: No description found for return value of 'hgafb_release'
> hgafb.c:406: warning: No description found for return value of 'hgafb_setcolreg'
> hgafb.c:425: warning: No description found for return value of 'hgafb_pan_display'
> hgafb.c:425: warning: expecting prototype for hga_pan_display(). Prototype was for hgafb_pan_display() instead
> hgafb.c:455: warning: No description found for return value of 'hgafb_blank'
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Ferenc Bakonyi <fero@drama.obuda.kando.hu>
> Cc: Helge Deller <deller@gmx.de>
> Cc: linux-nvidia@lists.surfsouth.com
> Cc: linux-fbdev@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> ---
>   drivers/video/fbdev/hgafb.c |   13 +++++++++++--
>   1 file changed, 11 insertions(+), 2 deletions(-)

applied.

Thanks!
Helge
diff mbox series

Patch

diff -- a/drivers/video/fbdev/hgafb.c b/drivers/video/fbdev/hgafb.c
--- a/drivers/video/fbdev/hgafb.c
+++ b/drivers/video/fbdev/hgafb.c
@@ -364,6 +364,8 @@  error:
  *	hgafb_open - open the framebuffer device
  *	@info: pointer to fb_info object containing info for current hga board
  *	@init: open by console system or userland.
+ *
+ *	Returns: %0
  */
 
 static int hgafb_open(struct fb_info *info, int init)
@@ -378,6 +380,8 @@  static int hgafb_open(struct fb_info *in
  *	hgafb_release - open the framebuffer device
  *	@info: pointer to fb_info object containing info for current hga board
  *	@init: open by console system or userland.
+ *
+ *	Returns: %0
  */
 
 static int hgafb_release(struct fb_info *info, int init)
@@ -399,6 +403,8 @@  static int hgafb_release(struct fb_info
  *	This callback function is used to set the color registers of a HGA
  *	board. Since we have only two fixed colors only @regno is checked.
  *	A zero is returned on success and 1 for failure.
+ *
+ *	Returns: %0
  */
 
 static int hgafb_setcolreg(u_int regno, u_int red, u_int green, u_int blue,
@@ -410,14 +416,15 @@  static int hgafb_setcolreg(u_int regno,
 }
 
 /**
- *	hga_pan_display - pan or wrap the display
+ *	hgafb_pan_display - pan or wrap the display
  *	@var:contains new xoffset, yoffset and vmode values
  *	@info:pointer to fb_info object containing info for current hga board
  *
  *	This function looks only at xoffset, yoffset and the %FB_VMODE_YWRAP
  *	flag in @var. If input parameters are correct it calls hga_pan() to
  *	program the hardware. @info->var is updated to the new values.
- *	A zero is returned on success and %-EINVAL for failure.
+ *
+ *	Returns: %0 on success or %-EINVAL for failure.
  */
 
 static int hgafb_pan_display(struct fb_var_screeninfo *var,
@@ -449,6 +456,8 @@  static int hgafb_pan_display(struct fb_v
  *		@blank_mode == 2 means suspend vsync,
  *		@blank_mode == 3 means suspend hsync,
  *		@blank_mode == 4 means powerdown.
+ *
+ * Returns: %0
  */
 
 static int hgafb_blank(int blank_mode, struct fb_info *info)