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

[Sebastian Reichel]

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 
> Tested-by: Tobias Schramm 

Thanks, queued.

-- Sebastian


signature.asc
Description: PGP signature

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 
Tested-by: Tobias Schramm 

Thanks,

Tobias

[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 
Signed-off-by: ChenTao 
---
 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