[PATCH] i2c: aspeed: fixed invalid clock parameters for very large divisors

Brendan Higgins brendanhiggins at google.com
Sat Sep 22 08:18:31 AEST 2018


On Fri, Sep 21, 2018 at 10:46 AM Jae Hyun Yoo
<jae.hyun.yoo at linux.intel.com> wrote:
>
> Hi Brendan,
>
> nit:
> Title in imperative mood. I'd put 'fix' instead of 'fixed'.
>
<snip>
>
> One minor issue is, 'base_clk_divisor' instead of 'base_clk' could avoid
> misreading on this code.
>
> With that, it looks nice to me. Thanks!
>
> Reviewed-by: Jae Hyun Yoo <jae.hyun.yoo at linux.intel.com>

Thanks for the review!


More information about the openbmc mailing list