Re: [PATCH v5 3/3] hwmon: pmbus: add MPQ8646 driver

From: Vincent Jardin

Date: Fri Jul 24 2026 - 06:24:43 EST


Hi Guenter,

> > + for (i = 0; i < ARRAY_SIZE(mpq8646_alarm_map); i++) {
> > + if (newly_set & mpq8646_alarm_map[i].mask)
> > + hwmon_notify_event(priv->hwmon_dev,
> > + mpq8646_alarm_map[i].type,
> > + mpq8646_alarm_map[i].attr,
> > + mpq8646_alarm_map[i].channel);
>
> This is wrong. This will require a new exported notification function
> in pmbus code.

OK, next, I'll try to factor the body of pmbus_fault_handler() into
an exported helper, something like pmbus_check_and_notify_faults(client).

It should help to revisit it.

Best regards,
Vincent