Messages in this thread | | | Date | Wed, 20 Mar 2019 07:57:46 -0700 | From | Guenter Roeck <> | Subject | Re: [PATCH -next] watchdog: sb_wdog: Make sbwdog_set and sbwdog_pet static |
| |
On Wed, Mar 20, 2019 at 09:42:50PM +0800, Yue Haibing wrote: > From: YueHaibing <yuehaibing@huawei.com> > > Fix sparse warning: > > drivers/watchdog/sb_wdog.c:70:6: warning: symbol 'sbwdog_set' was not declared. Should it be static? > drivers/watchdog/sb_wdog.c:84:6: warning: symbol 'sbwdog_pet' was not declared. Should it be static? > > Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> --- > drivers/watchdog/sb_wdog.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/sb_wdog.c b/drivers/watchdog/sb_wdog.c > index 3abae50..8264fced 100644 > --- a/drivers/watchdog/sb_wdog.c > +++ b/drivers/watchdog/sb_wdog.c > @@ -67,7 +67,7 @@ static DEFINE_SPINLOCK(sbwd_lock); > * > * wdog is the iomem address of the cfg register > */ > -void sbwdog_set(char __iomem *wdog, unsigned long t) > +static void sbwdog_set(char __iomem *wdog, unsigned long t) > { > spin_lock(&sbwd_lock); > __raw_writeb(0, wdog); > @@ -81,7 +81,7 @@ void sbwdog_set(char __iomem *wdog, unsigned long t) > * > * wdog is the iomem address of the cfg register > */ > -void sbwdog_pet(char __iomem *wdog) > +static void sbwdog_pet(char __iomem *wdog) > { > spin_lock(&sbwd_lock); > __raw_writeb(__raw_readb(wdog) | 1, wdog); > -- > 2.7.0 > >
| |