[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <74038a79-8c59-496c-9a61-08dc786f9c10@oss.qualcomm.com>
Date: Mon, 25 Aug 2025 09:32: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 6:25 PM, Jie Gan wrote:
>
>
> 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
Hi Suzuki, Mike, James
We already have the tag from dt-binding and dt maintainers. We haven't
additional modification for this patch series. Can you please help to
review the patch series from Coresight View?
Thanks,
Jie
>
>>
>> Regards
>>
>> Mike
>
>
Powered by blists - more mailing lists