From mboxrd@z Thu Jan 1 00:00:00 1970 From: boris.brezillon@free-electrons.com (Boris Brezillon) Date: Tue, 14 Jul 2015 13:16:28 +0200 Subject: [RFC PATCH 14/15] regulator: pwm: implement ->enable(), ->disable() and ->is_enabled methods In-Reply-To: <20150714110819.GU11162@sirena.org.uk> References: <1435738921-25027-1-git-send-email-boris.brezillon@free-electrons.com> <1435738921-25027-15-git-send-email-boris.brezillon@free-electrons.com> <20150714105019.GR11162@sirena.org.uk> <20150714130222.1bdfedc5@bbrezillon> <20150714110819.GU11162@sirena.org.uk> Message-ID: <20150714131628.0c87f0a4@bbrezillon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 14 Jul 2015 12:08:19 +0100 Mark Brown wrote: > On Tue, Jul 14, 2015 at 01:02:22PM +0200, Boris Brezillon wrote: > > Mark Brown wrote: > > > > On Wed, Jul 01, 2015 at 10:22:00AM +0200, Boris Brezillon wrote: > > > > Implement the ->enable(), ->disable() and ->is_enabled methods and remove > > > > the PWM call in ->set_voltage_sel(). > > > > This doesn't apply, please check and resend. > > > This series was made on top of Linus' tree (4.2-rc1 IIRC) and patch 14 > > and 15 were not meant to be applied without Thierry's approval (they > > depend on other changes in the PWM framework). > > I can rebase them on top of linux-next (or just on top of Linus' > > 4.2-rc2) if Thierry is okay with that, but I don't think rebasing them > > on your regulator's for-next branch is a good idea. > > What is the dependency for the enable patch? Oh, indeed. This patch has no dependency on the PWM changes. I'll send it on its own then. Thanks, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [RFC PATCH 14/15] regulator: pwm: implement ->enable(), ->disable() and ->is_enabled methods Date: Tue, 14 Jul 2015 13:16:28 +0200 Message-ID: <20150714131628.0c87f0a4@bbrezillon> References: <1435738921-25027-1-git-send-email-boris.brezillon@free-electrons.com> <1435738921-25027-15-git-send-email-boris.brezillon@free-electrons.com> <20150714105019.GR11162@sirena.org.uk> <20150714130222.1bdfedc5@bbrezillon> <20150714110819.GU11162@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150714110819.GU11162-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mark Brown Cc: Thierry Reding , linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Liam Girdwood , Bryan Wu , Richard Purdie , Jacek Anaszewski , linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Jean-Christophe Plagniol-Villard , Tomi Valkeinen , linux-fbdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Stephen Warren , Alexandre Courbot , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Maxime Ripard , Jingoo Han , Lee Jones , Doug Anderson List-Id: linux-leds@vger.kernel.org On Tue, 14 Jul 2015 12:08:19 +0100 Mark Brown wrote: > On Tue, Jul 14, 2015 at 01:02:22PM +0200, Boris Brezillon wrote: > > Mark Brown wrote: > > > > On Wed, Jul 01, 2015 at 10:22:00AM +0200, Boris Brezillon wrote: > > > > Implement the ->enable(), ->disable() and ->is_enabled methods and remove > > > > the PWM call in ->set_voltage_sel(). > > > > This doesn't apply, please check and resend. > > > This series was made on top of Linus' tree (4.2-rc1 IIRC) and patch 14 > > and 15 were not meant to be applied without Thierry's approval (they > > depend on other changes in the PWM framework). > > I can rebase them on top of linux-next (or just on top of Linus' > > 4.2-rc2) if Thierry is okay with that, but I don't think rebasing them > > on your regulator's for-next branch is a good idea. > > What is the dependency for the enable patch? Oh, indeed. This patch has no dependency on the PWM changes. I'll send it on its own then. Thanks, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Date: Tue, 14 Jul 2015 11:16:28 +0000 Subject: Re: [RFC PATCH 14/15] regulator: pwm: implement ->enable(), ->disable() and ->is_enabled methods Message-Id: <20150714131628.0c87f0a4@bbrezillon> List-Id: References: <1435738921-25027-1-git-send-email-boris.brezillon@free-electrons.com> <1435738921-25027-15-git-send-email-boris.brezillon@free-electrons.com> <20150714105019.GR11162@sirena.org.uk> <20150714130222.1bdfedc5@bbrezillon> <20150714110819.GU11162@sirena.org.uk> In-Reply-To: <20150714110819.GU11162@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Tue, 14 Jul 2015 12:08:19 +0100 Mark Brown wrote: > On Tue, Jul 14, 2015 at 01:02:22PM +0200, Boris Brezillon wrote: > > Mark Brown wrote: > > > > On Wed, Jul 01, 2015 at 10:22:00AM +0200, Boris Brezillon wrote: > > > > Implement the ->enable(), ->disable() and ->is_enabled methods and remove > > > > the PWM call in ->set_voltage_sel(). > > > > This doesn't apply, please check and resend. > > > This series was made on top of Linus' tree (4.2-rc1 IIRC) and patch 14 > > and 15 were not meant to be applied without Thierry's approval (they > > depend on other changes in the PWM framework). > > I can rebase them on top of linux-next (or just on top of Linus' > > 4.2-rc2) if Thierry is okay with that, but I don't think rebasing them > > on your regulator's for-next branch is a good idea. > > What is the dependency for the enable patch? Oh, indeed. This patch has no dependency on the PWM changes. I'll send it on its own then. Thanks, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com