Re: [PATCH 1/2] regulator: lt8722: Add driver for LT8722

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Nov 8, 2024 at 1:40 AM Ramon Cristopher M. Calam
<ramoncristopher.calam@xxxxxxxxxx> wrote:
>
> Add ADI LT8722 full bridge DC/DC converter driver.
>
> Signed-off-by: Ramon Cristopher M. Calam <ramoncristopher.calam@xxxxxxxxxx>
> ---
>  drivers/regulator/Kconfig            |  10 +
>  drivers/regulator/Makefile           |   1 +
>  drivers/regulator/lt8722-regulator.c | 701 +++++++++++++++++++++++++++
>  3 files changed, 712 insertions(+)
>  create mode 100644 drivers/regulator/lt8722-regulator.c
>
> diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
> index 39297f7d8177..d88b050ce166 100644
> --- a/drivers/regulator/Kconfig
> +++ b/drivers/regulator/Kconfig
> @@ -550,6 +550,16 @@ config REGULATOR_LP8788
>         help
>           This driver supports LP8788 voltage regulator chip.
>
> +config REGULATOR_LT8722
> +       tristate "LT8722 ultracompact full bridge driver with SPI"
> +       depends on SPI && OF
> +       help
> +         This driver controls an Analog Devices LT8722 ultracompact 4A, 15V,
> +         full bridge driver with SPI interface.
> +
> +         Say M here if you want to include support for the regulator as a
> +         module.
> +
>  config REGULATOR_LTC3589
>         tristate "LTC3589 8-output voltage regulator"
>         depends on I2C
> diff --git a/drivers/regulator/Makefile b/drivers/regulator/Makefile
> index 3d5a803dce8a..8f9e5aec74d2 100644
> --- a/drivers/regulator/Makefile
> +++ b/drivers/regulator/Makefile
> @@ -67,6 +67,7 @@ obj-$(CONFIG_REGULATOR_LP87565) += lp87565-regulator.o
>  obj-$(CONFIG_REGULATOR_LP8788) += lp8788-buck.o
>  obj-$(CONFIG_REGULATOR_LP8788) += lp8788-ldo.o
>  obj-$(CONFIG_REGULATOR_LP8755) += lp8755.o
> +obj-$(CONFIG_REGULATOR_LT8722) += lt8722-regulator.o
>  obj-$(CONFIG_REGULATOR_LTC3589) += ltc3589.o
>  obj-$(CONFIG_REGULATOR_LTC3676) += ltc3676.o
>  obj-$(CONFIG_REGULATOR_MAX14577) += max14577-regulator.o
> diff --git a/drivers/regulator/lt8722-regulator.c b/drivers/regulator/lt8722-regulator.c
> new file mode 100644
> index 000000000000..f5d378c0b1f2
> --- /dev/null
> +++ b/drivers/regulator/lt8722-regulator.c
> @@ -0,0 +1,701 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Analog Devices LT8722 Ultracompact Full Bridge Driver with SPI driver
> + *
> + * Copyright 2024 Analog Devices Inc.
> + */
> +
> +#include <asm/unaligned.h>
> +#include <linux/bitfield.h>
> +#include <linux/bits.h>
> +#include <linux/crc8.h>
> +#include <linux/delay.h>
> +#include <linux/gpio.h>
> +#include <linux/math.h>
> +#include <linux/module.h>
> +#include <linux/regulator/driver.h>
> +#include <linux/regulator/machine.h>
> +#include <linux/regulator/of_regulator.h>
> +#include <linux/spi/spi.h>
> +#include <linux/util_macros.h>
> +
> +/* Register map */
> +#define LT8722_SPIS_COMMAND            0x00
> +#define LT8722_SPIS_STATUS             0x01
> +#define        LT8722_SPIS_DAC_ILIMN           0x02
> +#define        LT8722_SPIS_DAC_ILIMP           0x03
> +#define        LT8722_SPIS_DAC                 0x04
> +#define        LT8722_SPIS_OV_CLAMP            0x05
> +#define        LT8722_SPIS_UV_CLAMP            0x06
> +#define        LT8722_SPIS_AMUX                0x07
> +
> +/* Register masks */
> +#define LT8722_SPIS_COMMAND_MASK       GENMASK(21, 0)
> +#define LT8722_SPIS_STATUS_MASK                GENMASK(10, 0)
> +#define LT8722_SPIS_DAC_ILIMN_MASK     GENMASK(8, 0)
> +#define LT8722_SPIS_DAC_ILIMP_MASK     GENMASK(8, 0)
> +#define LT8722_SPIS_DAC_MASK           GENMASK(31, 0)
> +#define LT8722_SPIS_OV_CLAMP_MASK      GENMASK(3, 0)
> +#define LT8722_SPIS_UV_CLAMP_MASK      GENMASK(3, 0)
> +#define LT8722_SPIS_AMUX_MASK          GENMASK(6, 0)
> +
> +/* SPIS_COMMAND register bit masks */
> +#define LT8722_EN_REQ_MASK             BIT(0)
> +#define LT8722_SWEN_REQ_MASK           BIT(1)
> +#define LT8722_SW_FRQ_SET_MASK         GENMASK(4, 2)
> +#define LT8722_SW_FRQ_ADJ_MASK         GENMASK(6, 5)
> +#define LT8722_SYS_DC_MASK             GENMASK(8, 7)
> +#define LT8722_VCC_VREG_MASK           BIT(9)
> +#define LT8722_SW_VC_IN_MASK           GENMASK(13, 11)
> +#define LT8722_SPI_RST_MASK            BIT(14)
> +#define LT8722_PWR_LIM_MASK            GENMASK(18, 15)
> +
> +#define LT8722_FAULTS_MASK             GENMASK(10, 5)
> +#define LT8722_UV_OV_MASK              GENMASK(23, 20)
> +#define LT8722_OC_MASK                 BIT(5)
> +#define LT8722_TSD_MASK                        BIT(6)
> +
> +#define LT8722_CRC8_POLY               0x07
> +#define LT8722_CRC_INIT                        0x00
> +
> +#define LT8722_READ_CMD                        0xF4
> +#define LT8722_WRITE_CMD               0xF2
> +#define LT8722_RW_CMD_SIZE             8
> +#define LT8722_DATA_SIZE               4
> +#define LT8722_DATA_POS                        2
> +#define LT8722_CRC_POS                 6
> +#define LT8722_ACK                     0xA5
> +#define LT8722_ACK_POS                 7
> +
> +#define LT8722_DAC_VREF                        2500000
> +#define LT8722_DAC_BITS                        25
> +#define LT8722_ILIM_STEP               13280
> +#define LT8722_RAMP_STEPS              5
> +
> +#define LT8722_MIN_DAC_CODE            0xFF000000
> +#define LT8722_MAX_DAC_CODE            0x00FFFFFF
> +#define LT8722_ILIMN_MIN_IOUT          -6786000
> +#define LT8722_ILIMN_MAX_IOUT          -664640
> +#define LT8722_ILIMP_MIN_IOUT          637440
> +#define LT8722_ILIMP_MAX_IOUT          6800000
> +#define LT8722_MIN_VOUT                        -20000000
> +#define LT8722_MAX_VOUT                        20000000
> +#define LT8722_MIN_IOUT                        -6786000
> +#define LT8722_MAX_IOUT                        6800000
> +
> +DECLARE_CRC8_TABLE(lt8722_crc8_table);
> +
> +struct lt8722_chip_info {
> +       struct spi_device *spi;
> +       struct regulator_dev *rdev;
> +       struct gpio_desc *en_gpio;
> +       struct gpio_desc *swen_gpio;
> +       int uv_clamp_uV;
> +       int ov_clamp_uV;
> +       int ilimn_uA;
> +       int ilimp_uA;
> +       int switch_freq_hz;
> +       const char *switch_freq_adjust;
> +       const char *duty_cycle_range;
> +       int vcc_vreg_mV;
> +       int peak_inductor_current_mA;
> +       int power_limit_mW;
> +};
> +
> +static const unsigned int lt8722_uv_clamp[] = {
> +       -20000000,
> +       -18750000,
> +       -17500000,
> +       -16250000,
> +       -15000000,
> +       -13750000,
> +       -12500000,
> +       -11250000,
> +       -10000000,
> +       -8750000,
> +       -7500000,
> +       -6250000,
> +       -5000000,
> +       -3750000,
> +       -2500000,
> +       -1250000,
> +};
> +
> +static const unsigned int lt8722_ov_clamp[] = {
> +       1250000,
> +       2500000,
> +       3750000,
> +       5000000,
> +       6250000,
> +       7500000,
> +       8750000,
> +       10000000,
> +       11250000,
> +       12500000,
> +       13750000,
> +       15000000,
> +       16250000,
> +       17500000,
> +       18750000,
> +       20000000,
> +};
> +
> +static const unsigned int lt8722_switch_freq[] = {
> +       500000,
> +       1000000,
> +       1500000,
> +       2000000,
> +       2500000,
> +       3000000,
> +};
> +
> +static const char * const lt8722_switch_freq_adjust[] = {
> +       "0%",
> +       "15%",
> +       "-15%",
> +};
> +
> +static const char * const lt8722_duty_cycle_range[] = {
> +       "20%-80%",
> +       "15%-85%",
> +       "10%-90%",
> +};
> +
> +static const unsigned int lt8722_vcc_vreg[] = {
> +       3100,
> +       3400,
> +};
> +
> +static const unsigned int lt8722_peak_inductor_current[] = {
> +       252,
> +       594,
> +       936,
> +       1278,
> +       1620,
> +       1962,
> +       2304,
> +       2646,
> +};
> +
> +static const unsigned int lt8722_power_limit[] = {
> +       2000,
> +       0,
> +       3000,
> +       3500,
> +};
> +
> +static s32 _lt8722_dac_to_uV(u32 dac_val)
> +{
> +       s64 tmp;
> +
> +       tmp = (s64)dac_val * LT8722_DAC_VREF;
> +       tmp = 16 * div_s64(tmp, BIT(LT8722_DAC_BITS));
> +
> +       return tmp;
> +}
> +
> +static s32 _lt8722_uV_to_dac(s32 uV)
> +{
> +       s64 tmp;
> +
> +       tmp = (s64)uV * BIT(LT8722_DAC_BITS);
> +       tmp = div_s64(tmp, LT8722_DAC_VREF * 16);
> +
> +       return tmp;
> +}
> +
> +static int lt8722_reg_read(struct spi_device *spi, u8 reg, u32 *val)
> +{
> +       int ret;
> +       struct spi_transfer t;
> +       u8 rx_buf[LT8722_RW_CMD_SIZE] = {0};
> +       u8 tx_buf[LT8722_RW_CMD_SIZE] = {0};
> +
> +       tx_buf[0] = LT8722_READ_CMD;
> +       tx_buf[1] = reg << 1;
> +       tx_buf[2] = crc8(lt8722_crc8_table, tx_buf, 2, LT8722_CRC_INIT);
> +
> +       t.tx_buf = tx_buf;
> +       t.rx_buf = rx_buf;
> +       t.len = LT8722_RW_CMD_SIZE;
> +
> +       ret = spi_sync_transfer(spi, &t, 1);
> +       if (ret < 0)
> +               return ret;
> +
> +       if (rx_buf[LT8722_CRC_POS] != crc8(lt8722_crc8_table, rx_buf,
> +                               LT8722_CRC_POS, LT8722_CRC_INIT))
> +               return -EIO;
> +
> +       if (rx_buf[LT8722_ACK_POS] != LT8722_ACK)
> +               return -EIO;
> +
> +       *val = get_unaligned_be32(&rx_buf[LT8722_DATA_POS]);
> +
> +       return 0;
> +}
> +
> +static int lt8722_reg_write(struct spi_device *spi, u8 reg, u32 val)
> +{
> +       int ret;
> +       struct spi_transfer t;
> +       u8 rx_buf[LT8722_RW_CMD_SIZE] = {0};
> +       u8 tx_buf[LT8722_RW_CMD_SIZE] = {0};
> +
> +       tx_buf[0] = LT8722_WRITE_CMD;
> +       tx_buf[1] = reg << 1;
> +
> +       put_unaligned_be32(val, &tx_buf[LT8722_DATA_POS]);
> +
> +       tx_buf[LT8722_CRC_POS] = crc8(lt8722_crc8_table, tx_buf, LT8722_CRC_POS,
> +                                     LT8722_CRC_INIT);
> +
> +       t.tx_buf = tx_buf;
> +       t.rx_buf = rx_buf;
> +       t.len = LT8722_RW_CMD_SIZE;
> +
> +       ret = spi_sync_transfer(spi, &t, 1);
> +       if (ret < 0)
> +               return ret;
> +
> +       if (rx_buf[LT8722_ACK_POS] != LT8722_ACK)
> +               return -EIO;
> +
> +       return 0;
> +}
> +
> +static int lt8722_reg_write_mask(struct spi_device *spi, u8 reg, u32 mask,
> +                                u32 val)
> +{
> +       int ret;
> +       u32 reg_val;
> +
> +       ret = lt8722_reg_read(spi, reg, &reg_val);
> +       if (ret < 0)
> +               return ret;
> +
> +       reg_val = (reg_val & ~mask) | (val & mask);
> +
> +       return lt8722_reg_write(spi, reg, reg_val);
> +}
> +
> +static int lt8722_set_voltage(struct regulator_dev *rdev, int min_uV,
> +                             int max_uV, unsigned int *selector)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +
> +       if (min_uV < LT8722_MIN_VOUT || max_uV > LT8722_MAX_VOUT)
> +               return -EINVAL;
> +
> +       *selector = _lt8722_uV_to_dac(min_uV);
> +
> +       return lt8722_reg_write(chip->spi, LT8722_SPIS_DAC, *selector);
> +}
> +
> +static int lt8722_get_voltage(struct regulator_dev *rdev)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +       int ret, dac_val;
> +
> +       ret = lt8722_reg_read(chip->spi, LT8722_SPIS_DAC, &dac_val);
> +       if (ret < 0)
> +               return ret;
> +
> +       dac_val = FIELD_GET(LT8722_SPIS_DAC_MASK, dac_val);
> +
> +       return _lt8722_dac_to_uV(dac_val);
> +}
> +
> +static int lt8722_get_prop_index(const u32 *table, size_t table_size, u32 value)
> +{
> +       int i;
> +
> +       for (i = 0; i < table_size; i++)
> +               if (table[i] == value)
> +                       break;

Nit pick: instead of break you can return the index
instead of checking again below if it is equal to
table_size, saving one check. Once you do that
then you can get rid of below if and simply return
EINVAL.

> +
> +       if (i == table_size)
> +               return -EINVAL;
> +
> +       return i;
> +}
> +
> +static int lt8722_parse_fw(struct lt8722_chip_info *chip,
> +                          struct regulator_init_data *init_data)
> +{
> +       int ret;
> +
> +       /* Override the min_uV constraint with the minimum output voltage */
> +       init_data->constraints.min_uV = LT8722_MIN_VOUT;
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,uv-clamp-microvolt",
> +                                      &chip->uv_clamp_uV);
> +       if (!ret) {
> +               ret = lt8722_get_prop_index(lt8722_uv_clamp,
> +                               ARRAY_SIZE(lt8722_uv_clamp), chip->uv_clamp_uV);
> +               if (ret < 0)
> +                       return ret;
> +
> +               ret = lt8722_reg_write(chip->spi, LT8722_SPIS_UV_CLAMP, ret);
> +               if (ret < 0)
> +                       return ret;
> +
> +               /* Override the min_uV constraint with the UV clamp value */
> +               init_data->constraints.min_uV = chip->uv_clamp_uV;
> +       }
> +
> +       /* Override the max_uV constraint with the maximum output voltage */
> +       init_data->constraints.max_uV = LT8722_MAX_VOUT;
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,ov-clamp-microvolt",
> +                                      &chip->ov_clamp_uV);
> +       if (!ret) {
> +               ret = lt8722_get_prop_index(lt8722_ov_clamp,
> +                               ARRAY_SIZE(lt8722_ov_clamp), chip->ov_clamp_uV);
> +               if (ret < 0)
> +                       return ret;
> +
> +               ret = lt8722_reg_write(chip->spi, LT8722_SPIS_OV_CLAMP, ret);
> +               if (ret < 0)
> +                       return ret;
> +
> +               /* Override the max_uV constraint with the OV clamp value */
> +               init_data->constraints.max_uV = chip->ov_clamp_uV;
> +       }
> +
> +       /* Override the min_uA constraint with the minimum output current */
> +       init_data->constraints.min_uA = LT8722_MIN_IOUT;
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,ilimn-microamp",
> +                                      &chip->ilimn_uA);
> +       if (!ret) {
> +               if (chip->ilimn_uA < LT8722_ILIMN_MIN_IOUT ||
> +                   chip->ilimn_uA > LT8722_ILIMN_MAX_IOUT)
> +                       return -EINVAL;
> +
> +               ret = div_s64(chip->ilimn_uA, -LT8722_ILIM_STEP);
> +
> +               ret = lt8722_reg_write(chip->spi, LT8722_SPIS_DAC_ILIMN, ret);
> +               if (ret < 0)
> +                       return ret;
> +
> +               /* Override the min_uA constraint with the ILIMN value */
> +               init_data->constraints.min_uA = chip->ilimn_uA;
> +       }
> +
> +       /* Override the max_uA constraint with the maximum output current */
> +       init_data->constraints.max_uA = LT8722_MAX_IOUT;
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,ilimp-microamp",
> +                                      &chip->ilimp_uA);
> +       if (!ret) {
> +               if (chip->ilimp_uA < LT8722_ILIMP_MIN_IOUT ||
> +                   chip->ilimp_uA > LT8722_ILIMP_MAX_IOUT)
> +                       return -EINVAL;
> +
> +               ret = div_s64(LT8722_MAX_IOUT - chip->ilimp_uA, LT8722_ILIM_STEP);
> +
> +               ret = lt8722_reg_write(chip->spi, LT8722_SPIS_DAC_ILIMP, ret);
> +               if (ret < 0)
> +                       return ret;
> +
> +               /* Override the max_uA constraint with the ILIMP value */
> +               init_data->constraints.max_uA = chip->ilimp_uA;
> +       }
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,switch-frequency-hz",
> +                                      &chip->switch_freq_hz);
> +       if (!ret) {
> +               ret = lt8722_get_prop_index(lt8722_switch_freq,
> +                                           ARRAY_SIZE(lt8722_switch_freq),
> +                                           chip->switch_freq_hz);
> +               if (ret < 0)
> +                       return ret;
> +
> +               ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                               LT8722_SW_FRQ_SET_MASK,
> +                               FIELD_PREP(LT8722_SW_FRQ_SET_MASK, ret));
> +               if (ret < 0)
> +                       return ret;
> +       }
> +
> +       ret = device_property_match_property_string(&chip->spi->dev,
> +                       "adi,switch-frequency-adjust",
> +                       lt8722_switch_freq_adjust,
> +                       ARRAY_SIZE(lt8722_switch_freq_adjust));
> +       if (ret >= 0) {
> +               ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                               LT8722_SW_FRQ_ADJ_MASK,
> +                               FIELD_PREP(LT8722_SW_FRQ_ADJ_MASK, ret));
> +               if (ret < 0)
> +                       return ret;
> +       }
> +
> +       ret = device_property_match_property_string(&chip->spi->dev,
> +                       "adi,duty-cycle-range", lt8722_duty_cycle_range,
> +                       ARRAY_SIZE(lt8722_duty_cycle_range));
> +       if (ret >= 0) {
> +               ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                               LT8722_SYS_DC_MASK,
> +                               FIELD_PREP(LT8722_SYS_DC_MASK, ret));
> +               if (ret < 0)
> +                       return ret;
> +       }
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,vcc-vreg-millivolt",
> +                                      &chip->vcc_vreg_mV);
> +       if (!ret) {
> +               ret = lt8722_get_prop_index(lt8722_vcc_vreg,
> +                               ARRAY_SIZE(lt8722_vcc_vreg), chip->vcc_vreg_mV);
> +               if (ret < 0)
> +                       return ret;
> +
> +               ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                               LT8722_VCC_VREG_MASK,
> +                               FIELD_PREP(LT8722_VCC_VREG_MASK, ret));
> +               if (ret < 0)
> +                       return ret;
> +       }
> +
> +       ret = device_property_read_u32(&chip->spi->dev,
> +                                      "adi,peak-inductor-current-milliamp",
> +                                      &chip->peak_inductor_current_mA);
> +       if (!ret) {
> +               ret = lt8722_get_prop_index(lt8722_peak_inductor_current,
> +                               ARRAY_SIZE(lt8722_peak_inductor_current),
> +                               chip->peak_inductor_current_mA);
> +               if (ret < 0)
> +                       return ret;
> +
> +               ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                               LT8722_SW_VC_IN_MASK,
> +                               FIELD_PREP(LT8722_SW_VC_IN_MASK, ret));
> +               if (ret < 0)
> +                       return ret;
> +       }
> +
> +       ret = device_property_read_u32(&chip->spi->dev, "adi,power-limit-milliwatt",
> +                                      &chip->power_limit_mW);
> +       if (!ret) {
> +               ret = lt8722_get_prop_index(lt8722_power_limit,
> +                                           ARRAY_SIZE(lt8722_power_limit),
> +                                           chip->power_limit_mW);
> +               if (ret < 0)
> +                       return ret;
> +
> +               ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                               LT8722_PWR_LIM_MASK,
> +                               FIELD_PREP(LT8722_PWR_LIM_MASK, ret));
> +               if (ret < 0)
> +                       return ret;
> +       }
> +
> +       return 0;
> +}
> +
> +static int lt8722_enable(struct regulator_dev *rdev)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +
> +       gpiod_set_value_cansleep(chip->en_gpio, 1);
> +
> +       return lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                       LT8722_EN_REQ_MASK,
> +                       FIELD_PREP(LT8722_EN_REQ_MASK, 0x1));
> +}
> +
> +static int lt8722_disable(struct regulator_dev *rdev)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +
> +       gpiod_set_value_cansleep(chip->en_gpio, 0);
> +
> +       return lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                       LT8722_EN_REQ_MASK,
> +                       FIELD_PREP(LT8722_EN_REQ_MASK, 0x0));
> +}
> +
> +static int lt8722_is_enabled(struct regulator_dev *rdev)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +       int ret;
> +       u32 reg_val;
> +       bool en_req, en_pin;
> +
> +       ret = lt8722_reg_read(chip->spi, LT8722_SPIS_COMMAND, &reg_val);
> +       if (ret < 0)
> +               return ret;
> +
> +       en_req = FIELD_GET(LT8722_EN_REQ_MASK, reg_val);
> +       en_pin = gpiod_get_value(chip->en_gpio);
> +
> +       return en_req && en_pin;
> +}
> +
> +static int lt8722_get_error_flags(struct regulator_dev *rdev,
> +                                 unsigned int *flags)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +       int ret;
> +       u32 reg_val;
> +
> +       ret = lt8722_reg_read(chip->spi, LT8722_SPIS_STATUS, &reg_val);
> +       if (ret < 0)
> +               return ret;
> +
> +       *flags = 0;
> +
> +       if (FIELD_GET(LT8722_OC_MASK, reg_val))
> +               *flags |= REGULATOR_ERROR_OVER_CURRENT;
> +
> +       if (FIELD_GET(LT8722_TSD_MASK, reg_val))
> +               *flags |= REGULATOR_ERROR_OVER_TEMP;
> +
> +       return 0;
> +}
> +
> +static int lt8722_set_soft_start(struct regulator_dev *rdev)
> +{
> +       struct lt8722_chip_info *chip = rdev_get_drvdata(rdev);
> +       s32 dac_val;
> +       int ret, i;
> +
> +       /* Clear faults before enabled VCC LDO and other device circuitry */
> +       ret = lt8722_reg_write(chip->spi, LT8722_SPIS_STATUS, 0x0);
> +       if (ret < 0)
> +               return ret;
> +
> +       ret = lt8722_enable(rdev);
> +       if (ret < 0)
> +               return ret;
> +
> +       /* Configure output voltage control DAC to 0xFF000000 */
> +       ret = lt8722_reg_write(chip->spi, LT8722_SPIS_DAC, LT8722_MIN_DAC_CODE);
> +       if (ret < 0)
> +               return ret;
> +
> +       /* Write all SPIS_STATUS register bits to 0 */
> +       ret = lt8722_reg_write(chip->spi, LT8722_SPIS_STATUS, 0x0);
> +       if (ret < 0)
> +               return ret;
> +
> +       fsleep(1000);
> +
> +       /* Ramp the output voltage control DAC from 0xFF000000 to 0x00000000 */
> +       for (i = 0; i < LT8722_RAMP_STEPS; i++) {
> +               dac_val = LT8722_MIN_DAC_CODE + 0x400000 * i;
> +
> +               ret = lt8722_reg_write(chip->spi, LT8722_SPIS_DAC, dac_val);
> +               if (ret < 0)
> +                       return ret;
> +
> +               fsleep(1000);
> +       }
> +
> +       /* Enable the PWM switching behavior */
> +       gpiod_set_value_cansleep(chip->swen_gpio, 1);
> +
> +       ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                       LT8722_SWEN_REQ_MASK,
> +                       FIELD_PREP(LT8722_SWEN_REQ_MASK, 0x1));
> +       if (ret < 0)
> +               return ret;
> +
> +       fsleep(160);
> +
> +       return 0;
> +}
> +
> +static const struct regulator_ops lt8722_regulator_ops = {
> +       .set_voltage = lt8722_set_voltage,
> +       .get_voltage = lt8722_get_voltage,
> +       .enable = lt8722_enable,
> +       .disable = lt8722_disable,
> +       .is_enabled = lt8722_is_enabled,
> +       .set_soft_start = lt8722_set_soft_start,
> +       .get_error_flags = lt8722_get_error_flags,
> +};
> +
> +static struct regulator_desc lt8722_regulator_desc = {
> +       .name = "lt8722",
> +       .ops = &lt8722_regulator_ops,
> +       .type = REGULATOR_VOLTAGE,
> +       .owner = THIS_MODULE,
> +};
> +
> +static int lt8722_probe(struct spi_device *spi)
> +{
> +       struct regulator_init_data *init_data;
> +       struct regulator_config config = { };
> +       struct lt8722_chip_info *chip;
> +       int ret;
> +
> +       init_data = of_get_regulator_init_data(&spi->dev, spi->dev.of_node,
> +                                              &lt8722_regulator_desc);
> +       if (!init_data)
> +               return -EINVAL;
> +
> +       chip = devm_kzalloc(&spi->dev, sizeof(*chip), GFP_KERNEL);
> +       if (!chip)
> +               return -ENOMEM;
> +
> +       crc8_populate_msb(lt8722_crc8_table, LT8722_CRC8_POLY);
> +
> +       chip->spi = spi;
> +
> +       chip->en_gpio = devm_gpiod_get(&spi->dev, "enable", GPIOD_OUT_LOW);
> +       if (IS_ERR(chip->en_gpio))
> +               return PTR_ERR(chip->en_gpio);
> +
> +       chip->swen_gpio = devm_gpiod_get(&spi->dev, "switch-enable",
> +                       GPIOD_OUT_LOW);
> +       if (IS_ERR(chip->swen_gpio))
> +               return PTR_ERR(chip->swen_gpio);
> +
> +       ret = lt8722_reg_write_mask(chip->spi, LT8722_SPIS_COMMAND,
> +                       LT8722_SPI_RST_MASK,
> +                       FIELD_PREP(LT8722_SPI_RST_MASK, 0x1));
> +       if (ret < 0)
> +               return ret;
> +
> +       ret = lt8722_parse_fw(chip, init_data);
> +       if (ret < 0)
> +               return ret;
> +
> +       config.dev = &spi->dev;
> +       config.init_data = init_data;
> +       config.driver_data = chip;
> +
> +       chip->rdev = devm_regulator_register(&spi->dev, &lt8722_regulator_desc,
> +                                            &config);
> +       if (IS_ERR(chip->rdev))
> +               return PTR_ERR(chip->rdev);
> +
> +       return 0;
> +}
> +
> +static const struct of_device_id lt8722_of_match[] = {
> +       { .compatible = "adi,lt8722", },
> +       { }
> +};
> +MODULE_DEVICE_TABLE(of, lt8722_of_match);
> +
> +static const struct spi_device_id lt8722_id[] = {
> +       { "lt8722" },
> +       { }
> +};
> +MODULE_DEVICE_TABLE(spi, lt8722_id);
> +
> +struct spi_driver lt8722_driver = {
> +       .driver = {
> +               .name = "lt8722",
> +               .of_match_table = lt8722_of_match,
> +       },
> +       .probe = lt8722_probe,
> +       .id_table = lt8722_id,
> +};
> +module_spi_driver(lt8722_driver);
> +
> +MODULE_AUTHOR("Ramon Cristopher Calam <ramoncristopher.calam@xxxxxxxxxx>");
> +MODULE_DESCRIPTION("LT8722 ultracompact full bridge driver with SPI driver");
> +MODULE_LICENSE("GPL");
> --
> 2.47.0
>
>





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux