Modern device tree bindings are supposed to be created as YAML-files
in accordance with dt-schema. This commit replaces SYSCON reboot-mode
legacy bare text bindings with YAML file. As before the bindings file
states that the corresponding dts node is supposed to be compatible
"syscon-reboot-mode" device and necessarily have an offset property
to determine which register from the regmap is supposed to keep the
mode on reboot.
Signed-off-by: NSerge Semin <Sergey.Semin@baikalelectronics.ru>
Reviewed-by: NRob Herring <robh@kernel.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

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: NHulk Robot <hulkci@huawei.com>
Signed-off-by: NChenTao <chentao107@huawei.com>
Acked-by: NTobias Schramm <t.schramm@manjaro.org>
Tested-by: NTobias Schramm <t.schramm@manjaro.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

We introduced some new locking here, but need to update the error
paths so they unlock before returning.

Fixes: 72d9cd9c ("power: bq25890: protect view of the chip's state")
Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Add POWER_SUPPLY_TYPE to the uevent env for power supply. Type is a
property of all power supplies and there is a sysfs entry for it but it
is not included in the properties array of the power supply so
explicitly add it to the udev env.
Signed-off-by: NMathew King <mathewk@chromium.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Reduce the number of touch points to add a new enum property to the
power_supply class by mapping the array of text values to the device
attribute descriptor. A new enum property can now added by creating an
array with the text values named POWER_SUPPLY_${PROPNAME}_TEXT and
adding POWER_SUPPLY_ENUM_ATTR(${PROPNAME}) to the power_supply_attrs
array.
Signed-off-by: NMathew King <mathewk@chromium.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Use designated initializers for the sysfs power supply text values. This
will help ensure that the text values are kept in sync with the enum
values from power_supply.h.
Signed-off-by: NMathew King <mathewk@chromium.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Make the device attribute list used to create sysfs attributes more
robust by decoupling the list order from order of the enum defined in
power_supply.h. This is done by using a designated initializer in the
POWER_SUPPLY_ATTR macro.
Signed-off-by: NMathew King <mathewk@chromium.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Extend bq->lock over whole updating of the chip's state. Might get
useful later for switching ADC modes correctly.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

regmap initialization will check I2C adapter functionality.
Remove redundant check in the driver.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Property list should not change, so mark it const.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Driver rejects unknown chips early in the probe(), so when
bq25890_power_supply_get_property() is made reachable, bq->chip_version
will already be set to correct value - there is no need to check
it again.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

This patch adds a driver for the CellWise cw2015 fuel gauge.

The CellWise cw2015 is a shuntless, single-cell Li-Ion fuel gauge used
in the pine64 Pinebook Pro laptop and some Raspberry Pi UPS HATs.
Signed-off-by: NTobias Schramm <t.schramm@manjaro.org>
Reviewed-by: NAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

This patch adds the dts binding schema for the cw2015 fuel gauge.
Signed-off-by: NTobias Schramm <t.schramm@manjaro.org>
Acked-by: NRob Herring <robh@kernel.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Signed-off-by: NTobias Schramm <t.schramm@manjaro.org>
Acked-by: NRob Herring <robh@kernel.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

In function power_supply_add_hwmon_sysfs(), psyhw->props is
allocated by bitmap_zalloc(). But this pointer is not deallocated
when devm_add_action fail,  which lead to a memory leak bug. To fix
this, we replace devm_add_action with devm_add_action_or_reset.

Cc: stable@kernel.org
Fixes: e67d4dfc ("power: supply: Add HWMON compatibility layer")
Signed-off-by: NQiushi Wu <wu000273@umn.edu>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Convert the power_supply.txt to power-supply.yaml.
This conversion entailed fixing up the binding to being yaml and dt
checker compliant.

Added a note in the power_supply.txt to reference the power-supply.yaml
Signed-off-by: NDan Murphy <dmurphy@ti.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Rework power_supply_hwmon_read_string() to check it's parameters.
This allows to extend it later with labels for other types of
measurements.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Initialize num_properties with length of the copied array instead
of relying on previously memcpy'd value. This makes it clear how
the array and the counter are related.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Currently HWMON emulation shows all labels (temp and ambient temp)
regardless if power supply supports reading the values. Check that at
least one property is enabled for each label.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

tempX_label files are swapped compared to what
power_supply_hwmon_temp_to_property() uses. Make them match.

Cc: stable@vger.kernel.org
Fixes: e67d4dfc ("power: supply: Add HWMON compatibility layer")
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Since tables pointed to by power_supply_desc->properties and
->usb_types are not expected to change after registration, mark
the pointers accordingly
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

This prepares the driver to work with the properties entry
in power_supply_desc marked as const.
Reviewed-by: NEnric Balletbo i Serra <enric.balletbo@collabora.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

