[PATCH v1 0/2] Enable CTCU device for QCS8300

Jie Gan posted 2 patches 8 months, 3 weeks ago
There is a newer version of this series
.../bindings/arm/qcom,coresight-ctcu.yaml     |   9 +-
arch/arm64/boot/dts/qcom/qcs8300.dtsi         | 153 ++++++++++++++++++
2 files changed, 160 insertions(+), 2 deletions(-)
[PATCH v1 0/2] Enable CTCU device for QCS8300
Posted by Jie Gan 8 months, 3 weeks ago
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.

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(-)

-- 
2.34.1
Re: [PATCH v1 0/2] Enable CTCU device for QCS8300
Posted by Jie Gan 7 months ago

On 3/27/2025 10:49 AM, 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 for the series.

dt-binding patch has been reviewed by Krzysztof.
dts patch has been acked by Konrad.

Thanks,
Jie

> 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(-)
>
Re: [PATCH v1 0/2] Enable CTCU device for QCS8300
Posted by Bjorn Andersson 6 months ago
On Wed, May 14, 2025 at 04:00:12PM +0800, Jie Gan wrote:
> 
> 
> On 3/27/2025 10:49 AM, 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 for the series.
> 
> dt-binding patch has been reviewed by Krzysztof.
> dts patch has been acked by Konrad.
> 

You don't seem to have the Coresight maintainers among the recipients of
the binding patch, so that's probably why it hasn't been merged yet -
and I can't merge the DTS change until the binding is accepted.

Please apply Krzysztof's R-b and send the binding alone to Coresight
maintainers, once they have accepted the binding please resubmit the dts
change and I'd be happy to merge it. (v2 sounds good on both)

PS. Please see go/upstream on how to adopt b4; "b4 prep --auto-to-cc"
would have saved you from this mistake.

Regards,
Bjorn

> Thanks,
> Jie
> 
> > 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(-)
> > 
>
Re: [PATCH v1 0/2] Enable CTCU device for QCS8300
Posted by Jie Gan 6 months ago

On 6/18/2025 10:50 AM, Bjorn Andersson wrote:
> On Wed, May 14, 2025 at 04:00:12PM +0800, Jie Gan wrote:
>>
>>
>> On 3/27/2025 10:49 AM, 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 for the series.
>>
>> dt-binding patch has been reviewed by Krzysztof.
>> dts patch has been acked by Konrad.
>>
> 
> You don't seem to have the Coresight maintainers among the recipients of
> the binding patch, so that's probably why it hasn't been merged yet -
> and I can't merge the DTS change until the binding is accepted.
> 
> Please apply Krzysztof's R-b and send the binding alone to Coresight
> maintainers, once they have accepted the binding please resubmit the dts
> change and I'd be happy to merge it. (v2 sounds good on both)
> 
> PS. Please see go/upstream on how to adopt b4; "b4 prep --auto-to-cc"
> would have saved you from this mistake.
> 

Thanks for the information.

Will re-send the patch series.

Thanks,
Jie

> Regards,
> Bjorn
> 
>> Thanks,
>> Jie
>>
>>> 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(-)
>>>
>>