Re: [PATCH v2] hwmon: (adm1275) Enable adm1278 VOUT sampling
On Tue, Oct 18, 2016 at 3:50 AM, Guenter Roeckwrote: > On Mon, Oct 17, 2016 at 06:38:53PM +1030, Joel Stanley wrote: >> From: Yi Li >> >> The adm1278 can optionally monitor the VOUT pin. This functioanltiy is >> not enabled at reset, so PMON_CONFIG needs to be modified in order to >> enable it. >> >> Signed-off-by: Yi Li >> Signed-off-by: Joel Stanley > > Applied to -next, after s/functioanltiy/functionality/ Thank you! And thanks for fixing the grammar and spelling. The author is not a native speaker, but I could have done a better job at checking. Cheers, Joel > > Thanks, > Guenter > >> --- >> v2: >> Return an error when unable to enable VOUT monitoring as suggestd by Guenter >> >> drivers/hwmon/pmbus/adm1275.c | 20 >> 1 file changed, 16 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/hwmon/pmbus/adm1275.c b/drivers/hwmon/pmbus/adm1275.c >> index 3baa4f4a8c5e..9940849351a1 100644 >> --- a/drivers/hwmon/pmbus/adm1275.c >> +++ b/drivers/hwmon/pmbus/adm1275.c >> @@ -499,15 +499,27 @@ static int adm1275_probe(struct i2c_client *client, >> pindex = 2; >> tindex = 3; >> >> - info->func[0] |= PMBUS_HAVE_PIN | PMBUS_HAVE_STATUS_INPUT; >> + info->func[0] |= PMBUS_HAVE_PIN | PMBUS_HAVE_STATUS_INPUT | >> + PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT; >> + >> + /* By default when reset VOUT is not enabled */ > > I also rephrased the comment a bit to make it easier to understand. > >> + if (!(config & ADM1278_VOUT_EN)) { >> + config |= ADM1278_VOUT_EN; >> + ret = i2c_smbus_write_byte_data(client, >> + ADM1275_PMON_CONFIG, >> + config); >> + if (ret < 0) { >> + dev_err(>dev, >> + "Failed to enable VOUT monitoring\n"); >> + return -ENODEV; >> + } >> + } >> + >> if (config & ADM1278_TEMP1_EN) >> info->func[0] |= >> PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP; >> if (config & ADM1278_VIN_EN) >> info->func[0] |= PMBUS_HAVE_VIN; >> - if (config & ADM1278_VOUT_EN) >> - info->func[0] |= >> - PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT; >> break; >> case adm1293: >> case adm1294: >> -- >> 2.9.3 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in >> the body of a message to majord...@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2] hwmon: (adm1275) Enable adm1278 VOUT sampling
On Mon, Oct 17, 2016 at 06:38:53PM +1030, Joel Stanley wrote: > From: Yi Li> > The adm1278 can optionally monitor the VOUT pin. This functioanltiy is > not enabled at reset, so PMON_CONFIG needs to be modified in order to > enable it. > > Signed-off-by: Yi Li > Signed-off-by: Joel Stanley Applied to -next, after s/functioanltiy/functionality/ Thanks, Guenter > --- > v2: > Return an error when unable to enable VOUT monitoring as suggestd by Guenter > > drivers/hwmon/pmbus/adm1275.c | 20 > 1 file changed, 16 insertions(+), 4 deletions(-) > > diff --git a/drivers/hwmon/pmbus/adm1275.c b/drivers/hwmon/pmbus/adm1275.c > index 3baa4f4a8c5e..9940849351a1 100644 > --- a/drivers/hwmon/pmbus/adm1275.c > +++ b/drivers/hwmon/pmbus/adm1275.c > @@ -499,15 +499,27 @@ static int adm1275_probe(struct i2c_client *client, > pindex = 2; > tindex = 3; > > - info->func[0] |= PMBUS_HAVE_PIN | PMBUS_HAVE_STATUS_INPUT; > + info->func[0] |= PMBUS_HAVE_PIN | PMBUS_HAVE_STATUS_INPUT | > + PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT; > + > + /* By default when reset VOUT is not enabled */ I also rephrased the comment a bit to make it easier to understand. > + if (!(config & ADM1278_VOUT_EN)) { > + config |= ADM1278_VOUT_EN; > + ret = i2c_smbus_write_byte_data(client, > + ADM1275_PMON_CONFIG, > + config); > + if (ret < 0) { > + dev_err(>dev, > + "Failed to enable VOUT monitoring\n"); > + return -ENODEV; > + } > + } > + > if (config & ADM1278_TEMP1_EN) > info->func[0] |= > PMBUS_HAVE_TEMP | PMBUS_HAVE_STATUS_TEMP; > if (config & ADM1278_VIN_EN) > info->func[0] |= PMBUS_HAVE_VIN; > - if (config & ADM1278_VOUT_EN) > - info->func[0] |= > - PMBUS_HAVE_VOUT | PMBUS_HAVE_STATUS_VOUT; > break; > case adm1293: > case adm1294: > -- > 2.9.3 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html