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

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

 



On Mon, Dec 13, 2021 at 03:28:12PM +0100, arthur@xxxxxxxxxxxx wrote:
> From: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx>
> 
> The IR38060, IR38164 and IR38263 can be supported using this driver.
> 
> Signed-off-by: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx>
> Signed-off-by: Arthur Heymans <arthur.heymans@xxxxxxxxxxxxx>

Series applied.

Thanks,
Guenter

> ---
>  .../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);



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux