[PATCH 0/3] add support for LTC7841 boost controller

[PATCH 0/3] add support for LTC7841 boost controller

[Mariel Tinaco]

This series introduces support for LTC7841 boost controller. 
The LTC7841 is similar to LTC7880, which is already supported
under LTC2978. It has reduced PMBUS registers as that of the LTC7880,
similar voltage range, and has only one channel. 

Mariel Tinaco (3):
  dt-bindings: hwmon: ltc2978: add support for ltc7841
  hwmon: (pmbus/ltc7841) add support for LTC7841 - docs
  hwmon: (pmbus/ltc2978) add support for ltc7841

 .../bindings/hwmon/lltc,ltc2978.yaml          |  2 ++
 Documentation/hwmon/ltc2978.rst               | 12 +++++++++++
 drivers/hwmon/pmbus/Kconfig                   |  6 +++---
 drivers/hwmon/pmbus/ltc2978.c                 | 20 +++++++++++++++++--
 4 files changed, 35 insertions(+), 5 deletions(-)


base-commit: eabb03810194b75417b09cff8a526d26939736ac
-- 
2.34.1

[PATCH 1/3] dt-bindings: hwmon: ltc2978: add support for ltc7841

[Mariel Tinaco]

Add LTC7841 to supported devices of LTC2978. It has similar set of
registers to LTC7880, differing only in number of output channels and
some unimplemented PMBUS status and functionalities.

Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>
---
 Documentation/devicetree/bindings/hwmon/lltc,ltc2978.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/hwmon/lltc,ltc2978.yaml b/Documentation/devicetree/bindings/hwmon/lltc,ltc2978.yaml
index 1f98da32f3fe..37e1dc9c7dd3 100644
--- a/Documentation/devicetree/bindings/hwmon/lltc,ltc2978.yaml
+++ b/Documentation/devicetree/bindings/hwmon/lltc,ltc2978.yaml
@@ -26,6 +26,7 @@ properties:
       - lltc,ltc3886
       - lltc,ltc3887
       - lltc,ltc3889
+      - lltc,ltc7841
       - lltc,ltc7880
       - lltc,ltm2987
       - lltc,ltm4664
@@ -50,6 +51,7 @@ properties:
       * ltc2977, ltc2979, ltc2980, ltm2987 : vout0 - vout7
       * ltc2978 : vout0 - vout7
       * ltc3880, ltc3882, ltc3884, ltc3886, ltc3887, ltc3889 : vout0 - vout1
+      * ltc7841 : vout0
       * ltc7880 : vout0 - vout1
       * ltc3883 : vout0
       * ltm4664 : vout0 - vout1
-- 
2.34.1

Re: [PATCH 1/3] dt-bindings: hwmon: ltc2978: add support for ltc7841

[Rob Herring (Arm)]

On Tue, 29 Oct 2024 09:37:32 +0800, Mariel Tinaco wrote:
> Add LTC7841 to supported devices of LTC2978. It has similar set of
> registers to LTC7880, differing only in number of output channels and
> some unimplemented PMBUS status and functionalities.
> 
> Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>
> ---
>  Documentation/devicetree/bindings/hwmon/lltc,ltc2978.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>

Re: [PATCH 1/3] dt-bindings: hwmon: ltc2978: add support for ltc7841

[Guenter Roeck]

On Tue, Oct 29, 2024 at 09:37:32AM +0800, Mariel Tinaco wrote:
> Add LTC7841 to supported devices of LTC2978. It has similar set of
> registers to LTC7880, differing only in number of output channels and
> some unimplemented PMBUS status and functionalities.
> 
> Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>

Applied.

Thanks,
Guenter

[PATCH 2/3] hwmon: (pmbus/ltc7841) add support for LTC7841 - docs

[Mariel Tinaco]

Add LTC7841 to compatible devices of LTC2978

Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>
---
 Documentation/hwmon/ltc2978.rst | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/Documentation/hwmon/ltc2978.rst b/Documentation/hwmon/ltc2978.rst
index edf24e5e1e11..651ca4904c66 100644
--- a/Documentation/hwmon/ltc2978.rst
+++ b/Documentation/hwmon/ltc2978.rst
@@ -1,3 +1,5 @@
+.. SPDX-License-Identifier: GPL-2.0
+
 Kernel driver ltc2978
 =====================
 
@@ -117,6 +119,14 @@ Supported chips:
 
     Datasheet: https://www.analog.com/en/products/ltc3889
 
