Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C538AC7EE2F for ; Fri, 3 Mar 2023 15:04:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230313AbjCCPEM (ORCPT ); Fri, 3 Mar 2023 10:04:12 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48252 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229586AbjCCPEJ (ORCPT ); Fri, 3 Mar 2023 10:04:09 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 17FF313DC5; Fri, 3 Mar 2023 07:04:05 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 7BDA8B8184D; Fri, 3 Mar 2023 15:04:03 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B5C5BC433EF; Fri, 3 Mar 2023 15:03:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1677855842; bh=jscmed4CEa0gSPVjLqvV8yD5UfNiux39u92dOUQxxhA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mDV4bFztSkMf+NRouWaRkZkvlrNZUQTudsBaqC3Ygzt9zRrXFMBndqcp3wGcXFxLl pUhpZeRv9nkITn4+Mqalr4h6kaRZ0QPgFOkysg1E2bz9MiyoKUxAWEbfMExoABozmX aiYhPmtLGqFVsluGhHSdlZ7ziYcWi9CLSrwAmwwUcy3uG+o/VN2mEaM8ke098ZUwCl 0ROAofMacPzCGl3B9cmL3Cx6ubwwxLA5FWKwZv+yFHalA1woZpWYDTayDWqS/f8/Jo AMlpxdmvMsSpGxJ32m5/aKUm2Yae4LMrs4OxMqriHFZzVTilDZifl+7iBehOfydC5c wEFuzgCHMh03g== Date: Fri, 3 Mar 2023 15:03:55 +0000 From: Lee Jones To: Julien Panis Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, corbet@lwn.net, hdegoede@redhat.com, eric.auger@redhat.com, jgg@ziepe.ca, razor@blackwall.org, suma.hegde@amd.com, stephen@networkplumber.org, arnd@arndb.de, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, eblanc@baylibre.com, jneanne@baylibre.com Subject: Re: [PATCH v1 2/4] mfd: tps6594: Add driver for TI TPS6594 PMIC Message-ID: <20230303150355.GQ2420672@google.com> References: <20230216114410.183489-1-jpanis@baylibre.com> <20230216114410.183489-3-jpanis@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230216114410.183489-3-jpanis@baylibre.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 16 Feb 2023, Julien Panis wrote: > This patch adds support for TPS6594 PMIC MFD core. It provides > communication through the I2C and SPI interfaces, and supports > protocols with embedded CRC data fields for safety applications. > > Signed-off-by: Julien Panis > --- > drivers/mfd/Kconfig | 32 ++ > drivers/mfd/Makefile | 3 + > drivers/mfd/tps6594-core.c | 413 ++++++++++++++ > drivers/mfd/tps6594-i2c.c | 240 +++++++++ > drivers/mfd/tps6594-spi.c | 125 +++++ > include/linux/mfd/tps6594.h | 1018 +++++++++++++++++++++++++++++++++++ > 6 files changed, 1831 insertions(+) > create mode 100644 drivers/mfd/tps6594-core.c > create mode 100644 drivers/mfd/tps6594-i2c.c > create mode 100644 drivers/mfd/tps6594-spi.c > create mode 100644 include/linux/mfd/tps6594.h > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 30db49f31866..3d2f5a10f54b 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -1690,6 +1690,38 @@ config MFD_TPS65912_SPI > If you say yes here you get support for the TPS65912 series of > PM chips with SPI interface. > > +config MFD_TPS6594 > + tristate > + select MFD_CORE > + select REGMAP > + select REGMAP_IRQ > + > +config MFD_TPS6594_I2C > + tristate "TI TPS6594 Power Management chip with I2C" > + select MFD_TPS6594 > + select REGMAP_I2C > + select CRC8 > + depends on I2C > + help > + If you say yes here you get support for the TPS6594 series of > + PM chips with I2C interface. > + > + This driver can also be built as a module. If so, the module > + will be called tps6594-i2c. > + > +config MFD_TPS6594_SPI > + tristate "TI TPS6594 Power Management chip with SPI" > + select MFD_TPS6594 > + select REGMAP_SPI > + select CRC8 > + depends on SPI_MASTER > + help > + If you say yes here you get support for the TPS6594 series of > + PM chips with SPI interface. > + > + This driver can also be built as a module. If so, the module > + will be called tps6594-spi. > + > config TWL4030_CORE > bool "TI TWL4030/TWL5030/TWL6030/TPS659x0 Support" > depends on I2C=y > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 457471478a93..d2cba7326b07 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -102,6 +102,9 @@ obj-$(CONFIG_MFD_TPS65910) += tps65910.o > obj-$(CONFIG_MFD_TPS65912) += tps65912-core.o > obj-$(CONFIG_MFD_TPS65912_I2C) += tps65912-i2c.o > obj-$(CONFIG_MFD_TPS65912_SPI) += tps65912-spi.o > +obj-$(CONFIG_MFD_TPS6594) += tps6594-core.o > +obj-$(CONFIG_MFD_TPS6594_I2C) += tps6594-i2c.o > +obj-$(CONFIG_MFD_TPS6594_SPI) += tps6594-spi.o > obj-$(CONFIG_MENELAUS) += menelaus.o > > obj-$(CONFIG_TWL4030_CORE) += twl-core.o twl4030-irq.o twl6030-irq.o > diff --git a/drivers/mfd/tps6594-core.c b/drivers/mfd/tps6594-core.c > new file mode 100644 > index 000000000000..a81a35324ab8 > --- /dev/null > +++ b/drivers/mfd/tps6594-core.c > @@ -0,0 +1,413 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > +/* > + * Core functions for TI TPS6594/TPS6593/LP8764X PMICs > + * > + * Copyright (C) 2022 BayLibre Incorporated - https://www.baylibre.com/ > + */ > + > +#include > +#include > +#include > +#include > + > +#include > +#include > + > +#define TPS6594_CRC_SYNC_TIMEOUT_MS 150 > + > +/* Completion to synchronize CRC feature enabling on all PMICs */ > +static DECLARE_COMPLETION(tps6594_crc_comp); > + > +static const struct resource tps6594_regulator_resources[] = { > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK1_OV, TPS6594_IRQ_NAME_BUCK1_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK1_UV, TPS6594_IRQ_NAME_BUCK1_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK1_SC, TPS6594_IRQ_NAME_BUCK1_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK1_ILIM, TPS6594_IRQ_NAME_BUCK1_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK2_OV, TPS6594_IRQ_NAME_BUCK2_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK2_UV, TPS6594_IRQ_NAME_BUCK2_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK2_SC, TPS6594_IRQ_NAME_BUCK2_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK2_ILIM, TPS6594_IRQ_NAME_BUCK2_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK3_OV, TPS6594_IRQ_NAME_BUCK3_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK3_UV, TPS6594_IRQ_NAME_BUCK3_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK3_SC, TPS6594_IRQ_NAME_BUCK3_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK3_ILIM, TPS6594_IRQ_NAME_BUCK3_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK4_OV, TPS6594_IRQ_NAME_BUCK4_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK4_UV, TPS6594_IRQ_NAME_BUCK4_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK4_SC, TPS6594_IRQ_NAME_BUCK4_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK4_ILIM, TPS6594_IRQ_NAME_BUCK4_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK5_OV, TPS6594_IRQ_NAME_BUCK5_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK5_UV, TPS6594_IRQ_NAME_BUCK5_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK5_SC, TPS6594_IRQ_NAME_BUCK5_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BUCK5_ILIM, TPS6594_IRQ_NAME_BUCK5_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO1_OV, TPS6594_IRQ_NAME_LDO1_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO1_UV, TPS6594_IRQ_NAME_LDO1_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO1_SC, TPS6594_IRQ_NAME_LDO1_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO1_ILIM, TPS6594_IRQ_NAME_LDO1_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO2_OV, TPS6594_IRQ_NAME_LDO2_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO2_UV, TPS6594_IRQ_NAME_LDO2_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO2_SC, TPS6594_IRQ_NAME_LDO2_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO2_ILIM, TPS6594_IRQ_NAME_LDO2_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO3_OV, TPS6594_IRQ_NAME_LDO3_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO3_UV, TPS6594_IRQ_NAME_LDO3_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO3_SC, TPS6594_IRQ_NAME_LDO3_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO3_ILIM, TPS6594_IRQ_NAME_LDO3_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO4_OV, TPS6594_IRQ_NAME_LDO4_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO4_UV, TPS6594_IRQ_NAME_LDO4_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO4_SC, TPS6594_IRQ_NAME_LDO4_SC), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_LDO4_ILIM, TPS6594_IRQ_NAME_LDO4_ILIM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VCCA_OV, TPS6594_IRQ_NAME_VCCA_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VCCA_UV, TPS6594_IRQ_NAME_VCCA_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VMON1_OV, TPS6594_IRQ_NAME_VMON1_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VMON1_UV, TPS6594_IRQ_NAME_VMON1_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VMON1_RV, TPS6594_IRQ_NAME_VMON1_RV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VMON2_OV, TPS6594_IRQ_NAME_VMON2_OV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VMON2_UV, TPS6594_IRQ_NAME_VMON2_UV), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VMON2_RV, TPS6594_IRQ_NAME_VMON2_RV), > +}; > + > +static const struct resource tps6594_pinctrl_resources[] = { > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO9, TPS6594_IRQ_NAME_GPIO9), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO10, TPS6594_IRQ_NAME_GPIO10), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO11, TPS6594_IRQ_NAME_GPIO11), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO1, TPS6594_IRQ_NAME_GPIO1), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO2, TPS6594_IRQ_NAME_GPIO2), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO3, TPS6594_IRQ_NAME_GPIO3), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO4, TPS6594_IRQ_NAME_GPIO4), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO5, TPS6594_IRQ_NAME_GPIO5), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO6, TPS6594_IRQ_NAME_GPIO6), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO7, TPS6594_IRQ_NAME_GPIO7), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_GPIO8, TPS6594_IRQ_NAME_GPIO8), > +}; > + > +static const struct resource tps6594_pfsm_resources[] = { > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_NPWRON_START, TPS6594_IRQ_NAME_NPWRON_START), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_ENABLE, TPS6594_IRQ_NAME_ENABLE), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_FSD, TPS6594_IRQ_NAME_FSD), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_SOFT_REBOOT, TPS6594_IRQ_NAME_SOFT_REBOOT), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BIST_PASS, TPS6594_IRQ_NAME_BIST_PASS), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_EXT_CLK, TPS6594_IRQ_NAME_EXT_CLK), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_TWARN, TPS6594_IRQ_NAME_TWARN), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_TSD_ORD, TPS6594_IRQ_NAME_TSD_ORD), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_BIST_FAIL, TPS6594_IRQ_NAME_BIST_FAIL), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_REG_CRC_ERR, TPS6594_IRQ_NAME_REG_CRC_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_RECOV_CNT, TPS6594_IRQ_NAME_RECOV_CNT), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_SPMI_ERR, TPS6594_IRQ_NAME_SPMI_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_NPWRON_LONG, TPS6594_IRQ_NAME_NPWRON_LONG), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_NINT_READBACK, TPS6594_IRQ_NAME_NINT_READBACK), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_NRSTOUT_READBACK, TPS6594_IRQ_NAME_NRSTOUT_READBACK), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_TSD_IMM, TPS6594_IRQ_NAME_TSD_IMM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_VCCA_OVP, TPS6594_IRQ_NAME_VCCA_OVP), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_PFSM_ERR, TPS6594_IRQ_NAME_PFSM_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_IMM_SHUTDOWN, TPS6594_IRQ_NAME_IMM_SHUTDOWN), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_ORD_SHUTDOWN, TPS6594_IRQ_NAME_ORD_SHUTDOWN), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_MCU_PWR_ERR, TPS6594_IRQ_NAME_MCU_PWR_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_SOC_PWR_ERR, TPS6594_IRQ_NAME_SOC_PWR_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_COMM_FRM_ERR, TPS6594_IRQ_NAME_COMM_FRM_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_COMM_CRC_ERR, TPS6594_IRQ_NAME_COMM_CRC_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_COMM_ADR_ERR, TPS6594_IRQ_NAME_COMM_ADR_ERR), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_EN_DRV_READBACK, TPS6594_IRQ_NAME_EN_DRV_READBACK), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_NRSTOUT_SOC_READBACK, > + TPS6594_IRQ_NAME_NRSTOUT_SOC_READBACK), > +}; > + > +static const struct resource tps6594_esm_resources[] = { > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_ESM_SOC_PIN, TPS6594_IRQ_NAME_ESM_SOC_PIN), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_ESM_SOC_FAIL, TPS6594_IRQ_NAME_ESM_SOC_FAIL), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_ESM_SOC_RST, TPS6594_IRQ_NAME_ESM_SOC_RST), > +}; > + > +static const struct resource tps6594_rtc_resources[] = { > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_TIMER, TPS6594_IRQ_NAME_TIMER), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_ALARM, TPS6594_IRQ_NAME_ALARM), > + DEFINE_RES_IRQ_NAMED(TPS6594_IRQ_POWER_UP, TPS6594_IRQ_NAME_POWERUP), > +}; > + > +static const struct mfd_cell tps6594_cells[] = { > + { > + .name = "tps6594-regulator", > + .resources = tps6594_regulator_resources, > + .num_resources = ARRAY_SIZE(tps6594_regulator_resources), > + }, > + { > + .name = "tps6594-pinctrl", > + .resources = tps6594_pinctrl_resources, > + .num_resources = ARRAY_SIZE(tps6594_pinctrl_resources), > + }, > + { > + .name = "tps6594-pfsm", > + .resources = tps6594_pfsm_resources, > + .num_resources = ARRAY_SIZE(tps6594_pfsm_resources), > + }, > + { > + .name = "tps6594-esm", > + .resources = tps6594_esm_resources, > + .num_resources = ARRAY_SIZE(tps6594_esm_resources), > + }, > + { > + .name = "tps6594-rtc", > + .resources = tps6594_rtc_resources, > + .num_resources = ARRAY_SIZE(tps6594_rtc_resources), > + }, > +}; Use MFD_CELL_RES() here. > +static const struct regmap_irq tps6594_irqs[] = { > + /* INT_BUCK1_2 register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK1_OV, 0, TPS6594_BIT_BUCKX_OV_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK1_UV, 0, TPS6594_BIT_BUCKX_UV_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK1_SC, 0, TPS6594_BIT_BUCKX_SC_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK1_ILIM, 0, TPS6594_BIT_BUCKX_ILIM_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK2_OV, 0, TPS6594_BIT_BUCKX_OV_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK2_UV, 0, TPS6594_BIT_BUCKX_UV_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK2_SC, 0, TPS6594_BIT_BUCKX_SC_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK2_ILIM, 0, TPS6594_BIT_BUCKX_ILIM_INT(1)), Perhaps some spaces between the blocks will make it easier on the eye. > + /* INT_BUCK3_4 register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK3_OV, 1, TPS6594_BIT_BUCKX_OV_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK3_UV, 1, TPS6594_BIT_BUCKX_UV_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK3_SC, 1, TPS6594_BIT_BUCKX_SC_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK3_ILIM, 1, TPS6594_BIT_BUCKX_ILIM_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK4_OV, 1, TPS6594_BIT_BUCKX_OV_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK4_UV, 1, TPS6594_BIT_BUCKX_UV_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK4_SC, 1, TPS6594_BIT_BUCKX_SC_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK4_ILIM, 1, TPS6594_BIT_BUCKX_ILIM_INT(3)), > + /* INT_BUCK5 register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK5_OV, 2, TPS6594_BIT_BUCKX_OV_INT(4)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK5_UV, 2, TPS6594_BIT_BUCKX_UV_INT(4)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK5_SC, 2, TPS6594_BIT_BUCKX_SC_INT(4)), > + REGMAP_IRQ_REG(TPS6594_IRQ_BUCK5_ILIM, 2, TPS6594_BIT_BUCKX_ILIM_INT(4)), > + /* INT_LDO1_2 register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO1_OV, 3, TPS6594_BIT_LDOX_OV_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO1_UV, 3, TPS6594_BIT_LDOX_UV_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO1_SC, 3, TPS6594_BIT_LDOX_SC_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO1_ILIM, 3, TPS6594_BIT_LDOX_ILIM_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO2_OV, 3, TPS6594_BIT_LDOX_OV_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO2_UV, 3, TPS6594_BIT_LDOX_UV_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO2_SC, 3, TPS6594_BIT_LDOX_SC_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO2_ILIM, 3, TPS6594_BIT_LDOX_ILIM_INT(1)), > + /* INT_LDO3_4 register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO3_OV, 4, TPS6594_BIT_LDOX_OV_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO3_UV, 4, TPS6594_BIT_LDOX_UV_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO3_SC, 4, TPS6594_BIT_LDOX_SC_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO3_ILIM, 4, TPS6594_BIT_LDOX_ILIM_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO4_OV, 4, TPS6594_BIT_LDOX_OV_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO4_UV, 4, TPS6594_BIT_LDOX_UV_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO4_SC, 4, TPS6594_BIT_LDOX_SC_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_LDO4_ILIM, 4, TPS6594_BIT_LDOX_ILIM_INT(3)), > + /* INT_VMON register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_VCCA_OV, 5, TPS6594_BIT_VCCA_OV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VCCA_UV, 5, TPS6594_BIT_VCCA_UV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VMON1_OV, 5, TPS6594_BIT_VMON1_OV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VMON1_UV, 5, TPS6594_BIT_VMON1_UV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VMON1_RV, 5, TPS6594_BIT_VMON1_RV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VMON2_OV, 5, TPS6594_BIT_VMON2_OV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VMON2_UV, 5, TPS6594_BIT_VMON2_UV_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VMON2_RV, 5, TPS6594_BIT_VMON2_RV_INT), > + /* INT_GPIO register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO9, 6, TPS6594_BIT_GPIO9_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO10, 6, TPS6594_BIT_GPIO10_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO11, 6, TPS6594_BIT_GPIO11_INT), > + /* INT_GPIO1_8 register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO1, 7, TPS6594_BIT_GPIOX_INT(0)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO2, 7, TPS6594_BIT_GPIOX_INT(1)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO3, 7, TPS6594_BIT_GPIOX_INT(2)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO4, 7, TPS6594_BIT_GPIOX_INT(3)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO5, 7, TPS6594_BIT_GPIOX_INT(4)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO6, 7, TPS6594_BIT_GPIOX_INT(5)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO7, 7, TPS6594_BIT_GPIOX_INT(6)), > + REGMAP_IRQ_REG(TPS6594_IRQ_GPIO8, 7, TPS6594_BIT_GPIOX_INT(7)), > + /* INT_STARTUP register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_NPWRON_START, 8, TPS6594_BIT_NPWRON_START_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_ENABLE, 8, TPS6594_BIT_ENABLE_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_FSD, 8, TPS6594_BIT_FSD_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_SOFT_REBOOT, 8, TPS6594_BIT_SOFT_REBOOT_INT), > + /* INT_MISC register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_BIST_PASS, 9, TPS6594_BIT_BIST_PASS_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_EXT_CLK, 9, TPS6594_BIT_EXT_CLK_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_TWARN, 9, TPS6594_BIT_TWARN_INT), > + /* INT_MODERATE_ERR register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_TSD_ORD, 10, TPS6594_BIT_TSD_ORD_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_BIST_FAIL, 10, TPS6594_BIT_BIST_FAIL_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_REG_CRC_ERR, 10, TPS6594_BIT_REG_CRC_ERR_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_RECOV_CNT, 10, TPS6594_BIT_RECOV_CNT_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_SPMI_ERR, 10, TPS6594_BIT_SPMI_ERR_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_NPWRON_LONG, 10, TPS6594_BIT_NPWRON_LONG_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_NINT_READBACK, 10, TPS6594_BIT_NINT_READBACK_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_NRSTOUT_READBACK, 10, TPS6594_BIT_NRSTOUT_READBACK_INT), > + /* INT_SEVERE_ERR register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_TSD_IMM, 11, TPS6594_BIT_TSD_IMM_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_VCCA_OVP, 11, TPS6594_BIT_VCCA_OVP_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_PFSM_ERR, 11, TPS6594_BIT_PFSM_ERR_INT), > + /* INT_FSM_ERR register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_IMM_SHUTDOWN, 12, TPS6594_BIT_IMM_SHUTDOWN_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_ORD_SHUTDOWN, 12, TPS6594_BIT_ORD_SHUTDOWN_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_MCU_PWR_ERR, 12, TPS6594_BIT_MCU_PWR_ERR_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_SOC_PWR_ERR, 12, TPS6594_BIT_SOC_PWR_ERR_INT), > + /* INT_COMM_ERR register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_COMM_FRM_ERR, 13, TPS6594_BIT_COMM_FRM_ERR_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_COMM_CRC_ERR, 13, TPS6594_BIT_COMM_CRC_ERR_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_COMM_ADR_ERR, 13, TPS6594_BIT_COMM_ADR_ERR_INT), > + /* INT_READBACK_ERR register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_EN_DRV_READBACK, 14, TPS6594_BIT_EN_DRV_READBACK_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_NRSTOUT_SOC_READBACK, 14, TPS6594_BIT_NRSTOUT_SOC_READBACK_INT), > + /* INT_ESM register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_ESM_SOC_PIN, 15, TPS6594_BIT_ESM_SOC_PIN_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_ESM_SOC_FAIL, 15, TPS6594_BIT_ESM_SOC_FAIL_INT), > + REGMAP_IRQ_REG(TPS6594_IRQ_ESM_SOC_RST, 15, TPS6594_BIT_ESM_SOC_RST_INT), > + /* RTC_STATUS register */ > + REGMAP_IRQ_REG(TPS6594_IRQ_TIMER, 16, TPS6594_BIT_TIMER), > + REGMAP_IRQ_REG(TPS6594_IRQ_ALARM, 16, TPS6594_BIT_ALARM), > + REGMAP_IRQ_REG(TPS6594_IRQ_POWER_UP, 16, TPS6594_BIT_POWER_UP), > +}; > + > +static const unsigned int tps6594_irq_reg[] = { > + TPS6594_REG_INT_BUCK1_2, > + TPS6594_REG_INT_BUCK3_4, > + TPS6594_REG_INT_BUCK5, > + TPS6594_REG_INT_LDO1_2, > + TPS6594_REG_INT_LDO3_4, > + TPS6594_REG_INT_VMON, > + TPS6594_REG_INT_GPIO, > + TPS6594_REG_INT_GPIO1_8, > + TPS6594_REG_INT_STARTUP, > + TPS6594_REG_INT_MISC, > + TPS6594_REG_INT_MODERATE_ERR, > + TPS6594_REG_INT_SEVERE_ERR, > + TPS6594_REG_INT_FSM_ERR, > + TPS6594_REG_INT_COMM_ERR, > + TPS6594_REG_INT_READBACK_ERR, > + TPS6594_REG_INT_ESM, > + TPS6594_REG_RTC_STATUS, > +}; > + > +static inline unsigned int tps6594_get_irq_reg(struct regmap_irq_chip_data *data, > + unsigned int base, int index) > +{ > + return tps6594_irq_reg[index]; > +}; > + > +static struct regmap_irq_chip tps6594_irq_chip = { > + .ack_base = TPS6594_REG_INT_BUCK1_2, > + .ack_invert = 1, > + .clear_ack = 1, > + .init_ack_masked = 1, > + .num_regs = ARRAY_SIZE(tps6594_irq_reg), > + .irqs = tps6594_irqs, > + .num_irqs = ARRAY_SIZE(tps6594_irqs), > + .get_irq_reg = tps6594_get_irq_reg, > +}; > + > +bool tps6594_is_volatile_reg(struct device *dev, unsigned int reg) > +{ > + return (reg >= TPS6594_REG_INT_TOP && reg <= TPS6594_REG_STAT_READBACK_ERR) || > + reg == TPS6594_REG_RTC_STATUS; > +} > +EXPORT_SYMBOL_GPL(tps6594_is_volatile_reg); > + > +static int tps6594_check_crc_mode(struct tps6594 *tps, bool primary_pmic) > +{ > + int ret; > + > + /* > + * Ensure that CRC is enabled. > + * Once CRC is enabled, it can't be disabled until next power cycle. > + */ > + tps->use_crc = true; > + ret = regmap_test_bits(tps->regmap, TPS6594_REG_SERIAL_IF_CONFIG, > + TPS6594_BIT_I2C1_SPI_CRC_EN); > + if (ret < 0) { > + tps->use_crc = false; > + } else if (ret == 0) { > + tps->use_crc = false; Will this value be used again after you return an error? > + ret = -EIO; > + } else { > + dev_info(tps->dev, "CRC feature enabled on %s PMIC", > + primary_pmic ? "primary" : "secondary"); > + ret = 0; I would consider reversing the logic of the if()s, default to 'false' then set 'true' in here before the print. > + } > + > + return ret; > +} > + > +static int tps6594_set_crc_feature(struct tps6594 *tps) > +{ > + int ret; > + > + /* Force PFSM I2C_2 trigger to enable CRC on primary PMIC */ > + ret = regmap_write_bits(tps->regmap, TPS6594_REG_FSM_I2C_TRIGGERS, > + TPS6594_BIT_TRIGGER_I2C(2), TPS6594_BIT_TRIGGER_I2C(2)); > + if (ret) > + return ret; > + > + /* Wait for PFSM to process trigger */ > + msleep(20); Is this the time specified in the datasheet? > + return tps6594_check_crc_mode(tps, true); > +} > + > +int tps6594_device_init(struct tps6594 *tps) > +{ > + struct device *dev = tps->dev; > + unsigned int prop; Since this only has a single use, better to rename it to something specific. > + unsigned long timeout = msecs_to_jiffies(TPS6594_CRC_SYNC_TIMEOUT_MS); > + int n_dev = ARRAY_SIZE(tps6594_cells); > + int ret; > + > + /* Keep PMIC in ACTIVE state */ > + ret = regmap_set_bits(tps->regmap, TPS6594_REG_FSM_NSLEEP_TRIGGERS, > + TPS6594_BIT_NSLEEP1B | TPS6594_BIT_NSLEEP2B); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to set PMIC state\n"); > + > + /* > + * CRC mode can be used with I2C or SPI protocols. > + * If this mode is specified for primary PMIC, it will also be applied to secondary PMICs > + * through SPMI serial interface. > + * In this multi-PMIC synchronization scheme, the primary PMIC is the controller device > + * on the SPMI bus, and the secondary PMICs are the target devices on the SPMI bus. > + */ > + prop = of_property_read_bool(dev->of_node, "ti,use-crc"); > + if (prop) { > + /* Enable CRC feature on primary PMIC */ > + ret = tps6594_set_crc_feature(tps); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to set crc feature\n"); "CRC" > + /* Notify secondary PMICs that CRC feature is enabled */ > + complete_all(&tps6594_crc_comp); > + } else { > + /* Wait for CRC feature enabling event, if any, from primary PMIC */ > + ret = wait_for_completion_interruptible_timeout(&tps6594_crc_comp, timeout); > + if (ret == 0) { > + dev_info(tps->dev, "CRC feature disabled"); > + } else if (ret > 0) { > + ret = tps6594_check_crc_mode(tps, false); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to set crc feature\n"); > + } else { > + return ret; > + } > + } > + > + tps6594_irq_chip.name = devm_kasprintf(dev, GFP_KERNEL, "%s-%ld-0x%02x", > + dev->driver->name, tps->chip_id, tps->reg); > + > + ret = devm_regmap_add_irq_chip(dev, tps->regmap, tps->irq, IRQF_SHARED | IRQF_ONESHOT, > + 0, &tps6594_irq_chip, &tps->irq_data); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to add regmap irq\n"); IRQ > + /* No RTC for LP8764X */ > + if (tps->chip_id == LP8764X) > + n_dev--; Super not keen on passing a larger structure with a reduced n_devs. It may work, but it's a hack. It also leaves the structure vulnerable since you now depend on it being ordered. Better to put the RTC in a separate structure and call mfd_add_device() again if LP8764X. > + ret = devm_mfd_add_devices(dev, PLATFORM_DEVID_AUTO, tps6594_cells, n_dev, > + NULL, 0, regmap_irq_get_domain(tps->irq_data)); > + if (ret) > + return dev_err_probe(dev, ret, "Failed to add child devices\n"); > + > + return 0; > +} > +EXPORT_SYMBOL_GPL(tps6594_device_init); > + > +MODULE_AUTHOR("Julien Panis "); > +MODULE_DESCRIPTION("TPS6594 MFD Driver"); Drop MFD from the description. > +MODULE_LICENSE("GPL"); > diff --git a/drivers/mfd/tps6594-i2c.c b/drivers/mfd/tps6594-i2c.c > new file mode 100644 > index 000000000000..fc159a32082a > --- /dev/null > +++ b/drivers/mfd/tps6594-i2c.c > @@ -0,0 +1,240 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > +/* > + * I2C access driver for TI TPS6594/TPS6593/LP8764X PMICs > + * > + * Copyright (C) 2022 BayLibre Incorporated - https://www.baylibre.com/ > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > + > +#include > + > +DECLARE_CRC8_TABLE(tps6594_i2c_crc_table); > + > +static int tps6594_i2c_transfer(struct i2c_adapter *adap, struct i2c_msg *msgs, int num) > +{ > + int ret = i2c_transfer(adap, msgs, num); > + > + if (ret == num) > + return 0; > + else if (ret < 0) > + return ret; > + else > + return -EIO; > +} > + > +static int tps6594_i2c_reg_read_with_crc(struct i2c_client *client, u8 page, u8 reg, u8 *val) > +{ > + struct i2c_msg msgs[2]; > + u8 buf_rx[] = {0, 0}; Prefer spaces after the '{' and before the '}' throughout (even below). > + /* I2C address = I2C base address + Page index */ > + const u8 addr = client->addr + page; > + /* > + * CRC is calculated from every bit included in the protocol > + * except the ACK bits from the target. Byte stream is: > + * - B0: (I2C_addr_7bits << 1) | WR_bit, with WR_bit = 0 > + * - B1: reg > + * - B2: (I2C_addr_7bits << 1) | RD_bit, with RD_bit = 1 > + * - B3: val > + * - B4: CRC from B0-B1-B2-B3 > + */ > + u8 crc_data[] = {addr << 1, reg, addr << 1 | 1, 0}; > + int ret; > + > + /* Write register */ > + msgs[0].addr = addr; > + msgs[0].flags = 0; > + msgs[0].len = 1; > + msgs[0].buf = ® > + > + /* Read data and CRC */ > + msgs[1].addr = msgs[0].addr; > + msgs[1].flags = I2C_M_RD; > + msgs[1].len = 2; > + msgs[1].buf = buf_rx; > + > + ret = tps6594_i2c_transfer(client->adapter, msgs, 2); > + if (ret < 0) > + return ret; > + > + crc_data[sizeof(crc_data) - 1] = *val = buf_rx[0]; > + if (buf_rx[1] != crc8(tps6594_i2c_crc_table, crc_data, sizeof(crc_data), CRC8_INIT_VALUE)) > + return -EIO; > + > + return ret; > +} > + > +static int tps6594_i2c_reg_write_with_crc(struct i2c_client *client, u8 page, u8 reg, u8 val) > +{ > + struct i2c_msg msg; > + u8 buf[] = {reg, val, 0}; > + /* I2C address = I2C base address + Page index */ > + const u8 addr = client->addr + page; > + /* > + * CRC is calculated from every bit included in the protocol > + * except the ACK bits from the target. Byte stream is: > + * - B0: (I2C_addr_7bits << 1) | WR_bit, with WR_bit = 0 > + * - B1: reg > + * - B2: val > + * - B3: CRC from B0-B1-B2 > + */ > + const u8 crc_data[] = {addr << 1, reg, val}; > + > + /* Write register, data and CRC */ > + msg.addr = addr; > + msg.flags = client->flags & I2C_M_TEN; > + msg.len = sizeof(buf); > + msg.buf = buf; > + > + buf[msg.len - 1] = crc8(tps6594_i2c_crc_table, crc_data, sizeof(crc_data), CRC8_INIT_VALUE); > + > + return tps6594_i2c_transfer(client->adapter, &msg, 1); > +} > + > +static int tps6594_i2c_read(void *context, const void *reg_buf, size_t reg_size, > + void *val_buf, size_t val_size) > +{ > + struct i2c_client *client = context; > + struct tps6594 *tps = i2c_get_clientdata(client); > + struct i2c_msg msgs[2]; > + const u8 *reg_bytes = reg_buf; > + u8 *val_bytes = val_buf; > + const u8 page = reg_bytes[1]; > + u8 reg = reg_bytes[0]; > + int ret = 0; > + int i; > + > + if (tps->use_crc) { > + /* > + * Auto-increment feature does not support CRC protocol. > + * Converts the bulk read operation into a series of single read operations. > + */ > + for (i = 0 ; ret == 0 && i < val_size ; i++) > + ret = tps6594_i2c_reg_read_with_crc(client, page, reg + i, val_bytes + i); > + > + return ret; > + } > + > + /* Write register: I2C address = I2C base address + Page index */ > + msgs[0].addr = client->addr + page; > + msgs[0].flags = 0; > + msgs[0].len = 1; > + msgs[0].buf = ® > + > + /* Read data */ > + msgs[1].addr = msgs[0].addr; > + msgs[1].flags = I2C_M_RD; > + msgs[1].len = val_size; > + msgs[1].buf = val_bytes; > + > + return tps6594_i2c_transfer(client->adapter, msgs, 2); > +} > + > +static int tps6594_i2c_write(void *context, const void *data, size_t count) > +{ > + struct i2c_client *client = context; > + struct tps6594 *tps = i2c_get_clientdata(client); > + struct i2c_msg msg; > + const u8 *bytes = data; > + u8 *buf; > + const u8 page = bytes[1]; > + const u8 reg = bytes[0]; > + int ret = 0; > + int i; > + > + if (tps->use_crc) { > + /* > + * Auto-increment feature does not support CRC protocol. > + * Converts the bulk write operation into a series of single write operations. > + */ > + for (i = 0 ; ret == 0 && i < count - 2 ; i++) > + ret = tps6594_i2c_reg_write_with_crc(client, page, reg + i, bytes[i + 2]); > + > + return ret; > + } > + > + /* Setup buffer: page byte is not sent */ > + buf = kzalloc(--count, GFP_KERNEL); > + if (!buf) > + return -ENOMEM; > + > + buf[0] = reg; > + for (i = 0 ; i < count - 1 ; i++) > + buf[i + 1] = bytes[i + 2]; > + > + /* Write register and data: I2C address = I2C base address + Page index */ > + msg.addr = client->addr + page; > + msg.flags = client->flags & I2C_M_TEN; > + msg.len = count; > + msg.buf = buf; > + > + ret = tps6594_i2c_transfer(client->adapter, &msg, 1); > + > + kfree(buf); > + return ret; > +} > + > +static const struct regmap_config tps6594_i2c_regmap_config = { > + .reg_bits = 16, > + .val_bits = 8, > + .max_register = TPS6594_REG_DWD_FAIL_CNT_REG, > + .volatile_reg = tps6594_is_volatile_reg, > + .read = tps6594_i2c_read, > + .write = tps6594_i2c_write, > +}; > + > +static const struct of_device_id tps6594_i2c_of_match_table[] = { > + { .compatible = "ti,tps6594", .data = (void *)TPS6594, }, > + { .compatible = "ti,tps6593", .data = (void *)TPS6593, }, > + { .compatible = "ti,lp8764x", .data = (void *)LP8764X, }, > + {} > +}; > +MODULE_DEVICE_TABLE(of, tps6594_i2c_of_match_table); > + > +static int tps6594_i2c_probe(struct i2c_client *client) > +{ > + struct device *dev = &client->dev; > + struct tps6594 *tps; > + const struct of_device_id *match; > + > + tps = devm_kzalloc(dev, sizeof(*tps), GFP_KERNEL); > + if (!tps) > + return -ENOMEM; > + > + i2c_set_clientdata(client, tps); > + > + tps->dev = dev; > + tps->reg = client->addr; > + tps->irq = client->irq; > + > + tps->regmap = devm_regmap_init(dev, NULL, client, &tps6594_i2c_regmap_config); > + if (IS_ERR(tps->regmap)) > + return dev_err_probe(dev, PTR_ERR(tps->regmap), "Failed to init regmap\n"); > + > + match = of_match_device(tps6594_i2c_of_match_table, dev); > + if (!match) > + return dev_err_probe(dev, PTR_ERR(match), "Failed to find matching chip ID\n"); > + tps->chip_id = (unsigned long)match->data; > + > + crc8_populate_msb(tps6594_i2c_crc_table, TPS6594_CRC8_POLYNOMIAL); > + > + return tps6594_device_init(tps); > +} > + > +static struct i2c_driver tps6594_i2c_driver = { > + .driver = { > + .name = "tps6594", > + .of_match_table = tps6594_i2c_of_match_table, > + }, > + .probe_new = tps6594_i2c_probe, > +}; > +module_i2c_driver(tps6594_i2c_driver); > + > +MODULE_AUTHOR("Julien Panis "); > +MODULE_DESCRIPTION("TPS6594 I2C Interface Driver"); > +MODULE_LICENSE("GPL"); > diff --git a/drivers/mfd/tps6594-spi.c b/drivers/mfd/tps6594-spi.c > new file mode 100644 > index 000000000000..799a383935c5 > --- /dev/null > +++ b/drivers/mfd/tps6594-spi.c > @@ -0,0 +1,125 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > +/* > + * SPI access driver for TI TPS6594/TPS6593/LP8764X PMICs > + * > + * Copyright (C) 2022 BayLibre Incorporated - https://www.baylibre.com/ > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > + > +#include > + > +#define TPS6594_SPI_PAGE_SHIFT 5 > +#define TPS6594_SPI_READ_BIT BIT(4) > + > +DECLARE_CRC8_TABLE(tps6594_spi_crc_table); > + > +static int tps6594_spi_reg_read(void *context, unsigned int reg, unsigned int *val) > +{ > + struct spi_device *spi = context; > + struct tps6594 *tps = spi_get_drvdata(spi); > + u8 buf[4] = {0}; > + size_t count_rx = 1; > + int ret; > + > + buf[0] = reg; > + buf[1] = TPS6594_REG_TO_PAGE(reg) << TPS6594_SPI_PAGE_SHIFT | TPS6594_SPI_READ_BIT; > + > + if (tps->use_crc) > + count_rx++; > + > + ret = spi_write_then_read(spi, buf, 2, buf + 2, count_rx); > + if (ret < 0) > + return ret; > + > + if (tps->use_crc && buf[3] != crc8(tps6594_spi_crc_table, buf, 3, CRC8_INIT_VALUE)) > + return -EIO; > + > + *val = buf[2]; > + > + return 0; > +} > + > +static int tps6594_spi_reg_write(void *context, unsigned int reg, unsigned int val) > +{ > + struct spi_device *spi = context; > + struct tps6594 *tps = spi_get_drvdata(spi); > + u8 buf[4] = {0}; > + size_t count = 3; > + > + buf[0] = reg; > + buf[1] = TPS6594_REG_TO_PAGE(reg) << TPS6594_SPI_PAGE_SHIFT; > + buf[2] = val; > + > + if (tps->use_crc) > + buf[3] = crc8(tps6594_spi_crc_table, buf, count++, CRC8_INIT_VALUE); > + > + return spi_write(spi, buf, count); > +} > + > +static const struct regmap_config tps6594_spi_regmap_config = { > + .reg_bits = 16, > + .val_bits = 8, > + .max_register = TPS6594_REG_DWD_FAIL_CNT_REG, > + .volatile_reg = tps6594_is_volatile_reg, > + .reg_read = tps6594_spi_reg_read, > + .reg_write = tps6594_spi_reg_write, > + .use_single_read = true, > + .use_single_write = true, > +}; > + > +static const struct of_device_id tps6594_spi_of_match_table[] = { > + { .compatible = "ti,tps6594", .data = (void *)TPS6594, }, > + { .compatible = "ti,tps6593", .data = (void *)TPS6593, }, > + { .compatible = "ti,lp8764x", .data = (void *)LP8764X, }, > + {} > +}; > +MODULE_DEVICE_TABLE(of, tps6594_spi_of_match_table); > + > +static int tps6594_spi_probe(struct spi_device *spi) > +{ > + struct device *dev = &spi->dev; > + struct tps6594 *tps; > + const struct of_device_id *match; > + > + tps = devm_kzalloc(dev, sizeof(*tps), GFP_KERNEL); > + if (!tps) > + return -ENOMEM; > + > + spi_set_drvdata(spi, tps); > + > + tps->dev = dev; > + tps->reg = spi->chip_select; > + tps->irq = spi->irq; > + > + tps->regmap = devm_regmap_init(dev, NULL, spi, &tps6594_spi_regmap_config); > + if (IS_ERR(tps->regmap)) > + return dev_err_probe(dev, PTR_ERR(tps->regmap), "Failed to init regmap\n"); > + > + match = of_match_device(tps6594_spi_of_match_table, dev); > + if (!match) > + return dev_err_probe(dev, PTR_ERR(match), "Failed to find matching chip ID\n"); > + tps->chip_id = (unsigned long)match->data; > + > + crc8_populate_msb(tps6594_spi_crc_table, TPS6594_CRC8_POLYNOMIAL); > + > + return tps6594_device_init(tps); > +} > + > +static struct spi_driver tps6594_spi_driver = { > + .driver = { > + .name = "tps6594", > + .of_match_table = tps6594_spi_of_match_table, > + }, > + .probe = tps6594_spi_probe, > +}; > +module_spi_driver(tps6594_spi_driver); > + > +MODULE_AUTHOR("Julien Panis "); > +MODULE_DESCRIPTION("TPS6594 SPI Interface Driver"); > +MODULE_LICENSE("GPL"); > diff --git a/include/linux/mfd/tps6594.h b/include/linux/mfd/tps6594.h > new file mode 100644 > index 000000000000..e2ffd4dc034d > --- /dev/null > +++ b/include/linux/mfd/tps6594.h > @@ -0,0 +1,1018 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* > + * Functions to access TPS6594 Power Management IC > + * > + * Copyright (C) 2022 BayLibre Incorporated - https://www.baylibre.com/ > + */ > + > +#ifndef __LINUX_MFD_TPS6594_H > +#define __LINUX_MFD_TPS6594_H > + > +#include > +#include > + > +struct regmap_irq_chip_data; > + > +/* Chip id list */ > +#define TPS6594 0 > +#define TPS6593 1 > +#define LP8764X 2 enum? > +/* Macro to get page index from register address */ > +#define TPS6594_REG_TO_PAGE(reg) ((reg) >> 8) > + > +/* Registers for page 0 of TPS6594 */ > +#define TPS6594_REG_DEV_REV 0x01 > + > +#define TPS6594_REG_NVM_CODE_1 0x02 > +#define TPS6594_REG_NVM_CODE_2 0x03 > + > +#define TPS6594_REG_BUCKX_CTRL(buck_inst) (0x04 + ((buck_inst) << 1)) > +#define TPS6594_REG_BUCKX_CONF(buck_inst) (0x05 + ((buck_inst) << 1)) > +#define TPS6594_REG_BUCKX_VOUT_1(buck_inst) (0x0e + ((buck_inst) << 1)) > +#define TPS6594_REG_BUCKX_VOUT_2(buck_inst) (0x0f + ((buck_inst) << 1)) > +#define TPS6594_REG_BUCKX_PG_WINDOW(buck_inst) (0x18 + (buck_inst)) > + > +#define TPS6594_REG_LDOX_CTRL(ldo_inst) (0x1d + (ldo_inst)) > +#define TPS6594_REG_LDORTC_CTRL 0x22 > +#define TPS6594_REG_LDOX_VOUT(ldo_inst) (0x23 + (ldo_inst)) > +#define TPS6594_REG_LDOX_PG_WINDOW(ldo_inst) (0x27 + (ldo_inst)) > + > +#define TPS6594_REG_VCCA_VMON_CTRL 0x2b > +#define TPS6594_REG_VCCA_PG_WINDOW 0x2c > +#define TPS6594_REG_VMON1_PG_WINDOW 0x2d > +#define TPS6594_REG_VMON1_PG_LEVEL 0x2e > +#define TPS6594_REG_VMON2_PG_WINDOW 0x2f > +#define TPS6594_REG_VMON2_PG_LEVEL 0x30 > + > +#define TPS6594_REG_GPIOX_CONF(gpio_inst) (0x31 + (gpio_inst)) > +#define TPS6594_REG_NPWRON_CONF 0x3c > +#define TPS6594_REG_GPIO_OUT_1 0x3d > +#define TPS6594_REG_GPIO_OUT_2 0x3e > +#define TPS6594_REG_GPIO_IN_1 0x3f > +#define TPS6594_REG_GPIO_IN_2 0x40 > +#define TPS6594_REG_GPIOX_OUT(gpio_inst) (TPS6594_REG_GPIO_OUT_1 + (gpio_inst) / 8) > +#define TPS6594_REG_GPIOX_IN(gpio_inst) (TPS6594_REG_GPIO_IN_1 + (gpio_inst) / 8) > + > +#define TPS6594_REG_GPIO_IN_1 0x3f > +#define TPS6594_REG_GPIO_IN_2 0x40 > + > +#define TPS6594_REG_RAIL_SEL_1 0x41 > +#define TPS6594_REG_RAIL_SEL_2 0x42 > +#define TPS6594_REG_RAIL_SEL_3 0x43 > + > +#define TPS6594_REG_FSM_TRIG_SEL_1 0x44 > +#define TPS6594_REG_FSM_TRIG_SEL_2 0x45 > +#define TPS6594_REG_FSM_TRIG_MASK_1 0x46 > +#define TPS6594_REG_FSM_TRIG_MASK_2 0x47 > +#define TPS6594_REG_FSM_TRIG_MASK_3 0x48 > + > +#define TPS6594_REG_MASK_BUCK1_2 0x49 > +#define TPS6594_REG_MASK_BUCK3_4 0x4a > +#define TPS6594_REG_MASK_BUCK5 0x4b > +#define TPS6594_REG_MASK_LDO1_2 0x4c > +#define TPS6594_REG_MASK_LDO3_4 0x4d > +#define TPS6594_REG_MASK_VMON 0x4e > +#define TPS6594_REG_MASK_GPIO1_8_FALL 0x4f > +#define TPS6594_REG_MASK_GPIO1_8_RISE 0x50 > +#define TPS6594_REG_MASK_GPIO9_11 0x51 > +#define TPS6594_REG_MASK_STARTUP 0x52 > +#define TPS6594_REG_MASK_MISC 0x53 > +#define TPS6594_REG_MASK_MODERATE_ERR 0x54 > +#define TPS6594_REG_MASK_FSM_ERR 0x56 > +#define TPS6594_REG_MASK_COMM_ERR 0x57 > +#define TPS6594_REG_MASK_READBACK_ERR 0x58 > +#define TPS6594_REG_MASK_ESM 0x59 > + > +#define TPS6594_REG_INT_TOP 0x5a > +#define TPS6594_REG_INT_BUCK 0x5b > +#define TPS6594_REG_INT_BUCK1_2 0x5c > +#define TPS6594_REG_INT_BUCK3_4 0x5d > +#define TPS6594_REG_INT_BUCK5 0x5e > +#define TPS6594_REG_INT_LDO_VMON 0x5f > +#define TPS6594_REG_INT_LDO1_2 0x60 > +#define TPS6594_REG_INT_LDO3_4 0x61 > +#define TPS6594_REG_INT_VMON 0x62 > +#define TPS6594_REG_INT_GPIO 0x63 > +#define TPS6594_REG_INT_GPIO1_8 0x64 > +#define TPS6594_REG_INT_STARTUP 0x65 > +#define TPS6594_REG_INT_MISC 0x66 > +#define TPS6594_REG_INT_MODERATE_ERR 0x67 > +#define TPS6594_REG_INT_SEVERE_ERR 0x68 > +#define TPS6594_REG_INT_FSM_ERR 0x69 > +#define TPS6594_REG_INT_COMM_ERR 0x6a > +#define TPS6594_REG_INT_READBACK_ERR 0x6b > +#define TPS6594_REG_INT_ESM 0x6c > + > +#define TPS6594_REG_STAT_BUCK1_2 0x6d > +#define TPS6594_REG_STAT_BUCK3_4 0x6e > +#define TPS6594_REG_STAT_BUCK5 0x6f > +#define TPS6594_REG_STAT_LDO1_2 0x70 > +#define TPS6594_REG_STAT_LDO3_4 0x71 > +#define TPS6594_REG_STAT_VMON 0x72 > +#define TPS6594_REG_STAT_STARTUP 0x73 > +#define TPS6594_REG_STAT_MISC 0x74 > +#define TPS6594_REG_STAT_MODERATE_ERR 0x75 > +#define TPS6594_REG_STAT_SEVERE_ERR 0x76 > +#define TPS6594_REG_STAT_READBACK_ERR 0x77 > + > +#define TPS6594_REG_PGOOD_SEL_1 0x78 > +#define TPS6594_REG_PGOOD_SEL_2 0x79 > +#define TPS6594_REG_PGOOD_SEL_3 0x7a > +#define TPS6594_REG_PGOOD_SEL_4 0x7b > + > +#define TPS6594_REG_PLL_CTRL 0x7c > + > +#define TPS6594_REG_CONFIG_1 0x7d > +#define TPS6594_REG_CONFIG_2 0x7e > + > +#define TPS6594_REG_ENABLE_DRV_REG 0x80 > + > +#define TPS6594_REG_MISC_CTRL 0x81 > + > +#define TPS6594_REG_ENABLE_DRV_STAT 0x82 > + > +#define TPS6594_REG_RECOV_CNT_REG_1 0x83 > +#define TPS6594_REG_RECOV_CNT_REG_2 0x84 > + > +#define TPS6594_REG_FSM_I2C_TRIGGERS 0x85 > +#define TPS6594_REG_FSM_NSLEEP_TRIGGERS 0x86 > + > +#define TPS6594_REG_BUCK_RESET_REG 0x87 > + > +#define TPS6594_REG_SPREAD_SPECTRUM_1 0x88 > + > +#define TPS6594_REG_FREQ_SEL 0x8a > + > +#define TPS6594_REG_FSM_STEP_SIZE 0x8b > + > +#define TPS6594_REG_LDO_RV_TIMEOUT_REG_1 0x8c > +#define TPS6594_REG_LDO_RV_TIMEOUT_REG_2 0x8d > + > +#define TPS6594_REG_USER_SPARE_REGS 0x8e > + > +#define TPS6594_REG_ESM_MCU_START_REG 0x8f > +#define TPS6594_REG_ESM_MCU_DELAY1_REG 0x90 > +#define TPS6594_REG_ESM_MCU_DELAY2_REG 0x91 > +#define TPS6594_REG_ESM_MCU_MODE_CFG 0x92 > +#define TPS6594_REG_ESM_MCU_HMAX_REG 0x93 > +#define TPS6594_REG_ESM_MCU_HMIN_REG 0x94 > +#define TPS6594_REG_ESM_MCU_LMAX_REG 0x95 > +#define TPS6594_REG_ESM_MCU_LMIN_REG 0x96 > +#define TPS6594_REG_ESM_MCU_ERR_CNT_REG 0x97 > +#define TPS6594_REG_ESM_SOC_START_REG 0x98 > +#define TPS6594_REG_ESM_SOC_DELAY1_REG 0x99 > +#define TPS6594_REG_ESM_SOC_DELAY2_REG 0x9a > +#define TPS6594_REG_ESM_SOC_MODE_CFG 0x9b > +#define TPS6594_REG_ESM_SOC_HMAX_REG 0x9c > +#define TPS6594_REG_ESM_SOC_HMIN_REG 0x9d > +#define TPS6594_REG_ESM_SOC_LMAX_REG 0x9e > +#define TPS6594_REG_ESM_SOC_LMIN_REG 0x9f > +#define TPS6594_REG_ESM_SOC_ERR_CNT_REG 0xa0 > + > +#define TPS6594_REG_REGISTER_LOCK 0xa1 > + > +#define TPS6594_REG_MANUFACTURING_VER 0xa6 > + > +#define TPS6594_REG_CUSTOMER_NVM_ID_REG 0xa7 > + > +#define TPS6594_REG_VMON_CONF_REG 0xa8 > + > +#define TPS6594_REG_SOFT_REBOOT_REG 0xab > + > +#define TPS6594_REG_RTC_SECONDS 0xb5 > +#define TPS6594_REG_RTC_MINUTES 0xb6 > +#define TPS6594_REG_RTC_HOURS 0xb7 > +#define TPS6594_REG_RTC_DAYS 0xb8 > +#define TPS6594_REG_RTC_MONTHS 0xb9 > +#define TPS6594_REG_RTC_YEARS 0xba > +#define TPS6594_REG_RTC_WEEKS 0xbb > + > +#define TPS6594_REG_ALARM_SECONDS 0xbc > +#define TPS6594_REG_ALARM_MINUTES 0xbd > +#define TPS6594_REG_ALARM_HOURS 0xbe > +#define TPS6594_REG_ALARM_DAYS 0xbf > +#define TPS6594_REG_ALARM_MONTHS 0xc0 > +#define TPS6594_REG_ALARM_YEARS 0xc1 > + > +#define TPS6594_REG_RTC_CTRL_1 0xc2 > +#define TPS6594_REG_RTC_CTRL_2 0xc3 > +#define TPS6594_REG_RTC_STATUS 0xc4 > +#define TPS6594_REG_RTC_INTERRUPTS 0xc5 > +#define TPS6594_REG_RTC_COMP_LSB 0xc6 > +#define TPS6594_REG_RTC_COMP_MSB 0xc7 > +#define TPS6594_REG_RTC_RESET_STATUS 0xc8 > + > +#define TPS6594_REG_SCRATCH_PAD_REG_1 0xc9 > +#define TPS6594_REG_SCRATCH_PAD_REG_2 0xca > +#define TPS6594_REG_SCRATCH_PAD_REG_3 0xcb > +#define TPS6594_REG_SCRATCH_PAD_REG_4 0xcc > + > +#define TPS6594_REG_PFSM_DELAY_REG_1 0xcd > +#define TPS6594_REG_PFSM_DELAY_REG_2 0xce > +#define TPS6594_REG_PFSM_DELAY_REG_3 0xcf > +#define TPS6594_REG_PFSM_DELAY_REG_4 0xd0 > + > +/* Registers for page 1 of TPS6594 */ > +#define TPS6594_REG_SERIAL_IF_CONFIG 0x11a > +#define TPS6594_REG_I2C1_ID 0x122 > +#define TPS6594_REG_I2C2_ID 0x123 > + > +/* Registers for page 4 of TPS6594 */ > +#define TPS6594_REG_WD_ANSWER_REG 0x401 > +#define TPS6594_REG_WD_QUESTION_ANSW_CNT 0x402 > +#define TPS6594_REG_WD_WIN1_CFG 0x403 > +#define TPS6594_REG_WD_WIN2_CFG 0x404 > +#define TPS6594_REG_WD_LONGWIN_CFG 0x405 > +#define TPS6594_REG_WD_MODE_REG 0x406 > +#define TPS6594_REG_WD_QA_CFG 0x407 > +#define TPS6594_REG_WD_ERR_STATUS 0x408 > +#define TPS6594_REG_WD_THR_CFG 0x409 > +#define TPS6594_REG_DWD_FAIL_CNT_REG 0x40a > + > +/* BUCKX_CTRL register field definition */ > +#define TPS6594_BIT_BUCK_EN BIT(0) > +#define TPS6594_BIT_BUCK_FPWM BIT(1) > +#define TPS6594_BIT_BUCK_FPWM_MP BIT(2) > +#define TPS6594_BIT_BUCK_VSEL BIT(3) > +#define TPS6594_BIT_BUCK_VMON_EN BIT(4) > +#define TPS6594_BIT_BUCK_PLDN BIT(5) > +#define TPS6594_BIT_BUCK_RV_SEL BIT(7) > + > +/* BUCKX_CONF register field definition */ > +#define TPS6594_MASK_BUCK_SLEW_RATE GENMASK(2, 0) > +#define TPS6594_MASK_BUCK_ILIM GENMASK(5, 3) > + > +/* BUCKX_PG_WINDOW register field definition */ > +#define TPS6594_MASK_BUCK_OV_THR GENMASK(2, 0) > +#define TPS6594_MASK_BUCK_UV_THR GENMASK(5, 3) > + > +/* BUCKX VSET */ > +#define TPS6594_MASK_BUCKS_VSET GENMASK(7, 0) > + > +/* LDOX_CTRL register field definition */ > +#define TPS6594_BIT_LDO_EN BIT(0) > +#define TPS6594_BIT_LDO_SLOW_RAMP BIT(1) > +#define TPS6594_BIT_LDO_VMON_EN BIT(4) > +#define TPS6594_MASK_LDO_PLDN GENMASK(6, 5) > +#define TPS6594_BIT_LDO_RV_SEL BIT(7) > + > +/* LDORTC_CTRL register field definition */ > +#define TPS6594_BIT_LDORTC_DIS BIT(0) > + > +/* LDOX_VOUT register field definition */ > +#define TPS6594_MASK_LDO123_VSET GENMASK(6, 1) > +#define TPS6594_MASK_LDO4_VSET GENMASK(6, 0) > +#define TPS6594_BIT_LDO_BYPASS BIT(7) > + > +/* LDOX_PG_WINDOW register field definition */ > +#define TPS6594_MASK_LDO_OV_THR GENMASK(2, 0) > +#define TPS6594_MASK_LDO_UV_THR GENMASK(5, 3) > + > +/* VCCA_VMON_CTRL register field definition */ > +#define TPS6594_BIT_VMON_EN BIT(0) > +#define TPS6594_BIT_VMON1_EN BIT(1) > +#define TPS6594_BIT_VMON1_RV_SEL BIT(2) > +#define TPS6594_BIT_VMON2_EN BIT(3) > +#define TPS6594_BIT_VMON2_RV_SEL BIT(4) > +#define TPS6594_BIT_VMON_DEGLITCH_SEL BIT(5) > + > +/* VCCA_PG_WINDOW register field definition */ > +#define TPS6594_MASK_VCCA_OV_THR GENMASK(2, 0) > +#define TPS6594_MASK_VCCA_UV_THR GENMASK(5, 3) > +#define TPS6594_BIT_VCCA_PG_SET BIT(6) > + > +/* VMONX_PG_WINDOW register field definition */ > +#define TPS6594_MASK_VMONX_OV_THR GENMASK(2, 0) > +#define TPS6594_MASK_VMONX_UV_THR GENMASK(5, 3) > +#define TPS6594_BIT_VMONX_RANGE BIT(6) > + > +/* GPIOX_CONF register field definition */ > +#define TPS6594_BIT_GPIO_DIR BIT(0) > +#define TPS6594_BIT_GPIO_OD BIT(1) > +#define TPS6594_BIT_GPIO_PU_SEL BIT(2) > +#define TPS6594_BIT_GPIO_PU_PD_EN BIT(3) > +#define TPS6594_BIT_GPIO_DEGLITCH_EN BIT(4) > +#define TPS6594_MASK_GPIO_SEL GENMASK(7, 5) > + > +/* NPWRON_CONF register field definition */ > +#define TPS6594_BIT_NRSTOUT_OD BIT(0) > +#define TPS6594_BIT_ENABLE_PU_SEL BIT(2) > +#define TPS6594_BIT_ENABLE_PU_PD_EN BIT(3) > +#define TPS6594_BIT_ENABLE_DEGLITCH_EN BIT(4) > +#define TPS6594_BIT_ENABLE_POL BIT(5) > +#define TPS6594_MASK_NPWRON_SEL GENMASK(7, 6) > + > +/* GPIO_OUT_X register field definition */ > +#define TPS6594_BIT_GPIOX_OUT(gpio_inst) BIT((gpio_inst) % 8) > + > +/* GPIO_IN_X register field definition */ > +#define TPS6594_BIT_GPIOX_IN(gpio_inst) BIT((gpio_inst) % 8) > +#define TPS6594_BIT_NPWRON_IN BIT(3) > + > +/* RAIL_SEL_1 register field definition */ > +#define TPS6594_MASK_BUCK1_GRP_SEL GENMASK(1, 0) > +#define TPS6594_MASK_BUCK2_GRP_SEL GENMASK(3, 2) > +#define TPS6594_MASK_BUCK3_GRP_SEL GENMASK(5, 4) > +#define TPS6594_MASK_BUCK4_GRP_SEL GENMASK(7, 6) > + > +/* RAIL_SEL_2 register field definition */ > +#define TPS6594_MASK_BUCK5_GRP_SEL GENMASK(1, 0) > +#define TPS6594_MASK_LDO1_GRP_SEL GENMASK(3, 2) > +#define TPS6594_MASK_LDO2_GRP_SEL GENMASK(5, 4) > +#define TPS6594_MASK_LDO3_GRP_SEL GENMASK(7, 6) > + > +/* RAIL_SEL_3 register field definition */ > +#define TPS6594_MASK_LDO4_GRP_SEL GENMASK(1, 0) > +#define TPS6594_MASK_VCCA_GRP_SEL GENMASK(3, 2) > +#define TPS6594_MASK_VMON1_GRP_SEL GENMASK(5, 4) > +#define TPS6594_MASK_VMON2_GRP_SEL GENMASK(7, 6) > + > +/* FSM_TRIG_SEL_1 register field definition */ > +#define TPS6594_MASK_MCU_RAIL_TRIG GENMASK(1, 0) > +#define TPS6594_MASK_SOC_RAIL_TRIG GENMASK(3, 2) > +#define TPS6594_MASK_OTHER_RAIL_TRIG GENMASK(5, 4) > +#define TPS6594_MASK_SEVERE_ERR_TRIG GENMASK(7, 6) > + > +/* FSM_TRIG_SEL_2 register field definition */ > +#define TPS6594_MASK_MODERATE_ERR_TRIG GENMASK(1, 0) > + > +/* FSM_TRIG_MASK_X register field definition */ > +#define TPS6594_BIT_GPIOX_FSM_MASK(gpio_inst) BIT(((gpio_inst) << 1) % 8) > +#define TPS6594_BIT_GPIOX_FSM_MASK_POL(gpio_inst) BIT(((gpio_inst) << 1) % 8 + 1) > + > +/* MASK_BUCKX register field definition */ > +#define TPS6594_BIT_BUCKX_OV_MASK(buck_inst) BIT(((buck_inst) << 2) % 8) > +#define TPS6594_BIT_BUCKX_UV_MASK(buck_inst) BIT(((buck_inst) << 2) % 8 + 1) > +#define TPS6594_BIT_BUCKX_ILIM_MASK(buck_inst) BIT(((buck_inst) << 2) % 8 + 3) > + > +/* MASK_LDOX register field definition */ > +#define TPS6594_BIT_LDOX_OV_MASK(ldo_inst) BIT(((ldo_inst) << 2) % 8) > +#define TPS6594_BIT_LDOX_UV_MASK(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 1) > +#define TPS6594_BIT_LDOX_ILIM_MASK(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 3) > + > +/* MASK_VMON register field definition */ > +#define TPS6594_BIT_VCCA_OV_MASK BIT(0) > +#define TPS6594_BIT_VCCA_UV_MASK BIT(1) > +#define TPS6594_BIT_VMON1_OV_MASK BIT(2) > +#define TPS6594_BIT_VMON1_UV_MASK BIT(3) > +#define TPS6594_BIT_VMON2_OV_MASK BIT(5) > +#define TPS6594_BIT_VMON2_UV_MASK BIT(6) > + > +/* MASK_GPIOX register field definition */ > +#define TPS6594_BIT_GPIOX_FALL_MASK(gpio_inst) BIT((gpio_inst) < 8 ? \ > + (gpio_inst) : (gpio_inst) % 8) > +#define TPS6594_BIT_GPIOX_RISE_MASK(gpio_inst) BIT((gpio_inst) < 8 ? \ > + (gpio_inst) : (gpio_inst) % 8 + 3) > + > +/* MASK_STARTUP register field definition */ > +#define TPS6594_BIT_NPWRON_START_MASK BIT(0) > +#define TPS6594_BIT_ENABLE_MASK BIT(1) > +#define TPS6594_BIT_FSD_MASK BIT(4) > +#define TPS6594_BIT_SOFT_REBOOT_MASK BIT(5) > + > +/* MASK_MISC register field definition */ > +#define TPS6594_BIT_BIST_PASS_MASK BIT(0) > +#define TPS6594_BIT_EXT_CLK_MASK BIT(1) > +#define TPS6594_BIT_TWARN_MASK BIT(3) > + > +/* MASK_MODERATE_ERR register field definition */ > +#define TPS6594_BIT_BIST_FAIL_MASK BIT(1) > +#define TPS6594_BIT_REG_CRC_ERR_MASK BIT(2) > +#define TPS6594_BIT_SPMI_ERR_MASK BIT(4) > +#define TPS6594_BIT_NPWRON_LONG_MASK BIT(5) > +#define TPS6594_BIT_NINT_READBACK_MASK BIT(6) > +#define TPS6594_BIT_NRSTOUT_READBACK_MASK BIT(7) > + > +/* MASK_FSM_ERR register field definition */ > +#define TPS6594_BIT_IMM_SHUTDOWN_MASK BIT(0) > +#define TPS6594_BIT_ORD_SHUTDOWN_MASK BIT(1) > +#define TPS6594_BIT_MCU_PWR_ERR_MASK BIT(2) > +#define TPS6594_BIT_SOC_PWR_ERR_MASK BIT(3) > + > +/* MASK_COMM_ERR register field definition */ > +#define TPS6594_BIT_COMM_FRM_ERR_MASK BIT(0) > +#define TPS6594_BIT_COMM_CRC_ERR_MASK BIT(1) > +#define TPS6594_BIT_COMM_ADR_ERR_MASK BIT(3) > +#define TPS6594_BIT_I2C2_CRC_ERR_MASK BIT(5) > +#define TPS6594_BIT_I2C2_ADR_ERR_MASK BIT(7) > + > +/* MASK_READBACK_ERR register field definition */ > +#define TPS6594_BIT_EN_DRV_READBACK_MASK BIT(0) > +#define TPS6594_BIT_NRSTOUT_SOC_READBACK_MASK BIT(3) > + > +/* MASK_ESM register field definition */ > +#define TPS6594_BIT_ESM_SOC_PIN_MASK BIT(0) > +#define TPS6594_BIT_ESM_SOC_FAIL_MASK BIT(1) > +#define TPS6594_BIT_ESM_SOC_RST_MASK BIT(2) > +#define TPS6594_BIT_ESM_MCU_PIN_MASK BIT(3) > +#define TPS6594_BIT_ESM_MCU_FAIL_MASK BIT(4) > +#define TPS6594_BIT_ESM_MCU_RST_MASK BIT(5) > + > +/* INT_TOP register field definition */ > +#define TPS6594_BIT_BUCK_INT BIT(0) > +#define TPS6594_BIT_LDO_VMON_INT BIT(1) > +#define TPS6594_BIT_GPIO_INT BIT(2) > +#define TPS6594_BIT_STARTUP_INT BIT(3) > +#define TPS6594_BIT_MISC_INT BIT(4) > +#define TPS6594_BIT_MODERATE_ERR_INT BIT(5) > +#define TPS6594_BIT_SEVERE_ERR_INT BIT(6) > +#define TPS6594_BIT_FSM_ERR_INT BIT(7) > + > +/* INT_BUCK register field definition */ > +#define TPS6594_BIT_BUCK1_2_INT BIT(0) > +#define TPS6594_BIT_BUCK3_4_INT BIT(1) > +#define TPS6594_BIT_BUCK5_INT BIT(2) > + > +/* INT_BUCKX register field definition */ > +#define TPS6594_BIT_BUCKX_OV_INT(buck_inst) BIT(((buck_inst) << 2) % 8) > +#define TPS6594_BIT_BUCKX_UV_INT(buck_inst) BIT(((buck_inst) << 2) % 8 + 1) > +#define TPS6594_BIT_BUCKX_SC_INT(buck_inst) BIT(((buck_inst) << 2) % 8 + 2) > +#define TPS6594_BIT_BUCKX_ILIM_INT(buck_inst) BIT(((buck_inst) << 2) % 8 + 3) > + > +/* INT_LDO_VMON register field definition */ > +#define TPS6594_BIT_LDO1_2_INT BIT(0) > +#define TPS6594_BIT_LDO3_4_INT BIT(1) > +#define TPS6594_BIT_VCCA_INT BIT(4) > + > +/* INT_LDOX register field definition */ > +#define TPS6594_BIT_LDOX_OV_INT(ldo_inst) BIT(((ldo_inst) << 2) % 8) > +#define TPS6594_BIT_LDOX_UV_INT(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 1) > +#define TPS6594_BIT_LDOX_SC_INT(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 2) > +#define TPS6594_BIT_LDOX_ILIM_INT(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 3) > + > +/* INT_VMON register field definition */ > +#define TPS6594_BIT_VCCA_OV_INT BIT(0) > +#define TPS6594_BIT_VCCA_UV_INT BIT(1) > +#define TPS6594_BIT_VMON1_OV_INT BIT(2) > +#define TPS6594_BIT_VMON1_UV_INT BIT(3) > +#define TPS6594_BIT_VMON1_RV_INT BIT(4) > +#define TPS6594_BIT_VMON2_OV_INT BIT(5) > +#define TPS6594_BIT_VMON2_UV_INT BIT(6) > +#define TPS6594_BIT_VMON2_RV_INT BIT(7) > + > +/* INT_GPIO register field definition */ > +#define TPS6594_BIT_GPIO9_INT BIT(0) > +#define TPS6594_BIT_GPIO10_INT BIT(1) > +#define TPS6594_BIT_GPIO11_INT BIT(2) > +#define TPS6594_BIT_GPIO1_8_INT BIT(3) > + > +/* INT_GPIOX register field definition */ > +#define TPS6594_BIT_GPIOX_INT(gpio_inst) BIT(gpio_inst) > + > +/* INT_STARTUP register field definition */ > +#define TPS6594_BIT_NPWRON_START_INT BIT(0) > +#define TPS6594_BIT_ENABLE_INT BIT(1) > +#define TPS6594_BIT_RTC_INT BIT(2) > +#define TPS6594_BIT_FSD_INT BIT(4) > +#define TPS6594_BIT_SOFT_REBOOT_INT BIT(5) > + > +/* INT_MISC register field definition */ > +#define TPS6594_BIT_BIST_PASS_INT BIT(0) > +#define TPS6594_BIT_EXT_CLK_INT BIT(1) > +#define TPS6594_BIT_TWARN_INT BIT(3) > + > +/* INT_MODERATE_ERR register field definition */ > +#define TPS6594_BIT_TSD_ORD_INT BIT(0) > +#define TPS6594_BIT_BIST_FAIL_INT BIT(1) > +#define TPS6594_BIT_REG_CRC_ERR_INT BIT(2) > +#define TPS6594_BIT_RECOV_CNT_INT BIT(3) > +#define TPS6594_BIT_SPMI_ERR_INT BIT(4) > +#define TPS6594_BIT_NPWRON_LONG_INT BIT(5) > +#define TPS6594_BIT_NINT_READBACK_INT BIT(6) > +#define TPS6594_BIT_NRSTOUT_READBACK_INT BIT(7) > + > +/* INT_SEVERE_ERR register field definition */ > +#define TPS6594_BIT_TSD_IMM_INT BIT(0) > +#define TPS6594_BIT_VCCA_OVP_INT BIT(1) > +#define TPS6594_BIT_PFSM_ERR_INT BIT(2) > + > +/* INT_FSM_ERR register field definition */ > +#define TPS6594_BIT_IMM_SHUTDOWN_INT BIT(0) > +#define TPS6594_BIT_ORD_SHUTDOWN_INT BIT(1) > +#define TPS6594_BIT_MCU_PWR_ERR_INT BIT(2) > +#define TPS6594_BIT_SOC_PWR_ERR_INT BIT(3) > +#define TPS6594_BIT_COMM_ERR_INT BIT(4) > +#define TPS6594_BIT_READBACK_ERR_INT BIT(5) > +#define TPS6594_BIT_ESM_INT BIT(6) > +#define TPS6594_BIT_WD_INT BIT(7) > + > +/* INT_COMM_ERR register field definition */ > +#define TPS6594_BIT_COMM_FRM_ERR_INT BIT(0) > +#define TPS6594_BIT_COMM_CRC_ERR_INT BIT(1) > +#define TPS6594_BIT_COMM_ADR_ERR_INT BIT(3) > +#define TPS6594_BIT_I2C2_CRC_ERR_INT BIT(5) > +#define TPS6594_BIT_I2C2_ADR_ERR_INT BIT(7) > + > +/* INT_READBACK_ERR register field definition */ > +#define TPS6594_BIT_EN_DRV_READBACK_INT BIT(0) > +#define TPS6594_BIT_NRSTOUT_SOC_READBACK_INT BIT(3) > + > +/* INT_ESM register field definition */ > +#define TPS6594_BIT_ESM_SOC_PIN_INT BIT(0) > +#define TPS6594_BIT_ESM_SOC_FAIL_INT BIT(1) > +#define TPS6594_BIT_ESM_SOC_RST_INT BIT(2) > +#define TPS6594_BIT_ESM_MCU_PIN_INT BIT(3) > +#define TPS6594_BIT_ESM_MCU_FAIL_INT BIT(4) > +#define TPS6594_BIT_ESM_MCU_RST_INT BIT(5) > + > +/* STAT_BUCKX register field definition */ > +#define TPS6594_BIT_BUCKX_OV_STAT(buck_inst) BIT(((buck_inst) << 2) % 8) > +#define TPS6594_BIT_BUCKX_UV_STAT(buck_inst) BIT(((buck_inst) << 2) % 8 + 1) > +#define TPS6594_BIT_BUCKX_ILIM_STAT(buck_inst) BIT(((buck_inst) << 2) % 8 + 3) > + > +/* STAT_LDOX register field definition */ > +#define TPS6594_BIT_LDOX_OV_STAT(ldo_inst) BIT(((ldo_inst) << 2) % 8) > +#define TPS6594_BIT_LDOX_UV_STAT(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 1) > +#define TPS6594_BIT_LDOX_ILIM_STAT(ldo_inst) BIT(((ldo_inst) << 2) % 8 + 3) > + > +/* STAT_VMON register field definition */ > +#define TPS6594_BIT_VCCA_OV_STAT BIT(0) > +#define TPS6594_BIT_VCCA_UV_STAT BIT(1) > +#define TPS6594_BIT_VMON1_OV_STAT BIT(2) > +#define TPS6594_BIT_VMON1_UV_STAT BIT(3) > +#define TPS6594_BIT_VMON2_OV_STAT BIT(5) > +#define TPS6594_BIT_VMON2_UV_STAT BIT(6) > + > +/* STAT_STARTUP register field definition */ > +#define TPS6594_BIT_ENABLE_STAT BIT(1) > + > +/* STAT_MISC register field definition */ > +#define TPS6594_BIT_EXT_CLK_STAT BIT(1) > +#define TPS6594_BIT_TWARN_STAT BIT(3) > + > +/* STAT_MODERATE_ERR register field definition */ > +#define TPS6594_BIT_TSD_ORD_STAT BIT(0) > + > +/* STAT_SEVERE_ERR register field definition */ > +#define TPS6594_BIT_TSD_IMM_STAT BIT(0) > +#define TPS6594_BIT_VCCA_OVP_STAT BIT(1) > + > +/* STAT_READBACK_ERR register field definition */ > +#define TPS6594_BIT_EN_DRV_READBACK_STAT BIT(0) > +#define TPS6594_BIT_NINT_READBACK_STAT BIT(1) > +#define TPS6594_BIT_NRSTOUT_READBACK_STAT BIT(2) > +#define TPS6594_BIT_NRSTOUT_SOC_READBACK_STAT BIT(3) > + > +/* PGOOD_SEL_1 register field definition */ > +#define TPS6594_MASK_PGOOD_SEL_BUCK1 GENMASK(1, 0) > +#define TPS6594_MASK_PGOOD_SEL_BUCK2 GENMASK(3, 2) > +#define TPS6594_MASK_PGOOD_SEL_BUCK3 GENMASK(5, 4) > +#define TPS6594_MASK_PGOOD_SEL_BUCK4 GENMASK(7, 6) > + > +/* PGOOD_SEL_2 register field definition */ > +#define TPS6594_MASK_PGOOD_SEL_BUCK5 GENMASK(1, 0) > + > +/* PGOOD_SEL_3 register field definition */ > +#define TPS6594_MASK_PGOOD_SEL_LDO1 GENMASK(1, 0) > +#define TPS6594_MASK_PGOOD_SEL_LDO2 GENMASK(3, 2) > +#define TPS6594_MASK_PGOOD_SEL_LDO3 GENMASK(5, 4) > +#define TPS6594_MASK_PGOOD_SEL_LDO4 GENMASK(7, 6) > + > +/* PGOOD_SEL_4 register field definition */ > +#define TPS6594_BIT_PGOOD_SEL_VCCA BIT(0) > +#define TPS6594_BIT_PGOOD_SEL_VMON1 BIT(1) > +#define TPS6594_BIT_PGOOD_SEL_VMON2 BIT(2) > +#define TPS6594_BIT_PGOOD_SEL_TDIE_WARN BIT(3) > +#define TPS6594_BIT_PGOOD_SEL_NRSTOUT BIT(4) > +#define TPS6594_BIT_PGOOD_SEL_NRSTOUT_SOC BIT(5) > +#define TPS6594_BIT_PGOOD_POL BIT(6) > +#define TPS6594_BIT_PGOOD_WINDOW BIT(7) > + > +/* PLL_CTRL register field definition */ > +#define TPS6594_MASK_EXT_CLK_FREQ GENMASK(1, 0) > + > +/* CONFIG_1 register field definition */ > +#define TPS6594_BIT_TWARN_LEVEL BIT(0) > +#define TPS6594_BIT_TSD_ORD_LEVEL BIT(1) > +#define TPS6594_BIT_I2C1_HS BIT(3) > +#define TPS6594_BIT_I2C2_HS BIT(4) > +#define TPS6594_BIT_EN_ILIM_FSM_CTRL BIT(5) > +#define TPS6594_BIT_NSLEEP1_MASK BIT(6) > +#define TPS6594_BIT_NSLEEP2_MASK BIT(7) > + > +/* CONFIG_2 register field definition */ > +#define TPS6594_BIT_BB_CHARGER_EN BIT(0) > +#define TPS6594_BIT_BB_ICHR BIT(1) > +#define TPS6594_MASK_BB_VEOC GENMASK(3, 2) > +#define TPS6594_BB_EOC_RDY BIT(7) > + > +/* ENABLE_DRV_REG register field definition */ > +#define TPS6594_BIT_ENABLE_DRV BIT(0) > + > +/* MISC_CTRL register field definition */ > +#define TPS6594_BIT_NRSTOUT BIT(0) > +#define TPS6594_BIT_NRSTOUT_SOC BIT(1) > +#define TPS6594_BIT_LPM_EN BIT(2) > +#define TPS6594_BIT_CLKMON_EN BIT(3) > +#define TPS6594_BIT_AMUXOUT_EN BIT(4) > +#define TPS6594_BIT_SEL_EXT_CLK BIT(5) > +#define TPS6594_MASK_SYNCCLKOUT_FREQ_SEL GENMASK(7, 6) > + > +/* ENABLE_DRV_STAT register field definition */ > +#define TPS6594_BIT_EN_DRV_IN BIT(0) > +#define TPS6594_BIT_NRSTOUT_IN BIT(1) > +#define TPS6594_BIT_NRSTOUT_SOC_IN BIT(2) > +#define TPS6594_BIT_FORCE_EN_DRV_LOW BIT(3) > +#define TPS6594_BIT_SPMI_LPM_EN BIT(4) > + > +/* RECOV_CNT_REG_1 register field definition */ > +#define TPS6594_MASK_RECOV_CNT GENMASK(3, 0) > + > +/* RECOV_CNT_REG_2 register field definition */ > +#define TPS6594_MASK_RECOV_CNT_THR GENMASK(3, 0) > +#define TPS6594_BIT_RECOV_CNT_CLR BIT(4) > + > +/* FSM_I2C_TRIGGERS register field definition */ > +#define TPS6594_BIT_TRIGGER_I2C(bit) BIT(bit) > + > +/* FSM_NSLEEP_TRIGGERS register field definition */ > +#define TPS6594_BIT_NSLEEP1B BIT(0) > +#define TPS6594_BIT_NSLEEP2B BIT(1) > + > +/* BUCK_RESET_REG register field definition */ > +#define TPS6594_BIT_BUCKX_RESET(buck_inst) BIT(buck_inst) > + > +/* SPREAD_SPECTRUM_1 register field definition */ > +#define TPS6594_MASK_SS_DEPTH GENMASK(1, 0) > +#define TPS6594_BIT_SS_EN BIT(2) > + > +/* FREQ_SEL register field definition */ > +#define TPS6594_BIT_BUCKX_FREQ_SEL(buck_inst) BIT(buck_inst) > + > +/* FSM_STEP_SIZE register field definition */ > +#define TPS6594_MASK_PFSM_DELAY_STEP GENMASK(4, 0) > + > +/* LDO_RV_TIMEOUT_REG_1 register field definition */ > +#define TPS6594_MASK_LDO1_RV_TIMEOUT GENMASK(3, 0) > +#define TPS6594_MASK_LDO2_RV_TIMEOUT GENMASK(7, 4) > + > +/* LDO_RV_TIMEOUT_REG_2 register field definition */ > +#define TPS6594_MASK_LDO3_RV_TIMEOUT GENMASK(3, 0) > +#define TPS6594_MASK_LDO4_RV_TIMEOUT GENMASK(7, 4) > + > +/* USER_SPARE_REGS register field definition */ > +#define TPS6594_BIT_USER_SPARE(bit) BIT(bit) > + > +/* ESM_MCU_START_REG register field definition */ > +#define TPS6594_BIT_ESM_MCU_START BIT(0) > + > +/* ESM_MCU_MODE_CFG register field definition */ > +#define TPS6594_MASK_ESM_MCU_ERR_CNT_TH GENMASK(3, 0) > +#define TPS6594_BIT_ESM_MCU_ENDRV BIT(5) > +#define TPS6594_BIT_ESM_MCU_EN BIT(6) > +#define TPS6594_BIT_ESM_MCU_MODE BIT(7) > + > +/* ESM_MCU_ERR_CNT_REG register field definition */ > +#define TPS6594_MASK_ESM_MCU_ERR_CNT GENMASK(4, 0) > + > +/* ESM_SOC_START_REG register field definition */ > +#define TPS6594_BIT_ESM_SOC_START BIT(0) > + > +/* ESM_SOC_MODE_CFG register field definition */ > +#define TPS6594_MASK_ESM_SOC_ERR_CNT_TH GENMASK(3, 0) > +#define TPS6594_BIT_ESM_SOC_ENDRV BIT(5) > +#define TPS6594_BIT_ESM_SOC_EN BIT(6) > +#define TPS6594_BIT_ESM_SOC_MODE BIT(7) > + > +/* ESM_SOC_ERR_CNT_REG register field definition */ > +#define TPS6594_MASK_ESM_SOC_ERR_CNT GENMASK(4, 0) > + > +/* REGISTER_LOCK register field definition */ > +#define TPS6594_BIT_REGISTER_LOCK_STATUS BIT(0) > + > +/* VMON_CONF register field definition */ > +#define TPS6594_MASK_VMON1_SLEW_RATE GENMASK(2, 0) > +#define TPS6594_MASK_VMON2_SLEW_RATE GENMASK(5, 3) > + > +/* SOFT_REBOOT_REG register field definition */ > +#define TPS6594_BIT_SOFT_REBOOT BIT(0) > + > +/* RTC_SECONDS & ALARM_SECONDS register field definition */ > +#define TPS6594_MASK_SECOND_0 GENMASK(3, 0) > +#define TPS6594_MASK_SECOND_1 GENMASK(6, 4) > + > +/* RTC_MINUTES & ALARM_MINUTES register field definition */ > +#define TPS6594_MASK_MINUTE_0 GENMASK(3, 0) > +#define TPS6594_MASK_MINUTE_1 GENMASK(6, 4) > + > +/* RTC_HOURS & ALARM_HOURS register field definition */ > +#define TPS6594_MASK_HOUR_0 GENMASK(3, 0) > +#define TPS6594_MASK_HOUR_1 GENMASK(5, 4) > +#define TPS6594_BIT_PM_NAM BIT(7) > + > +/* RTC_DAYS & ALARM_DAYS register field definition */ > +#define TPS6594_MASK_DAY_0 GENMASK(3, 0) > +#define TPS6594_MASK_DAY_1 GENMASK(5, 4) > + > +/* RTC_MONTHS & ALARM_MONTHS register field definition */ > +#define TPS6594_MASK_MONTH_0 GENMASK(3, 0) > +#define TPS6594_BIT_MONTH_1 BIT(4) > + > +/* RTC_YEARS & ALARM_YEARS register field definition */ > +#define TPS6594_MASK_YEAR_0 GENMASK(3, 0) > +#define TPS6594_MASK_YEAR_1 GENMASK(7, 4) > + > +/* RTC_WEEKS register field definition */ > +#define TPS6594_MASK_WEEK GENMASK(2, 0) > + > +/* RTC_CTRL_1 register field definition */ > +#define TPS6594_BIT_STOP_RTC BIT(0) > +#define TPS6594_BIT_ROUND_30S BIT(1) > +#define TPS6594_BIT_AUTO_COMP BIT(2) > +#define TPS6594_BIT_MODE_12_24 BIT(3) > +#define TPS6594_BIT_SET_32_COUNTER BIT(5) > +#define TPS6594_BIT_GET_TIME BIT(6) > +#define TPS6594_BIT_RTC_V_OPT BIT(7) > + > +/* RTC_CTRL_2 register field definition */ > +#define TPS6594_BIT_XTAL_EN BIT(0) > +#define TPS6594_MASK_XTAL_SEL GENMASK(2, 1) > +#define TPS6594_BIT_LP_STANDBY_SEL BIT(3) > +#define TPS6594_BIT_FAST_BIST BIT(4) > +#define TPS6594_MASK_STARTUP_DEST GENMASK(6, 5) > +#define TPS6594_BIT_FIRST_STARTUP_DONE BIT(7) > + > +/* RTC_STATUS register field definition */ > +#define TPS6594_BIT_RUN BIT(1) > +#define TPS6594_BIT_TIMER BIT(5) > +#define TPS6594_BIT_ALARM BIT(6) > +#define TPS6594_BIT_POWER_UP BIT(7) > + > +/* RTC_INTERRUPTS register field definition */ > +#define TPS6594_MASK_EVERY GENMASK(1, 0) > +#define TPS6594_BIT_IT_TIMER BIT(2) > +#define TPS6594_BIT_IT_ALARM BIT(3) > + > +/* RTC_RESET_STATUS register field definition */ > +#define TPS6594_BIT_RESET_STATUS_RTC BIT(0) > + > +/* SERIAL_IF_CONFIG register field definition */ > +#define TPS6594_BIT_I2C_SPI_SEL BIT(0) > +#define TPS6594_BIT_I2C1_SPI_CRC_EN BIT(1) > +#define TPS6594_BIT_I2C2_CRC_EN BIT(2) > +#define TPS6594_MASK_T_CRC GENMASK(7, 3) > + > +/* WD_QUESTION_ANSW_CNT register field definition */ > +#define TPS6594_MASK_WD_QUESTION GENMASK(3, 0) > +#define TPS6594_MASK_WD_ANSW_CNT GENMASK(5, 4) > + > +/* WD_MODE_REG register field definition */ > +#define TPS6594_BIT_WD_RETURN_LONGWIN BIT(0) > +#define TPS6594_BIT_WD_MODE_SELECT BIT(1) > +#define TPS6594_BIT_WD_PWRHOLD BIT(2) > + > +/* WD_QA_CFG register field definition */ > +#define TPS6594_MASK_WD_QUESTION_SEED GENMASK(3, 0) > +#define TPS6594_MASK_WD_QA_LFSR GENMASK(5, 4) > +#define TPS6594_MASK_WD_QA_FDBK GENMASK(7, 6) > + > +/* WD_ERR_STATUS register field definition */ > +#define TPS6594_BIT_WD_LONGWIN_TIMEOUT_INT BIT(0) > +#define TPS6594_BIT_WD_TIMEOUT BIT(1) > +#define TPS6594_BIT_WD_TRIG_EARLY BIT(2) > +#define TPS6594_BIT_WD_ANSW_EARLY BIT(3) > +#define TPS6594_BIT_WD_SEQ_ERR BIT(4) > +#define TPS6594_BIT_WD_ANSW_ERR BIT(5) > +#define TPS6594_BIT_WD_FAIL_INT BIT(6) > +#define TPS6594_BIT_WD_RST_INT BIT(7) > + > +/* WD_THR_CFG register field definition */ > +#define TPS6594_MASK_WD_RST_TH GENMASK(2, 0) > +#define TPS6594_MASK_WD_FAIL_TH GENMASK(5, 3) > +#define TPS6594_BIT_WD_EN BIT(6) > +#define TPS6594_BIT_WD_RST_EN BIT(7) > + > +/* WD_FAIL_CNT_REG register field definition */ > +#define TPS6594_MASK_WD_FAIL_CNT GENMASK(3, 0) > +#define TPS6594_BIT_WD_FIRST_OK BIT(5) > +#define TPS6594_BIT_WD_BAD_EVENT BIT(6) > + > +/* CRC8 polynomial for I2C & SPI protocols */ > +#define TPS6594_CRC8_POLYNOMIAL 0x07 > + > +/* IRQs */ > +enum tps6594_irqs { > + /* INT_BUCK1_2 register */ > + TPS6594_IRQ_BUCK1_OV, > + TPS6594_IRQ_BUCK1_UV, > + TPS6594_IRQ_BUCK1_SC, > + TPS6594_IRQ_BUCK1_ILIM, > + TPS6594_IRQ_BUCK2_OV, > + TPS6594_IRQ_BUCK2_UV, > + TPS6594_IRQ_BUCK2_SC, > + TPS6594_IRQ_BUCK2_ILIM, > + /* INT_BUCK3_4 register */ > + TPS6594_IRQ_BUCK3_OV, > + TPS6594_IRQ_BUCK3_UV, > + TPS6594_IRQ_BUCK3_SC, > + TPS6594_IRQ_BUCK3_ILIM, > + TPS6594_IRQ_BUCK4_OV, > + TPS6594_IRQ_BUCK4_UV, > + TPS6594_IRQ_BUCK4_SC, > + TPS6594_IRQ_BUCK4_ILIM, > + /* INT_BUCK5 register */ > + TPS6594_IRQ_BUCK5_OV, > + TPS6594_IRQ_BUCK5_UV, > + TPS6594_IRQ_BUCK5_SC, > + TPS6594_IRQ_BUCK5_ILIM, > + /* INT_LDO1_2 register */ > + TPS6594_IRQ_LDO1_OV, > + TPS6594_IRQ_LDO1_UV, > + TPS6594_IRQ_LDO1_SC, > + TPS6594_IRQ_LDO1_ILIM, > + TPS6594_IRQ_LDO2_OV, > + TPS6594_IRQ_LDO2_UV, > + TPS6594_IRQ_LDO2_SC, > + TPS6594_IRQ_LDO2_ILIM, > + /* INT_LDO3_4 register */ > + TPS6594_IRQ_LDO3_OV, > + TPS6594_IRQ_LDO3_UV, > + TPS6594_IRQ_LDO3_SC, > + TPS6594_IRQ_LDO3_ILIM, > + TPS6594_IRQ_LDO4_OV, > + TPS6594_IRQ_LDO4_UV, > + TPS6594_IRQ_LDO4_SC, > + TPS6594_IRQ_LDO4_ILIM, > + /* INT_VMON register */ > + TPS6594_IRQ_VCCA_OV, > + TPS6594_IRQ_VCCA_UV, > + TPS6594_IRQ_VMON1_OV, > + TPS6594_IRQ_VMON1_UV, > + TPS6594_IRQ_VMON1_RV, > + TPS6594_IRQ_VMON2_OV, > + TPS6594_IRQ_VMON2_UV, > + TPS6594_IRQ_VMON2_RV, > + /* INT_GPIO register */ > + TPS6594_IRQ_GPIO9, > + TPS6594_IRQ_GPIO10, > + TPS6594_IRQ_GPIO11, > + /* INT_GPIO1_8 register */ > + TPS6594_IRQ_GPIO1, > + TPS6594_IRQ_GPIO2, > + TPS6594_IRQ_GPIO3, > + TPS6594_IRQ_GPIO4, > + TPS6594_IRQ_GPIO5, > + TPS6594_IRQ_GPIO6, > + TPS6594_IRQ_GPIO7, > + TPS6594_IRQ_GPIO8, > + /* INT_STARTUP register */ > + TPS6594_IRQ_NPWRON_START, > + TPS6594_IRQ_ENABLE, > + TPS6594_IRQ_FSD, > + TPS6594_IRQ_SOFT_REBOOT, > + /* INT_MISC register */ > + TPS6594_IRQ_BIST_PASS, > + TPS6594_IRQ_EXT_CLK, > + TPS6594_IRQ_TWARN, > + /* INT_MODERATE_ERR register */ > + TPS6594_IRQ_TSD_ORD, > + TPS6594_IRQ_BIST_FAIL, > + TPS6594_IRQ_REG_CRC_ERR, > + TPS6594_IRQ_RECOV_CNT, > + TPS6594_IRQ_SPMI_ERR, > + TPS6594_IRQ_NPWRON_LONG, > + TPS6594_IRQ_NINT_READBACK, > + TPS6594_IRQ_NRSTOUT_READBACK, > + /* INT_SEVERE_ERR register */ > + TPS6594_IRQ_TSD_IMM, > + TPS6594_IRQ_VCCA_OVP, > + TPS6594_IRQ_PFSM_ERR, > + /* INT_FSM_ERR register */ > + TPS6594_IRQ_IMM_SHUTDOWN, > + TPS6594_IRQ_ORD_SHUTDOWN, > + TPS6594_IRQ_MCU_PWR_ERR, > + TPS6594_IRQ_SOC_PWR_ERR, > + /* INT_COMM_ERR register */ > + TPS6594_IRQ_COMM_FRM_ERR, > + TPS6594_IRQ_COMM_CRC_ERR, > + TPS6594_IRQ_COMM_ADR_ERR, > + TPS6594_IRQ_I2C2_CRC_ERR, > + TPS6594_IRQ_I2C2_ADR_ERR, > + /* INT_READBACK_ERR register */ > + TPS6594_IRQ_EN_DRV_READBACK, > + TPS6594_IRQ_NRSTOUT_SOC_READBACK, > + /* INT_ESM register */ > + TPS6594_IRQ_ESM_SOC_PIN, > + TPS6594_IRQ_ESM_SOC_FAIL, > + TPS6594_IRQ_ESM_SOC_RST, > + /* RTC_STATUS register */ > + TPS6594_IRQ_TIMER, > + TPS6594_IRQ_ALARM, > + TPS6594_IRQ_POWER_UP, > +}; > + > +#define TPS6594_IRQ_NAME_BUCK1_OV "buck1_ov" > +#define TPS6594_IRQ_NAME_BUCK1_UV "buck1_uv" > +#define TPS6594_IRQ_NAME_BUCK1_SC "buck1_sc" > +#define TPS6594_IRQ_NAME_BUCK1_ILIM "buck1_ilim" > +#define TPS6594_IRQ_NAME_BUCK2_OV "buck2_ov" > +#define TPS6594_IRQ_NAME_BUCK2_UV "buck2_uv" > +#define TPS6594_IRQ_NAME_BUCK2_SC "buck2_sc" > +#define TPS6594_IRQ_NAME_BUCK2_ILIM "buck2_ilim" > +#define TPS6594_IRQ_NAME_BUCK3_OV "buck3_ov" > +#define TPS6594_IRQ_NAME_BUCK3_UV "buck3_uv" > +#define TPS6594_IRQ_NAME_BUCK3_SC "buck3_sc" > +#define TPS6594_IRQ_NAME_BUCK3_ILIM "buck3_ilim" > +#define TPS6594_IRQ_NAME_BUCK4_OV "buck4_ov" > +#define TPS6594_IRQ_NAME_BUCK4_UV "buck4_uv" > +#define TPS6594_IRQ_NAME_BUCK4_SC "buck4_sc" > +#define TPS6594_IRQ_NAME_BUCK4_ILIM "buck4_ilim" > +#define TPS6594_IRQ_NAME_BUCK5_OV "buck5_ov" > +#define TPS6594_IRQ_NAME_BUCK5_UV "buck5_uv" > +#define TPS6594_IRQ_NAME_BUCK5_SC "buck5_sc" > +#define TPS6594_IRQ_NAME_BUCK5_ILIM "buck5_ilim" > +#define TPS6594_IRQ_NAME_LDO1_OV "ldo1_ov" > +#define TPS6594_IRQ_NAME_LDO1_UV "ldo1_uv" > +#define TPS6594_IRQ_NAME_LDO1_SC "ldo1_sc" > +#define TPS6594_IRQ_NAME_LDO1_ILIM "ldo1_ilim" > +#define TPS6594_IRQ_NAME_LDO2_OV "ldo2_ov" > +#define TPS6594_IRQ_NAME_LDO2_UV "ldo2_uv" > +#define TPS6594_IRQ_NAME_LDO2_SC "ldo2_sc" > +#define TPS6594_IRQ_NAME_LDO2_ILIM "ldo2_ilim" > +#define TPS6594_IRQ_NAME_LDO3_OV "ldo3_ov" > +#define TPS6594_IRQ_NAME_LDO3_UV "ldo3_uv" > +#define TPS6594_IRQ_NAME_LDO3_SC "ldo3_sc" > +#define TPS6594_IRQ_NAME_LDO3_ILIM "ldo3_ilim" > +#define TPS6594_IRQ_NAME_LDO4_OV "ldo4_ov" > +#define TPS6594_IRQ_NAME_LDO4_UV "ldo4_uv" > +#define TPS6594_IRQ_NAME_LDO4_SC "ldo4_sc" > +#define TPS6594_IRQ_NAME_LDO4_ILIM "ldo4_ilim" > +#define TPS6594_IRQ_NAME_VCCA_OV "vcca_ov" > +#define TPS6594_IRQ_NAME_VCCA_UV "vcca_uv" > +#define TPS6594_IRQ_NAME_VMON1_OV "vmon1_ov" > +#define TPS6594_IRQ_NAME_VMON1_UV "vmon1_uv" > +#define TPS6594_IRQ_NAME_VMON1_RV "vmon1_rv" > +#define TPS6594_IRQ_NAME_VMON2_OV "vmon2_ov" > +#define TPS6594_IRQ_NAME_VMON2_UV "vmon2_uv" > +#define TPS6594_IRQ_NAME_VMON2_RV "vmon2_rv" > +#define TPS6594_IRQ_NAME_GPIO9 "gpio9" > +#define TPS6594_IRQ_NAME_GPIO10 "gpio10" > +#define TPS6594_IRQ_NAME_GPIO11 "gpio11" > +#define TPS6594_IRQ_NAME_GPIO1 "gpio1" > +#define TPS6594_IRQ_NAME_GPIO2 "gpio2" > +#define TPS6594_IRQ_NAME_GPIO3 "gpio3" > +#define TPS6594_IRQ_NAME_GPIO4 "gpio4" > +#define TPS6594_IRQ_NAME_GPIO5 "gpio5" > +#define TPS6594_IRQ_NAME_GPIO6 "gpio6" > +#define TPS6594_IRQ_NAME_GPIO7 "gpio7" > +#define TPS6594_IRQ_NAME_GPIO8 "gpio8" > +#define TPS6594_IRQ_NAME_NPWRON_START "npwron_start" > +#define TPS6594_IRQ_NAME_ENABLE "enable" > +#define TPS6594_IRQ_NAME_FSD "fsd" > +#define TPS6594_IRQ_NAME_SOFT_REBOOT "soft_reboot" > +#define TPS6594_IRQ_NAME_BIST_PASS "bist_pass" > +#define TPS6594_IRQ_NAME_EXT_CLK "ext_clk" > +#define TPS6594_IRQ_NAME_TWARN "twarn" > +#define TPS6594_IRQ_NAME_TSD_ORD "tsd_ord" > +#define TPS6594_IRQ_NAME_BIST_FAIL "bist_fail" > +#define TPS6594_IRQ_NAME_REG_CRC_ERR "reg_crc_err" > +#define TPS6594_IRQ_NAME_RECOV_CNT "recov_cnt" > +#define TPS6594_IRQ_NAME_SPMI_ERR "spmi_err" > +#define TPS6594_IRQ_NAME_NPWRON_LONG "npwron_long" > +#define TPS6594_IRQ_NAME_NINT_READBACK "nint_readback" > +#define TPS6594_IRQ_NAME_NRSTOUT_READBACK "nrstout_readback" > +#define TPS6594_IRQ_NAME_TSD_IMM "tsd_imm" > +#define TPS6594_IRQ_NAME_VCCA_OVP "vcca_ovp" > +#define TPS6594_IRQ_NAME_PFSM_ERR "pfsm_err" > +#define TPS6594_IRQ_NAME_IMM_SHUTDOWN "imm_shutdown" > +#define TPS6594_IRQ_NAME_ORD_SHUTDOWN "ord_shutdown" > +#define TPS6594_IRQ_NAME_MCU_PWR_ERR "mcu_pwr_err" > +#define TPS6594_IRQ_NAME_SOC_PWR_ERR "soc_pwr_err" > +#define TPS6594_IRQ_NAME_COMM_FRM_ERR "comm_frm_err" > +#define TPS6594_IRQ_NAME_COMM_CRC_ERR "comm_crc_err" > +#define TPS6594_IRQ_NAME_COMM_ADR_ERR "comm_adr_err" > +#define TPS6594_IRQ_NAME_EN_DRV_READBACK "en_drv_readback" > +#define TPS6594_IRQ_NAME_NRSTOUT_SOC_READBACK "nrstout_soc_readback" > +#define TPS6594_IRQ_NAME_ESM_SOC_PIN "esm_soc_pin" > +#define TPS6594_IRQ_NAME_ESM_SOC_FAIL "esm_soc_fail" > +#define TPS6594_IRQ_NAME_ESM_SOC_RST "esm_soc_rst" > +#define TPS6594_IRQ_NAME_TIMER "timer" > +#define TPS6594_IRQ_NAME_ALARM "alarm" > +#define TPS6594_IRQ_NAME_POWERUP "powerup" > + > +/** > + * struct tps6594 - device private data structure > + * > + * @dev: MFD device "Parent device" > + * @chip_id: chip ID > + * @reg: I2C slave address or SPI chip select number > + * @use_crc: if true, use CRC for I2C and SPI interface protocols > + * @regmap: regmap for accessing the device registers > + * @irq: irq generated by the device > + * @irq_data: regmap irq data used for the irq chip > + */ > +struct tps6594 { > + struct device *dev; > + unsigned long chip_id; > + unsigned short reg; > + bool use_crc; > + struct regmap *regmap; > + int irq; > + struct regmap_irq_chip_data *irq_data; > +}; > + > +bool tps6594_is_volatile_reg(struct device *dev, unsigned int reg); > +int tps6594_device_init(struct tps6594 *tps); > + > +#endif /* __LINUX_MFD_TPS6594_H */ > -- > 2.37.3 > -- Lee Jones [李琼斯]