Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1528414

Re: [PATCH 1/2] power: supply: add sbs-charger driver

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

On Wed, Nov 23, 2016 at 12:20:43PM +0100, Nicolas Saenz Julienne wrote:
> This adds support for sbs-charger compilant chips as defined here:
> http://sbs-forum.org/specs/sbc110.pdf
> 
> This was tested on a arm board connected to an LTC41000 battery charger
> chip.
> 
> Signed-off-by: Nicolas Saenz Julienne <nicolas.saenz@prodys.net>
> ---
>  v1 -> v2:
>  - add spec link in header
>  - use proper gpio/interrupt interface
>  - update regmap configuration (max register & endianness)
>  - dropped oldschool .supplied_to assignments
>  - use devm_* APIs

Thanks. Looks almost fine now.

>  drivers/power/supply/Kconfig       |   6 +
>  drivers/power/supply/Makefile      |   1 +
>  drivers/power/supply/sbs-charger.c | 264 +++++++++++++++++++++++++++++++++++++
>  3 files changed, 271 insertions(+)
>  create mode 100644 drivers/power/supply/sbs-charger.c
> 
> [...]
>
> +static const struct regmap_config sbs_regmap = {
> +	.reg_bits	= 8,
> +	.val_bits	= 16,
> +	.max_register	= SBS_CHARGER_REG_ALARM_WARNING,
> +	.volatile_reg	= sbs_volatile_reg,
> +	.val_format_endian = REGMAP_ENDIAN_LITTLE, /* since based on SMBus */
> +};

Please provide at least a readable_reg marking the
range from 0x00-0x10 unreadable. You can check this
with

cat /sys/kernel/debug/regmap/sbs-charger/registers

> [...]

-- Sebastian

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] power: supply: add sbs-charger driver Nicolas Saenz Julienne <nicolas.saenz@prodys.net> - 2016-11-23 12:30 +0100
  [PATCH 1/2] power: supply: add sbs-charger driver Nicolas Saenz Julienne <nicolas.saenz@prodys.net> - 2016-11-23 12:30 +0100
    Re: [PATCH 1/2] power: supply: add sbs-charger driver Sebastian Reichel <sre@kernel.org> - 2016-11-23 14:40 +0100
  [PATCH 2/2] dt-bindings: power: add bindings for sbs-charger Nicolas Saenz Julienne <nicolas.saenz@prodys.net> - 2016-11-23 12:30 +0100
    Re: [PATCH 2/2] dt-bindings: power: add bindings for sbs-charger Sebastian Reichel <sre@kernel.org> - 2016-11-23 14:50 +0100

csiph-web