[U-Boot] [PATCH 4/7] dm: pmic: add s2mps11 PMIC I/O driver

Simon Glass sjg at chromium.org
Tue Sep 1 02:33:21 CEST 2015


On 28 August 2015 at 07:59, Przemyslaw Marczak <p.marczak at samsung.com> wrote:
> This driver allows I/O operations on the Samsung S2MPS11 PMIC,
> which provides lots of LDO/BUCK outputs.
>
> To enable it, update defconfig with:
> - CONFIG_DM_PMIC_S2MPS11
> and additional, if were not defined:
> - CONFIG_CMD_PMIC
> - CONFIG_ERRNO_STR
>
> The binding info: doc/device-tree-bindings/pmic/s2mps11.txt
>
> Signed-off-by: Przemyslaw Marczak <p.marczak at samsung.com>
> ---
>  doc/device-tree-bindings/pmic/s2mps11.txt |  17 +++++
>  drivers/power/pmic/Kconfig                |  14 ++++
>  drivers/power/pmic/Makefile               |   1 +
>  drivers/power/pmic/s2mps11.c              |  60 ++++++++++++++++
>  include/power/s2mps11.h                   | 109 ++++++++++++++++++++++++++++++
>  5 files changed, 201 insertions(+)
>  create mode 100644 doc/device-tree-bindings/pmic/s2mps11.txt
>  create mode 100644 drivers/power/pmic/s2mps11.c
>  create mode 100644 include/power/s2mps11.h

Reviewed-by: Simon Glass <sjg at chromium.org>

(2 nits below)

>
> diff --git a/doc/device-tree-bindings/pmic/s2mps11.txt b/doc/device-tree-bindings/pmic/s2mps11.txt
> new file mode 100644
> index 0000000..db8d624
> --- /dev/null
> +++ b/doc/device-tree-bindings/pmic/s2mps11.txt
> @@ -0,0 +1,17 @@
> +SAMSUBNG, S2MPS11 PMIC
> +
> +This file describes the binding info for the PMIC driver:
> +- drivers/power/pmic/s2mps11.c
> +
> +Required properties:
> +- compatible: "samsung,s2mps11-pmic"
> +- reg = 0x66
> +
> +With those two properties, the pmic device can be used for read/write only.
> +
> +Example:
> +
> +s2mps11 at 66 {
> +       compatible = "samsung,s2mps11-pmic";
> +       reg = <0x66>;
> +};
> diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig
> index fc6a374..c66b352 100644
> --- a/drivers/power/pmic/Kconfig
> +++ b/drivers/power/pmic/Kconfig
> @@ -24,6 +24,20 @@ config DM_PMIC_MAX77686
>         This config enables implementation of driver-model pmic uclass features
>         for PMIC MAX77686. The driver implements read/write operations.
>
> +config DM_PMIC_S2MPS11

nit: I don't think this should have a DM_ prefix. Remember that all
the CONFIG_DM_... options are going to be removed one day. It is
intended to select the DM version of a subsystem. In your case you
only have a DM version of the driver, so just CONFIG_PMIC_S2MPS11
seems fine.

