Re: [PATCH v2 0/2] Enable CTCU device for QCS8300

[Jie Gan]

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@xxxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxx/
> > > > > >
> > > > > > 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