Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp3875642rdb; Thu, 28 Dec 2023 03:02:50 -0800 (PST) X-Google-Smtp-Source: AGHT+IFf7GcZFLFNK9fPpJudc/ve5fsFInJAc61wn604YwCqVMbHwP47FBXL1rWOBwDWKn+wKlqd X-Received: by 2002:a05:6358:41a1:b0:170:c921:bf85 with SMTP id w33-20020a05635841a100b00170c921bf85mr13101869rwc.27.1703761369682; Thu, 28 Dec 2023 03:02:49 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703761369; cv=none; d=google.com; s=arc-20160816; b=MNAH0Js8elYcDFEEaFBjmriNW6SgyuO5lJby3rCnG0psoLfqljWkwg/NxOxeM5BHLN c/CBHS8JrPWjiy6ppJPA/f83LJQJYc0o0o+NcZalffs//vdiJN7iRBEdMHcxYJq+uRgz VHNxslyuohA2oNf5UTuVVK2ORl/zjLY4mn97Ri7s+TwFv+k051mzoMonL0ImbTh7SAVF VXU5LRUc9M+qIn0la6DfqZ313rv/qa0Lk9sY1du51iHa6c76nV9+dxyHqwEy4BVDpUR3 Ps0nVDkm7Mq2VAxhgj2MhJlOQrBHIbY422iP+aLno5VLAdEbbvFG+zvUDakC+F5cdOW5 JSmw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id; bh=Ke61WYoi/W56ccmh49cH26LG9+VDGGuTxXSz46tDTKo=; fh=cd84gYTyLEt5EB7oAt01XWhHRLhYRa0vHjspgdTHXQM=; b=W0BDiQMw4OEM1Jd4lAG+ALyytln80j5LzmUF4lUvWRDh0PtzS7b25GYfRl5PQjWZWD lxFR5ZRGwJZ9z44hY/MwJ+6zEAqx4BPNPFPmMnzw05WpD/i+TcojxSc9s9GY8xLG3kB9 A73e2MDv/zoeSO7iSH+AxR6U/fcnCIB5e/Ed2PQwb3XJZ/gF76ST139icg+fumAUETvi rWzYr1WhbbmqDGNYnvKgAMr6dGtPe1TiwiBir9KPVEmcnVkEYefPn1jeHxzhVlO0X2Xt DplAO7mvGJLzkz3BrxIkXffOJoZmLfUTnddwufKl4v963ja3ASjzdN0XBJ3g2WzfeGfe ybLg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-12601-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-12601-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id a35-20020a056a001d2300b006d9bde5c1aasi6007820pfx.167.2023.12.28.03.02.49 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Dec 2023 03:02:49 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-12601-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-12601-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-12601-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 4B2382813EE for ; Thu, 28 Dec 2023 11:02:49 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id AB1556FCA; Thu, 28 Dec 2023 11:02:41 +0000 (UTC) X-Original-To: linux-kernel@vger.kernel.org Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 066D06FB6; Thu, 28 Dec 2023 11:02:38 +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 B804D2F4; Thu, 28 Dec 2023 03:03:23 -0800 (PST) Received: from [192.168.151.22] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 981AA3F5A1; Thu, 28 Dec 2023 03:02:33 -0800 (PST) Message-ID: Date: Thu, 28 Dec 2023 11:02:34 +0000 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: [PATCH v1 1/2] dt-bindings: arm: coresight: Update the pattern of ete node name To: Krzysztof Kozlowski , Jinlong Mao , Suzuki K Poulose Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Tingwei Zhang , Yuanfang Zhang , Tao Zhang , coresight@lists.linaro.org, Mathieu Poirier , Mike Leach , Leo Yan , Andy Gross , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley References: <20231220140538.13136-1-quic_jinlmao@quicinc.com> <20231220140538.13136-2-quic_jinlmao@quicinc.com> <79f88d35-17cc-43b0-bb22-3c854f89d961@linaro.org> <8e5e9603-456b-4956-be03-b866feeeafb4@quicinc.com> <16932826-fcc2-49d3-95ab-201eff729360@quicinc.com> <9d13e1ce-38b1-4cdd-83ba-eca0c3091ce1@quicinc.com> <578d6319-5ab5-45e5-8457-00116c7c84ae@linaro.org> Content-Language: en-US From: James Clark In-Reply-To: <578d6319-5ab5-45e5-8457-00116c7c84ae@linaro.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 26/12/2023 09:36, Krzysztof Kozlowski wrote: > On 26/12/2023 02:50, Jinlong Mao wrote: >> >> >> On 12/21/2023 4:44 PM, Krzysztof Kozlowski wrote: >>> On 21/12/2023 09:36, Jinlong Mao wrote: >>>> >>>> >>>> On 12/21/2023 4:17 PM, Krzysztof Kozlowski wrote: >>>>> On 21/12/2023 09:15, Jinlong Mao wrote: >>>>>> >>>>>> >>>>>> On 12/21/2023 4:12 PM, Krzysztof Kozlowski wrote: >>>>>>> On 21/12/2023 04:28, Jinlong Mao wrote: >>>>>>>>>> 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. >>>>>>> >>>>>>> This is obvious and was not my question. > > You already said it here... > >>>>>> >>>>>> Do you mean why the pattern match of the node name is added here ? >>>>> >>>>> Yes, especially that it is requiring a non-generic name. >>>>> >>>>>> >>>>>> This node should not have the node name match, right ? >>>>> >>>>> Usually. For sure shouldn't be for non-generic names. >>>>> >>>> Hi Suzuki, >>>> >>>> Can we remove the pattern match of the node name and use a generic name >>>> "ete" for the ete DT nodes ? >>> >>> "ete" is not a generic name. What is generic here? It's an acronym of >>> some specific device name. >>> >> >> The device full name is embedded trace extension. So use ETE as the name >> here. > > That's obvious and my comment was not about it. Second time... This is > my unlucky day... I said, why do you even want to enforce name which is > not generic, since the names should be generic? > I think we can just drop the enforced name if it's getting in the way. It doesn't really do anything and other Coresight bindings don't have it anyway. > I assume you read the DT specification: > https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation > > > Best regards, > Krzysztof > I couldn't find anything in that list that would be a good fit for a name, and it seems like all of the Coresight devices have already been added with non generic names (like funnel and replicator etc), so it might be a bit late now. But if we drop the enforced name then it's probably fine. James