+  * Linear Technology LTC7841
+
+    Prefix: 'ltc7841'
+
+    Addresses scanned: -
+
+    Datasheet: https://www.analog.com/en/products/ltc7841
+
   * Linear Technology LTC7880
 
     Prefix: 'ltc7880'
@@ -290,6 +300,7 @@ in[N]_label		"vout[1-8]".
 			  LTC7880, LTM4644, LTM4675, LTM4676, LTM4677, LTM4678,
 			  LTM4680, LTM4700: N=2-3
 			- LTC3883: N=2
+			- LTC7841: N=2
 
 in[N]_input		Measured output voltage.
 
@@ -420,6 +431,7 @@ curr[N]_label		"iout[1-4]".
 			  LTM4664, LTM4675, LTM4676, LTM4677, LTM4678, LTM4680,
 			  LTM4700: N=2-3
 			- LTC3883: N=2
+			- LTC7841: N=2
 
 curr[N]_input		Measured output current.
 
-- 
2.34.1

Re: [PATCH 2/3] hwmon: (pmbus/ltc7841) add support for LTC7841 - docs

[Guenter Roeck]

On Tue, Oct 29, 2024 at 09:37:33AM +0800, Mariel Tinaco wrote:
> Add LTC7841 to compatible devices of LTC2978
> 
> Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>

For my reference:

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Re: [PATCH 2/3] hwmon: (pmbus/ltc7841) add support for LTC7841 - docs

[Guenter Roeck]

On Tue, Oct 29, 2024 at 09:37:33AM +0800, Mariel Tinaco wrote:
> Add LTC7841 to compatible devices of LTC2978
> 
> Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>

Applied.

Thanks,
Guenter

[PATCH 3/3] hwmon: (pmbus/ltc2978) add support for ltc7841

[Mariel Tinaco]

Add support for LTC7841. The LTC7841 is a high performance
PolyPhase® single output synchronous boost converter controller.
Multiphase operation reduces input and output capacitor
requirements and allows the use of smaller inductors than the
single-phase equivalent.

The relevant registers in the LTC7841 are similar to the
LTC7880, only reduced by some amount. So it's just a matter of adding
the chip id. The device also doesn't support polling, on top of the
reduced register set, so a separate case for setting the chip info is
added.

Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>
---
 drivers/hwmon/pmbus/Kconfig   |  6 +++---
 drivers/hwmon/pmbus/ltc2978.c | 20 ++++++++++++++++++--
 2 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
index a4f02cad92fd..f6d352841953 100644
--- a/drivers/hwmon/pmbus/Kconfig
+++ b/drivers/hwmon/pmbus/Kconfig
@@ -224,9 +224,9 @@ config SENSORS_LTC2978_REGULATOR
 	depends on SENSORS_LTC2978 && REGULATOR
 	help
 	  If you say yes here you get regulator support for Linear Technology
-	  LTC3880, LTC3883, LTC3884, LTC3886, LTC3887, LTC3889, LTC7880,
-	  LTM4644, LTM4675, LTM4676, LTM4677, LTM4678, LTM4680, LTM4686,
-	  and LTM4700.
+	  LTC3880, LTC3883, LTC3884, LTC3886, LTC3887, LTC3889, LTC7841,
+	  LTC7880, LTM4644, LTM4675, LTM4676, LTM4677, LTM4678, LTM4680,
+	  LTM4686, and LTM4700.
 
 config SENSORS_LTC3815
 	tristate "Linear Technologies LTC3815"
