[U-Boot] [PATCH v3 1/2] power: pmic: lp87565: Add the basic pmic support

[Simon Glass]

On 7 June 2017 at 07:38, Keerthy <j-keerthy at ti.com> wrote:
> Add support to bind the regulators/child nodes with the pmic.
>
> Signed-off-by: Keerthy <j-keerthy at ti.com>
> ---
>
> Changes in v3:
>
> Returned values from dm_i2c_write and dm_i2c_read as is.
>
> Changes in v2:
>
>   * Used dev_read_subnode function to fetch the regulators node.
>
>
>  drivers/power/pmic/Kconfig   |  7 ++++
>  drivers/power/pmic/Makefile  |  1 +
>  drivers/power/pmic/lp87565.c | 87 ++++++++++++++++++++++++++++++++++++++++++++
>  include/power/lp87565.h      | 12 ++++++
>  4 files changed, 107 insertions(+)
>  create mode 100644 drivers/power/pmic/lp87565.c
>  create mode 100644 include/power/lp87565.h

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

nit below

>
> diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig
> index 3f50c12..e3f9e4d 100644
> --- a/drivers/power/pmic/Kconfig
> +++ b/drivers/power/pmic/Kconfig
> @@ -188,6 +188,13 @@ config PMIC_LP873X
>         The LP873X is a PMIC containing couple of LDOs and couple of SMPS.
>         This driver binds the pmic children.
>
> +config PMIC_LP87565
> +       bool "Enable driver for Texas Instruments LP87565 PMIC"
> +       depends on DM_PMIC
> +       ---help---
> +       The LP87565 is a PMIC containing a bunch of SMPS.

What's an SMPS? Can you write that out in full please?

> +       This driver binds the pmic children.