Re: staging: vme_user: Replace the "<<" with BIT macro

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

 



On Wed, Dec 14, 2022 at 05:05:38PM +0530, Abhirup Deb wrote:
> Replace the "<<" operator with BIT macro, in accordance to the
> checkpatch.pl script and Linux kernel coding-style guidelines.
> Issues reported by checkpatch.pl:
> 	CHECK: Prefer using the BIT macro
> 
> Signed-off-by: Abhirup Deb <abhirupdeb@xxxxxxxxxxxxxxxxxx>
> ---
>  drivers/staging/vme_user/vme.h | 26 ++++++++++++++------------
>  1 file changed, 14 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/staging/vme_user/vme.h b/drivers/staging/vme_user/vme.h
> index 98da8d039d60..faa9816046a9 100644
> --- a/drivers/staging/vme_user/vme.h
> +++ b/drivers/staging/vme_user/vme.h
> @@ -2,6 +2,8 @@
>  #ifndef _VME_H_
>  #define _VME_H_
>  
> +#include <linux/bitops.h>
> +
>  /* Resource Type */
>  enum vme_resource_type {
>  	VME_MASTER,
> @@ -54,20 +56,20 @@ enum vme_resource_type {
>  #define VME_R_ROBIN_MODE	0x1
>  #define VME_PRIORITY_MODE	0x2
>  
> -#define VME_DMA_PATTERN			(1<<0)
> -#define VME_DMA_PCI			(1<<1)
> -#define VME_DMA_VME			(1<<2)
> +#define VME_DMA_PATTERN		BIT(0)
> +#define VME_DMA_PCI			BIT(1)
> +#define VME_DMA_VME			BIT(2)
>  
> -#define VME_DMA_PATTERN_BYTE		(1<<0)
> -#define VME_DMA_PATTERN_WORD		(1<<1)
> -#define VME_DMA_PATTERN_INCREMENT	(1<<2)
> +#define VME_DMA_PATTERN_BYTE		BIT(0)
> +#define VME_DMA_PATTERN_WORD		BIT(1)
> +#define VME_DMA_PATTERN_INCREMENT	BIT(2)
>  
> -#define VME_DMA_VME_TO_MEM		(1<<0)
> -#define VME_DMA_MEM_TO_VME		(1<<1)
> -#define VME_DMA_VME_TO_VME		(1<<2)
> -#define VME_DMA_MEM_TO_MEM		(1<<3)
> -#define VME_DMA_PATTERN_TO_VME		(1<<4)
> -#define VME_DMA_PATTERN_TO_MEM		(1<<5)
> +#define VME_DMA_VME_TO_MEM		BIT(0)
> +#define VME_DMA_MEM_TO_VME		BIT(1)
> +#define VME_DMA_VME_TO_VME		BIT(2)
> +#define VME_DMA_MEM_TO_MEM		BIT(3)
> +#define VME_DMA_PATTERN_TO_VME		BIT(4)
> +#define VME_DMA_PATTERN_TO_MEM		BIT(5)
>  
>  struct vme_dma_attr {
>  	u32 type;
> -- 
> 2.31.1
> 
> 

Hi,

This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him
a patch that has triggered this response.  He used to manually respond
to these common problems, but in order to save his sanity (he kept
writing the same thing over and over, yet to different people), I was
created.  Hopefully you will not take offence and will fix the problem
in your patch and resubmit it so that it can be accepted into the Linux
kernel tree.

You are receiving this message because of the following common error(s)
as indicated below:

- This looks like a new version of a previously submitted patch, but you
  did not list below the --- line any changes from the previous version.
  Please read the section entitled "The canonical patch format" in the
  kernel file, Documentation/process/submitting-patches.rst for what
  needs to be done here to properly describe this.

If you wish to discuss this problem further, or you have questions about
how to resolve this issue, please feel free to respond to this email and
Greg will reply once he has dug out from the pending patches received
from other developers.

thanks,

greg k-h's patch email bot




[Index of Archives]     [Linux Driver Development]     [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