> +       bool "Enable Driver Model for PMIC Samsung S2MPS11"
> +       depends on DM_PMIC
> +       ---help---
> +       The Samsung S2MPS11 PMIC provides:
> +        - 38 adjustable LDO regulators
> +        - 9 High-Efficiency Buck Converters
> +        - 1 BuckBoost Converter
> +        - RTC with two alarms
> +        - Backup battery charger
> +        - I2C Configuration Interface
> +       This driver provides access to I/O interface only.
> +       Binding info: doc/device-tree-bindings/pmic/s2mps11.txt
> +
>  config DM_PMIC_SANDBOX
>         bool "Enable Driver Model for emulated Sandbox PMIC "
>         depends on DM_PMIC
> diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile
> index 99c5778..1b58d98 100644
> --- a/drivers/power/pmic/Makefile
> +++ b/drivers/power/pmic/Makefile
> @@ -8,6 +8,7 @@
>  obj-$(CONFIG_DM_PMIC) += pmic-uclass.o
>  obj-$(CONFIG_DM_PMIC_MAX77686) += max77686.o
>  obj-$(CONFIG_DM_PMIC_PFUZE100) += pfuze100.o
> +obj-$(CONFIG_DM_PMIC_S2MPS11) += s2mps11.o
>  obj-$(CONFIG_DM_PMIC_SANDBOX) += sandbox.o i2c_pmic_emul.o
>  obj-$(CONFIG_PMIC_TPS65090) += tps65090.o
>  obj-$(CONFIG_PMIC_S5M8767) += s5m8767.o
> diff --git a/drivers/power/pmic/s2mps11.c b/drivers/power/pmic/s2mps11.c
> new file mode 100644
> index 0000000..7e28402
> --- /dev/null
> +++ b/drivers/power/pmic/s2mps11.c
> @@ -0,0 +1,60 @@
> +/*
> + *  Copyright (C) 2015 Samsung Electronics
> + *  Przemyslaw Marczak  <p.marczak at samsung.com>
> + *
> + * SPDX-License-Identifier:    GPL-2.0+
> + */
> +
> +#include <common.h>
> +#include <fdtdec.h>
> +#include <errno.h>
> +#include <dm.h>
> +#include <i2c.h>
> +#include <power/pmic.h>
> +#include <power/s2mps11.h>
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +static int s2mps11_reg_count(struct udevice *dev)
> +{
> +       return S2MPS11_REG_COUNT;
> +}
> +
> +static int s2mps11_write(struct udevice *dev, uint reg, const uint8_t *buff,
> +                        int len)
> +{
> +       if (dm_i2c_write(dev, reg, buff, len)) {
> +               error("write error to device: %p register: %#x!", dev, reg);
> +               return -EIO;
> +       }
> +
> +       return 0;
> +}
> +
> +static int s2mps11_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
> +{
> +       if (dm_i2c_read(dev, reg, buff, len)) {
> +               error("read error from device: %p register: %#x!", dev, reg);
> +               return -EIO;
> +       }
> +
> +       return 0;
> +}
> +
> +static struct dm_pmic_ops s2mps11_ops = {
> +       .reg_count = s2mps11_reg_count,
> +       .read = s2mps11_read,
> +       .write = s2mps11_write,
> +};
> +
> +static const struct udevice_id s2mps11_ids[] = {
> +       { .compatible = "samsung,s2mps11-pmic" },
> +       { }
> +};
> +
> +U_BOOT_DRIVER(pmic_s2mps11) = {
> +       .name = "s2mps11_pmic",
> +       .id = UCLASS_PMIC,
> +       .of_match = s2mps11_ids,
> +       .ops = &s2mps11_ops,
> +};
> diff --git a/include/power/s2mps11.h b/include/power/s2mps11.h
> new file mode 100644
> index 0000000..885a816
> --- /dev/null
> +++ b/include/power/s2mps11.h
> @@ -0,0 +1,109 @@
> +#ifndef __INCLUDE__S2MPS11__H__

nit: I don't think you need __INCLUDE__ here

