Received-SPF: pass (google.com: domain of linux-kernel+bounces-28449-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <41429e0b-e7f5-40a3-8dff-ebf1d26c9aac@quicinc.com>
Date: Wed, 17 Jan 2024 09:58:43 +0800
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 0/2] coresight: core: Add device name support
Content-Language: en-US
To: Suzuki K Poulose <suzuki.poulose@arm.com>,
        Mike Leach
	<mike.leach@linaro.org>, James Clark <james.clark@arm.com>,
        Leo Yan
	<leo.yan@linaro.org>,
        Alexander Shishkin
	<alexander.shishkin@linux.intel.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley
	<conor+dt@kernel.org>,
        Maxime Coquelin <mcoquelin.stm32@gmail.com>,
        Alexandre
 Torgue <alexandre.torgue@foss.st.com>,
        Bjorn Andersson
	<andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Mathieu
 Poirier <mathieu.poirier@linaro.org>
CC: <coresight@lists.linaro.org>, <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-stm32@st-md-mailman.stormreply.com>,
        <linux-arm-msm@vger.kernel.org>
References: <20240115164252.26510-1-quic_jinlmao@quicinc.com>
 <cef041b7-70ac-47c9-888c-696a0cec04d1@arm.com>
From: Jinlong Mao <quic_jinlmao@quicinc.com>
In-Reply-To: <cef041b7-70ac-47c9-888c-696a0cec04d1@arm.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit


On 1/16/2024 9:43 PM, Suzuki K Poulose wrote:
> On 15/01/2024 16:42, Mao Jinlong wrote:
>> With current design, the name of the non-cpu bounded coresight
>> component is the device type with the number. And with 'ls' command
>> we can get the register address of the component. But from these
>> information, we can't know what the HW or system the component belongs
>> to. Add device-name in DT to support it.
>>
>> cti_sys0 -> ../../../devices/platform/soc@0/138f0000.cti/cti_sys0
>> cti_sys1 -> ../../../devices/platform/soc@0/13900000.cti/cti_sys1
>> tpdm0 -> ../../../devices/platform/soc@0/10b0d000.tpdm/tpdm0
>> tpdm1 -> ../../../devices/platform/soc@0/10c28000.tpdm/tpdm1
>> tpdm2 -> ../../../devices/platform/soc@0/10c29000.tpdm/tpdm2
> 
> Please could you rebase this on for-next/queue (which has all bells and 
> whistles enabled to report Warnings), fixing the reported issue by 
> kernel test robot ?
> 
I will rebase the patch.

Thanks
Jinlong Mao
> Suzuki
> 
> 
>>
>> Change since V1:
>> 1. Change coresight-name to device name.
>> 2. Add the device-name in coresight dt bindings.
>>
>> Mao Jinlong (2):
>>    coresight: core: Add device name support
>>    dt-bindings: arm: Add device-name in the coresight components
>>
>>   .../bindings/arm/arm,coresight-catu.yaml      |  5 +++
>>   .../bindings/arm/arm,coresight-cpu-debug.yaml |  5 +++
>>   .../bindings/arm/arm,coresight-cti.yaml       |  5 +++
>>   .../arm/arm,coresight-dummy-sink.yaml         |  5 +++
>>   .../arm/arm,coresight-dummy-source.yaml       |  5 +++
>>   .../arm/arm,coresight-dynamic-funnel.yaml     |  5 +++
>>   .../arm/arm,coresight-dynamic-replicator.yaml |  5 +++
>>   .../bindings/arm/arm,coresight-etb10.yaml     |  5 +++
>>   .../bindings/arm/arm,coresight-etm.yaml       |  5 +++
>>   .../arm/arm,coresight-static-funnel.yaml      |  5 +++
>>   .../arm/arm,coresight-static-replicator.yaml  |  5 +++
>>   .../bindings/arm/arm,coresight-stm.yaml       |  5 +++
>>   .../bindings/arm/arm,coresight-tmc.yaml       |  5 +++
>>   .../bindings/arm/arm,coresight-tpiu.yaml      |  5 +++
>>   .../bindings/arm/qcom,coresight-tpda.yaml     |  5 +++
>>   .../bindings/arm/qcom,coresight-tpdm.yaml     |  5 +++
>>   drivers/hwtracing/coresight/coresight-core.c  | 33 ++++++++++---------
>>   .../hwtracing/coresight/coresight-platform.c  | 31 +++++++++++++++++
>>   include/linux/coresight.h                     |  1 +
>>   19 files changed, 130 insertions(+), 15 deletions(-)
>>
>