[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <c78c1c19-cf11-4146-acda-fd435add6808@foss.st.com>
Date: Fri, 25 Apr 2025 14:50:06 +0200
From: Fabrice Gasnier <fabrice.gasnier@...s.st.com>
To: Lee Jones <lee@...nel.org>
CC: <ukleinek@...nel.org>, <alexandre.torgue@...s.st.com>,
<krzk+dt@...nel.org>, <conor+dt@...nel.org>, <jic23@...nel.org>,
<daniel.lezcano@...aro.org>, <tglx@...utronix.de>, <robh@...nel.org>,
<catalin.marinas@....com>, <will@...nel.org>,
<devicetree@...r.kernel.org>, <wbg@...nel.org>,
<linux-stm32@...md-mailman.stormreply.com>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<linux-iio@...r.kernel.org>, <linux-pwm@...r.kernel.org>,
<olivier.moysan@...s.st.com>
Subject: Re: [PATCH v4 2/8] mfd: stm32-lptimer: add support for stm32mp25
On 4/24/25 15:01, Lee Jones wrote:
> On Fri, 04 Apr 2025, Lee Jones wrote:
>
>> On Fri, 14 Mar 2025, Fabrice Gasnier wrote:
>>
>>> Add support for STM32MP25 SoC.
>>> A new hardware configuration register (HWCFGR2) has been added, to gather
>>> number of capture/compare channels, autonomous mode and input capture
>>> capability. The full feature set is implemented in LPTIM1/2/3/4. LPTIM5
>>> supports a smaller set of features. This can now be read from HWCFGR
>>> registers.
>>>
>>> Add new registers to the stm32-lptimer.h: CCMR1, CCR2, HWCFGR1/2 and VERR.
>>> Update the stm32_lptimer data struct so signal the number of
>>> capture/compare channels to the child devices.
>>> Also Remove some unused bit masks (CMPOK_ARROK / CMPOKCF_ARROKCF).
>>>
>>> Signed-off-by: Fabrice Gasnier <fabrice.gasnier@...s.st.com>
>>> ---
>>> Changes in V4:
>>> - Add DIEROK, ARROK status flags, and their clear flags.
>>> Changes in V2:
>>> - rely on fallback compatible as no specific .data is associated to the
>>> driver. Compatibility is added by reading hardware configuration
>>> registers.
>>> - read version register, to be used by clockevent child driver
>>> - rename register/bits definitions
>>> ---
>>> drivers/mfd/stm32-lptimer.c | 33 ++++++++++++++++++++++++++-
>>> include/linux/mfd/stm32-lptimer.h | 37 ++++++++++++++++++++++++++++---
>>
>> At least the Clocksource driver depends on this.
>>
>> I need Acks from the other Maintainers before I can merge this.
>
> Suggest you resubmit the set as a [RESEND] to re-gain traction.
>
Hi Lee,
Thanks for suggesting.
I recently found I needed to add a small delay in clocksource driver. So
I just have sent a V5.
Best Regards,
Fabrice
Powered by blists - more mailing lists