> +#define __INCLUDE__S2MPS11__H__
> +
> +enum s2mps11_reg {
> +       S2MPS11_REG_ID = 0,
> +       S2MPS11_REG_INT1,
> +       S2MPS11_REG_INT2,
> +       S2MPS11_REG_INT3,
> +       S2MPS11_REG_INT1M,
> +       S2MPS11_REG_INT2M,
> +       S2MPS11_REG_INT3M,
> +       S2MPS11_REG_STATUS1,
> +       S2MPS11_REG_STATUS2,
> +       S2MPS11_REG_OFFSRC,
> +       S2MPS11_REG_PWRONSRC,
> +       S2MPS11_REG_RTC_CTRL,
> +       S2MPS11_REG_CTRL1,
> +       S2MPS11_REG_ETC_TEST,
> +       S2MPS11_REG_RSVD3,
> +       S2MPS11_REG_BU_CHG,
> +       S2MPS11_REG_RAMP,
> +       S2MPS11_REG_RAMP_BUCK,
> +       S2MPS11_REG_LDO1_8,
> +       S2MPS11_REG_LDO9_16,
> +       S2MPS11_REG_LDO17_24,
> +       S2MPS11_REG_LDO25_32,
> +       S2MPS11_REG_LDO33_38,
> +       S2MPS11_REG_LDO1_8_OVC,
> +       S2MPS11_REG_LDO9_16_OVC,
> +       S2MPS11_REG_LDO17_24_OVC,
> +       S2MPS11_REG_LDO25_32_OVC,
> +       S2MPS11_REG_LDO33_38_OVC,
> +       S2MPS11_REG_RESERVED1,
> +       S2MPS11_REG_RESERVED2,
> +       S2MPS11_REG_RESERVED3,
> +       S2MPS11_REG_RESERVED4,
> +       S2MPS11_REG_RESERVED5,
> +       S2MPS11_REG_RESERVED6,
> +       S2MPS11_REG_RESERVED7,
> +       S2MPS11_REG_RESERVED8,
> +       S2MPS11_REG_WDRSTEN_CTRL,
> +       S2MPS11_REG_B1CTRL1,
> +       S2MPS11_REG_B1CTRL2,
> +       S2MPS11_REG_B2CTRL1,
> +       S2MPS11_REG_B2CTRL2,
> +       S2MPS11_REG_B3CTRL1,
> +       S2MPS11_REG_B3CTRL2,
> +       S2MPS11_REG_B4CTRL1,
> +       S2MPS11_REG_B4CTRL2,
> +       S2MPS11_REG_B5CTRL1,
> +       S2MPS11_REG_BUCK5_SW,
> +       S2MPS11_REG_B5CTRL2,
> +       S2MPS11_REG_B5CTRL3,
> +       S2MPS11_REG_B5CTRL4,
> +       S2MPS11_REG_B5CTRL5,
> +       S2MPS11_REG_B6CTRL1,
> +       S2MPS11_REG_B6CTRL2,
> +       S2MPS11_REG_B7CTRL1,
> +       S2MPS11_REG_B7CTRL2,
> +       S2MPS11_REG_B8CTRL1,
> +       S2MPS11_REG_B8CTRL2,
> +       S2MPS11_REG_B9CTRL1,
> +       S2MPS11_REG_B9CTRL2,
> +       S2MPS11_REG_B10CTRL1,
> +       S2MPS11_REG_B10CTRL2,
> +       S2MPS11_REG_L1CTRL,
> +       S2MPS11_REG_L2CTRL,
> +       S2MPS11_REG_L3CTRL,
> +       S2MPS11_REG_L4CTRL,
> +       S2MPS11_REG_L5CTRL,
> +       S2MPS11_REG_L6CTRL,
> +       S2MPS11_REG_L7CTRL,
> +       S2MPS11_REG_L8CTRL,
> +       S2MPS11_REG_L9CTRL,
> +       S2MPS11_REG_L10CTRL,
> +       S2MPS11_REG_L11CTRL,
> +       S2MPS11_REG_L12CTRL,
> +       S2MPS11_REG_L13CTRL,
> +       S2MPS11_REG_L14CTRL,
> +       S2MPS11_REG_L15CTRL,
> +       S2MPS11_REG_L16CTRL,
> +       S2MPS11_REG_L17CTRL,
> +       S2MPS11_REG_L18CTRL,
> +       S2MPS11_REG_L19CTRL,
> +       S2MPS11_REG_L20CTRL,
> +       S2MPS11_REG_L21CTRL,
> +       S2MPS11_REG_L22CTRL,
> +       S2MPS11_REG_L23CTRL,
> +       S2MPS11_REG_L24CTRL,
> +       S2MPS11_REG_L25CTRL,
> +       S2MPS11_REG_L26CTRL,
> +       S2MPS11_REG_L27CTRL,
> +       S2MPS11_REG_L28CTRL,
> +       S2MPS11_REG_L29CTRL,
> +       S2MPS11_REG_L30CTRL,
> +       S2MPS11_REG_L31CTRL,
> +       S2MPS11_REG_L32CTRL,
> +       S2MPS11_REG_L33CTRL,
> +       S2MPS11_REG_L34CTRL,
> +       S2MPS11_REG_L35CTRL,
> +       S2MPS11_REG_L36CTRL,
> +       S2MPS11_REG_L37CTRL,
> +       S2MPS11_REG_L38CTRL,
> +       S2MPS11_REG_COUNT,
> +};
> +
> +#define S2MPS11_LDO26_ENABLE   0xec
> +
> +#endif
> --
> 1.9.1
>

Regards,
Simon


More information about the U-Boot mailing list