[PATCH v1 1/2] power: supply: sbs-battery: Silence warning about unknown chemistry

From: Dmitry Osipenko
Date: Mon May 10 2021 - 18:09:37 EST


Older variants of controller don't support reporting type of the battery.
Make warning message about unknown chemistry to be printed only once in
order to stop flooding kernel log with the message on each request of the
property. This patch fixes the noisy messages on Asus Transformer TF101.

Tested-by: Antoni Aloy Torrens <aaloytorrens@xxxxxxxxx> # TF101
Tested-by: Nikola Milosavljević <mnidza@xxxxxxxxxxx> # TF101
Signed-off-by: Dmitry Osipenko <digetx@xxxxxxxxx>
---
drivers/power/supply/sbs-battery.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/power/supply/sbs-battery.c b/drivers/power/supply/sbs-battery.c
index 8d7a10730e43..b71fbf543428 100644
--- a/drivers/power/supply/sbs-battery.c
+++ b/drivers/power/supply/sbs-battery.c
@@ -814,7 +814,7 @@ static int sbs_get_chemistry(struct i2c_client *client,
val->intval = POWER_SUPPLY_TECHNOLOGY_UNKNOWN;

if (val->intval == POWER_SUPPLY_TECHNOLOGY_UNKNOWN)
- dev_warn(&client->dev, "Unknown chemistry: %s\n", chemistry);
+ dev_warn_once(&client->dev, "Unknown chemistry: %s\n", chemistry);

return 0;
}
--
2.30.2