[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <42bee6c5-6e1f-2b41-3013-023bd95fe8b7@quicinc.com>
Date: Wed, 12 Jul 2023 12:00:05 +0530
From: Anusha Canchi <quic_anusha@...cinc.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
CC: <agross@...nel.org>, <andersson@...nel.org>,
<bhupesh.sharma@...aro.org>, <conor+dt@...nel.org>,
<davem@...emloft.net>, <devicetree@...r.kernel.org>,
<herbert@...dor.apana.org.au>, <konrad.dybcio@...aro.org>,
<krzysztof.kozlowski+dt@...aro.org>,
<linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
<linux-crypto@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<mturquette@...libre.com>, <p.zabel@...gutronix.de>,
<quic_arajkuma@...cinc.com>, <quic_gokulsri@...cinc.com>,
<quic_kathirav@...cinc.com>, <quic_poovendh@...cinc.com>,
<quic_sjaganat@...cinc.com>, <quic_srichara@...cinc.com>,
<robh+dt@...nel.org>, <sboyd@...nel.org>,
<thara.gopinath@...il.com>
Subject: Re: [PATCH V4 3/4] dt-bindings: qcom-qce: add SoC compatible string
for ipq9574
On 6/23/2023 6:23 PM, Krzysztof Kozlowski wrote:
> On 23/06/2023 13:55, Anusha Rao wrote:
>>> Document the compatible string for ipq9574.
>>>
>>> Acked-by: Conor Dooley <conor.dooley@...rochip.com>
>>> Reviewed-by: Bhupesh Sharma <bhupesh.sharma@...aro.org>
>>> Signed-off-by: Anusha Rao <quic_anusha@...cinc.com>
>>> ---
>>> Changes in V4:
>>> - Picked up Reviewed-by tag.
>> A gentle reminder to pick the dt-binding patch.
>> As the dts change is picked, this patch is required to resolve dt-bindings check issues.
> One patchset with four patches targeting three different subsystems, so
> no wonder it gets missed. You will usually receive better results with
> splitting such patchsets per subsystems.
>
> One more thing is lack of proper subject prefix which indicates the
> subsystem. Without it why anyone would pick it up? For example me, I
> would just ignore it for my subsystem...
Ok..Referred previous commit titles of this file and added subject
prefix. Do you suggest to repost it with
"dt-bindings: crypto: qcom-qce: add SoC compatible string for ipq9574" ?
Thanks,
Anusha
>
> Best regards,
> Krzysztof
>
Powered by blists - more mailing lists