From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754020Ab3KTPfm (ORCPT ); Wed, 20 Nov 2013 10:35:42 -0500 Received: from cassiel.sirena.org.uk ([80.68.93.111]:43101 "EHLO cassiel.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752111Ab3KTPfk (ORCPT ); Wed, 20 Nov 2013 10:35:40 -0500 Date: Wed, 20 Nov 2013 15:35:07 +0000 From: Mark Brown To: Krzysztof Kozlowski Cc: MyungJoo Ham , Chanwoo Choi , Samuel Ortiz , Lee Jones , Anton Vorontsov , David Woodhouse , Liam Girdwood , Grant Likely , Rob Herring , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Pawel Moll , Stephen Warren , Ian Campbell , Rob Landley , linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Bartlomiej Zolnierkiewicz , Marek Szyprowski , Kyungmin Park Message-ID: <20131120153507.GH2674@sirena.org.uk> References: <1384956732-19526-1-git-send-email-k.kozlowski@samsung.com> <1384956732-19526-5-git-send-email-k.kozlowski@samsung.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="8Dt/lkN5I0m8i0o7" Content-Disposition: inline In-Reply-To: <1384956732-19526-5-git-send-email-k.kozlowski@samsung.com> X-Cookie: Drive defensively. Buy a tank. User-Agent: Mutt/1.5.21 (2010-09-15) X-SA-Exim-Connect-IP: 94.175.92.69 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH v2 4/5] regulator: max14577: Add regulator driver for Maxim 14577 X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:57:07 +0000) X-SA-Exim-Scanned: Yes (on cassiel.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --8Dt/lkN5I0m8i0o7 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Nov 20, 2013 at 03:12:11PM +0100, Krzysztof Kozlowski wrote: > + size = sizeof(struct regulator_dev *) * pdata->num_regulators; > + info->regulators = devm_kzalloc(&pdev->dev, size, GFP_KERNEL); > + if (!info->regulators) { > + dev_err(&pdev->dev, "Cannot allocate memory for regulators\n"); > + return -ENOMEM; > + } The set of regulators in the silicon doesn't vary - you should unconditionally register all the regulators. --8Dt/lkN5I0m8i0o7 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.22 (GNU/Linux) iQIcBAEBAgAGBQJSjNaoAAoJELSic+t+oim9Ks4QAITh22bu4KFUkdernccb4Pe+ J3BxtFhinCS1WFLgiQZp8qu8i9xTjHxxtzw+CZn8pF4/mZFnTGOptmajeoOwmbGw hyzP2yWq0EEJafAv6fPrvYxckm7cS6iqVW0SGdIXGP4BtRpaUFjc43mOcC/Vu1au httLzgaLFlOjH3LQEl3BYOvQSS+lu7i5UakhveW2RYITa4054YrqrOI8yxCpTuIK f/WichGNoOCgy/+LaVJGo+B3m9xLmLdBlckDHrTVOmyaVoR5cjvPd5MKuuseN1ur HvN7Kp1Y442/6rtBEcY1PZOmWgLwru+ZPtGIg0eKJ5OMN3TcJMVeK+T2HNqZ5P33 oigbzrn652rZ1K75ZneQL49KBFDDIEX7380xPdIGSoudqFgNQ7+LBhrFh8xKMTbm 9VKtAUJmkC+F1h+lUTeLlJyqFLFxNqWTMeG8rNeS3jD1iP0OIj925nHAPTWRXOEV Y+EI21o+K4n9KJLYa2yFf42OaScnnkE4BguUL3S+qFzntagbXB7HZxfXlm3/SFkG dpvg+Lsd5QiP9MpupSNitXoTRt+XSCCucTJPqZjaOLWMUe78ARcTvqE9zLVIyYnN 9MLsb5OmSaqsA15S+2/oFuNDrzKO24//h56nLsyoET0uolEA7LhRwk8kLDC+EZ1p w5Hr6Qk1V8oTalQOHTio =eHY1 -----END PGP SIGNATURE----- --8Dt/lkN5I0m8i0o7--