mfd: ti_am335x_tscadc: Reword the comment explaining the dividers
authorMiquel Raynal <miquel.raynal@bootlin.com>
Fri, 15 Oct 2021 08:14:31 +0000 (10:14 +0200)
committerLee Jones <lee.jones@linaro.org>
Wed, 20 Oct 2021 16:24:53 +0000 (17:24 +0100)
The comment misses the main information which is that we assume that a
sample takes 15 ADC clock cycles to be generated. Let's take the
occasion to rework a little bit this comment.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Link: https://lore.kernel.org/r/20211015081506.933180-14-miquel.raynal@bootlin.com
drivers/mfd/ti_am335x_tscadc.c

index a211b35..7aeabac 100644 (file)
@@ -199,12 +199,12 @@ static    int ti_tscadc_probe(struct platform_device *pdev)
        pm_runtime_get_sync(&pdev->dev);
 
        /*
-        * The TSC_ADC_Subsystem has 2 clock domains
-        * OCP_CLK and ADC_CLK.
-        * The ADC clock is expected to run at target of 3MHz,
-        * and expected to capture 12-bit data at a rate of 200 KSPS.
-        * The TSC_ADC_SS controller design assumes the OCP clock is
-        * at least 6x faster than the ADC clock.
+        * The TSC_ADC_Subsystem has 2 clock domains: OCP_CLK and ADC_CLK.
+        * ADCs produce a 12-bit sample every 15 ADC_CLK cycles.
+        * am33xx ADCs expect to capture 200ksps.
+        * We need the ADC clocks to run at 3MHz.
+        * This frequency is valid since TSC_ADC_SS controller design
+        * assumes the OCP clock is at least 6x faster than the ADC clock.
         */
        clk = devm_clk_get(&pdev->dev, "adc_tsc_fck");
        if (IS_ERR(clk)) {