Re: [PATCH 1/2] Staging: ks7010: mark symbols static where possible
From: Arnd Bergmann
Date: Sat Sep 24 2016 - 05:20:09 EST
On Saturday, September 24, 2016 9:34:41 AM CEST Baoyou Xie wrote:
> On 24 September 2016 at 01:10, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Fri, Sep 23, 2016 at 09:22:29PM +0800, Baoyou Xie wrote:
> > > We get 3 warnings when building kernel with W=1:
> > > drivers/staging/ks7010/ks7010_sdio.c:90:6: warning: no previous
> > prototype for 'ks_wlan_hw_sleep_doze_request' [-Wmissing-prototypes]
> > > drivers/staging/ks7010/ks7010_sdio.c:120:6: warning: no previous
> > prototype for 'ks_wlan_hw_sleep_wakeup_request' [-Wmissing-prototypes]
> > > drivers/staging/ks7010/ks7010_sdio.c:172:5: warning: no previous
> > prototype for '_ks_wlan_hw_power_save' [-Wmissing-prototypes]
> > >
> > > In fact, these functions are only used in the file in which they are
> > > declared and don't need a declaration, but can be made static.
> > > so this patch marks these functions with 'static'.
> > >
> > > Signed-off-by: Baoyou Xie <baoyou.xie@xxxxxxxxxx>
> > > ---
> > > drivers/staging/ks7010/ks7010_sdio.c | 6 +++---
> > > 1 file changed, 3 insertions(+), 3 deletions(-)
> > Does not apply cleanly, are you sure you are using the latest tree?
> I worked on linux next-20160922 tag, could you tell me how to find the
> valid tree?
The file saw a couple of small changes in next-20160923, so I think
this was just unfortunate timing but you used the right tree.
staging can be moving very rapidly at times, so if you want to
be sure to get the right tree of the minute, you can
rebase on top of
I usually just base on top of the linux-next of the day, which tends
to work well enough.