linux-kernel - Re: [PATCH v2] i2c documentation: rename variable "register" to "reg"

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] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 27 Aug 2014 12:35:43 -0700
From:	Randy Dunlap <rdunlap@...radead.org>
To:	José Manuel Alarcón Roldán 
	<jose.alarcon.roldan@...il.com>
CC:	wsa@...-dreams.de, linux-doc@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] i2c documentation: rename variable "register" to "reg"

On 08/27/14 11:12, José Manuel Alarcón Roldán wrote:
> The example code provided with the i2c device interface documentation
> won't compile since it uses the reserved word "register" to name a
> variable.
> 
> The compiler fails with this error message:
> 
>  error: expected identifier or '(' before '=' token
>    __u8 register = 0x20; /* Device register to access */
>                  ^
> 
> Rename the variable "register" to simply "reg"
> in the example code.
> 
> Another couple of typos has been fixed as well.
> 
> 
> Signed-off-by: Jose Alarcon Roldan <jose.alarcon.roldan@...il.com>
> ---
>  Documentation/i2c/dev-interface | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/i2c/dev-interface b/Documentation/i2c/dev-interface
> index 3e742ba..744ce70 100644
> --- a/Documentation/i2c/dev-interface
> +++ b/Documentation/i2c/dev-interface
> @@ -57,12 +57,12 @@ Well, you are all set up now. You can now use
> SMBus commands or plain

The 2 lines above should have been on one line.  Looks like your mail client is
splitting lines where they shouldn't be split.  Some clients have config
options for that...   (I fixed it in your patch.)
Please see if Documentation/email-clients.txt can help you any.

>  I2C to communicate with your device. SMBus commands are preferred if
>  the device supports them. Both are illustrated below.
> 
> -  __u8 register = 0x10; /* Device register to access */
> +  __u8 reg = 0x10; /* Device register to access */
>    __s32 res;
>    char buf[10];
> 
>    /* Using SMBus commands */
> -  res = i2c_smbus_read_word_data(file, register);
> +  res = i2c_smbus_read_word_data(file, reg);
>    if (res < 0) {
>      /* ERROR HANDLING: i2c transaction failed */
>    } else {
> @@ -70,11 +70,11 @@ the device supports them. Both are illustrated below.
>    }
> 
>    /* Using I2C Write, equivalent of

The line above should end with a space character as in the current source file
that is being patched.  Maybe your email client stripped it.  Please fix your
email client for future patches.

> -     i2c_smbus_write_word_data(file, register, 0x6543) */
> -  buf[0] = register;
> +  i2c_smbus_write_word_data(file, reg, 0x6543) */

The line above should be indented more since it is a continuation
of the comment above it.  I fixed that.

> +  buf[0] = reg;
>    buf[1] = 0x43;
>    buf[2] = 0x65;
> -  if (write(file, buf, 3) ! =3) {
> +  if (write(file, buf, 3) != 3) {
>      /* ERROR HANDLING: i2c transaction failed */
>    }
> 

Patch applied.  Or would you like to see if you can send the patch
with all of these minor annoying problems fixed?

Thanks.

-- 
~Randy
--
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/