Re: [PATCH v2 1/2] ARM: dts: imx6q-cm-fx6: fix the operation points

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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@xxxxxxxxxxxxxx> 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@xxxxxxx>
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux