[PATCH] staging/android: mark sync_timeline_create() static

[PATCH] staging/android: mark sync_timeline_create() static

[Baoyou Xie]

We get 1 warning when building kernel with W=1:
drivers/staging/android/sw_sync.c:56:23: warning: no previous prototype for 'sync_timeline_create' [-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/android/sw_sync.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/android/sw_sync.c b/drivers/staging/android/sw_sync.c
index 115c917..99a197c 100644
--- a/drivers/staging/android/sw_sync.c
+++ b/drivers/staging/android/sw_sync.c
@@ -53,7 +53,7 @@ static inline struct sync_pt *fence_to_sync_pt(struct fence *fence)
  * Creates a new sync_timeline. Returns the sync_timeline object or NULL in
  * case of error.
  */
-struct sync_timeline *sync_timeline_create(const char *name)
+static struct sync_timeline *sync_timeline_create(const char *name)
 {
 	struct sync_timeline *obj;
 
-- 
2.7.4

Re: [PATCH] staging/android: mark sync_timeline_create() static

[Sudip Mukherjee]

On Sat, Sep 03, 2016 at 02:34:13PM +0800, Baoyou Xie wrote:
> We get 1 warning when building kernel with W=1:
> drivers/staging/android/sw_sync.c:56:23: warning: no previous prototype for 'sync_timeline_create' [-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/android/sw_sync.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

which tree are you using?

This file has been removed from staging by:
35538d7822e8 ("dma-buf/sw_sync: de-stage SW_SYNC")

regards
sudip

Re: [PATCH] staging/android: mark sync_timeline_create() static

[Greg KH]

On Mon, Sep 05, 2016 at 08:40:25AM +0800, Baoyou Xie wrote:
> 
> 
> On 4 September 2016 at 23:41, Sudip Mukherjee <sudipm.mukherjee@gmail.com>
> wrote:
> 
>     On Sat, Sep 03, 2016 at 02:34:13PM +0800, Baoyou Xie wrote:
>     > We get 1 warning when building kernel with W=1:
>     > drivers/staging/android/sw_sync.c:56:23: warning: no previous prototype
>     for 'sync_timeline_create' [-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/android/sw_sync.c | 2 +-
>     >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>     which tree are you using?
> 
> 
> I using master tree with 4.8-rc1

That's a quick way to ensure that your patches will not apply, as that
tree does not contain any of the patches submitted since then.

Please always work against linux-next or my staging-next tree for
staging patches.

thanks,

greg k-h