Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 28 Aug 2020 17:59:11 +0200
From:   Sebastian Reichel <sebastian.reichel@collabora.com>
To:     Andreas Kemnade <andreas@kemnade.info>
Cc:     lee.jones@linaro.org, linux-kernel@vger.kernel.org,
        linux-pm@vger.kernel.org, b.galvani@gmail.com, phh@phh.me,
        letux-kernel@openphoenux.org, Randy Dunlap <rdunlap@infradead.org>
Subject: Re: [PATCH v3] power: supply: Add support for RN5T618/RC5T619
 charger and fuel gauge
Message-ID: <20200828155911.dvuwdzzp4gxdlr3a@earth.universe>
References: <20200828114906.10056-1-andreas@kemnade.info>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="a4lvmlxu4smauyfa"
Content-Disposition: inline
In-Reply-To: <20200828114906.10056-1-andreas@kemnade.info>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--a4lvmlxu4smauyfa
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi,

On Fri, Aug 28, 2020 at 01:49:06PM +0200, Andreas Kemnade wrote:
> Both chips have charger and a fuel gauge.
>=20
> This adds basic support for displaying the state of the battery and the
> input power, settings are not modified. There are some defaults set via
> OTP.
>=20
> Charging also starts after plugging USB.
>=20
> Known issues of the fuel gauge: There are drivers in the wild which disab=
le
> the fuel gauge at shutdown. If a kernel is booted without fuel gauge
> support, after such a driver has been used, the fuel gauge will stay off
> and decalibrate.
> If this driver is used after that, it might display wrong values for char=
ge
> level.
>=20
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
> ---
> Changes in v3:
> - punctuation fixed
>=20
> Changes in v2:
> - patch 2/2 is already accepted, so it is not included, this
>   one can be applied independantly
> - cleanup of sign handling
> - adp properties fixed

Thanks, queued.

-- Sebastian

