* [PATCH] staging: media: omap4iss: mark omap4iss_flush() static
@ 2016-09-04 6:41 Baoyou Xie
2016-09-05 11:49 ` Arnd Bergmann
2016-09-05 11:49 ` Laurent Pinchart
0 siblings, 2 replies; 3+ messages in thread
From: Baoyou Xie @ 2016-09-04 6:41 UTC (permalink / raw)
To: laurent.pinchart, mchehab, gregkh
Cc: linux-media, devel, linux-kernel, arnd, baoyou.xie, xie.baoyou
We get 1 warning when building kernel with W=1:
drivers/staging/media/omap4iss/iss.c:64:6: warning: no previous prototype for 'omap4iss_flush' [-Wmissing-prototypes]
In fact, this function is only used in the file in which it is
declared and don't need a declaration, but can be made static.
so this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
---
drivers/staging/media/omap4iss/iss.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/omap4iss/iss.c b/drivers/staging/media/omap4iss/iss.c
index 6ceb4eb..e27c7a9 100644
--- a/drivers/staging/media/omap4iss/iss.c
+++ b/drivers/staging/media/omap4iss/iss.c
@@ -61,7 +61,7 @@ static void iss_print_status(struct iss_device *iss)
* See this link for reference:
* http://www.mail-archive.com/linux-omap@vger.kernel.org/msg08149.html
*/
-void omap4iss_flush(struct iss_device *iss)
+static void omap4iss_flush(struct iss_device *iss)
{
iss_reg_write(iss, OMAP4_ISS_MEM_TOP, ISS_HL_REVISION, 0);
iss_reg_read(iss, OMAP4_ISS_MEM_TOP, ISS_HL_REVISION);
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] staging: media: omap4iss: mark omap4iss_flush() static
2016-09-04 6:41 [PATCH] staging: media: omap4iss: mark omap4iss_flush() static Baoyou Xie
@ 2016-09-05 11:49 ` Arnd Bergmann
2016-09-05 11:49 ` Laurent Pinchart
1 sibling, 0 replies; 3+ messages in thread
From: Arnd Bergmann @ 2016-09-05 11:49 UTC (permalink / raw)
To: Baoyou Xie
Cc: laurent.pinchart, mchehab, gregkh, linux-media, devel,
linux-kernel, xie.baoyou
On Sunday, September 4, 2016 2:41:41 PM CEST Baoyou Xie wrote:
> We get 1 warning when building kernel with W=1:
> drivers/staging/media/omap4iss/iss.c:64:6: warning: no previous prototype for 'omap4iss_flush' [-Wmissing-prototypes]
>
> In fact, this function is only used in the file in which it is
> declared and don't need a declaration, but can be made static.
> so this patch marks this function with 'static'.
>
> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] staging: media: omap4iss: mark omap4iss_flush() static
2016-09-04 6:41 [PATCH] staging: media: omap4iss: mark omap4iss_flush() static Baoyou Xie
2016-09-05 11:49 ` Arnd Bergmann
@ 2016-09-05 11:49 ` Laurent Pinchart
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Pinchart @ 2016-09-05 11:49 UTC (permalink / raw)
To: Baoyou Xie
Cc: mchehab, gregkh, linux-media, devel, linux-kernel, arnd, xie.baoyou
Hi Baoyou,
Thank you for the patch.
On Sunday 04 Sep 2016 14:41:41 Baoyou Xie wrote:
> We get 1 warning when building kernel with W=1:
> drivers/staging/media/omap4iss/iss.c:64:6: warning: no previous prototype
> for 'omap4iss_flush' [-Wmissing-prototypes]
>
> In fact, this function is only used in the file in which it is
> declared and don't need a declaration, but can be made static.
> so this patch marks this function with 'static'.
>
> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> drivers/staging/media/omap4iss/iss.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/omap4iss/iss.c
> b/drivers/staging/media/omap4iss/iss.c index 6ceb4eb..e27c7a9 100644
> --- a/drivers/staging/media/omap4iss/iss.c
> +++ b/drivers/staging/media/omap4iss/iss.c
> @@ -61,7 +61,7 @@ static void iss_print_status(struct iss_device *iss)
> * See this link for reference:
> * http://www.mail-archive.com/linux-omap@vger.kernel.org/msg08149.html
> */
> -void omap4iss_flush(struct iss_device *iss)
> +static void omap4iss_flush(struct iss_device *iss)
> {
> iss_reg_write(iss, OMAP4_ISS_MEM_TOP, ISS_HL_REVISION, 0);
> iss_reg_read(iss, OMAP4_ISS_MEM_TOP, ISS_HL_REVISION);
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-09-05 11:54 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-04 6:41 [PATCH] staging: media: omap4iss: mark omap4iss_flush() static Baoyou Xie
2016-09-05 11:49 ` Arnd Bergmann
2016-09-05 11:49 ` Laurent Pinchart
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.