Received-SPF: pass (google.com: domain of linux-kernel+bounces-7809-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <8e5e9603-456b-4956-be03-b866feeeafb4@quicinc.com>
Date: Thu, 21 Dec 2023 11:28:19 +0800
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v1 1/2] dt-bindings: arm: coresight: Update the pattern of
 ete node name
Content-Language: en-US
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Mathieu Poirier
	<mathieu.poirier@linaro.org>,
        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>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson
	<andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Rob Herring
	<robh+dt@kernel.org>,
        Krzysztof Kozlowski
	<krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>
CC: <linux-arm-msm@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>,
        Tingwei Zhang <quic_tingweiz@quicinc.com>,
        Yuanfang Zhang <quic_yuanfang@quicinc.com>,
        Tao Zhang
	<quic_taozha@quicinc.com>, <coresight@lists.linaro.org>
References: <20231220140538.13136-1-quic_jinlmao@quicinc.com>
 <20231220140538.13136-2-quic_jinlmao@quicinc.com>
 <79f88d35-17cc-43b0-bb22-3c854f89d961@linaro.org>
From: Jinlong Mao <quic_jinlmao@quicinc.com>
In-Reply-To: <79f88d35-17cc-43b0-bb22-3c854f89d961@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit


On 12/20/2023 11:50 PM, Krzysztof Kozlowski wrote:
> On 20/12/2023 15:05, Mao Jinlong wrote:
>> Update the suffix for ete node name to be with "-".
>>
>> Signed-off-by: Mao Jinlong <quic_jinlmao@quicinc.com>
>> ---
>>   .../bindings/arm/arm,embedded-trace-extension.yaml          | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
>> index f725e6940993..cbf583d34029 100644
>> --- a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
>> +++ b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml
>> @@ -23,7 +23,7 @@ description: |
>>   
>>   properties:
>>     $nodename:
>> -    pattern: "^ete([0-9a-f]+)$"
>> +    pattern: "^ete-([0-9a-f]+)$"
> 
> My concerns are not resolved. Why is it here in the first place?

Hi Krzysztof,

ETE is acronym of embedded trace extension. The number of the name is 
the same as the number of the CPU it belongs to.

Hi Suzuki,

Please help to comment on this.

Thanks
Jinlong Mao

> 
> Best regards,
> Krzysztof
>