Re: [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema

Re: [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema

[Krzysztof Kozlowski]

On Mon, Jun 29, 2026 at 12:58:37PM +0000, Charan Pedumuru wrote:
> Convert STMicroelectronics STiH41x PWM/Capture controller binding
> to DT schema.
> Changes during conversion:
> - Fix compatible string from "st,pwm" to "st,sti-pwm" to match the
>   actual hardware variant naming convention used across STi bindings.
> - Drop pinctrl-names from the required list as pinctrl properties are
>   inherited and validated by the pinctrl schema.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

Best regards,
Krzysztof

Re: [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 1679 bytes --]

On Mon, Jun 29, 2026 at 12:58:37PM +0000, Charan Pedumuru wrote:
> Convert STMicroelectronics STiH41x PWM/Capture controller binding
> to DT schema.
> Changes during conversion:
> - Fix compatible string from "st,pwm" to "st,sti-pwm" to match the
>   actual hardware variant naming convention used across STi bindings.
> - Drop pinctrl-names from the required list as pinctrl properties are
>   inherited and validated by the pinctrl schema.
> 
> Signed-off-by: Charan Pedumuru <charan.pedumuru@gmail.com>
> ---
> Changes in v5:
> - Enforce stricter validation of the clock-names property.
> - Link to v4: https://patch.msgid.link/20260625-st-pwm-v4-1-958d7d6bdf39@gmail.com
> 
> Changes in v4:
> - Fix compatible to "st,sti-pwm" in examples.
> - Modify subject line to point to the correct YAML file.
> - Link to v3: https://patch.msgid.link/20260625-st-pwm-v3-1-ace9bff88529@gmail.com
> 
> Changes in v3:
> - Fix compatible to "st,sti-pwm".
> - Drop pinctrl-names from required.
> - Modify the commit message to explain the changes made.
> - Change maintainer to "Lee Jones <lee.jones@linaro.org>".

I wonder if Lee has a comment about that. In v2 the maintainer was

	Ajit Pal Singh <ajitpal.singh@st.com>

and I don't see Lee agree to be named maintainer. The only rationale I
can find was

	Also the maintainer email doesn't exist now. I will change that
	too with other maintainer.

in
https://lore.kernel.org/all/9379bfdb-9a9c-4fea-924c-6c1dc8e997c1@gmail.com/
. At some point in the past lee.jones@linaro.org didn't exist either. I
added that address to the list of recipents to test.

Otherwise looks fine.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 1177 bytes --]

Hello,

On Mon, Aug 10, 2026 at 10:32:52AM +0200, Uwe Kleine-König wrote:
> > Changes in v3:
> > [...]
> > - Change maintainer to "Lee Jones <lee.jones@linaro.org>".
> 
> I wonder if Lee has a comment about that. In v2 the maintainer was
> 
> 	Ajit Pal Singh <ajitpal.singh@st.com>
> 
> and I don't see Lee agree to be named maintainer. The only rationale I
> can find was
> 
> 	Also the maintainer email doesn't exist now. I will change that
> 	too with other maintainer.
> 
> in
> https://lore.kernel.org/all/9379bfdb-9a9c-4fea-924c-6c1dc8e997c1@gmail.com/
> . At some point in the past lee.jones@linaro.org didn't exist either. I
> added that address to the list of recipents to test.

FTR: I got a Delivery Status Notification:

	Final-Recipient: rfc822; lee.jones@linaro.org
	Action: failed
	Status: 5.1.3
	Diagnostic-Code: smtp; The email account that you tried to reach does not exist. Please try double-checking the recipient's email address for typos or unnecessary spaces. For more information, go to https://support.google.com/mail/?p=NoSuchUser

I marked the patch as "Changes requested" in patchwork accordingly.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema

[Lee Jones]

On Mon, 10 Aug 2026, Uwe Kleine-König wrote:

> Hello,
> 
> On Mon, Aug 10, 2026 at 10:32:52AM +0200, Uwe Kleine-König wrote:
> > > Changes in v3:
> > > [...]
> > > - Change maintainer to "Lee Jones <lee.jones@linaro.org>".
> > 
> > I wonder if Lee has a comment about that. In v2 the maintainer was
> > 
> > 	Ajit Pal Singh <ajitpal.singh@st.com>
> > 
> > and I don't see Lee agree to be named maintainer. The only rationale I
> > can find was
> > 
> > 	Also the maintainer email doesn't exist now. I will change that
> > 	too with other maintainer.
> > 
> > in
> > https://lore.kernel.org/all/9379bfdb-9a9c-4fea-924c-6c1dc8e997c1@gmail.com/
> > . At some point in the past lee.jones@linaro.org didn't exist either. I
> > added that address to the list of recipents to test.
> 
> FTR: I got a Delivery Status Notification:
> 
> 	Final-Recipient: rfc822; lee.jones@linaro.org
> 	Action: failed
> 	Status: 5.1.3
> 	Diagnostic-Code: smtp; The email account that you tried to reach does not exist. Please try double-checking the recipient's email address for typos or unnecessary spaces. For more information, go to https://support.google.com/mail/?p=NoSuchUser
> 
> I marked the patch as "Changes requested" in patchwork accordingly.

Right, and I don't have much to do with this driver anymore.

If you have to put one, put the PWM maintainer(s).

-- 
Lee Jones