devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>
To: george.moussalem@outlook.com,
	"Uwe Kleine-König" <ukleinek@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Baruch Siach" <baruch@tkos.co.il>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konradybcio@kernel.org>
Cc: linux-arm-msm@vger.kernel.org, linux-pwm@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Devi Priya <quic_devipriy@quicinc.com>,
	Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Baruch Siach <baruch.siach@siklu.com>
Subject: Re: [PATCH v16 0/9] Add PWM support for IPQ chipsets
Date: Fri, 3 Oct 2025 14:34:31 +0530	[thread overview]
Message-ID: <3cb50330-effc-4089-b80c-d454bccc63f3@oss.qualcomm.com> (raw)
In-Reply-To: <20251001-ipq-pwm-v16-0-300f237e0e68@outlook.com>


On 10/1/2025 7:34 PM, George Moussalem via B4 Relay wrote:
> Add PWM driver and binding support for IPQ chipsets.
> Also, add nodes to add support for pwm in ipq6018, ipq5018, ipq5332, and
> ipq9574.
>
> I've picked up work based on Devi's last submission (v15) which dates
> back to 05 October 2023 as below SoCs are still active.
>
> V16:
>
>    Removed reg description in bindings as the offset is not relative to
>    the TCSR region anymore since simple-mfd support was dropped and PWM
>    nodes defined as their own nodes, not child nodes. Updated the example
>    too.
>
>    Dropped patch to add simple-mfd support to the qcom,tcsr bindings

George, thanks for taking up this series.

Can you elaborate on why the qcom,tcsr binding change is dropped and 
made the PWM as an independent node? IIUC, it should be child of TCSR 
node. In V15, I see the qcom,tcsr binding is applied and *dropped only* 
due to dependencies / lack of information on the "pwm-cells" property.

Thanks,

Kathiravan T.


  parent reply	other threads:[~2025-10-03  9:04 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-01 14:04 [PATCH v16 0/9] Add PWM support for IPQ chipsets George Moussalem via B4 Relay
2025-10-01 14:04 ` [PATCH v16 1/9] dt-bindings: pwm: add IPQ6018 binding George Moussalem via B4 Relay
2025-10-01 14:04 ` [PATCH v16 2/9] pwm: driver for qualcomm ipq6018 pwm block George Moussalem via B4 Relay
2025-10-01 14:04 ` [PATCH v16 3/9] dt-bindings: pwm: qcom,ipq6018-pwm: Add compatible for ipq5018 George Moussalem via B4 Relay
2025-10-05 17:07   ` George Moussalem
2025-10-08 13:31     ` Rob Herring
2025-10-08 13:31   ` Rob Herring (Arm)
2025-10-01 14:04 ` [PATCH v16 4/9] dt-bindings: pwm: qcom,ipq6018-pwm: Add compatible for ipq5332 George Moussalem via B4 Relay
2025-10-08 13:28   ` Rob Herring (Arm)
2025-10-01 14:04 ` [PATCH v16 5/9] dt-bindings: pwm: qcom,ipq6018-pwm: Add compatible for ipq9574 George Moussalem via B4 Relay
2025-10-08 13:28   ` Rob Herring (Arm)
2025-10-01 14:04 ` [PATCH v16 6/9] arm64: dts: qcom: ipq6018: add pwm node George Moussalem via B4 Relay
2025-10-02  2:05   ` Dmitry Baryshkov
2025-10-01 14:04 ` [PATCH v16 7/9] arm64: dts: qcom: ipq5018: " George Moussalem via B4 Relay
2025-10-02  2:06   ` Dmitry Baryshkov
2025-10-01 14:04 ` [PATCH v16 8/9] arm64: dts: qcom: ipq5332: " George Moussalem via B4 Relay
2025-10-02  2:06   ` Dmitry Baryshkov
2025-10-01 14:04 ` [PATCH v16 9/9] arm64: dts: qcom: ipq9574: " George Moussalem via B4 Relay
2025-10-02  2:21   ` Dmitry Baryshkov
2025-10-01 23:42 ` [PATCH v16 0/9] Add PWM support for IPQ chipsets Rob Herring (Arm)
2025-10-03  7:06   ` George Moussalem
2025-10-06 12:06     ` Konrad Dybcio
2025-10-03  9:04 ` Kathiravan Thirumoorthy [this message]
2025-10-03  9:30   ` George Moussalem
2025-10-03  9:39     ` Kathiravan Thirumoorthy

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=3cb50330-effc-4089-b80c-d454bccc63f3@oss.qualcomm.com \
    --to=kathiravan.thirumoorthy@oss.qualcomm.com \
    --cc=andersson@kernel.org \
    --cc=baruch.siach@siklu.com \
    --cc=baruch@tkos.co.il \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=george.moussalem@outlook.com \
    --cc=konradybcio@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pwm@vger.kernel.org \
    --cc=quic_devipriy@quicinc.com \
    --cc=robh@kernel.org \
    --cc=ukleinek@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;
as well as URLs for NNTP newsgroup(s).