From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH v4 2/3] regulator: tps6586x: add and use correct voltage table Date: Wed, 4 Dec 2013 10:48:10 +0100 Message-ID: <20131204094809.GR19943@ulmo.nvidia.com> References: <127fe07e18665f93aa8d7c03e03164f1a049ddb6.1386108712.git.stefan@agner.ch> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="EguBBKnZWdUQS9Kz" Return-path: Content-Disposition: inline In-Reply-To: <127fe07e18665f93aa8d7c03e03164f1a049ddb6.1386108712.git.stefan-XLVq0VzYD2Y@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stefan Agner Cc: swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org, dev-8ppwABl0HbeELgA04lAiVw@public.gmane.org, lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, kai.poggensee-RM9K5IK7kjKj5M59NBduVrNAH6kLmebB@public.gmane.org, sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org --EguBBKnZWdUQS9Kz Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Dec 03, 2013 at 11:18:47PM +0100, Stefan Agner wrote: [...] > diff --git a/drivers/regulator/tps6586x-regulator.c b/drivers/regulator/tps6586x-regulator.c [...] > + /* Search version specific table first */ > + if (table) { > + for (i = 0; i < num; i++) { > + ri = &table[i]; > + if (ri->desc.id == id) > + return ri; > + } > + } Ah... there you go. Looks good: Reviewed-by: Thierry Reding --EguBBKnZWdUQS9Kz Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJSnvpZAAoJEN0jrNd/PrOhI3MP/iCQEYBVNh2NsQ1FNFlSgJnV dXXQQLBqIxMDp+ExE356aNANGWRoWPIBkDzEp/IMA2pM1nHQyu3XKjUe7aXNdd8f kxuUMiXnc/V/fx+p6Qsdb6EoF7qPQTQ+HLglVblhK7SiQCktifcoo+LmCFt/ZH0a Tw4WAFYES2qmt25YedF22eShIplXBXXGCjRgYAq4FdDKY0HLHFVnxpFmYNJg1XPJ qf2b4trMNOG/TwcaR2CLCho7TEgTVKPAWZdvnycc5QwO7xBoGclZTWzHcizKHNmI jAu5eEXvkztUGfKZenRIFfGcgys3TEQ4FrTWDOQa2r51sbLvYN5eCa8h3oM+Fy6C DJGfGykJyVE5nimkDKnykQdwkLr+kAMZH1E3Oj/BqDg9+27VZ53dXOEmi312wapL 4VyC/RbLiz6wZNfEvXpajtd60eS0EgLaySjSGTYCUbpjFDe9fhmiTs+e6kHWnD8t HeFC1x49DBL7JupYH0cYA46t3xJUpGsmByogoHllJdAk7LzKuPCYq38myO/ngIu8 mcpuZmx0B9w9DBKQhYVJwKAflaVR6lL29CZq8/pIWqgxr4gGI5KQCp6D20rHZ5g9 JtR+pc/EQCu+0GeEatJ3cYF4/PX8NMeDoI8n2/uwKU+vftZ1O93NUaAPeULWFzR2 rfOTldJFLk7ibpkK9ojA =oAlW -----END PGP SIGNATURE----- --EguBBKnZWdUQS9Kz--