Thread (77 messages) 77 messages, 4 authors, 2021-09-19

Re: [PATCH v2 33/46] mfd: ti_am335x_tscadc: Move control register configuration

From: Grygorii Strashko <grygorii.strashko@ti.com>
Date: 2021-09-03 13:24:54
Also in: linux-devicetree, linux-iio, linux-omap


On 03/09/2021 00:51, Miquel Raynal wrote:
quoted hunk ↗ jump to hunk
The datasheet states that most of the configuration should be set in the
control register in the first place, before actually enabling the
hardware. So far only half of the configuration was made in the first
step, which does not make really sense and would complicating the code
when introducing support for the am437x hardware.

Let's move that register write a bit below to enclose more configuration.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
  drivers/mfd/ti_am335x_tscadc.c | 2 ++
  1 file changed, 2 insertions(+)
diff --git a/drivers/mfd/ti_am335x_tscadc.c b/drivers/mfd/ti_am335x_tscadc.c
index 29ada9da8826..a0db3e4ff265 100644
--- a/drivers/mfd/ti_am335x_tscadc.c
+++ b/drivers/mfd/ti_am335x_tscadc.c
@@ -239,6 +239,8 @@ static	int ti_tscadc_probe(struct platform_device *pdev)
  	}
  	regmap_write(tscadc->regmap, REG_CTRL, tscadc->ctrl);
  
+	regmap_write(tscadc->regmap, REG_CTRL, tscadc->ctrl);
+
Strange change - above 2 lines a the same !?
  	tscadc_idle_config(tscadc);
  
  	/* Enable the TSC module enable bit */
-- 
Best regards,
grygorii
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help