From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfram Sang Subject: Re: [PATCH] i2c: mux: demux-pinctrl: use struct_size() in devm_kzalloc() Date: Fri, 4 Jan 2019 19:06:12 +0100 Message-ID: <20190104180612.GE3009@kunai> References: <20190104174327.GA30932@embeddedor> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="5oH/S/bF6lOfqCQb" Return-path: Content-Disposition: inline In-Reply-To: <20190104174327.GA30932@embeddedor> Sender: linux-kernel-owner@vger.kernel.org To: "Gustavo A. R. Silva" Cc: Wolfram Sang , Peter Rosin , linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-i2c@vger.kernel.org --5oH/S/bF6lOfqCQb Content-Type: text/plain; charset=us-ascii Content-Disposition: inline > Instead of leaving these open-coded and prone to type mistakes, we can > now use the new struct_size() helper: I am okay with this patch in general. However, I think the name of the helper 'struct_size' is a bit unfortunate. I really wondered on the first read why it needs 3 arguments to find out a struct size. I only understood on the second read. I think 'trailing_array_struct_size' would be way more clearer what it does. --5oH/S/bF6lOfqCQb Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAlwvoJQACgkQFA3kzBSg Kbab8g/8Csm4yMDQXB3AiC2iWdWtjRM0k03QWDCvN3dVV1VGWbpuO6534EsmKF6g vLHMtJJ0oZv5d7PiNKfmX6Pit4brrBYqW1cQ0EyWTbgSSm8HBP6WQFqy7v0QADSi 7vCUJeoY8SppZiM3zNxM5mtusbGfVBROmk6buEAF+gvzTUG2TiqJv/o5qPdG+iC4 MECZEzfPG93VXz+PsL2NILbpX7KV8c1EFeXlL2AU7AxNOkl781hlrOYIlj1f9A/l bKo4UKLbsWGPh1kdwSNr/IbDjEWOSY9BdEsfj4NS39EfetdF6Pn4HiyV5s65g8yf entP8bOUgXG8AMLxU9e9B6ydUniMy2ASvELV/XDXkcHAAFhbALylAgCN4G6zO0vH cFP8G0IrhWmNULL3pTQm2BjHZxdj8x1nG38qqUZkmzZ7+0RULOcmzK2U+ddc8KBp BJMOOAFzAPo6BiZlNdjR1wHPThUUaCDfzBhY1qn5D+RAPXPJYM33nLfYdkLwrFvo ZalWEZJgrtW7Q1cvsfK5Kh4sUM905AYDcZTpLvTJIBLrV6YvNAB9+2m1CHvNKr0M idaS/anHyz/2K5PlhLZ1i7UuyRRX34vy39hLa5CwfUKt46votKoGyw9Bbd+AWlRE PBxOBzeRoqQB/uj1tkYtCVqf/1dYvmKC7rqyWQBI7UuYRRj1v74= =d9UO -----END PGP SIGNATURE----- --5oH/S/bF6lOfqCQb--