Thread (12 messages) 12 messages, 5 authors, 2026-04-02

Re: [PATCH 0/2] Add PWM support Amlogic S7 S7D S6

From: Xianwei Zhao <xianwei.zhao@amlogic.com>
Date: 2026-03-31 07:10:27
Also in: linux-amlogic, linux-devicetree, linux-pwm, lkml

Hi Martin,
     I confirmed with Junyi Zhao that the current implementation counts 
from zero, so this submission is correct.
We agree this should be fixed and will address it in a follow-up patch.
Thanks for pointing it out.

On 2026/3/31 05:54, Martin Blumenstingl wrote:
Hi Xianwei Zhao,

thanks for your contribution!

On Thu, Mar 26, 2026 at 7:35 AM Xianwei Zhao via B4 Relay
[off-list ref]  wrote:
quoted
Add bindings and driver support Amlogic S7/S7D/S6 SoCs.
There is an old report that got lost, stating that the current
pwm-meson driver has an off-by-one error with the hi and lo fields:
[0]
Since you are working on bringing up a new platform: is this something
you can verify in your lab?
To be clear: I'm not expecting you to work on this ad-hoc or bring a
patch into this series. However, it would be great if you could verify
if the findings from [0] are correct and send an updated patch in
future.

Thank you and best regards
Martin
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help