media: ov8856: Check reading clock frequency succeeded
Instead of blindly trusting getting the clock frequency succeeded end then testing it against a pre-defined value, verify reading the value succeeded. Fixes: 879347f0c258 ("media: ov8856: Add support for OV8856 sensor") Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
This commit is contained in:
committed by
Mauro Carvalho Chehab
parent
bf27502b1f
commit
fa4bb7d3ac
@ -1106,7 +1106,10 @@ static int ov8856_check_hwcfg(struct device *dev)
|
||||
if (!fwnode)
|
||||
return -ENXIO;
|
||||
|
||||
fwnode_property_read_u32(fwnode, "clock-frequency", &mclk);
|
||||
ret = fwnode_property_read_u32(fwnode, "clock-frequency", &mclk);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
if (mclk != OV8856_MCLK) {
|
||||
dev_err(dev, "external clock %d is not supported", mclk);
|
||||
return -EINVAL;
|
||||
|
Reference in New Issue
Block a user