Re: [PATCH v4 25/48] mfd: ti_am335x_tscadc: Use the new HZ_PER_MHZ macro

[PATCH v4 00/48] TI AM437X ADC1 · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 01/48] clk: ti: am43xx: Add clkctrl data for am43xx ADC1 · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 02/48] dt-bindings: mfd: ti,am3359-tscadc: Add a yaml description for this MFD · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 03/48] dt-bindings: touchscreen: ti,am3359-tsc: New yaml description · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 04/48] dt-bindings: iio: adc: ti,am3359-adc: New yaml description · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 05/48] dt-bindings: touchscreen: ti,am3359-tsc: Remove deprecated text file · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 06/48] dt-bindings: mfd: ti,am3359-tscadc: Describe am4372 MFD compatible · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 07/48] dt-bindings: iio: adc: ti,am3359-adc: Describe am4372 ADC compatible · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 08/48] mfd: ti_am335x_tscadc: Ensure a balanced number of node get/put · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 09/48] mfd: ti_am335x_tscadc: Replace license text with SPDX tag · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 10/48] mfd: ti_am335x_tscadc: Fix style · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 11/48] mfd: ti_am335x_tscadc: Drop extra spacing when declaring stack variables · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 12/48] mfd: ti_am335x_tscadc: Get rid of useless gotos · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 13/48] mfd: ti_am335x_tscadc: Reword the comment explaining the dividers · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 14/48] mfd: ti_am335x_tscadc: Don't search the tree for our clock · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 15/48] mfd: ti_am335x_tscadc: Simplify divisor calculation · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 16/48] mfd: ti_am335x_tscadc: Move the driver structure allocation earlier · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 17/48] mfd: ti_am335x_tscadc: Use driver data · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 19/48] mfd: ti_am335x_tscadc: Drop useless variables from the driver structure · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 18/48] mfd: ti_am335x_tscadc: Mimic the probe from resume() · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 20/48] mfd: ti_am335x_tscadc: Always provide an idle configuration · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 21/48] mfd: ti_am335x_tscadc: Reorder the initialization steps · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 23/48] mfd: ti_am335x_tscadc: Replace the header license text with SPDX tag · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 24/48] mfd: ti_am335x_tscadc: Fix header spacing · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 22/48] mfd: ti_am335x_tscadc: Gather the ctrl register logic in one place · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 25/48] mfd: ti_am335x_tscadc: Use the new HZ_PER_MHZ macro · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
Re: [PATCH v4 25/48] mfd: ti_am335x_tscadc: Use the new HZ_PER_MHZ macro · Jonathan Cameron <jic23@kernel.org> · 2021-09-29
Re: [PATCH v4 25/48] mfd: ti_am335x_tscadc: Use the new HZ_PER_MHZ macro · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-10-01
[PATCH v4 27/48] mfd: ti_am335x_tscadc: Use BIT(), GENMASK() and FIELD_PREP() when relevant · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 26/48] mfd: ti_am335x_tscadc: Drop unused definitions from the header · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 28/48] mfd: ti_am335x_tscadc: Clarify the maximum values for DT entries · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 29/48] mfd: ti_am335x_tscadc: Drop useless definitions from the header · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 30/48] mfd: ti_am335x_tscadc: Rename the subsystem enable macro · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 31/48] mfd: ti_am335x_tscadc: Add TSC prefix in certain macros · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 32/48] mfd: ti_am335x_tscadc: Rename a variable · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 33/48] mfd: ti_am335x_tscadc: Fix an error message · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 34/48] mfd: ti_am335x_tscadc: Add a boolean to clarify the presence of a touchscreen · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 35/48] mfd: ti_am335x_tscadc: Introduce a helper to deal with the type of hardware · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 36/48] mfd: ti_am335x_tscadc: Add ADC1/magnetic reader support · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
Re: [PATCH v4 36/48] mfd: ti_am335x_tscadc: Add ADC1/magnetic reader support · kernel test robot <hidden> · 2021-09-28
[PATCH v4 37/48] mfd: ti_am335x_tscadc: Support the correctly spelled DT property · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 38/48] iio: adc: ti_am335x_adc: Wait the idle state to avoid stalls · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 39/48] iio: adc: ti_am335x_adc: Replace license text with SPDX tag · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 40/48] iio: adc: ti_am335x_adc: Fix style · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 41/48] iio: adc: ti_am335x_adc: Get rid of useless gotos · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 42/48] iio: adc: ti_am335x_adc: Gather the checks on the delays · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 44/48] iio: adc: ti_am335x_adc: Add the scale information · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 46/48] ARM: dts: am437x-cm-t43: Use a correctly spelled DT property · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 45/48] iio: adc: ti_am335x_adc: Add the am437x compatible · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 43/48] iio: adc: ti_am335x_adc: Add a unit to the timeout delay · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 47/48] ARM: dts: am43xx: Describe the magnetic reader/ADC1 hardware module · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28
[PATCH v4 48/48] ARM: dts: am437x-gp-evm: enable ADC1 · Miquel Raynal <miquel.raynal@bootlin.com> · 2021-09-28

From: Miquel Raynal <miquel.raynal@bootlin.com>
Date: 2021-10-01 07:11:44
Also in: linux-iio, linux-omap, lkml

Hi Jonathan,

jic23@kernel.org wrote on Wed, 29 Sep 2021 16:29:05 +0100:

On Tue, 28 Sep 2021 15:31:20 +0200
Miquel Raynal [off-list ref] wrote:

quoted

Before adding another frequency with even more zeroes, use the
HZ_PER_MHZ macro to clarify the number.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

You missed tag I gave on v3.

Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

Oh right, sorry about that.

Cheers,
Miquèl

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help