Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933777AbcKVWE2 (ORCPT ); Tue, 22 Nov 2016 17:04:28 -0500 Received: from mail.kmu-office.ch ([178.209.48.109]:49309 "EHLO mail.kmu-office.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755882AbcKVWE1 (ORCPT ); Tue, 22 Nov 2016 17:04:27 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Date: Tue, 22 Nov 2016 13:56:46 -0800 From: Stefan Agner To: Lukasz Majewski Cc: Thierry Reding , Sascha Hauer , Boris Brezillon , linux-pwm@vger.kernel.org, linux-kernel@vger.kernel.org, Fabio Estevam , Fabio Estevam , Lothar Wassmann , Bhuvanchandra DV , kernel@pengutronix.de Subject: Re: [PATCH v3 06/11] pwm: imx: Move PWMv2 wait for fifo slot code to a separate function In-Reply-To: <1477984230-18071-7-git-send-email-l.majewski@majess.pl> References: <1477984230-18071-1-git-send-email-l.majewski@majess.pl> <1477984230-18071-7-git-send-email-l.majewski@majess.pl> Message-ID: <9a3ad088c9fb9e533508563ad633aea6@agner.ch> User-Agent: Roundcube Webmail/1.1.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2893 Lines: 93 On 2016-11-01 00:10, Lukasz Majewski wrote: > The code, which waits for fifo slot, has been extracted from > imx_pwm_config_v2 function and moved to new one - imx_pwm_wait_fifo_slot(). > > This change reduces the overall size of imx_pwm_config_v2() and prepares > it for atomic PWM operation. > > Suggested-by: Stefan Agner > Suggested-by: Boris Brezillon > Signed-off-by: Lukasz Majewski Reviewed-by: Stefan Agner > --- > Changes for v3: > - None > > Changes for v2: > - None > --- > drivers/pwm/pwm-imx.c | 43 +++++++++++++++++++++++++------------------ > 1 file changed, 25 insertions(+), 18 deletions(-) > > diff --git a/drivers/pwm/pwm-imx.c b/drivers/pwm/pwm-imx.c > index b4e5803..ebe9b0c 100644 > --- a/drivers/pwm/pwm-imx.c > +++ b/drivers/pwm/pwm-imx.c > @@ -137,18 +137,36 @@ static void imx_pwm_sw_reset(struct pwm_chip *chip) > dev_warn(dev, "software reset timeout\n"); > } > > +static void imx_pwm_wait_fifo_slot(struct pwm_chip *chip, > + struct pwm_device *pwm) > +{ > + struct imx_chip *imx = to_imx_chip(chip); > + struct device *dev = chip->dev; > + unsigned int period_ms; > + int fifoav; > + u32 sr; > + > + sr = readl(imx->mmio_base + MX3_PWMSR); > + fifoav = sr & MX3_PWMSR_FIFOAV_MASK; > + if (fifoav == MX3_PWMSR_FIFOAV_4WORDS) { > + period_ms = DIV_ROUND_UP(pwm_get_period(pwm), > + NSEC_PER_MSEC); > + msleep(period_ms); > + > + sr = readl(imx->mmio_base + MX3_PWMSR); > + if (fifoav == (sr & MX3_PWMSR_FIFOAV_MASK)) > + dev_warn(dev, "there is no free FIFO slot\n"); > + } > +} > > static int imx_pwm_config_v2(struct pwm_chip *chip, > struct pwm_device *pwm, int duty_ns, int period_ns) > { > struct imx_chip *imx = to_imx_chip(chip); > - struct device *dev = chip->dev; > unsigned long long c; > unsigned long period_cycles, duty_cycles, prescale; > - unsigned int period_ms; > bool enable = pwm_is_enabled(pwm); > - int fifoav; > - u32 cr, sr; > + u32 cr; > > /* > * i.MX PWMv2 has a 4-word sample FIFO. > @@ -157,21 +175,10 @@ static int imx_pwm_config_v2(struct pwm_chip *chip, > * wait for a full PWM cycle to get a relinquished FIFO slot > * when the controller is enabled and the FIFO is fully loaded. > */ > - if (enable) { > - sr = readl(imx->mmio_base + MX3_PWMSR); > - fifoav = sr & MX3_PWMSR_FIFOAV_MASK; > - if (fifoav == MX3_PWMSR_FIFOAV_4WORDS) { > - period_ms = DIV_ROUND_UP(pwm_get_period(pwm), > - NSEC_PER_MSEC); > - msleep(period_ms); > - > - sr = readl(imx->mmio_base + MX3_PWMSR); > - if (fifoav == (sr & MX3_PWMSR_FIFOAV_MASK)) > - dev_warn(dev, "there is no free FIFO slot\n"); > - } > - } else { > + if (enable) > + imx_pwm_wait_fifo_slot(chip, pwm); > + else > imx_pwm_sw_reset(chip); > - } > > c = clk_get_rate(imx->clk_per); > c = c * period_ns;