Devicetree
 help / color / mirror / Atom feed
From: "Uwe Kleine-König" <ukleinek@kernel.org>
To: Charan Pedumuru <charan.pedumuru@gmail.com>,
	 Lee Jones <lee@kernel.org>, Lee Jones <lee.jones@linaro.org>
Cc: Rob Herring <robh@kernel.org>,
	 Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-pwm@vger.kernel.org,  devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema
Date: Mon, 10 Aug 2026 10:32:50 +0200	[thread overview]
Message-ID: <anmK4os44Uz27oR9@monoceros> (raw)
In-Reply-To: <20260629-st-pwm-v5-1-a93f2bfec38c@gmail.com>

[-- 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 --]

  parent reply	other threads:[~2026-08-10  8:32 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-29 12:58 [PATCH v5] dt-bindings: pwm: st,sti-pwm: convert to DT schema Charan Pedumuru
2026-06-29 13:31 ` sashiko-bot
2026-06-30  6:32 ` Krzysztof Kozlowski
2026-08-10  8:32 ` Uwe Kleine-König [this message]
2026-08-10  9:04   ` Uwe Kleine-König
2026-08-10 11:03     ` 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=anmK4os44Uz27oR9@monoceros \
    --to=ukleinek@kernel.org \
    --cc=charan.pedumuru@gmail.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=lee.jones@linaro.org \
    --cc=lee@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=robh@kernel.org \
    /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