media: ccs-pll: Check for too high VT PLL multiplier in dual PLL case

The check for VT PLL upper limit in dual PLL case was missing. Add it now.

Fixes: 6c7469e46b60 ("media: ccs-pll: Add trivial dual PLL support")
Cc: stable@vger.kernel.org
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>

diff --git a/drivers/media/i2c/ccs-pll.c b/drivers/media/i2c/ccs-pll.c
index 2399cd6509b757601e9ac007aa9e1033066c69e5..266fcd160da6ef127451d0eb981736951753aa7b 100644 (file)
--- a/drivers/media/i2c/ccs-pll.c
+++ b/drivers/media/i2c/ccs-pll.c
@@ -312,6 +312,11 @@ __ccs_pll_calculate_vt_tree(struct device *dev,
        dev_dbg(dev, "more_mul2: %u\n", more_mul);
 
        pll_fr->pll_multiplier = mul * more_mul;
+       if (pll_fr->pll_multiplier > lim_fr->max_pll_multiplier) {
+               dev_dbg(dev, "pll multiplier %u too high\n",
+                       pll_fr->pll_multiplier);
+               return -EINVAL;
+       }
 
        if (pll_fr->pll_multiplier * pll_fr->pll_ip_clk_freq_hz >
            lim_fr->max_pll_op_clk_freq_hz)