This prepares the driver to work with the properties entry
in power_supply_desc marked as const.
Reviewed-by: NEnric Balletbo i Serra <enric.balletbo@collabora.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Reduce power_supply_show_usb_type() parameter count by folding
power_supply_desc dereference into the function.  This makes following
patch making usb_types const easier.
Signed-off-by: NMichał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Add reboot handler for Oxford OX820 chips as reboot currenly hangs on
those boards. Code is based on ox820_assert_system_reset() found in
https://github.com/kref/linux-oxnas.git in
arch/arm/mach-oxnas/mach-ox820.c line 181.
Signed-off-by: NDaniel Golle <daniel@makrotopia.org>
Acked-by: NNeil Armstrong <narmstrong@baylibre.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

The max170{42,47,50,55} family of fuel gauges all provide time-to-empty
estimation. As such, let's export this as a property.
Signed-off-by: NGeordan Neukum <gneukum1@gmail.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

In the axp288_charger_probe(), when get irq failed, the function
platform_get_irq() logs an error message, so remove redundant
message here.
Signed-off-by: NTang Bin <tangbin@cmss.chinamobile.com>
Signed-off-by: NShengju Zhang <zhangshengju@cmss.chinamobile.com>
Reviewed-by: NHans de Goede <hdegoede@redhat.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Add new property to allow to get the voltage measured during boot time.
Signed-off-by: NYuanjiang Yu <yuanjiang.yu@unisoc.com>
Signed-off-by: NBaolin Wang <baolin.wang7@gmail.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Add new properties to get present current and voltage of the fuel gauge.
Signed-off-by: NYuanjiang Yu <yuanjiang.yu@unisoc.com>
Signed-off-by: NBaolin Wang <baolin.wang7@gmail.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

The battery full capacity can be affected by the temperature or the
servicing time or other factors, so some platforms will track the
real battery full capacity in charger manager service. Thus we should
allow to change the battery full capacity by setting the
'POWER_SUPPLY_PROP_ENERGY_FULL_DESIGN' property as writeable.
Signed-off-by: NYuanjiang Yu <yuanjiang.yu@unisoc.com>
Signed-off-by: NBaolin Wang <baolin.wang7@gmail.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

There is no thermal zone should be created for the SC27XX FGU power supply,
thus set the 'no_thermal' flag as true.
Signed-off-by: NYuanjiang Yu <yuanjiang.yu@unisoc.com>
Signed-off-by: NBaolin Wang <baolin.wang7@gmail.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Fix the following coccicheck warning:

drivers/power/supply/ab8500_fg.c:2402:5-24: WARNING: Comparison to bool
Signed-off-by: NJason Yan <yanaijie@huawei.com>
Reviewed-by: NLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Instead of hardcode the mask, it should be depends
on which generation of pon it was.
Signed-off-by: NKejia Hu <kejia.hu@codethink.co.uk>
Fixes: fce5430f ("reset: qcom-pon: Add support for gen2 pon")
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

In the pm860x_battery_probe(), when get irq failed, the function
platform_get_irq() logs an dev_err message, so remove redundant
message here.
Signed-off-by: NTang Bin <tangbin@cmss.chinamobile.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

    The Meegopad T02 is a PC in stick format and doesn't have a battery,
    it is reported with a random and constant battery charge but as
    discharging to userspace.

    Add it to the blacklist to avoid the bogus battery status reporting.
Signed-off-by: NRafael Gandolfi <rafirafi.at@gmail.com>
Reviewed-by: NHans de Goede <hdegoede@redhat.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

Fix the following gcc warning:

drivers/power/supply/max14656_charger_detector.c:142:6: warning:
variable ‘ret’ set but not used [-Wunused-but-set-variable]
  int ret = 0;
      ^~~
Reported-by: NHulk Robot <hulkci@huawei.com>
Signed-off-by: NJason Yan <yanaijie@huawei.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

regmap is a library function that gets selected by drivers that need
it. No driver modules should depend on it. Depending on REGMAP_I2C makes
this driver only build if another driver already selected REGMAP_I2C,
as the symbol can't be selected through the menu kernel configuration.

Fixes: 2219a935 ("power_supply: Add TI BQ24257 charger driver")
Signed-off-by: NEnric Balletbo i Serra <enric.balletbo@collabora.com>
Signed-off-by: NSebastian Reichel <sebastian.reichel@collabora.com>

This sorts the actual field names too, potentially causing even more
chaos and confusion at merge time if you have edited the MAINTAINERS
file.  But the end result is a more consistent layout, and hopefully
it's a one-time pain minimized by doing this just before the -rc1
release.

This was entirely scripted:

  ./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS --order
Requested-by: NJoe Perches <joe@perches.com>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>

They are all supposed to be sorted, but people who add new entries don't
always know the alphabet.  Plus sometimes the entry names get edited,
and people don't then re-order the entry.

Let's see how painful this will be for merging purposes (the MAINTAINERS
file is often edited in various different trees), but Joe claims there's
relatively few patches in -next that touch this, and doing it just
before -rc1 is likely the best time.  Fingers crossed.

This was scripted with

  /scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS

but then I also ended up manually upper-casing a few entry names that
stood out when looking at the end result.
Requested-by: NJoe Perches <joe@perches.com>
Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>