9 Jan
2020
9 Jan
'20
9:38 p.m.
On Fri, Dec 27, 2019 at 04:20:56PM +0100, Peter Seiderer wrote:
@@ -338,7 +338,8 @@ static unsigned long clk_aic32x4_div_recalc_rate(struct clk_hw *hw,
unsigned int val;
- regmap_read(div->regmap, div->reg, &val);
- if (regmap_read(div->regmap, div->reg, &val))
return 0;
Is this the best fix - shouldn't we be returning an error here? We don't know what the value programmed into the device actually is so zero might be wrong, and we still have the risk that the value we read from the device may be zero if the device is misprogrammed.