linux-kernel - Re: [RESEND v7 0/6] clockevent: add low power STM32 timer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 16 Jun 2020 11:44:01 +0000
From:   Benjamin GAIGNARD <benjamin.gaignard@...com>
To:     Fabrice GASNIER <fabrice.gasnier@...com>,
        "lee.jones@...aro.org" <lee.jones@...aro.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
        Alexandre TORGUE <alexandre.torgue@...com>,
        "linux@...linux.org.uk" <linux@...linux.org.uk>,
        "daniel.lezcano@...aro.org" <daniel.lezcano@...aro.org>,
        "tglx@...utronix.de" <tglx@...utronix.de>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-stm32@...md-mailman.stormreply.com" 
        <linux-stm32@...md-mailman.stormreply.com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [RESEND v7 0/6] clockevent: add low power STM32 timer



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
>