media: i2c: imx214: Separate legacy link frequency check from PLL calculation
Make it easier to drop the legacy support from the driver later. Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: André Apitzsch <git@apitzsch.eu> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
This commit is contained in:
committed by
Hans Verkuil
parent
db967cc41f
commit
880c2395eb
@@ -1362,18 +1362,16 @@ static int imx214_parse_fwnode(struct imx214 *imx214)
|
||||
u64 freq = bus_cfg->link_frequencies[i];
|
||||
struct ccs_pll pll;
|
||||
|
||||
if (!imx214_pll_calculate(imx214, &pll, freq))
|
||||
break;
|
||||
if (freq == IMX214_DEFAULT_LINK_FREQ_LEGACY) {
|
||||
dev_warn(dev,
|
||||
"link-frequencies %d not supported, please review your DT. Continuing anyway\n",
|
||||
IMX214_DEFAULT_LINK_FREQ);
|
||||
freq = IMX214_DEFAULT_LINK_FREQ;
|
||||
if (imx214_pll_calculate(imx214, &pll, freq))
|
||||
continue;
|
||||
bus_cfg->link_frequencies[i] = freq;
|
||||
break;
|
||||
}
|
||||
|
||||
if (!imx214_pll_calculate(imx214, &pll, freq))
|
||||
break;
|
||||
}
|
||||
|
||||
if (i == bus_cfg->nr_of_link_frequencies)
|
||||
|
||||
Reference in New Issue
Block a user