[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <bbbec517-cc2b-444b-b76b-a4930902d667@oss.qualcomm.com>
Date: Tue, 5 Aug 2025 18:25:57 +0800
From: Jie Gan <jie.gan@....qualcomm.com>
To: Mike Leach <mike.leach@...aro.org>
Cc: Suzuki K Poulose <suzuki.poulose@....com>,
James Clark <james.clark@...aro.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley
<conor+dt@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>, coresight@...ts.linaro.org,
linux-arm-kernel@...ts.infradead.org, linux-arm-msm@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] Enable CTCU device for QCS8300
On 8/5/2025 5:53 PM, Mike Leach wrote:
> Hi Jie,
>
> On Tue, 5 Aug 2025 at 05:11, Jie Gan <jie.gan@....qualcomm.com> wrote:
>>
>>
>>
>> On 7/28/2025 9:08 AM, Jie Gan wrote:
>>>
>>>
>>> On 7/15/2025 8:41 AM, Jie Gan wrote:
>>>>
>>>>
>>>> On 6/24/2025 5:59 PM, Jie Gan wrote:
>>>>> Enable CTCU device for QCS8300 platform. Add a fallback mechnasim in
>>>>> binding to utilize
>>>>> the compitable of the SA8775p platform becuase the CTCU for QCS8300
>>>>> shares same
>>>>> configurations as SA8775p platform.
>>>>
>>>> Gentle ping.
>>>
>>> Gentle ping.
>>
>> Gentle ping.
>> Hi Coresight maintainers,
>>
>> Can you please help to review the patch?
>>
>> Thanks,
>> Jie
>>
>>>
>>> Thanks,
>>> Jie
>>>
>>>>
>>>> Hi Suzuki, Mike, James, Rob
>>>>
>>>> Can you plz help to review the patch from Coresight view?
>>>>
>>>> Thanks,
>>>> Jie
>>>>
>>>>>
>>>>> Changes in V2:
>>>>> 1. Add Krzysztof's R-B tag for dt-binding patch.
>>>>> 2. Add Konrad's Acked-by tag for dt patch.
>>>>> 3. Rebased on tag next-20250623.
>>>>> 4. Missed email addresses for coresight's maintainers in V1, loop them.
>>>>> Link to V1 - https://lore.kernel.org/all/20250327024943.3502313-1-
>>>>> jie.gan@....qualcomm.com/
>>>>>
>>>>> Jie Gan (2):
>>>>> dt-bindings: arm: add CTCU device for QCS8300
>>>>> arm64: dts: qcom: qcs8300: Add CTCU and ETR nodes
>>>>>
>>>>> .../bindings/arm/qcom,coresight-ctcu.yaml | 9 +-
>>>>> arch/arm64/boot/dts/qcom/qcs8300.dtsi | 153 ++++++++++++++++++
>>>>> 2 files changed, 160 insertions(+), 2 deletions(-)
>>>>>
>>>>
>>>>
>>>
>>
>
> You need to send a new patch addressing the comments made by Krzysztof..
Hi Mike,
I just proposed an idea to add a common compatible for CTCU device, its
not about the patch series itself. We dropped the idea and prefer to add
the board specific compatible for each platform.
Thanks,
Jie
>
> Regards
>
> Mike
Powered by blists - more mailing lists