diff mbox series

video: fbdev: omap: Add prototype for hwa742_update_window_async() to avoid build warning

Message ID YpMXF9luBc4Sjr9h@ls3530 (mailing list archive)
State Accepted, archived
Headers show
Series video: fbdev: omap: Add prototype for hwa742_update_window_async() to avoid build warning | expand

Commit Message

Helge Deller May 29, 2022, 6:47 a.m. UTC
The symbol hwa742_update_window_async() is exported, but there is no
prototype defined for it. That's why gcc complains:

drivers-video-fbdev-omap-hwa742.c:warning:no-previous-prototype-for-hwa742_update_window_async

Add the prototype, but I wonder if we couldn't drop exporting the symbol
instead. Since omapfb_update_window_async() is exported the same way,
are there any users outside of the tree?

Signed-off-by: Helge Deller <deller@gmx.de>

Comments

Janusz Krzysztofik May 29, 2022, 11:14 a.m. UTC | #1
Hi Helge,

On Sunday, 29 May 2022 08:47:51 CEST Helge Deller wrote:
> The symbol hwa742_update_window_async() is exported, but there is no
> prototype defined for it. That's why gcc complains:
> 
> drivers-video-fbdev-omap-hwa742.c:warning:no-previous-prototype-for-hwa742_update_window_async
> 
> Add the prototype, but I wonder if we couldn't drop exporting the symbol
> instead. Since omapfb_update_window_async() is exported the same way,
> are there any users outside of the tree?

It looks like there are no non-local users of any of those two functions 
either, so we could make them static, and I'm not aware of any out of tree 
users, but let's ask more deeply involved people.  Tony, Aaro, Tomi, can 
you please confirm?

Thanks,
Janusz

> 
> Signed-off-by: Helge Deller <deller@gmx.de>
> 
> diff --git a/drivers/video/fbdev/omap/omapfb.h b/drivers/video/fbdev/omap/omapfb.h
> index d930152c289c..0c753ccbc1c0 100644
> --- a/drivers/video/fbdev/omap/omapfb.h
> +++ b/drivers/video/fbdev/omap/omapfb.h
> @@ -229,5 +229,9 @@ extern int  omapfb_update_window_async(struct fb_info *fbi,
>  				       struct omapfb_update_window *win,
>  				       void (*callback)(void *),
>  				       void *callback_data);
> +extern int  hwa742_update_window_async(struct fb_info *fbi,
> +				       struct omapfb_update_window *win,
> +				       void (*callback)(void *),
> +				       void *callback_data);
> 
>  #endif /* __OMAPFB_H */
>
diff mbox series

Patch

diff --git a/drivers/video/fbdev/omap/omapfb.h b/drivers/video/fbdev/omap/omapfb.h
index d930152c289c..0c753ccbc1c0 100644
--- a/drivers/video/fbdev/omap/omapfb.h
+++ b/drivers/video/fbdev/omap/omapfb.h
@@ -229,5 +229,9 @@  extern int  omapfb_update_window_async(struct fb_info *fbi,
 				       struct omapfb_update_window *win,
 				       void (*callback)(void *),
 				       void *callback_data);
+extern int  hwa742_update_window_async(struct fb_info *fbi,
+				       struct omapfb_update_window *win,
+				       void (*callback)(void *),
+				       void *callback_data);

 #endif /* __OMAPFB_H */