Re: [PATCH v3 2/2] hwmon: (pmbus/isl68137): Add Renesas RAA229639 and RAA229640

From: Krzysztof Kozlowski

Date: Fri Sep 25 2026 - 09:07:24 EST


On Mon, Sep 21, 2026 at 03:16:36PM +0800, Colin Huang wrote:
> Add support for the Renesas RAA229639 and RAA229640 PMBus
> voltage regulator devices.
>
> Both devices are supported by the existing raa_dmpvr2_2rail_nontc
> driver variant, so no functional changes are required beyond
> adding the corresponding device IDs and OF match entries.
>
> Signed-off-by: Colin Huang <u8813345@xxxxxxxxx>
> ---
> drivers/hwmon/pmbus/isl68137.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/hwmon/pmbus/isl68137.c b/drivers/hwmon/pmbus/isl68137.c
> index 2f7f825bfb69..15b4e3f15f11 100644
> --- a/drivers/hwmon/pmbus/isl68137.c
> +++ b/drivers/hwmon/pmbus/isl68137.c
> @@ -456,6 +456,8 @@ static const struct i2c_device_id raa_dmpvr_id[] = {
> { .name = "raa229004", .driver_data = raa_dmpvr2_2rail },
> { .name = "raa229141", .driver_data = raa_dmpvr2_2rail_pmbus },
> { .name = "raa229621", .driver_data = raa_dmpvr2_2rail },
> + { .name = "raa229639", .driver_data = raa_dmpvr2_2rail_nontc },
> + { .name = "raa229640", .driver_data = raa_dmpvr2_2rail_nontc },

The entire point of fallback is to avoid this change. Why do you need
this?

Best regards,
Krzysztof