[PATCH v2 1/2] ARM: dts: imx6q-cm-fx6: fix the operation points
Christopher Spinrath
christopher.spinrath at rwth-aachen.de
Sun Jun 12 16:04:15 PDT 2016
Hi Fabio,
thanks again for your review!
On 06/13/2016 12:49 AM, Fabio Estevam wrote:
> On Sun, Jun 12, 2016 at 12:19 PM, <christopher.spinrath at rwth-aachen.de> wrote:
>
>> +&cpu0 {
>> + /* Although the imx6q fuse indicates that 1.2GHz operation is possible,
>> + * the module behaves unstable at this frequency. Hence, remove the
>> + * 1.2GHz operation point here.
>> + */
>
> Just one minor comment.
>
> From Documentation/CodingStyle:
>
> "The preferred style for long (multi-line) comments is:
>
> /*
> * This is the preferred style for multi-line
> * comments in the Linux kernel source code.
> * Please use it consistently.
> *
> * Description: A column of asterisks on the left side,
> * with beginning and ending almost-blank lines.
> */
> "
>
ups, I took only a quick look and picked the comment style for net/ ...
I will fix it and read more carefully next time.
Thanks,
Christopher
> Reviewed-by: Fabio Estevam <fabio.estevam at nxp.com>
>
More information about the linux-arm-kernel
mailing list