[PATCH 05/10] opp: Add support to parse "operating-points-v2" bindings
Bartlomiej Zolnierkiewicz
b.zolnierkie at samsung.com
Fri Jul 24 11:02:33 PDT 2015
Hi,
On Thursday, July 09, 2015 10:48:46 AM Viresh Kumar wrote:
> On 08-07-15, 15:41, Bartlomiej Zolnierkiewicz wrote:
> > Isn't using u32 for storing frequency (in Hz) too small by today's
> > standards?
> >
> > [ Please note that the old v1 binding uses kHz not Hz. ]
>
> Hmm, probably yes we need it to be u64. Thanks for pointing out.
There is also a minor issue with of_init_opp_table() documentation
(the function can now return -EINVAL in some cases). Except these
two things the patch looks fine and once it is fixed you can add:
Reviewed-by: Bartlomiej Zolnierkiewicz <b.zolnierkie at samsung.com>
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
More information about the linux-arm-kernel
mailing list