From: Naman Jain <quic_namajain@quicinc.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
Danila Tikhonov <danila@jiaxyga.com>, <andersson@kernel.org>,
<konrad.dybcio@linaro.org>, <djakov@kernel.org>,
<robh@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
<conor+dt@kernel.org>
Cc: <linux-arm-msm@vger.kernel.org>, <linux-pm@vger.kernel.org>,
<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] dt-bindings: interconnect: Add Qualcomm SM7150 DT bindings
Date: Mon, 19 Feb 2024 17:57:28 +0530 [thread overview]
Message-ID: <34734d65-6be7-4fab-98b7-357726751af5@quicinc.com> (raw)
In-Reply-To: <f522e679-3543-46ee-8b2d-92e384427382@linaro.org>
On 2/19/2024 4:44 PM, Krzysztof Kozlowski wrote:
> On 19/02/2024 10:48, Naman Jain wrote:
>> On 2/19/2024 12:02 AM, Danila Tikhonov wrote:
>>> The Qualcomm SM7150 platform has several bus fabrics that could be
>>> controlled and tuned dynamically according to the bandwidth demand.
>>
>> Please add what you are trying to do with this patch.
>>
>> Ref:
>> https://docs.kernel.org/process/submitting-patches.html#describe-your-changes
>
> The reason for this change is explained in commit msg, so what exactly
> are you missing?
>
> Best regards,
> Krzysztof
>
I was under the impression that we should have explained/mentioned the
subject of this patch in the description of the commit msg. It seems
that it's not required. Thanks for correcting me.
Regards,
Naman Jain
next prev parent reply other threads:[~2024-02-19 12:27 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-18 18:32 [PATCH 0/2] Add interconnect support for SM7150 SoC Danila Tikhonov
2024-02-18 18:32 ` [PATCH 1/2] dt-bindings: interconnect: Add Qualcomm SM7150 DT bindings Danila Tikhonov
2024-02-18 18:52 ` Krzysztof Kozlowski
2024-02-18 19:02 ` Danila Tikhonov
2024-02-19 7:11 ` Krzysztof Kozlowski
2024-02-19 9:48 ` Naman Jain
2024-02-19 11:14 ` Krzysztof Kozlowski
2024-02-19 12:27 ` Naman Jain [this message]
2024-02-18 18:32 ` [PATCH 2/2] interconnect: qcom: Add SM7150 driver support Danila Tikhonov
2024-02-19 10:14 ` Konrad Dybcio
2024-02-19 9:48 ` [PATCH 0/2] Add interconnect support for SM7150 SoC Naman Jain
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=34734d65-6be7-4fab-98b7-357726751af5@quicinc.com \
--to=quic_namajain@quicinc.com \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=danila@jiaxyga.com \
--cc=devicetree@vger.kernel.org \
--cc=djakov@kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@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