Re: [PATCH v3] arm64: dts: qcom: Add coresight node for SM8650

From: Konrad Dybcio
Date: Fri Jan 03 2025 - 07:45:02 EST


On 3.01.2025 12:27 PM, Yuanfang Zhang wrote:
> Add coresight components: Funnel, ETE and ETF for SM8650.
>
> Signed-off-by: Yuanfang Zhang <quic_yuanfang@xxxxxxxxxxx>
> ---

[...]

> +
> + funnel@13810000 {
> + compatible = "arm,coresight-dynamic-funnel", "arm,primecell";
> +
> + reg = <0x0 0x13810000 0x0 0x1000>;

This now looks good, please just re-sort these nodes (by address instead
of adding them at the end).

Also, did you give this patch a try on a production-fused device?

Konrad