All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lee Jones <lee.jones@linaro.org>
To: Fabrice Gasnier <fabrice.gasnier@st.com>
Cc: thierry.reding@gmail.com, alexandre.torgue@st.com,
	benjamin.gaignard@linaro.org, robh+dt@kernel.org,
	mark.rutland@arm.com, linux@armlinux.org.uk,
	mcoquelin.stm32@gmail.com, benjamin.gaignard@st.com,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-pwm@vger.kernel.org
Subject: Re: [PATCH v3 3/6] pwm: stm32: add capture support
Date: Mon, 16 Apr 2018 13:24:26 +0100	[thread overview]
Message-ID: <20180416122426.72lhi3hrk2g7y526@dell> (raw)
In-Reply-To: <1522404084-24903-4-git-send-email-fabrice.gasnier@st.com>

On Fri, 30 Mar 2018, Fabrice Gasnier wrote:

> Add support for PMW input mode on pwm-stm32. STM32 timers support
> period and duty cycle capture as long as they have at least two PWM
> channels. One capture channel is used for period (rising-edge), one
> for duty-cycle (falling-edge).
> When there's only one channel available, only period can be captured.
> Duty-cycle is simply zero'ed in such a case.
> 
> Capture requires exclusive access (e.g. no pwm output running at the
> same time, to protect common prescaler).
> Timer DMA burst mode (from MFD core) is being used, to take two
> snapshots of capture registers (upon each period rising edge).
> 
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
> Reviewed-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> Acked-by: Thierry Reding <thierry.reding@gmail.com>
> ---
> Changes in v3:
> - update stm32_timers_dma_burst_read() call: don't pass ddata structure,
>   use MFD parent device structure instead since MFD core update.
> 
> Changes in v2:
> - DMA handling has been moved to MFD core. Rework capture routines to
>   use it.
> ---
>  drivers/pwm/pwm-stm32.c          | 176 +++++++++++++++++++++++++++++++++++++++
>  include/linux/mfd/stm32-timers.h |  11 +++

For my own reference:
  Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org>

>  2 files changed, 187 insertions(+)

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

WARNING: multiple messages have this Message-ID (diff)
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 3/6] pwm: stm32: add capture support
Date: Mon, 16 Apr 2018 13:24:26 +0100	[thread overview]
Message-ID: <20180416122426.72lhi3hrk2g7y526@dell> (raw)
In-Reply-To: <1522404084-24903-4-git-send-email-fabrice.gasnier@st.com>

On Fri, 30 Mar 2018, Fabrice Gasnier wrote:

> Add support for PMW input mode on pwm-stm32. STM32 timers support
> period and duty cycle capture as long as they have at least two PWM
> channels. One capture channel is used for period (rising-edge), one
> for duty-cycle (falling-edge).
> When there's only one channel available, only period can be captured.
> Duty-cycle is simply zero'ed in such a case.
> 
> Capture requires exclusive access (e.g. no pwm output running at the
> same time, to protect common prescaler).
> Timer DMA burst mode (from MFD core) is being used, to take two
> snapshots of capture registers (upon each period rising edge).
> 
> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@st.com>
> Reviewed-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> Acked-by: Thierry Reding <thierry.reding@gmail.com>
> ---
> Changes in v3:
> - update stm32_timers_dma_burst_read() call: don't pass ddata structure,
>   use MFD parent device structure instead since MFD core update.
> 
> Changes in v2:
> - DMA handling has been moved to MFD core. Rework capture routines to
>   use it.
> ---
>  drivers/pwm/pwm-stm32.c          | 176 +++++++++++++++++++++++++++++++++++++++
>  include/linux/mfd/stm32-timers.h |  11 +++

For my own reference:
  Acked-for-MFD-by: Lee Jones <lee.jones@linaro.org>

>  2 files changed, 187 insertions(+)

-- 
Lee Jones [???]
Linaro Services Technical Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2018-04-16 12:24 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-30 10:01 [PATCH v3 0/6] Add support for PWM input capture on STM32 Fabrice Gasnier
2018-03-30 10:01 ` Fabrice Gasnier
2018-03-30 10:01 ` Fabrice Gasnier
2018-03-30 10:01 ` [PATCH v3 1/6] dt-bindings: mfd: stm32-timers: add support for dmas Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-04-16 12:23   ` Lee Jones
2018-04-16 12:23     ` Lee Jones
2018-04-16 12:23     ` Lee Jones
2018-04-16 12:23       ` Lee Jones
2018-03-30 10:01 ` [PATCH v3 2/6] " Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-04-16 12:22   ` Lee Jones
2018-04-16 12:22     ` Lee Jones
2018-04-16 12:46     ` Fabrice Gasnier
2018-04-16 12:46       ` Fabrice Gasnier
2018-04-16 12:46       ` Fabrice Gasnier
2018-04-16 14:47       ` Lee Jones
2018-04-16 14:47         ` Lee Jones
2018-04-16 15:12         ` Fabrice Gasnier
2018-04-16 15:12           ` Fabrice Gasnier
2018-04-16 15:12           ` Fabrice Gasnier
2018-03-30 10:01 ` [PATCH v3 3/6] pwm: stm32: add capture support Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-04-16 12:24   ` Lee Jones [this message]
2018-04-16 12:24     ` Lee Jones
2018-03-30 10:01 ` [PATCH v3 4/6] pwm: stm32: improve capture by tuning counter prescaler Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01 ` [PATCH v3 5/6] pwm: stm32: use input prescaler to improve period capture Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-04-16 12:25   ` Lee Jones
2018-04-16 12:25     ` Lee Jones
2018-03-30 10:01 ` [PATCH v3 6/6] ARM: dts: stm32: Enable pwm3 input capture on stm32f429i-eval Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier
2018-03-30 10:01   ` Fabrice Gasnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180416122426.72lhi3hrk2g7y526@dell \
    --to=lee.jones@linaro.org \
    --cc=alexandre.torgue@st.com \
    --cc=benjamin.gaignard@linaro.org \
    --cc=benjamin.gaignard@st.com \
    --cc=devicetree@vger.kernel.org \
    --cc=fabrice.gasnier@st.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.