diff --git a/drivers/hwmon/pmbus/ltc2978.c b/drivers/hwmon/pmbus/ltc2978.c
index 73a86f4d6472..a6eb4d4b5487 100644
--- a/drivers/hwmon/pmbus/ltc2978.c
+++ b/drivers/hwmon/pmbus/ltc2978.c
@@ -23,7 +23,8 @@ enum chips {
 	/* Managers */
 	ltc2972, ltc2974, ltc2975, ltc2977, ltc2978, ltc2979, ltc2980,
 	/* Controllers */
-	ltc3880, ltc3882, ltc3883, ltc3884, ltc3886, ltc3887, ltc3889, ltc7132, ltc7880,
+	ltc3880, ltc3882, ltc3883, ltc3884, ltc3886, ltc3887, ltc3889, ltc7132,
+	ltc7841, ltc7880,
 	/* Modules */
 	ltm2987, ltm4664, ltm4675, ltm4676, ltm4677, ltm4678, ltm4680, ltm4686,
 	ltm4700,
@@ -50,7 +51,7 @@ enum chips {
 #define LTC3880_MFR_CLEAR_PEAKS		0xe3
 #define LTC3880_MFR_TEMPERATURE2_PEAK	0xf4
 
-/* LTC3883, LTC3884, LTC3886, LTC3889, LTC7132, LTC7880 */
+/* LTC3883, LTC3884, LTC3886, LTC3889, LTC7132, LTC7841 and LTC7880 only */
 #define LTC3883_MFR_IIN_PEAK		0xe1
 
 /* LTC2975 only */
@@ -80,6 +81,7 @@ enum chips {
 #define LTC3887_ID			0x4700
 #define LTC3889_ID			0x4900
 #define LTC7132_ID			0x4CE0
+#define LTC7841_ID			0x40D0
 #define LTC7880_ID			0x49E0
 #define LTM2987_ID_A			0x8010	/* A/B for two die IDs */
 #define LTM2987_ID_B			0x8020
@@ -548,6 +550,7 @@ static const struct i2c_device_id ltc2978_id[] = {
 	{"ltc3887", ltc3887},
 	{"ltc3889", ltc3889},
 	{"ltc7132", ltc7132},
+	{"ltc7841", ltc7841},
 	{"ltc7880", ltc7880},
 	{"ltm2987", ltm2987},
 	{"ltm4664", ltm4664},
@@ -654,6 +657,8 @@ static int ltc2978_get_id(struct i2c_client *client)
 		return ltc3889;
 	else if (chip_id == LTC7132_ID)
 		return ltc7132;
+	else if (chip_id == LTC7841_ID)
+		return ltc7841;
 	else if (chip_id == LTC7880_ID)
 		return ltc7880;
 	else if (chip_id == LTM2987_ID_A || chip_id == LTM2987_ID_B)
@@ -854,6 +859,16 @@ static int ltc2978_probe(struct i2c_client *client)
 		  | PMBUS_HAVE_POUT
 		  | PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP;
 		break;
+	case ltc7841:
+		data->features |= FEAT_CLEAR_PEAKS;
+		info->read_word_data = ltc3883_read_word_data;
+		info->pages = LTC3883_NUM_PAGES;
+		info->func[0] = PMBUS_HAVE_VIN | PMBUS_HAVE_IIN
+		  | PMBUS_HAVE_STATUS_INPUT
+		  | PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT
+		  | PMBUS_HAVE_IOUT
+		  | PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP;
+		break;
 	default:
 		return -ENODEV;
 	}
@@ -907,6 +922,7 @@ static const struct of_device_id ltc2978_of_match[] = {
 	{ .compatible = "lltc,ltc3887" },
 	{ .compatible = "lltc,ltc3889" },
 	{ .compatible = "lltc,ltc7132" },
+	{ .compatible = "lltc,ltc7841" },
 	{ .compatible = "lltc,ltc7880" },
 	{ .compatible = "lltc,ltm2987" },
 	{ .compatible = "lltc,ltm4664" },
-- 
2.34.1

Re: [PATCH 3/3] hwmon: (pmbus/ltc2978) add support for ltc7841

[Guenter Roeck]

On Tue, Oct 29, 2024 at 09:37:34AM +0800, Mariel Tinaco wrote:
> Add support for LTC7841. The LTC7841 is a high performance
> PolyPhase® single output synchronous boost converter controller.
> Multiphase operation reduces input and output capacitor
> requirements and allows the use of smaller inductors than the
> single-phase equivalent.
> 
> The relevant registers in the LTC7841 are similar to the
> LTC7880, only reduced by some amount. So it's just a matter of adding
> the chip id. The device also doesn't support polling, on top of the
> reduced register set, so a separate case for setting the chip info is
> added.
> 
> Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>

For my reference:

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Waiting for dt approvals.

Thanks,
Guenter

Re: [PATCH 3/3] hwmon: (pmbus/ltc2978) add support for ltc7841

[Guenter Roeck]

On Tue, Oct 29, 2024 at 09:37:34AM +0800, Mariel Tinaco wrote:
> Add support for LTC7841. The LTC7841 is a high performance
> PolyPhase® single output synchronous boost converter controller.
> Multiphase operation reduces input and output capacitor
> requirements and allows the use of smaller inductors than the
> single-phase equivalent.
> 
> The relevant registers in the LTC7841 are similar to the
> LTC7880, only reduced by some amount. So it's just a matter of adding
> the chip id. The device also doesn't support polling, on top of the
> reduced register set, so a separate case for setting the chip info is
> added.
> 
> Signed-off-by: Mariel Tinaco <Mariel.Tinaco@analog.com>

Applied.

Thanks,
Guenter