Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sat, 27 Jun 2020 22:33:57 +0200
From:   Wolfram Sang <wsa@kernel.org>
To:     Lee Jones <lee.jones@linaro.org>
Cc:     arnd@arndb.de, gregkh@linuxfoundation.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 06/10] misc: eeprom: eeprom_93cx6: Repair function arg
 descriptions
Message-ID: <20200627203357.GA954@ninjato>
References: <20200626130525.389469-1-lee.jones@linaro.org>
 <20200626130525.389469-7-lee.jones@linaro.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="TB36FDmn/VVEgNH/"
Content-Disposition: inline
In-Reply-To: <20200626130525.389469-7-lee.jones@linaro.org>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--TB36FDmn/VVEgNH/
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

> @@ -270,7 +270,7 @@ EXPORT_SYMBOL_GPL(eeprom_93cx6_readb);
>   * @eeprom: Pointer to eeprom structure
>   * @byte: Index from where we should start reading
>   * @data: target pointer where the information will have to be stored
> - * @words: Number of bytes that should be read.
> + * @bytes: Number of bytes that should be read.

Now we have 'byte' and 'bytes' here as arguments which is confusing. I
think renaming 'words' into 'num_bytes' would be even better.


--TB36FDmn/VVEgNH/
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAl73rTEACgkQFA3kzBSg
KbaBzA//dLI5VkiLGrU7srp/U5c4psR79hClXFkbf/Ki9UWEBgl011V9+LTe+lQk
r4OI7bKMBBZbw/U4TV27Eb2wD6fZ3TWbVEF5X/hWTEbAnP3ZgteH+CgV3Qp3antd
k6ZF1ENnu7bGCIksTCyrHiEf2o7eeE7k9uYIN1fGCc2HMeqJTfaXZfiuQWPLcIsq
Bm+Fbz+K8wG5HO8xi8QRWU3QQSAgLPsHaeHLVH4wWgtnmw7LI9FZgeYmlPSHa9Gu
x39+p2WNzGuna4FmepxQ4CZ6qBtJiPqvgBnVbz7q//mlqwIb66WxyydGXEi3FdLG
noFhSZx9eQKJiDN4Msy/2z1aa6+4JMP7sEfNJGosYg+V2a4B6ZsBjFBp3PAriGzz
oOOLRQh+bbXaZ5Loam0Eyh99fVFPmHb+9heoqzT8x1gH4Fbyzj6jjXJnlPToJaOc
ralvHcf7UH7/wXkO3gBWrOuEsV0IbI6yqds4LmCwA3Yjf3JfbdWsJMl7TMOYAc6q
XlYwFrpTcLyi6Jh2JQYSXu4mYMwEfRaRIsy5bVMH+q9hNlbQ06Y8Bl0dfRV66vl1
+VzMZ3WjyRQ4b16hs/Gx6bV0zU0pzvG14Dld4XPj6vKlrqnqFjs+aTVLBHnn+3Jv
9/lG7OLbdliAy/12VJAw4RKKCX2oYgia9bmxDP1D5UMUy5eXMNk=
=NGax
-----END PGP SIGNATURE-----

--TB36FDmn/VVEgNH/--