spi: rzv2m-csi: Rework CSI_CKS_MAX definition
authorFabrizio Castro <fabrizio.castro.jz@renesas.com>
Sat, 15 Jul 2023 01:04:00 +0000 (02:04 +0100)
committerMark Brown <broonie@kernel.org>
Mon, 17 Jul 2023 11:41:42 +0000 (12:41 +0100)
commitaecf9fbdb7a4dc6d83e8d9984c8d9dc074d8ea2e
tree3d8dbb6ca33dd12ca9280aa1f5229bd7ef06e2d4
parent74e27ce8d23c3aeb1a9fdcaf6261462506bbbfc3
spi: rzv2m-csi: Rework CSI_CKS_MAX definition

Clock "csiclk" gets divided by 2 * CSI_CLKSEL_CKS in order to generate
the serial clock (output from master), with CSI_CLKSEL_CKS ranging from
0x1 (that means "csiclk" is divided by 2) to 0x3FFF ("csiclk" is divided
by 32766). CSI_CKS_MAX is used for referring to the setting
corresponding to the maximum frequency divider.
Value 0x3FFF for CSI_CKS_MAX doesn't really means much to the reader
without an explanation and a more readable definition.

Add a comment with a meaningful description and also replace value
0x3FFF with the corresponding GENMASK, to make it very clear what the
macro means.

Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Link: https://lore.kernel.org/r/20230715010407.1751715-4-fabrizio.castro.jz@renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-rzv2m-csi.c