spi: rzv2h-rspi: Simplify clock rate search function signatures

The spr_min and spr_max parameters passed to
rzv2h_rspi_find_rate_variable() and rzv2h_rspi_find_rate_fixed() were
always called with RSPI_SPBR_SPR_MIN and RSPI_SPBR_SPR_MAX respectively.
There is no need to pass these as parameters since the valid SPR range
is fixed by the hardware.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Link: https://patch.msgid.link/20260410080517.2405700-4-prabhakar.mahadev-lad.rj@bp.renesas.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/spi/spi-rzv2h-rspi.c b/drivers/spi/spi-rzv2h-rspi.c
index f62223e106c05efa17be67e1948bb00bd179c177..56f2689bb04810c5ad2c3668d274ee80ddfd9769 100644 (file)
--- a/drivers/spi/spi-rzv2h-rspi.c
+++ b/drivers/spi/spi-rzv2h-rspi.c
@@ -88,9 +88,9 @@ struct rzv2h_rspi_best_clock {
 };
 
 struct rzv2h_rspi_info {
-       void (*find_tclk_rate)(struct clk *clk, u32 hz, u8 spr_min, u8 spr_max,
+       void (*find_tclk_rate)(struct clk *clk, u32 hz,
                               struct rzv2h_rspi_best_clock *best_clk);
-       void (*find_pclk_rate)(struct clk *clk, u32 hz, u8 spr_low, u8 spr_high,
+       void (*find_pclk_rate)(struct clk *clk, u32 hz,
                               struct rzv2h_rspi_best_clock *best_clk);
        const char *tclk_name;
        unsigned int fifo_size;
@@ -413,7 +413,6 @@ static inline u32 rzv2h_rspi_calc_bitrate(unsigned long tclk_rate, u8 spr,
 }
 
 static void rzv2h_rspi_find_rate_variable(struct clk *clk, u32 hz,
-                                         u8 spr_min, u8 spr_max,
                                          struct rzv2h_rspi_best_clock *best)
 {
        long clk_rate, clk_min_rate, clk_max_rate;
@@ -464,7 +463,7 @@ static void rzv2h_rspi_find_rate_variable(struct clk *clk, u32 hz,
                 * minimum SPR that is in the valid range.
                 */
                min_rate_spr = DIV_ROUND_CLOSEST(clk_min_rate, rate_div) - 1;
-               if (min_rate_spr > spr_max)
+               if (min_rate_spr > RSPI_SPBR_SPR_MAX)
                        continue;
 
                /*
@@ -474,14 +473,14 @@ static void rzv2h_rspi_find_rate_variable(struct clk *clk, u32 hz,
                 * maximum SPR that is in the valid range.
                 */
                max_rate_spr = DIV_ROUND_CLOSEST(clk_max_rate, rate_div) - 1;
-               if (max_rate_spr < spr_min)
+               if (max_rate_spr < RSPI_SPBR_SPR_MIN)
                        break;
 
-               if (min_rate_spr < spr_min)
-                       min_rate_spr = spr_min;
+               if (min_rate_spr < RSPI_SPBR_SPR_MIN)
+                       min_rate_spr = RSPI_SPBR_SPR_MIN;
 
-               if (max_rate_spr > spr_max)
-                       max_rate_spr = spr_max;
+               if (max_rate_spr > RSPI_SPBR_SPR_MAX)
+                       max_rate_spr = RSPI_SPBR_SPR_MAX;
 
                for (spr = min_rate_spr; spr <= max_rate_spr; spr++) {
                        clk_rate = (spr + 1) * rate_div;
@@ -512,7 +511,6 @@ static void rzv2h_rspi_find_rate_variable(struct clk *clk, u32 hz,
 }
 
 static void rzv2h_rspi_find_rate_fixed(struct clk *clk, u32 hz,
-                                      u8 spr_min, u8 spr_max,
                                       struct rzv2h_rspi_best_clock *best)
 {
        unsigned long clk_rate;
@@ -545,7 +543,7 @@ static void rzv2h_rspi_find_rate_fixed(struct clk *clk, u32 hz,
                 */
                if (!spr && !brdv)
                        continue;
-               if (spr >= spr_min && spr <= spr_max)
+               if (spr >= RSPI_SPBR_SPR_MIN && spr <= RSPI_SPBR_SPR_MAX)
                        goto clock_found;
        }
 
@@ -575,12 +573,10 @@ static u32 rzv2h_rspi_setup_clock(struct rzv2h_rspi_priv *rspi, u32 hz)
        };
        int ret;
 
-       rspi->info->find_tclk_rate(rspi->tclk, hz, RSPI_SPBR_SPR_MIN,
-                                  RSPI_SPBR_SPR_MAX, &best_clock);
+       rspi->info->find_tclk_rate(rspi->tclk, hz, &best_clock);
 
        if (best_clock.error && rspi->info->find_pclk_rate)
-               rspi->info->find_pclk_rate(rspi->pclk, hz, RSPI_SPBR_SPR_MIN,
-                                          RSPI_SPBR_SPR_MAX, &best_clock);
+               rspi->info->find_pclk_rate(rspi->pclk, hz, &best_clock);
 
        if (!best_clock.clk_rate)
                return -EINVAL;