From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexandre Belloni Subject: Re: [PATCH 1/2] PWM: atmel: fix incorrect CDTY value after enabling Date: Tue, 26 May 2015 13:51:55 +0200 Message-ID: <20150526115155.GF3199@piout.net> References: <1432559996-4415-1-git-send-email-alexandre.belloni@free-electrons.com> <5564366D.9090707@atmel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <5564366D.9090707@atmel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Nicolas Ferre Cc: linux-pwm@vger.kernel.org, Thierry Reding , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: linux-pwm@vger.kernel.org On 26/05/2015 at 11:01:33 +0200, Nicolas Ferre wrote : > Le 25/05/2015 15:19, Alexandre Belloni a =E9crit : > > CUPD is not flushed before enabling the channel so it will update CDTY/= CPRD just > > after one period. So we always set CUPD, even when the channel is not e= nabled. > > = > > Signed-off-by: Alexandre Belloni > = > Acked-by: Nicolas Ferre > = > BTW, is a "stable tag" needed for this one? > = I had a look and both can probably be backported up to 3.14. -- = Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: alexandre.belloni@free-electrons.com (Alexandre Belloni) Date: Tue, 26 May 2015 13:51:55 +0200 Subject: [PATCH 1/2] PWM: atmel: fix incorrect CDTY value after enabling In-Reply-To: <5564366D.9090707@atmel.com> References: <1432559996-4415-1-git-send-email-alexandre.belloni@free-electrons.com> <5564366D.9090707@atmel.com> Message-ID: <20150526115155.GF3199@piout.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 26/05/2015 at 11:01:33 +0200, Nicolas Ferre wrote : > Le 25/05/2015 15:19, Alexandre Belloni a ?crit : > > CUPD is not flushed before enabling the channel so it will update CDTY/CPRD just > > after one period. So we always set CUPD, even when the channel is not enabled. > > > > Signed-off-by: Alexandre Belloni > > Acked-by: Nicolas Ferre > > BTW, is a "stable tag" needed for this one? > I had a look and both can probably be backported up to 3.14. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751061AbbEZNqQ (ORCPT ); Tue, 26 May 2015 09:46:16 -0400 Received: from down.free-electrons.com ([37.187.137.238]:57795 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753177AbbEZNqN (ORCPT ); Tue, 26 May 2015 09:46:13 -0400 Date: Tue, 26 May 2015 13:51:55 +0200 From: Alexandre Belloni To: Nicolas Ferre Cc: Thierry Reding , linux-pwm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] PWM: atmel: fix incorrect CDTY value after enabling Message-ID: <20150526115155.GF3199@piout.net> References: <1432559996-4415-1-git-send-email-alexandre.belloni@free-electrons.com> <5564366D.9090707@atmel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <5564366D.9090707@atmel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 26/05/2015 at 11:01:33 +0200, Nicolas Ferre wrote : > Le 25/05/2015 15:19, Alexandre Belloni a écrit : > > CUPD is not flushed before enabling the channel so it will update CDTY/CPRD just > > after one period. So we always set CUPD, even when the channel is not enabled. > > > > Signed-off-by: Alexandre Belloni > > Acked-by: Nicolas Ferre > > BTW, is a "stable tag" needed for this one? > I had a look and both can probably be backported up to 3.14. -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com