[U-Boot] [PATCH 2/2] clk: change the type of return value to long

[Masahiro Yamada]

2016-01-14 5:10 GMT+09:00 Simon Glass <sjg at chromium.org>:
> Hi Masahiro,
>
> On 12 January 2016 at 00:40, Masahiro Yamada
> <yamada.masahiro at socionext.com> wrote:
>> The comments in include/clk.h state "or -ve error code" for these
>> functions, and actually the functions return negative error code
>> here and there.  Returning unsigned value is not suitable.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
>> ---
>>
>>  drivers/clk/clk-uclass.c  |  8 ++++----
>>  drivers/clk/clk_rk3036.c  |  6 +++---
>>  drivers/clk/clk_rk3288.c  |  6 +++---
>>  drivers/clk/clk_sandbox.c |  9 ++++-----
>>  include/clk.h             | 16 ++++++++--------
>>  5 files changed, 22 insertions(+), 23 deletions(-)
>
> This limits us to about 2GHz which seems a bit dangerous. We are
> starting to see things around that level. We can use IS_ERR_VALUE().
>

OK, please disregard this patch.



-- 
Best Regards
Masahiro Yamada