Re: [PATCH v2 1/6] staging: ft1000: spaces required around that '=', '<' and '=='

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

 



On Sat, Nov 01, 2014 at 10:47:22AM +0800, Chen Weixiang wrote:
> Remove following code style errors from ft1000/ft1000-usb/ft1000_debug.c
> and ft1000/ft1000-pcmcia/ft1000_hw.c:
> ERROR: spaces required around that '=' (ctx:VxV)
> ERROR: spaces required around that '<' (ctx:VxV)
> ERROR: spaces required around that '==' (ctx:VxV)
> 

many whitespace errors in the patch.
checkpatch gives total: 8 errors, 23 warnings, 125 lines checked

thanks
sudip

> Signed-off-by: Chen Weixiang <weixiang.chen@xxxxxxxxx>
> ---
>  drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c |  6 ++---
>  drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 28 ++++++++++++------------
>  2 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
> index 44575c7..fc28dee 100644
> --- a/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
> +++ b/drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c
> @@ -812,11 +812,11 @@ static void ft1000_send_cmd (struct net_device *dev, u16 *ptempbuffer, int size,
>  
>      /* Make sure SLOWQ doorbell is clear */
>      tempword = ft1000_read_reg(dev, FT1000_REG_DOORBELL);
> -    i=0;
> +    i = 0;
>      while (tempword & FT1000_DB_DPRAM_TX) {
>          mdelay(10);
>          i++;
> -        if (i==10) {
> +        if (i == 10) {
>              spin_unlock_irqrestore(&info->dpram_lock, flags);
>              return;
>          }
> @@ -1221,7 +1221,7 @@ static void ft1000_proc_drvmsg(struct net_device *dev)
>  				ppseudo_hdr->portsrc = 0;
>  				/* Calculate new checksum */
>                  ppseudo_hdr->checksum = *pmsg++;
> -                for (i=1; i<7; i++) {
> +                for (i = 1; i < 7; i++) {
>                      ppseudo_hdr->checksum ^= *pmsg++;
>                  }
>  				pmsg = (u16 *) & tempbuffer[16];
> diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c b/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c
> index 0f347ab..02bfb3e 100644
> --- a/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c
> +++ b/drivers/staging/ft1000/ft1000-usb/ft1000_debug.c
> @@ -193,7 +193,7 @@ int ft1000_create_dev(struct ft1000_usb *dev)
>  
>      /* initialize application information */
>      dev->appcnt = 0;
> -    for (i=0; i<MAX_NUM_APP; i++) {
> +    for (i = 0; i < MAX_NUM_APP; i++) {
>          dev->app_info[i].nTxMsg = 0;
>          dev->app_info[i].nRxMsg = 0;
>          dev->app_info[i].nTxMsgReject = 0;
> @@ -260,7 +260,7 @@ void ft1000_destroy_dev(struct net_device *netdev)
>  					   dev->DeviceName);
>  
>          /* Make sure we free any memory reserve for slow Queue */
> -        for (i=0; i<MAX_NUM_APP; i++) {
> +        for (i = 0; i < MAX_NUM_APP; i++) {
>              while (list_empty(&dev->app_info[i].app_sqlist) == 0) {
>                  pdpram_blk = list_entry(dev->app_info[i].app_sqlist.next, struct dpram_blk, list);
>                  list_del(&pdpram_blk->list);
> @@ -318,7 +318,7 @@ static int ft1000_open(struct inode *inode, struct file *file)
>      }
>  
>      /* Search for available application info block */
> -    for (i=0; i<MAX_NUM_APP; i++) {
> +    for (i = 0; i < MAX_NUM_APP; i++) {
>          if ((dev->app_info[i].fileobject == NULL)) {
>              break;
>          }
> @@ -369,7 +369,7 @@ static unsigned int ft1000_poll_dev(struct file *file, poll_table *wait)
>      }
>  
>      /* Search for matching file object */
> -    for (i=0; i<MAX_NUM_APP; i++) {
> +    for (i = 0; i < MAX_NUM_APP; i++) {
>          if (dev->app_info[i].fileobject == &file->f_owner) {
>              /* DEBUG("FT1000:ft1000_ioctl: Message is for AppId = %d\n", dev->app_info[i].app_id); */
>              break;
> @@ -411,7 +411,7 @@ static long ft1000_ioctl(struct file *file, unsigned int command,
>      void __user *argp = (void __user *)argument;
>  	struct ft1000_info *info;
>      struct ft1000_usb *ft1000dev;
> -    int result=0;
> +    int result = 0;
>      int cmd;
>      int i;
>      u16 tempword;
> @@ -426,8 +426,8 @@ static long ft1000_ioctl(struct file *file, unsigned int command,
>                            0x00,0x00,0x02,0x37,0x00,0x00,0x00,0x08,0x00,0x00,0x00,0x01,0x00,0x01,0x7f,0x00,
>                            0x00,0x01,0x00,0x00};
>  
> -    unsigned short ledStat=0;
> -    unsigned short conStat=0;
> +    unsigned short ledStat = 0;
> +    unsigned short conStat = 0;
>  
>      /* DEBUG("ft1000_ioctl called\n"); */
>  
> @@ -454,7 +454,7 @@ static long ft1000_ioctl(struct file *file, unsigned int command,
>              }
>              if (tempword == DSPBCMSGID) {
>                  /* Search for matching file object */
> -                for (i=0; i<MAX_NUM_APP; i++) {
> +                for (i = 0; i < MAX_NUM_APP; i++) {
>                      if (ft1000dev->app_info[i].fileobject == &file->f_owner) {
>                          ft1000dev->app_info[i].DspBCMsgFlag = 1;
>                          DEBUG("FT1000:ft1000_ioctl:Registered for broadcast messages\n");
> @@ -581,12 +581,12 @@ static long ft1000_ioctl(struct file *file, unsigned int command,
>                      result = -EFAULT;
>                  } else {
>                      /* Check if this message came from a registered application */
> -                    for (i=0; i<MAX_NUM_APP; i++) {
> +                    for (i = 0; i < MAX_NUM_APP; i++) {
>                          if (ft1000dev->app_info[i].fileobject == &file->f_owner) {
>                              break;
>                          }
>                      }
> -                    if (i==MAX_NUM_APP) {
> +                    if (i == MAX_NUM_APP) {
>                          DEBUG("FT1000:No matching application fileobject\n");
>                          result = -EINVAL;
>  			kfree(dpram_data);
> @@ -646,7 +646,7 @@ static long ft1000_ioctl(struct file *file, unsigned int command,
>                              /* Calculate new checksum */
>                              ppseudo_hdr->checksum = *pmsg++;
>                              /* DEBUG("checksum = 0x%x\n", ppseudo_hdr->checksum); */
> -                            for (i=1; i<7; i++) {
> +                            for (i = 1; i < 7; i++) {
>                                  ppseudo_hdr->checksum ^= *pmsg++;
>                                  /* DEBUG("checksum = 0x%x\n", ppseudo_hdr->checksum); */
>                              }
> @@ -682,7 +682,7 @@ static long ft1000_ioctl(struct file *file, unsigned int command,
>              }
>  
>              /* Search for matching file object */
> -            for (i=0; i<MAX_NUM_APP; i++) {
> +            for (i = 0; i < MAX_NUM_APP; i++) {
>                  if (ft1000dev->app_info[i].fileobject == &file->f_owner) {
>                      /*DEBUG("FT1000:ft1000_ioctl: Message is for AppId = %d\n", ft1000dev->app_info[i].app_id); */
>                      break;
> @@ -766,14 +766,14 @@ static int ft1000_release(struct inode *inode, struct file *file)
>      }
>  
>      /* Search for matching file object */
> -    for (i=0; i<MAX_NUM_APP; i++) {
> +    for (i = 0; i < MAX_NUM_APP; i++) {
>          if (ft1000dev->app_info[i].fileobject == &file->f_owner) {
>              /* DEBUG("FT1000:ft1000_ioctl: Message is for AppId = %d\n", ft1000dev->app_info[i].app_id); */
>              break;
>          }
>      }
>  
> -    if (i==MAX_NUM_APP)
> +    if (i == MAX_NUM_APP)
>  	    return 0;
>  
>      while (list_empty(&ft1000dev->app_info[i].app_sqlist) == 0) {
> -- 
> 2.1.2
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux