Hi Wolfram, Kaneko-san,
On Fri, Jul 14, 2017 at 4:25 PM, Wolfram Sang <[email protected]> wrote:
> On Thu, Apr 20, 2017 at 02:46:33AM +0900, Yoshihiro Kaneko wrote:
>> From: Takeshi Kihara <[email protected]>
>>
>> In .recalc_rate of struct clk_ops, it is desirable to return 0 if an
>> error occurs, but -EINVAL is returned. This patch fixes it.
>>
>> Fixes: 5b1defde7054 ("clk: renesas: cpg-mssr: Extract common R-Car Gen3
>> support code")
>> Signed-off-by: Takeshi Kihara <[email protected]>
>> Signed-off-by: Yoshihiro Kaneko <[email protected]>
>
> Reviewed-by: Wolfram Sang <[email protected]>
Why is it desirable to return 0 if an error occurs? Because the return type is
unsigned long?
Is this routine allowed to fail? I don't see any handling of zero by
its callers.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
