[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