linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin GAIGNARD <benjamin.gaignard@st.com>
To: Fabrice GASNIER <fabrice.gasnier@st.com>,
	"lee.jones@linaro.org" <lee.jones@linaro.org>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	"mcoquelin.stm32@gmail.com" <mcoquelin.stm32@gmail.com>,
	Alexandre TORGUE <alexandre.torgue@st.com>,
	"linux@armlinux.org.uk" <linux@armlinux.org.uk>,
	"daniel.lezcano@linaro.org" <daniel.lezcano@linaro.org>,
	"tglx@linutronix.de" <tglx@linutronix.de>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linux-stm32@st-md-mailman.stormreply.com" 
	<linux-stm32@st-md-mailman.stormreply.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RESEND v7 0/6] clockevent: add low power STM32 timer
Date: Tue, 16 Jun 2020 11:44:01 +0000	[thread overview]
Message-ID: <50df6585-d9df-4cb3-5589-aab2c97cad51@st.com> (raw)
In-Reply-To: <20200603125439.23275-1-benjamin.gaignard@st.com>



On 6/3/20 2:54 PM, Benjamin Gaignard wrote:
> This series add low power timer as boadcast clockevent device.
> Low power timer could runs even when CPUs are in idle mode and
> could wakeup them.
>
> Lee has acked the MFD part.
> Clocksource driver still need to be reviewed by maintainers.
> Add missing part of the bindings to describe interrupt.

Hi Lee,

I think everything has been reviewed and acked in this series.
May I have forgot something that prevent it to be merge ?

Benjamin

>
> version 7 resend:
> - with Daniel ack for driver patch
> - with Rob review for bindings patch
>
> version 7:
> - rebased on top of v5.7-rc2
>
> version 6:
> - simplify binding, DT and code to use only one interrupt
>
> version 5:
> - document interrupts and interrupt-names bindings
> - use a different wake up interrupt
> - add device-tree patch
> - make STM32MP157 select low power timer configuration flag
> - enable fast_io in regmap configuration
>
> version 4:
> - move defines in mfd/stm32-lptimer.h
> - change compatible and subnode names
> - document wakeup-source property
> - reword commit message
> - make driver Kconfig depends of MFD_STM32_LPTIMER
> - remove useless include
> - remove rate and clk fields from the private structure
> - to add comments about the registers sequence in stm32_clkevent_lp_set_timer
> - rework probe function and use devm_request_irq()
> - do not allow module to be removed
>
> version 3:
> - fix timer set sequence
> - don't forget to free irq on remove function
> - use devm_kzalloc to simplify errors handling in probe function
>
> version 2:
> - stm32 clkevent driver is now a child of the stm32 lp timer node
> - add a probe function and adpat the driver to use regmap provide
>    by it parent
> - stop using timer_of helpers
>
>
> Benjamin Gaignard (6):
>    dt-bindings: mfd: Document STM32 low power timer bindings
>    ARM: dts: stm32: Add timer subnodes on stm32mp15 SoCs
>    mfd: stm32: Add defines to be used for clkevent purpose
>    mfd: stm32: enable regmap fast_io for stm32-lptimer
>    clocksource: Add Low Power STM32 timers driver
>    ARM: mach-stm32: select low power timer for STM32MP157
>
>   .../devicetree/bindings/mfd/st,stm32-lptimer.yaml  |   5 +
>   arch/arm/boot/dts/stm32mp151.dtsi                  |  35 ++++
>   arch/arm/mach-stm32/Kconfig                        |   1 +
>   drivers/clocksource/Kconfig                        |   4 +
>   drivers/clocksource/Makefile                       |   1 +
>   drivers/clocksource/timer-stm32-lp.c               | 221 +++++++++++++++++++++
>   drivers/mfd/stm32-lptimer.c                        |   1 +
>   include/linux/mfd/stm32-lptimer.h                  |   5 +
>   8 files changed, 273 insertions(+)
>   create mode 100644 drivers/clocksource/timer-stm32-lp.c
>

  parent reply	other threads:[~2020-06-16 11:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-03 12:54 [RESEND v7 0/6] clockevent: add low power STM32 timer Benjamin Gaignard
2020-06-03 12:54 ` [RESEND v7 1/6] dt-bindings: mfd: Document STM32 low power timer bindings Benjamin Gaignard
2020-06-03 12:54 ` [RESEND v7 2/6] ARM: dts: stm32: Add timer subnodes on stm32mp15 SoCs Benjamin Gaignard
2020-06-03 12:54 ` [RESEND v7 3/6] mfd: stm32: Add defines to be used for clkevent purpose Benjamin Gaignard
2020-06-03 12:54 ` [RESEND v7 4/6] mfd: stm32: enable regmap fast_io for stm32-lptimer Benjamin Gaignard
2020-06-03 12:54 ` [RESEND v7 5/6] clocksource: Add Low Power STM32 timers driver Benjamin Gaignard
2020-06-03 12:54 ` [RESEND v7 6/6] ARM: mach-stm32: select low power timer for STM32MP157 Benjamin Gaignard
2020-06-16 11:44 ` Benjamin GAIGNARD [this message]
2020-06-18 10:21 ` [GIT PULL] Immutable branch between MFD and Clocksource due for the v5.9 merge window Lee Jones

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=50df6585-d9df-4cb3-5589-aab2c97cad51@st.com \
    --to=benjamin.gaignard@st.com \
    --cc=alexandre.torgue@st.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=fabrice.gasnier@st.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-stm32@st-md-mailman.stormreply.com \
    --cc=linux@armlinux.org.uk \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=tglx@linutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).