On Tue, Jul 25, 2023 at 02:54:25PM +0200, Naresh Solanki wrote: > From: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx> > > Refactor the pmbus_is_enabled() function to return the raw status > without any additional processing as its already done in > _pmbus_is_enabled function. > > Signed-off-by: Patrick Rudolph <patrick.rudolph@xxxxxxxxxxxxx> > Signed-off-by: Naresh Solanki <Naresh.Solanki@xxxxxxxxxxxxx> Applied. I rephrased the commit message so don't be surprised that it looks different. Thanks, Guenter > --- > drivers/hwmon/pmbus/pmbus_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > > base-commit: 55612007f16b5d7b1fb83a7b0f5bb686829db7c7 > > diff --git a/drivers/hwmon/pmbus/pmbus_core.c b/drivers/hwmon/pmbus/pmbus_core.c > index fa06325f5a7c..42fb7286805b 100644 > --- a/drivers/hwmon/pmbus/pmbus_core.c > +++ b/drivers/hwmon/pmbus/pmbus_core.c > @@ -2768,7 +2768,7 @@ static int __maybe_unused pmbus_is_enabled(struct device *dev, u8 page) > ret = _pmbus_is_enabled(dev, page); > mutex_unlock(&data->update_lock); > > - return !!(ret & PB_OPERATION_CONTROL_ON); > + return ret; > } > > #define to_dev_attr(_dev_attr) \
