On Thu, Feb 7, 2013 at 6:23 PM, Wolfgang Denk <w...@denx.de> wrote:
> Dear Sonic Zhang,
>
> In message <1360223258-6945-10-git-send-email-sonic....@gmail.com> you wrote:
>> From: Scott Jiang <scott.jiang.li...@gmail.com>
>>
>> Spi driver for bf60x is different from old one, so implement a new
>> driver for it.
>
> Checkpatch issues, please fix.
>
Could the checkpatch warning "line over 80 characters" in head file
Macro definition be ignored? I think put the explanation and the
definition on the same line is more clean than to chopping to 80.

#define SPI_CTL_FIFO0               0x00000000    /* FCWM: TFIFO empty
or RFIFO Full */
#define SPI_CTL_FIFO1               0x00010000    /* FCWM: TFIFO 75%
or more empty or RFIFO 75% or more full */
#define SPI_CTL_FIFO2               0x00020000    /* FCWM: TFIFO 50%
or more empty or RFIFO 50% or more full */


Regards,

Sonic
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to