>=20
>  drivers/power/supply/Kconfig         |   8 +
>  drivers/power/supply/Makefile        |   1 +
>  drivers/power/supply/rn5t618_power.c | 556 +++++++++++++++++++++++++++
>  3 files changed, 565 insertions(+)
>  create mode 100644 drivers/power/supply/rn5t618_power.c
>=20
> diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
> index 44d3c8512fb8..1117a79860b6 100644
> --- a/drivers/power/supply/Kconfig
> +++ b/drivers/power/supply/Kconfig
> @@ -739,4 +739,12 @@ config CHARGER_WILCO
>  	  information can be found in
>  	  Documentation/ABI/testing/sysfs-class-power-wilco
> =20
> +config RN5T618_POWER
> +	tristate "RN5T618 charger/fuel gauge support"
> +	depends on MFD_RN5T618
> +	help
> +	  Say Y here to have support for RN5T618 PMIC family fuel gauge and cha=
rger.
> +	  This driver can also be built as a module. If so, the module will be
> +	  called rn5t618_power.
> +
>  endif # POWER_SUPPLY
> diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile
> index b9644663e435..23866b6ccdae 100644
> --- a/drivers/power/supply/Makefile
> +++ b/drivers/power/supply/Makefile
> @@ -95,3 +95,4 @@ obj-$(CONFIG_CHARGER_UCS1002)	+=3D ucs1002_power.o
>  obj-$(CONFIG_CHARGER_BD70528)	+=3D bd70528-charger.o
>  obj-$(CONFIG_CHARGER_BD99954)	+=3D bd99954-charger.o
>  obj-$(CONFIG_CHARGER_WILCO)	+=3D wilco-charger.o
> +obj-$(CONFIG_RN5T618_POWER)	+=3D rn5t618_power.o
> diff --git a/drivers/power/supply/rn5t618_power.c b/drivers/power/supply/=
rn5t618_power.c
> new file mode 100644
> index 000000000000..424d2817bee5
> --- /dev/null
> +++ b/drivers/power/supply/rn5t618_power.c
> @@ -0,0 +1,556 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Power supply driver for the RICOH RN5T618 power management chip family
> + *
> + * Copyright (C) 2020 Andreas Kemnade
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/device.h>
> +#include <linux/bitops.h>
> +#include <linux/errno.h>
> +#include <linux/init.h>
> +#include <linux/interrupt.h>
> +#include <linux/module.h>
> +#include <linux/mfd/rn5t618.h>
> +#include <linux/platform_device.h>
> +#include <linux/power_supply.h>
> +#include <linux/regmap.h>
> +#include <linux/slab.h>
> +
> +#define CHG_STATE_ADP_INPUT 0x40
> +#define CHG_STATE_USB_INPUT 0x80
> +#define CHG_STATE_MASK	0x1f
> +#define CHG_STATE_CHG_OFF	0
> +#define CHG_STATE_CHG_READY_VADP	1
> +#define CHG_STATE_CHG_TRICKLE	2
> +#define CHG_STATE_CHG_RAPID	3
> +#define CHG_STATE_CHG_COMPLETE	4
> +#define CHG_STATE_SUSPEND	5
> +#define CHG_STATE_VCHG_OVER_VOL	6
> +#define CHG_STATE_BAT_ERROR	7
> +#define CHG_STATE_NO_BAT	8
> +#define CHG_STATE_BAT_OVER_VOL	9
> +#define CHG_STATE_BAT_TEMP_ERR	10
> +#define CHG_STATE_DIE_ERR	11
> +#define CHG_STATE_DIE_SHUTDOWN	12
> +#define CHG_STATE_NO_BAT2	13
> +#define CHG_STATE_CHG_READY_VUSB	14
> +
> +#define FG_ENABLE 1
> +
> +struct rn5t618_power_info {
> +	struct rn5t618 *rn5t618;
> +	struct platform_device *pdev;
> +	struct power_supply *battery;
> +	struct power_supply *usb;
> +	struct power_supply *adp;
> +	int irq;
> +};
> +
> +static enum power_supply_property rn5t618_usb_props[] =3D {
> +	POWER_SUPPLY_PROP_STATUS,
> +	POWER_SUPPLY_PROP_ONLINE,
> +};
> +
> +static enum power_supply_property rn5t618_adp_props[] =3D {
> +	POWER_SUPPLY_PROP_STATUS,
> +	POWER_SUPPLY_PROP_ONLINE,
> +};
> +
> +
> +static enum power_supply_property rn5t618_battery_props[] =3D {
> +	POWER_SUPPLY_PROP_STATUS,
> +	POWER_SUPPLY_PROP_PRESENT,
> +	POWER_SUPPLY_PROP_VOLTAGE_NOW,
> +	POWER_SUPPLY_PROP_CURRENT_NOW,
> +	POWER_SUPPLY_PROP_CAPACITY,
> +	POWER_SUPPLY_PROP_TEMP,
> +	POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW,
> +	POWER_SUPPLY_PROP_TIME_TO_FULL_NOW,
> +	POWER_SUPPLY_PROP_TECHNOLOGY,
> +	POWER_SUPPLY_PROP_CHARGE_FULL,
> +	POWER_SUPPLY_PROP_CHARGE_NOW,
> +};
> +
> +static int rn5t618_battery_read_doublereg(struct rn5t618_power_info *inf=
o,
> +					  u8 reg, u16 *result)
> +{
> +	int ret, i;
> +	u8 data[2];
> +	u16 old, new;
> +
> +	old =3D 0;
> +	/* Prevent races when registers are changing. */
> +	for (i =3D 0; i < 3; i++) {
> +		ret =3D regmap_bulk_read(info->rn5t618->regmap,
> +				       reg, data, sizeof(data));
> +		if (ret)
> +			return ret;
> +
> +		new =3D data[0] << 8;
> +		new |=3D data[1];
> +		if (new =3D=3D old)
> +			break;
> +
> +		old =3D new;
> +	}
> +
> +	*result =3D new;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_decode_status(unsigned int status)
> +{
> +	switch (status & CHG_STATE_MASK) {
> +	case CHG_STATE_CHG_OFF:
> +	case CHG_STATE_SUSPEND:
> +	case CHG_STATE_VCHG_OVER_VOL:
> +	case CHG_STATE_DIE_SHUTDOWN:
> +		return POWER_SUPPLY_STATUS_DISCHARGING;
> +
> +	case CHG_STATE_CHG_TRICKLE:
> +	case CHG_STATE_CHG_RAPID:
> +		return POWER_SUPPLY_STATUS_CHARGING;
> +
> +	case CHG_STATE_CHG_COMPLETE:
> +		return POWER_SUPPLY_STATUS_FULL;
> +
> +	default:
> +		return POWER_SUPPLY_STATUS_NOT_CHARGING;
> +	}
> +}
> +
> +static int rn5t618_battery_status(struct rn5t618_power_info *info,
> +				  union power_supply_propval *val)
> +{
> +	unsigned int v;
> +	int ret;
> +
> +	ret =3D regmap_read(info->rn5t618->regmap, RN5T618_CHGSTATE, &v);
> +	if (ret)
> +		return ret;
> +
> +	val->intval =3D POWER_SUPPLY_STATUS_UNKNOWN;
> +
> +	if (v & 0xc0) { /* USB or ADP plugged */
> +		val->intval =3D rn5t618_decode_status(v);
> +	} else
> +		val->intval =3D POWER_SUPPLY_STATUS_DISCHARGING;
> +
> +	return ret;
> +}
> +
> +static int rn5t618_battery_present(struct rn5t618_power_info *info,
> +				   union power_supply_propval *val)
> +{
> +	unsigned int v;
> +	int ret;
> +
> +	ret =3D regmap_read(info->rn5t618->regmap, RN5T618_CHGSTATE, &v);
> +	if (ret)
> +		return ret;
> +
> +	v &=3D CHG_STATE_MASK;
> +	if ((v =3D=3D CHG_STATE_NO_BAT) || (v =3D=3D CHG_STATE_NO_BAT2))
> +		val->intval =3D 0;
> +	else
> +		val->intval =3D 1;
> +
> +	return ret;
> +}
> +
> +static int rn5t618_battery_voltage_now(struct rn5t618_power_info *info,
> +				       union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_VOLTAGE_1, &res);
> +	if (ret)
> +		return ret;
> +
> +	val->intval =3D res * 2 * 2500 / 4095 * 1000;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_current_now(struct rn5t618_power_info *info,
> +				       union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_CC_AVEREG1, &res);
> +	if (ret)
> +		return ret;
> +
> +	/* current is negative when discharging */
> +	val->intval =3D sign_extend32(res, 13) * 1000;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_capacity(struct rn5t618_power_info *info,
> +				    union power_supply_propval *val)
> +{
> +	unsigned int v;
> +	int ret;
> +
> +	ret =3D regmap_read(info->rn5t618->regmap, RN5T618_SOC, &v);
> +	if (ret)
> +		return ret;
> +
> +	val->intval =3D v;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_temp(struct rn5t618_power_info *info,
> +				union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_TEMP_1, &res);
> +	if (ret)
> +		return ret;
> +
> +	val->intval =3D sign_extend32(res, 11) * 10 / 16;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_tte(struct rn5t618_power_info *info,
> +			       union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_TT_EMPTY_H, &res);
> +	if (ret)
> +		return ret;
> +
> +	if (res =3D=3D 65535)
> +		return -ENODATA;
> +
> +	val->intval =3D res * 60;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_ttf(struct rn5t618_power_info *info,
> +			       union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_TT_FULL_H, &res);
> +	if (ret)
> +		return ret;
> +
> +	if (res =3D=3D 65535)
> +		return -ENODATA;
> +
> +	val->intval =3D res * 60;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_charge_full(struct rn5t618_power_info *info,
> +				       union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_FA_CAP_H, &res);
> +	if (ret)
> +		return ret;
> +
> +	val->intval =3D res * 1000;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_charge_now(struct rn5t618_power_info *info,
> +				      union power_supply_propval *val)
> +{
> +	u16 res;
> +	int ret;
> +
> +	ret =3D rn5t618_battery_read_doublereg(info, RN5T618_RE_CAP_H, &res);
> +	if (ret)
> +		return ret;
> +
> +	val->intval =3D res * 1000;
> +
> +	return 0;
> +}
> +
> +static int rn5t618_battery_get_property(struct power_supply *psy,
> +					enum power_supply_property psp,
> +					union power_supply_propval *val)
> +{
> +	int ret =3D 0;
> +	struct rn5t618_power_info *info =3D power_supply_get_drvdata(psy);
> +
> +	switch (psp) {
> +	case POWER_SUPPLY_PROP_STATUS:
> +		ret =3D rn5t618_battery_status(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_PRESENT:
> +		ret =3D rn5t618_battery_present(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_VOLTAGE_NOW:
> +		ret =3D rn5t618_battery_voltage_now(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_CURRENT_NOW:
> +		ret =3D rn5t618_battery_current_now(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_CAPACITY:
> +		ret =3D rn5t618_battery_capacity(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_TEMP:
> +		ret =3D rn5t618_battery_temp(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW:
> +		ret =3D rn5t618_battery_tte(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_TIME_TO_FULL_NOW:
> +		ret =3D rn5t618_battery_ttf(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_TECHNOLOGY:
> +		val->intval =3D POWER_SUPPLY_TECHNOLOGY_LION;
> +		break;
> +	case POWER_SUPPLY_PROP_CHARGE_FULL:
> +		ret =3D rn5t618_battery_charge_full(info, val);
> +		break;
> +	case POWER_SUPPLY_PROP_CHARGE_NOW:
> +		ret =3D rn5t618_battery_charge_now(info, val);
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return ret;
> +}
> +
> +static int rn5t618_adp_get_property(struct power_supply *psy,
> +				    enum power_supply_property psp,
> +				    union power_supply_propval *val)
> +{
> +	struct rn5t618_power_info *info =3D power_supply_get_drvdata(psy);
> +	unsigned int chgstate;
> +	bool online;
> +	int ret;
> +
> +	ret =3D regmap_read(info->rn5t618->regmap, RN5T618_CHGSTATE, &chgstate);
> +	if (ret)
> +		return ret;
> +
> +	online =3D !!(chgstate & CHG_STATE_ADP_INPUT);
> +
> +	switch (psp) {
> +	case POWER_SUPPLY_PROP_ONLINE:
> +		val->intval =3D online;
> +		break;
> +	case POWER_SUPPLY_PROP_STATUS:
> +		if (!online) {
> +			val->intval =3D POWER_SUPPLY_STATUS_NOT_CHARGING;
> +			break;
> +		}
> +		val->intval =3D rn5t618_decode_status(chgstate);
> +		if (val->intval !=3D POWER_SUPPLY_STATUS_CHARGING)
> +			val->intval =3D POWER_SUPPLY_STATUS_NOT_CHARGING;
> +
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return 0;
> +}
> +
> +static int rn5t618_usb_get_property(struct power_supply *psy,
> +				    enum power_supply_property psp,
> +				    union power_supply_propval *val)
> +{
> +	struct rn5t618_power_info *info =3D power_supply_get_drvdata(psy);
> +	unsigned int chgstate;
> +	bool online;
> +	int ret;
> +
> +	ret =3D regmap_read(info->rn5t618->regmap, RN5T618_CHGSTATE, &chgstate);
> +	if (ret)
> +		return ret;
> +
> +	online =3D !!(chgstate & CHG_STATE_USB_INPUT);
> +
> +	switch (psp) {
> +	case POWER_SUPPLY_PROP_ONLINE:
> +		val->intval =3D online;
> +		break;
> +	case POWER_SUPPLY_PROP_STATUS:
> +		if (!online) {
> +			val->intval =3D POWER_SUPPLY_STATUS_NOT_CHARGING;
> +			break;
> +		}
> +		val->intval =3D rn5t618_decode_status(chgstate);
> +		if (val->intval !=3D POWER_SUPPLY_STATUS_CHARGING)
> +			val->intval =3D POWER_SUPPLY_STATUS_NOT_CHARGING;
> +
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return 0;
> +}
> +
> +static const struct power_supply_desc rn5t618_battery_desc =3D {
> +	.name                   =3D "rn5t618-battery",
> +	.type                   =3D POWER_SUPPLY_TYPE_BATTERY,
> +	.properties             =3D rn5t618_battery_props,
> +	.num_properties         =3D ARRAY_SIZE(rn5t618_battery_props),
> +	.get_property           =3D rn5t618_battery_get_property,
> +};
> +
> +static const struct power_supply_desc rn5t618_adp_desc =3D {
> +	.name                   =3D "rn5t618-adp",
> +	.type                   =3D POWER_SUPPLY_TYPE_MAINS,
> +	.properties             =3D rn5t618_adp_props,
> +	.num_properties         =3D ARRAY_SIZE(rn5t618_adp_props),
> +	.get_property           =3D rn5t618_adp_get_property,
> +};
> +
> +static const struct power_supply_desc rn5t618_usb_desc =3D {
> +	.name                   =3D "rn5t618-usb",
> +	.type                   =3D POWER_SUPPLY_TYPE_USB,
> +	.properties             =3D rn5t618_usb_props,
> +	.num_properties         =3D ARRAY_SIZE(rn5t618_usb_props),
> +	.get_property           =3D rn5t618_usb_get_property,
> +};
> +
> +static irqreturn_t rn5t618_charger_irq(int irq, void *data)
> +{
> +	struct device *dev =3D data;
> +	struct rn5t618_power_info *info =3D dev_get_drvdata(dev);
> +
> +	unsigned int ctrl, stat1, stat2, err;
> +
> +	regmap_read(info->rn5t618->regmap, RN5T618_CHGERR_IRR, &err);
> +	regmap_read(info->rn5t618->regmap, RN5T618_CHGCTRL_IRR, &ctrl);
> +	regmap_read(info->rn5t618->regmap, RN5T618_CHGSTAT_IRR1, &stat1);
> +	regmap_read(info->rn5t618->regmap, RN5T618_CHGSTAT_IRR2, &stat2);
> +
> +	regmap_write(info->rn5t618->regmap, RN5T618_CHGERR_IRR, 0);
> +	regmap_write(info->rn5t618->regmap, RN5T618_CHGCTRL_IRR, 0);
> +	regmap_write(info->rn5t618->regmap, RN5T618_CHGSTAT_IRR1, 0);
> +	regmap_write(info->rn5t618->regmap, RN5T618_CHGSTAT_IRR2, 0);
> +
> +	dev_dbg(dev, "chgerr: %x chgctrl: %x chgstat: %x chgstat2: %x\n",
> +		err, ctrl, stat1, stat2);
> +
> +	power_supply_changed(info->usb);
> +	power_supply_changed(info->adp);
> +	power_supply_changed(info->battery);
> +
> +	return IRQ_HANDLED;
> +}
> +
> +static int rn5t618_power_probe(struct platform_device *pdev)
> +{
> +	int ret =3D 0;
> +	unsigned int v;
> +	struct power_supply_config psy_cfg =3D {};
> +	struct rn5t618_power_info *info;
> +
> +	info =3D devm_kzalloc(&pdev->dev, sizeof(*info), GFP_KERNEL);
> +	if (!info)
> +		return -ENOMEM;
> +
> +	info->pdev =3D pdev;
> +	info->rn5t618 =3D dev_get_drvdata(pdev->dev.parent);
> +	info->irq =3D -1;
> +
> +	platform_set_drvdata(pdev, info);
> +
> +	ret =3D regmap_read(info->rn5t618->regmap, RN5T618_CONTROL, &v);
> +	if (ret)
> +		return ret;
> +
> +	if (!(v & FG_ENABLE)) {
> +		/* E.g. the vendor kernels of various Kobo and Tolino Ebook
> +		 * readers disable the fuel gauge on shutdown. If a kernel
> +		 * without fuel gauge support is booted after that, the fuel
> +		 * gauge will get decalibrated.
> +		 */
> +		dev_info(&pdev->dev, "Fuel gauge not enabled, enabling now\n");
> +		dev_info(&pdev->dev, "Expect unprecise results\n");
> +		regmap_update_bits(info->rn5t618->regmap, RN5T618_CONTROL,
> +				   FG_ENABLE, FG_ENABLE);
> +	}
> +
> +	psy_cfg.drv_data =3D info;
> +	info->battery =3D devm_power_supply_register(&pdev->dev,
> +						   &rn5t618_battery_desc,
> +						   &psy_cfg);
> +	if (IS_ERR(info->battery)) {
> +		ret =3D PTR_ERR(info->battery);
> +		dev_err(&pdev->dev, "failed to register battery: %d\n", ret);
> +		return ret;
> +	}
> +
> +	info->adp =3D devm_power_supply_register(&pdev->dev,
> +					       &rn5t618_adp_desc,
> +					       &psy_cfg);
> +	if (IS_ERR(info->adp)) {
> +		ret =3D PTR_ERR(info->adp);
> +		dev_err(&pdev->dev, "failed to register adp: %d\n", ret);
> +		return ret;
> +	}
> +
> +	info->usb =3D devm_power_supply_register(&pdev->dev,
> +					       &rn5t618_usb_desc,
> +					       &psy_cfg);
> +	if (IS_ERR(info->usb)) {
> +		ret =3D PTR_ERR(info->usb);
> +		dev_err(&pdev->dev, "failed to register usb: %d\n", ret);
> +		return ret;
> +	}
> +
> +	if (info->rn5t618->irq_data)
> +		info->irq =3D regmap_irq_get_virq(info->rn5t618->irq_data,
> +						RN5T618_IRQ_CHG);
> +
> +	if (info->irq < 0)
> +		info->irq =3D -1;
> +	else {
> +		ret =3D devm_request_threaded_irq(&pdev->dev, info->irq, NULL,
> +						rn5t618_charger_irq,
> +						IRQF_ONESHOT,
> +						"rn5t618_power",
> +						&pdev->dev);
> +
> +		if (ret < 0) {
> +			dev_err(&pdev->dev, "request IRQ:%d fail\n",
> +				info->irq);
> +			info->irq =3D -1;
> +		}
> +	}
> +
> +	return 0;
> +}
> +
> +static struct platform_driver rn5t618_power_driver =3D {
> +	.driver =3D {
> +		.name   =3D "rn5t618-power",
> +	},
> +	.probe =3D rn5t618_power_probe,
> +};
> +
> +module_platform_driver(rn5t618_power_driver);
> +MODULE_ALIAS("platform:rn5t618-power");
> +MODULE_DESCRIPTION("Power supply driver for RICOH RN5T618");
> +MODULE_LICENSE("GPL");
> --=20
> 2.20.1
>=20

--a4lvmlxu4smauyfa
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAl9JKc8ACgkQ2O7X88g7
+pqUdA/+PaWcI6V6JUbtE6wI119cXB7VSXHDuGK2W9kH5h54RL6yGFnOjHsQZY0m
OxtSorZ6JuQSM8ZUwB9liLh0Y+/VqYkj+CunSo8LCRZYKnNMeOof68UWKVOsE9zY
CaNJZh75/oEr1fBKZwXVMhp1819s+MiZtQVEWR4Rn6V8BkF/U6/QevOufYRz1NQe
c9N5J/2VD8W1X/nFx/5C7HDHStP6iaQnKd+RF48+SfrltYlCUyTKGI1SNr7Mtssp
ZjzV/2IwzNG9ujphttmffG6Q15PVEdikFFJ8odxWects2rg/IErNo9Z6p1JfGCZM
+fZrJ8fJv/7ByMBuFm5dWcuCk1/YjVpGhsuJeAP6Ts4/Feu8lZu+ne/11qenrV2/
2pLNFIxn9sv7s93/cTiqv6F+zPMBUPE2+ofze1+Hmfh+yybLGJWH5N4b7rIoja0C
f/mzNQ3VwlqNVxlKvfRsL4ueTKiWYoK+lPB65KkyZk5tSGUa4cbIxIbg2ivBbobY
KmmjXctlbyqpIONisRBvGaXZOk+nLICDRo6PHwAs2UNKyAIJUMeVkmGn0DVq8/H4
XMFPGGx2EJF+U6pCIoN1e6tQ+47pJLXAdsRrB8EITsWyBv+mofk80EZZS5z8gy6F
gK9RY091xNbaDVVvOqq4Sfj5CUryUMdTvVdgxk4qxJDLqV28PjY=
=Csos
-----END PGP SIGNATURE-----

--a4lvmlxu4smauyfa--