Re: [PATCH 1/3] hwmon/ir38064: Add support for IR38060, IR38164 IR38263

[Guenter Roeck <linux@xxxxxxxxxxxx>]

On 12/13/21 2:19 AM, arthur@xxxxxxxxxxxx wrote:
> From: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx>

Description goes here. the same applies to the other patches in the series.

Guenter

> Signed-off-by: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx>
> Signed-off-by: Arthur Heymans <arthur.heymans@xxxxxxxxxxxxx>
> ---
>   .../devicetree/bindings/trivial-devices.yaml  |  6 ++++
>   Documentation/hwmon/ir38064.rst               | 28 +++++++++++++++++--
>   drivers/hwmon/pmbus/Kconfig                   |  4 +--
>   drivers/hwmon/pmbus/ir38064.c                 |  5 +++-
>   4 files changed, 38 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/trivial-devices.yaml b/Documentation/devicetree/bindings/trivial-devices.yaml
> index 791079021f1b..38c3264b3dcb 100644
> --- a/Documentation/devicetree/bindings/trivial-devices.yaml
> +++ b/Documentation/devicetree/bindings/trivial-devices.yaml
> @@ -121,8 +121,14 @@ properties:
>             - ibm,cffps2
>               # Infineon IR36021 digital POL buck controller
>             - infineon,ir36021
> +          # Infineon IR38060 Voltage Regulator
> +          - infineon,ir38060
>               # Infineon IR38064 Voltage Regulator
>             - infineon,ir38064
> +          # Infineon IR38164 Voltage Regulator
> +          - infineon,ir38164
> +          # Infineon IR38263 Voltage Regulator
> +          - infineon,ir38263
>               # Infineon SLB9635 (Soft-) I2C TPM (old protocol, max 100khz)
>             - infineon,slb9635tt
>               # Infineon SLB9645 I2C TPM (new protocol, max 400khz)
> diff --git a/Documentation/hwmon/ir38064.rst b/Documentation/hwmon/ir38064.rst
> index c455d755a267..e1148f21ea2a 100644
> --- a/Documentation/hwmon/ir38064.rst
> +++ b/Documentation/hwmon/ir38064.rst
> @@ -3,14 +3,38 @@ Kernel driver ir38064
>   
>   Supported chips:
>   
> +  * Infineon IR38060
> +
> +    Prefix: 'IR38060'
> +    Addresses scanned: -
> +
> +    Datasheet: Publicly available at the Infineon website
> +      https://www.infineon.com/dgdl/Infineon-IR38060M-DS-v03_16-EN.pdf?fileId=5546d4625c167129015c3291ea9a4cee
> +
>     * Infineon IR38064
>   
>       Prefix: 'ir38064'
>       Addresses scanned: -
>   
> -    Datasheet: Publicly available at the Infineon webiste
> +    Datasheet: Publicly available at the Infineon website
>         https://www.infineon.com/dgdl/Infineon-IR38064MTRPBF-DS-v03_07-EN.pdf?fileId=5546d462584d1d4a0158db0d9efb67ca
>   
> +  * Infineon IR38164
> +
> +    Prefix: 'ir38164'
> +    Addresses scanned: -
> +
> +    Datasheet: Publicly available at the Infineon website
> +      https://www.infineon.com/dgdl/Infineon-IR38164M-DS-v02_02-EN.pdf?fileId=5546d462636cc8fb01640046efea1248
> +
> +  * Infineon ir38263
> +
> +    Prefix: 'ir38263'
> +    Addresses scanned: -
> +
> +    Datasheet:  Publicly available at the Infineon website
> +      https://www.infineon.com/dgdl/Infineon-IR38263M-DataSheet-v03_05-EN.pdf?fileId=5546d4625b62cd8a015bcf81f90a6e52
> +
>   Authors:
>         - Maxim Sloyko <maxims@xxxxxxxxxx>
>         - Patrick Venture <venture@xxxxxxxxxx>
> @@ -18,7 +42,7 @@ Authors:
>   Description
>   -----------
>   
> -IR38064 is a Single-input Voltage, Synchronous Buck Regulator, DC-DC Converter.
> +IR38x6x are a Single-input Voltage, Synchronous Buck Regulator, DC-DC Converter.
>   
>   Usage Notes
>   -----------
> diff --git a/drivers/hwmon/pmbus/Kconfig b/drivers/hwmon/pmbus/Kconfig
> index ffb609cee3a4..d2b178a902f0 100644
> --- a/drivers/hwmon/pmbus/Kconfig
> +++ b/drivers/hwmon/pmbus/Kconfig
> @@ -123,10 +123,10 @@ config SENSORS_IR36021
>   	  be called ir36021.
>   
>   config SENSORS_IR38064
> -	tristate "Infineon IR38064"
> +	tristate "Infineon IR38064 and compatibles"
>   	help
>   	  If you say yes here you get hardware monitoring support for Infineon
> -	  IR38064.
> +	  IR38060, IR38064, IR38164 and IR38263.
>   
>   	  This driver can also be built as a module. If so, the module will
>   	  be called ir38064.
> diff --git a/drivers/hwmon/pmbus/ir38064.c b/drivers/hwmon/pmbus/ir38064.c
> index 1fb7f1248639..4e91d3e54a4a 100644
> --- a/drivers/hwmon/pmbus/ir38064.c
> +++ b/drivers/hwmon/pmbus/ir38064.c
> @@ -41,7 +41,10 @@ static int ir38064_probe(struct i2c_client *client)
>   }
>   
>   static const struct i2c_device_id ir38064_id[] = {
> +	{"ir38060", 0},
>   	{"ir38064", 0},
> +	{"ir38164", 0},
> +	{"ir38263", 0},
>   	{}
>   };
>   
> @@ -59,6 +62,6 @@ static struct i2c_driver ir38064_driver = {
>   module_i2c_driver(ir38064_driver);
>   
>   MODULE_AUTHOR("Maxim Sloyko <maxims@xxxxxxxxxx>");
> -MODULE_DESCRIPTION("PMBus driver for Infineon IR38064");
> +MODULE_DESCRIPTION("PMBus driver for Infineon IR38064 and comaptible chips");
>   MODULE_LICENSE("GPL");
>   MODULE_IMPORT_NS(PMBUS);