Messages in this thread | | | Subject | Re: [PATCH -next] watchdog: sb_wdog: Make sbwdog_set and sbwdog_pet static | From | Mukesh Ojha <> | Date | Thu, 21 Mar 2019 00:57:00 +0530 |
| |
On 3/20/2019 7:12 PM, 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> > --- > 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);
Acked-by: Mukesh Ojha <mojha@codeaurora.org>
Thanks.
Mukesh
| |