media: ov5640: Fix check for PLL1 exceeding max allowed rate
authorAdam Ford <aford173@gmail.com>
Tue, 29 Oct 2019 12:42:11 +0000 (13:42 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sat, 4 Jan 2020 07:08:58 +0000 (08:08 +0100)
commit2e3df204f9af42a47823ee955c08950373417420
tree564d711ea501af2597c31ebb8de6b2c4c6f76906
parent5554c80e18780db8457e222e998ef56030f687bf
media: ov5640: Fix check for PLL1 exceeding max allowed rate

The variable _rate is by ov5640_compute_sys_clk() which returns
zero if the PLL exceeds 1GHz.  Unfortunately, the check to see
if the max PLL1 output is checking 'rate' and not '_rate' and
'rate' does not ever appear to be 0.

This patch changes the check against the returned value of
'_rate' to determine if the PLL1 output exceeds 1GHz.

Fixes: aa2882481cad ("media: ov5640: Adjust the clock based on the expected rate")
Signed-off-by: Adam Ford <aford173@gmail.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/i2c/ov5640.c