From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 85754134A0; Tue, 16 Jan 2024 09:33:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 4303A2F4; Tue, 16 Jan 2024 01:34:20 -0800 (PST) Received: from [10.57.46.197] (unknown [10.57.46.197]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 636543F6C4; Tue, 16 Jan 2024 01:33:29 -0800 (PST) Message-ID: Date: Tue, 16 Jan 2024 09:33:27 +0000 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/2] dt-bindings: arm: coresight: Remove pattern match of ETE node name Content-Language: en-GB To: Mao Jinlong , Mike Leach , James Clark , Leo Yan , Alexander Shishkin , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Konrad Dybcio , Mathieu Poirier Cc: coresight@lists.linaro.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, Tingwei Zhang , Yuanfang Zhang , Tao Zhang References: <20240116064505.487-1-quic_jinlmao@quicinc.com> <20240116064505.487-2-quic_jinlmao@quicinc.com> From: Suzuki K Poulose In-Reply-To: <20240116064505.487-2-quic_jinlmao@quicinc.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 16/01/2024 06:45, Mao Jinlong wrote: > Remove pattern match of ETE node name. Use ete with the number as the > name for ete nodes. > > Signed-off-by: Mao Jinlong > --- > .../bindings/arm/arm,embedded-trace-extension.yaml | 6 ++---- > 1 file changed, 2 insertions(+), 4 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..ed78cc7ae94a 100644 > --- a/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml > +++ b/Documentation/devicetree/bindings/arm/arm,embedded-trace-extension.yaml > @@ -22,8 +22,6 @@ description: | > with any optional connection graph as per the coresight bindings. > > properties: > - $nodename: > - pattern: "^ete([0-9a-f]+)$" > compatible: > items: > - const: arm,embedded-trace-extension > @@ -55,13 +53,13 @@ examples: > > # An ETE node without legacy CoreSight connections > - | > - ete0 { > + ete-0 { Why do we need the number ? why not simply "ete" as Krzysztof suggested ? Suzuki