From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BCBA4C433E0 for ; Thu, 18 Feb 2021 22:54:32 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5D7EF64DFD for ; Thu, 18 Feb 2021 22:54:32 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5D7EF64DFD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=VsalKyNgp3KdMJmXhbL2PHZFHgpG/mbezPLQhKSl610=; b=DyBeZl7Om21dJXd7a+RNbRLb+ DNFfYGlJI+KbspLpeHfXJUGJXwCvoFA4EIkqMUJJnGBFyE9UONugJn2ybdlLZeESKN5MNq3fTnzUR cOCEo5MWFq8TAbHZndj2Mz3ygai0Tz0OE8ikQ1p3WaD35uk0ksw0YJtlRM6pvWg8TkTwCkBSNfOER UWms+/ugFvmGPOrca+VcGqGvapjjcZcUZHdj/39/Ljo+3eaVK0Y14PMQvCLEIBevrCvMk6dFbD3ye 9KBTJra8blD/O4m1SpDDebkb+f+tKB3bvcSKU5nEMFc6+K/6XXpYZx/7OWJQtI3c+2OWR9mTjLJ5o zJpdB7gJA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1lCs9h-0005ze-UD; Thu, 18 Feb 2021 22:52:18 +0000 Received: from foss.arm.com ([217.140.110.172]) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1lCs9d-0005ye-VQ for linux-arm-kernel@lists.infradead.org; Thu, 18 Feb 2021 22:52:15 +0000 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 791EEED1; Thu, 18 Feb 2021 14:52:05 -0800 (PST) Received: from [10.57.61.241] (unknown [10.57.61.241]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D9D4A3F73D; Thu, 18 Feb 2021 14:52:03 -0800 (PST) Subject: Re: [PATCH V3 06/14] dts: bindings: Document device tree bindings for ETE To: Rob Herring References: <1611737738-1493-1-git-send-email-anshuman.khandual@arm.com> <1611737738-1493-7-git-send-email-anshuman.khandual@arm.com> <20210209190031.GA4102836@robh.at.kernel.org> <4d0e6b88-72c2-be23-f43a-3f541f9ebb86@arm.com> <20210218183335.GA915713@robh.at.kernel.org> From: Suzuki K Poulose Message-ID: <952f91ef-7fd2-a3d4-06d8-b7f433aa4c76@arm.com> Date: Thu, 18 Feb 2021 22:51:56 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.1 MIME-Version: 1.0 In-Reply-To: <20210218183335.GA915713@robh.at.kernel.org> Content-Language: en-GB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210218_175214_253376_966D0A5A X-CRM114-Status: GOOD ( 27.86 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, mathieu.poirier@linaro.org, Anshuman Khandual , coresight@lists.linaro.org, linux-kernel@vger.kernel.org, lcherian@marvell.com, linux-arm-kernel@lists.infradead.org, mike.leach@linaro.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2/18/21 6:33 PM, Rob Herring wrote: > On Wed, Feb 10, 2021 at 12:33:44PM +0000, Suzuki K Poulose wrote: >> Hi Rob >> >> On 2/9/21 7:00 PM, Rob Herring wrote: >>> On Wed, Jan 27, 2021 at 02:25:30PM +0530, Anshuman Khandual wrote: >>>> From: Suzuki K Poulose >>>> >>>> Document the device tree bindings for Embedded Trace Extensions. >>>> ETE can be connected to legacy coresight components and thus >>>> could optionally contain a connection graph as described by >>>> the CoreSight bindings. >>>> >>>> Cc: devicetree@vger.kernel.org >>>> Cc: Mathieu Poirier >>>> Cc: Mike Leach >>>> Cc: Rob Herring >>>> Signed-off-by: Suzuki K Poulose >>>> Signed-off-by: Anshuman Khandual >>>> --- >>>> Changes in V3: >>>> >>>> - Fixed all DT yaml semantics problems >>>> >>>> Documentation/devicetree/bindings/arm/ete.yaml | 74 ++++++++++++++++++++++++++ >>>> 1 file changed, 74 insertions(+) >>>> create mode 100644 Documentation/devicetree/bindings/arm/ete.yaml >>>> >>>> diff --git a/Documentation/devicetree/bindings/arm/ete.yaml b/Documentation/devicetree/bindings/arm/ete.yaml >>>> new file mode 100644 >>>> index 0000000..edc1fe2 >>>> --- /dev/null >>>> +++ b/Documentation/devicetree/bindings/arm/ete.yaml >>>> @@ -0,0 +1,74 @@ >>>> +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause >>>> +# Copyright 2021, Arm Ltd >>>> +%YAML 1.2 >>>> +--- >>>> +$id: "http://devicetree.org/schemas/arm/ete.yaml#" >>>> +$schema: "http://devicetree.org/meta-schemas/core.yaml#" >>>> + >>>> +title: ARM Embedded Trace Extensions >>>> + >>>> +maintainers: >>>> + - Suzuki K Poulose >>>> + - Mathieu Poirier >>>> + >>>> +description: | >>>> + Arm Embedded Trace Extension(ETE) is a per CPU trace component that >>>> + allows tracing the CPU execution. It overlaps with the CoreSight ETMv4 >>>> + architecture and has extended support for future architecture changes. >>>> + The trace generated by the ETE could be stored via legacy CoreSight >>>> + components (e.g, TMC-ETR) or other means (e.g, using a per CPU buffer >>>> + Arm Trace Buffer Extension (TRBE)). Since the ETE can be connected to >>>> + legacy CoreSight components, a node must be listed per instance, along >>>> + with any optional connection graph as per the coresight bindings. >>>> + See bindings/arm/coresight.txt. >>>> + >>>> +properties: >>>> + $nodename: >>>> + pattern: "^ete([0-9a-f]+)$" >>>> + compatible: >>>> + items: >>>> + - const: arm,embedded-trace-extension >>>> + >>>> + cpu: >>> >>> We've already established 'cpus' for this purpose. >>> >> >> Please see : https://lkml.kernel.org/r/9417218b-6eda-373b-a2cb-869089ffc7cd@arm.com >> for my response in the previous version to this and the one with out-ports. > > Okay, fair enough. > >> >>>> + description: | >>>> + Handle to the cpu this ETE is bound to. >>>> + $ref: /schemas/types.yaml#/definitions/phandle >>>> + >>>> + out-ports: >>>> + type: object >>> >>> Replace with: $ref: /schemas/graph.yaml#/properties/ports >> >> So, just to confirm again : >> The CoreSight graph bindings expect the input ports and output ports >> grouped under in-ports{} and out-ports{} respectively to avoid having >> to specify the direction of the ports in the individual "port" nodes. >> i.e >> >> in-ports { >> >> property: ports >> OR >> property: port >> >> required: >> OneOf: >> ports >> port > > No, 'ports' as a child of in-ports is not correct. There should only be > 'port(@[0-9a-f]+)?' nodes. That's why you need the above $ref added. The > $ref doesn't define the node name is 'ports', but what a 'ports' or > 'foo-ports' contains. Sorry, it is my bad. We don't expect ports{} under in-ports. So your suggestion is the accurate one. I will respin. > >> } >> >> out-ports { >> >> # same as above >> } >> >> So thats why I added out-ports as a new object, where the ports/port >> could be a child node. >> >> Ideally the definition of out-ports /in-ports should go to a common schema >> for CoreSight bindings, when we move to Yaml for the existing bindings, >> which will follow in a separate series, later. > > Yes, maybe, but I'm not sure something common is going to help here. > You'll still have to describe what each 'port' node does in each device > specific binding. For CoreSight components the end-point of the ports are system specific. i.e, it could be anything on the other end. There is no fixed end-point connection. e.g, ETM could be connected to a Replicator or a Funnel. Same as here above for ETE. Thus the driver must parse the endpoints and make the connection path from devices to other devices. Anyways, will come to that in a different series. I will fix the in-ports/out-ports for the next version. Thanks for your guidance. Cheers Suzuki > > Rob > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel