From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753483AbcD0KlA (ORCPT ); Wed, 27 Apr 2016 06:41:00 -0400 Received: from mga01.intel.com ([192.55.52.88]:45630 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750747AbcD0Kk6 (ORCPT ); Wed, 27 Apr 2016 06:40:58 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,541,1455004800"; d="asc'?scan'208";a="92817506" From: Felipe Balbi To: Mark Brown Cc: Jisheng Zhang , mathias.nyman@linux.intel.com, gregkh@linuxfoundation.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, yendapally.reddy@broadcom.com Subject: Re: [RESEND PATCH v2 7/7] usb: xhci: plat: add vbus regulator control In-Reply-To: <20160427103521.GJ3217@sirena.org.uk> References: <1461675460-2295-1-git-send-email-jszhang@marvell.com> <1461675460-2295-8-git-send-email-jszhang@marvell.com> <87bn4vpq7j.fsf@intel.com> <20160427095738.GG3217@sirena.org.uk> <87oa8vmjqg.fsf@intel.com> <20160427103521.GJ3217@sirena.org.uk> User-Agent: Notmuch/0.21+96~g9bbc54b (http://notmuchmail.org) Emacs/25.0.90.3 (x86_64-pc-linux-gnu) Date: Wed, 27 Apr 2016 13:38:53 +0300 Message-ID: <87k2jjmj42.fsf@intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi, Mark Brown writes: > On Wed, Apr 27, 2016 at 01:25:27PM +0300, Felipe Balbi wrote: >> Mark Brown writes: > >> > this to be just a normal regulator_get(). > >> jokes aside, this regulator is optional because not all platforms >> require a SW controlled regulator, no ? Will normal regulator_get() give >> us a dummy regulator in case it's not listed in DT/ACPI ? > > Yes we do that, but even regulators that are not software controlled okay, good. > should really be described anyway since it's a much simpler rule for okay, we'll wait until all vendors update their ACPI tables ;-) > people to understand, it ensures that we can just scale up on systems > where there does happen to be software control and it makes all the > resulting code much simpler and hence less error prone if we're not > randomly ignoring some errors. =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJXIJa+AAoJEIaOsuA1yqREllkQAKqmaqv5PzWNryNW7fwIq+lC IyKQgpCGiHdvEfPKa5p8vO8HgGGHFOXYCygDsRU9lauHkZwEFZ8lCIsdUj+ayJ2Q i8RIfxe8VwAJH/iME0wapt2QvVGpaoqJsODdoZNm2YDu6CgELSKpbATgRWJq9mRP 0cCVTM+kRgsJ1DoquFtD+zhBYi93tjXfhIKJEd0LA/gvJnuC1UcfDV+dBuBzfFt9 BXPfHHnJRnt0fAooLKYIYmzq5l5CcIy53KVuCD8QX8Suz/758A3wChuM+7MrKSqr FWoF/Tp6Xaxd9jvHTVEnt4djdjK9eMk3S4yvA9YkMugQRmxB8u3XB9AMo4/jsneP UDJGvEyhStLXD2clRERU5673yanGV+b3w27Uad7HHF6sIbObxuuz8vXDSY4WbZ8x mn7DA2+6f9uPD7iyZG2ToTOwBZsvzm5ZJ6Xy0B4azSkn8lFI1GZPoIso1+os5A7N f3Hp6Ph0EOPpNlIVQxRNMTLBiFHfqRduP5a29zZ+nWS22LPFUlVR94r9JoHTArGJ tD1mLSVDt3DMh4ikH/qtdrTPKJG30ZXO4E63cB1/tw/TM045XXYyGBMh68VYw6i2 3vXPA0KdXk3q6Ub45zKfIxoru+4ILz7SSOBla0mlunYCRckItJyDbs9Fgr/tyL7Q 52/whUx+CrTYj/BPr3Z4 =OKyB -----END PGP SIGNATURE----- --=-=-=-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: felipe.balbi@linux.intel.com (Felipe Balbi) Date: Wed, 27 Apr 2016 13:38:53 +0300 Subject: [RESEND PATCH v2 7/7] usb: xhci: plat: add vbus regulator control In-Reply-To: <20160427103521.GJ3217@sirena.org.uk> References: <1461675460-2295-1-git-send-email-jszhang@marvell.com> <1461675460-2295-8-git-send-email-jszhang@marvell.com> <87bn4vpq7j.fsf@intel.com> <20160427095738.GG3217@sirena.org.uk> <87oa8vmjqg.fsf@intel.com> <20160427103521.GJ3217@sirena.org.uk> Message-ID: <87k2jjmj42.fsf@intel.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Mark Brown writes: > On Wed, Apr 27, 2016 at 01:25:27PM +0300, Felipe Balbi wrote: >> Mark Brown writes: > >> > this to be just a normal regulator_get(). > >> jokes aside, this regulator is optional because not all platforms >> require a SW controlled regulator, no ? Will normal regulator_get() give >> us a dummy regulator in case it's not listed in DT/ACPI ? > > Yes we do that, but even regulators that are not software controlled okay, good. > should really be described anyway since it's a much simpler rule for okay, we'll wait until all vendors update their ACPI tables ;-) > people to understand, it ensures that we can just scale up on systems > where there does happen to be software control and it makes all the > resulting code much simpler and hence less error prone if we're not > randomly ignoring some errors. -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 818 bytes Desc: not available URL: