linux-kernel - Re: [PATCH] Staging/bcm: Fix whitespace/comments in Ioctl.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 16 Sep 2014 11:01:41 +0200
From:	Matthias Beyer <mail@...ermatthias.de>
To:	Martin Kelly <martin@...tingkelly.com>
Cc:	devel@...verdev.osuosl.org, gregkh@...uxfoundation.org,
	klmckinney1@...il.com, linux-kernel@...r.kernel.org,
	Martin Kelly <martkell@...zon.com>
Subject: Re: [PATCH] Staging/bcm: Fix whitespace/comments in Ioctl.h

Reviewed-by: Matthias Beyer <mail@...ermatthias.de>

On 15-09-2014 21:16:15, Martin Kelly wrote:
> Cleanup whitespace and comments in Ioctl.h in a few ways:
> - > 80 character cleanup
> - Comment clarification
> - More consistent vertical alignment
> 
> Signed-off-by: Martin Kelly <martkell@...zon.com>
> ---
>  drivers/staging/bcm/Ioctl.h | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/staging/bcm/Ioctl.h b/drivers/staging/bcm/Ioctl.h
> index 797f862..fa5f867 100644
> --- a/drivers/staging/bcm/Ioctl.h
> +++ b/drivers/staging/bcm/Ioctl.h
> @@ -87,9 +87,9 @@ struct bcm_user_thread_req {
>  #define IOCTL_BCM_FLASH2X_SECTION_WRITE		_IOW(BCM_IOCTL, 0x866, int)
>  #define IOCTL_BCM_GET_FLASH2X_SECTION_BITMAP	_IOR(BCM_IOCTL, 0x867, int)
>  #define IOCTL_BCM_SET_ACTIVE_SECTION		_IOW(BCM_IOCTL, 0x868, int)
> -#define	IOCTL_BCM_IDENTIFY_ACTIVE_SECTION	_IO(BCM_IOCTL, 0x869)
> +#define IOCTL_BCM_IDENTIFY_ACTIVE_SECTION	_IO(BCM_IOCTL, 0x869)
>  #define IOCTL_BCM_COPY_SECTION			_IOW(BCM_IOCTL, 0x870, int)
> -#define	IOCTL_BCM_GET_FLASH_CS_INFO		_IOR(BCM_IOCTL, 0x871, int)
> +#define IOCTL_BCM_GET_FLASH_CS_INFO		_IOR(BCM_IOCTL, 0x871, int)
>  #define IOCTL_BCM_SELECT_DSD			_IOW(BCM_IOCTL, 0x872, int)
>  #define IOCTL_BCM_NVM_RAW_READ			_IOR(BCM_IOCTL, 0x875, int)
>  #define IOCTL_BCM_CNTRLMSG_MASK			_IOW(BCM_IOCTL, 0x874, int)
> @@ -130,7 +130,7 @@ struct bcm_bulk_wrm_buffer {
>  };
>  
>  enum bcm_flash2x_section_val {
> -	NO_SECTION_VAL = 0, /* no section is chosen when absolute offset is given for RD/WR */
> +	NO_SECTION_VAL = 0, /* no section chosen when absolute offset is given for RD/WR */
>  	ISO_IMAGE1,
>  	ISO_IMAGE2,
>  	DSD0,
> @@ -152,11 +152,11 @@ enum bcm_flash2x_section_val {
>   * Structure used for READ/WRITE Flash Map2.x
>   */
>  struct bcm_flash2x_readwrite {
> -	enum bcm_flash2x_section_val Section; /* which section has to be read/written */
> -	u32 offset;	     /* Offset within Section. */
> -	u32 numOfBytes;	     /* NOB from the offset */
> +	enum bcm_flash2x_section_val Section; /* section to be read/written */
> +	u32 offset; /* offset within section. */
> +	u32 numOfBytes; /* number of bytes from the offset */
>  	u32 bVerify;
> -	void __user *pDataBuff;	     /* Buffer for reading/writing */
> +	void __user *pDataBuff; /* buffer for reading/writing */
>  };
>  
>  /*
> @@ -207,20 +207,20 @@ struct bcm_time_elapsed {
>  };
>  
>  enum {
> -	WIMAX_IDX = 0,  /* To access WiMAX chip GPIO's for GPIO_MULTI_INFO or GPIO_MULTI_MODE */
> -	HOST_IDX,	/* To access Host chip GPIO's for GPIO_MULTI_INFO or GPIO_MULTI_MODE */
> +	WIMAX_IDX = 0, /* To access WiMAX chip GPIO's for GPIO_MULTI_INFO or GPIO_MULTI_MODE */
> +	HOST_IDX, /* To access Host chip GPIO's for GPIO_MULTI_INFO or GPIO_MULTI_MODE */
>  	MAX_IDX
>  };
>  
>  struct bcm_gpio_multi_info {
>  	unsigned int uiGPIOCommand; /* 1 for set and 0 for get */
> -	unsigned int uiGPIOMask;    /* set the correspondig bit to 1 to access GPIO */
> -	unsigned int uiGPIOValue;   /* 0 or 1; value to be set when command is 1. */
> +	unsigned int uiGPIOMask; /* set the corresponding bit to 1 to access GPIO */
> +	unsigned int uiGPIOValue; /* 0 or 1; value to be set when command is 1. */
>  } __packed;
>  
>  struct bcm_gpio_multi_mode {
> -	unsigned int uiGPIOMode;    /* 1 for OUT mode, 0 for IN mode */
> -	unsigned int uiGPIOMask;    /* GPIO mask to set mode */
> +	unsigned int uiGPIOMode; /* 1 for OUT mode, 0 for IN mode */
> +	unsigned int uiGPIOMask; /* GPIO mask to set mode */
>  } __packed;
>  
>  #endif
> -- 
> 2.1.0
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

-- 
Mit freundlichen Grüßen,
Kind regards,
Matthias Beyer

Proudly sent with mutt.
Happily signed with gnupg.

Content of type "application/pgp-signature" skipped