Commit a0e3573e authored by Trevor Gamblin's avatar Trevor Gamblin Committed by Jonathan Cameron

iio: adc: bcm_iproc_adc: make use of regmap_clear_bits()

Instead of using regmap_update_bits() and passing val = 0, use
regmap_clear_bits().
Suggested-by: default avatarUwe Kleine-König <u.kleine-koenig@baylibre.com>
Signed-off-by: default avatarTrevor Gamblin <tgamblin@baylibre.com>
Acked-by: default avatarUwe Kleine-König <u.kleine-koenig@baylibre.com>
Link: https://patch.msgid.link/20240617-review-v3-7-88d1338c4cca@baylibre.comSigned-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 6e195872
...@@ -357,8 +357,8 @@ static int iproc_adc_enable(struct iio_dev *indio_dev) ...@@ -357,8 +357,8 @@ static int iproc_adc_enable(struct iio_dev *indio_dev)
int ret; int ret;
/* Set i_amux = 3b'000, select channel 0 */ /* Set i_amux = 3b'000, select channel 0 */
ret = regmap_update_bits(adc_priv->regmap, IPROC_ANALOG_CONTROL, ret = regmap_clear_bits(adc_priv->regmap, IPROC_ANALOG_CONTROL,
IPROC_ADC_CHANNEL_SEL_MASK, 0); IPROC_ADC_CHANNEL_SEL_MASK);
if (ret) { if (ret) {
dev_err(&indio_dev->dev, dev_err(&indio_dev->dev,
"failed to write IPROC_ANALOG_CONTROL %d\n", ret); "failed to write IPROC_ANALOG_CONTROL %d\n", ret);
...@@ -543,8 +543,8 @@ static int iproc_adc_probe(struct platform_device *pdev) ...@@ -543,8 +543,8 @@ static int iproc_adc_probe(struct platform_device *pdev)
if (adc_priv->irqno < 0) if (adc_priv->irqno < 0)
return adc_priv->irqno; return adc_priv->irqno;
ret = regmap_update_bits(adc_priv->regmap, IPROC_REGCTL2, ret = regmap_clear_bits(adc_priv->regmap, IPROC_REGCTL2,
IPROC_ADC_AUXIN_SCAN_ENA, 0); IPROC_ADC_AUXIN_SCAN_ENA);
if (ret) { if (ret) {
dev_err(&pdev->dev, "failed to write IPROC_REGCTL2 %d\n", ret); dev_err(&pdev->dev, "failed to write IPROC_REGCTL2 %d\n", ret);
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment