From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752505AbcEKOvG (ORCPT ); Wed, 11 May 2016 10:51:06 -0400 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:44212 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752088AbcEKOvA (ORCPT ); Wed, 11 May 2016 10:51:00 -0400 Date: Wed, 11 May 2016 15:50:46 +0100 From: Mark Brown To: Henry Chen Cc: Rob Herring , Liam Girdwood , Matthias Brugger , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Stephen Boyd , Bjorn Andersson , Laxman Dewangan , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Message-ID: <20160511145046.GI6261@sirena.org.uk> References: <1462959090-2877-1-git-send-email-henryc.chen@mediatek.com> <1462959090-2877-3-git-send-email-henryc.chen@mediatek.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="7IgncvKP0CVPV/ZZ" Content-Disposition: inline In-Reply-To: <1462959090-2877-3-git-send-email-henryc.chen@mediatek.com> X-Cookie: When in doubt, follow your heart. User-Agent: Mutt/1.6.0 (2016-04-01) X-SA-Exim-Connect-IP: 94.175.94.161 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH RESEND 2/2] regulator: mt6397: Add buck change mode regulator interface for mt6397 X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --7IgncvKP0CVPV/ZZ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, May 11, 2016 at 05:31:30PM +0800, Henry Chen wrote: > +The possible values for "regulator-supported-modes" are: > + 0: PWM mode, mapping to regulator fast mode, allow the > + converter to remain in the PWM mode regardless of the load current, so > + that the noise spectrum of the converter can be minimized for certain > + highly-noise-sensitive handset applications. > + 1: Auto mode, mapping to regulator normal mode, mode switch by current > + loading. At light load currents, the converter automatically enters > + PFM mode to save power and improve light load efficiency. At heavy > + load currents, the converter automatically enters PWM mode to minimiz > + the noise spectrum of the converter. The details of how this maps onto Linux shouldn't be in the bindings. > + regulator-allow-change-mode; > + regulator-supported-modes =3D <0 1>; These are problematic too as covered in my other mail.=20 > -static struct regulator_ops mt6397_volt_range_ops =3D { > +static const struct regulator_ops mt6397_volt_range_ops =3D { The constification is fine but an unrelated change. Otherwise this looks fine from a code point of view, let's get that merged separately. --7IgncvKP0CVPV/ZZ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJXM0bFAAoJECTWi3JdVIfQWUIH/iYT3d/hj9humaXhx493iF37 DIeyno9cdT4e1Fc8W6Qr7Tr/lAOn5j9mBTilozxp0Y5lPk5SmbZhQnLWuVGBcetQ Zq1VjqZ8pksAWhNwb3j6M/UyOuFmHRQZRpoV35xbLj4JTWdATrBAb0wqJkSmQa0c 5BI1NS0hMaq0DcH5CODK2uP5u4ka5fMllT/UrWtq+FskWLIarhM8vrMvrqNzQewp +DVV8UftScFV20RUtv/barWrVGUtXzOf6+XMs3FuZi21XUG4I+QCpNCdHxHzYkcB A2ZDQHUKR/7c9DVd+Q6bliNZ6Bgjy52gsnXQMJc618j2/Hf+hvkF4xbezdeZfps= =Rydm -----END PGP SIGNATURE----- --7IgncvKP0CVPV/ZZ--