[PATCH -next] power/supply/cw2015: Make some symbols static

[PATCH -next] power/supply/cw2015: Make some symbols static

[ChenTao]

Fix the following warning:

drivers/power/supply/cw2015_battery.c:96:5: warning:
'cw_update_profile' was not declared. Should it be static?
drivers/power/supply/cw2015_battery.c:712:1: warning:
'cw_bat_pm_ops' was not declared. Should it be static?

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: ChenTao <chentao107@huawei.com>
---
 drivers/power/supply/cw2015_battery.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/power/supply/cw2015_battery.c b/drivers/power/supply/cw2015_battery.c
index 8603da6f4593..19f62ea957ee 100644
--- a/drivers/power/supply/cw2015_battery.c
+++ b/drivers/power/supply/cw2015_battery.c
@@ -93,7 +93,7 @@ static int cw_read_word(struct cw_battery *cw_bat, u8 reg, u16 *val)
 	return 0;
 }
 
-int cw_update_profile(struct cw_battery *cw_bat)
+static int cw_update_profile(struct cw_battery *cw_bat)
 {
 	int ret;
 	unsigned int reg_val;
@@ -709,7 +709,7 @@ static int __maybe_unused cw_bat_resume(struct device *dev)
 	return 0;
 }
 
-SIMPLE_DEV_PM_OPS(cw_bat_pm_ops, cw_bat_suspend, cw_bat_resume);
+static SIMPLE_DEV_PM_OPS(cw_bat_pm_ops, cw_bat_suspend, cw_bat_resume);
 
 static int cw_bat_remove(struct i2c_client *client)
 {
-- 
2.22.0

Re: [PATCH -next] power/supply/cw2015: Make some symbols static

[Tobias Schramm]

> Fix the following warning:
> 
> drivers/power/supply/cw2015_battery.c:96:5: warning:
> 'cw_update_profile' was not declared. Should it be static?
> drivers/power/supply/cw2015_battery.c:712:1: warning:
> 'cw_bat_pm_ops' was not declared. Should it be static?
> 

LGTM.

Acked-by: Tobias Schramm <t.schramm@manjaro.org>
Tested-by: Tobias Schramm <t.schramm@manjaro.org>

Thanks,

Tobias

Re: [PATCH -next] power/supply/cw2015: Make some symbols static

[Sebastian Reichel]

[-- Attachment #1: Type: text/plain, Size: 514 bytes --]

Hi,

On Wed, May 06, 2020 at 09:49:26PM +0200, Tobias Schramm wrote:
> > Fix the following warning:
> > 
> > drivers/power/supply/cw2015_battery.c:96:5: warning:
> > 'cw_update_profile' was not declared. Should it be static?
> > drivers/power/supply/cw2015_battery.c:712:1: warning:
> > 'cw_bat_pm_ops' was not declared. Should it be static?
> > 
> 
> LGTM.
> 
> Acked-by: Tobias Schramm <t.schramm@manjaro.org>
> Tested-by: Tobias Schramm <t.schramm@manjaro.org>

Thanks, queued.

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]