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=-8.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,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 E49EEC4360C for ; Fri, 27 Sep 2019 17:16:18 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 BAE0E217D9 for ; Fri, 27 Sep 2019 17:16:18 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="i2ixDEee"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="TQkpoYvw"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="WNGMtdwl" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BAE0E217D9 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-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=bombadil.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=XN54ZSa7tK3/nnKV6DmhSMOYyrDnwLnPmYTkQu8WVyg=; b=i2ixDEeevP1JCKQNadxqQmXC+ YZenyY4cKp1gr+HGXo20vw6COw3Bj8l1kGKTHZGwt+drhdHASKPHKsvkpDw3K4lRj9xE6wCDWsGds 0B2bWLbULpyfC/3DK261izimN1Pa6KVcPHTiV4g5QAQfrn13kFuFHm6Z8pcMQsyiUVFlUp6K1MWaR eW9ccJiK+DmMnLTmhB3VZ0IG/DWTpTr9zQMJfK01518zxoBfXwciZ9OX0IoJGP0SoHo2USukf1SP/ OW/ZzZ9hcyW3AzvkNWn1SHloOPdL9St0sYwS1oDF/3YbMYspZAr9udVtRjOA1bO5VFKgm8s1RJvk5 jOru0nlYA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.2 #3 (Red Hat Linux)) id 1iDtqs-0000F1-At; Fri, 27 Sep 2019 17:16:18 +0000 Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.92.2 #3 (Red Hat Linux)) id 1iDtqo-0000EV-6T for linux-arm-kernel@lists.infradead.org; Fri, 27 Sep 2019 17:16:15 +0000 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 42A50611C5; Fri, 27 Sep 2019 17:16:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1569604573; bh=j8UGCvps8OpySydggrLehleyZDNW7x2YH12aTZwi19g=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=TQkpoYvwoetCV+O6hrmQMiUMKs6qc1rDflKiVf9YbF95qKMP9BASYX3vN1Is6KpgO h7x/Xf0zkosSsLADcVNiXd7bCNfLMIoTi1AaCtKmU4CNzFwl3GUwlOYLKYfsbT4iWF LbYYkbGZLaRfDaqXuHauAb59QQM4nS94WFx7/9RA= Received: from [192.168.1.12] (cpe-76-167-70-25.natsow.res.rr.com [76.167.70.25]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: daidavid1@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 04A4C60767; Fri, 27 Sep 2019 17:16:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1569604572; bh=j8UGCvps8OpySydggrLehleyZDNW7x2YH12aTZwi19g=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=WNGMtdwlbwlVn35Z2XmuPkgHKF/DWbdYdaFAMsD766oU7BmXzOw2h/qrqp8cMoZA8 r9/0+VFjXN/0R3q8vWrdd6o0Y74XSoNPiAYs5Ir5OOjlsfoiALAFocb1ItUt8XrF1M iSwzdXumLZUcEDxofcviK6Rya0UaSPlM28pztfVY= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 04A4C60767 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=daidavid1@codeaurora.org Subject: Re: [RFC PATCH] interconnect: Replace of_icc_get() with icc_get() and reduce DT binding To: Stephen Boyd , Bjorn Andersson References: <20190925054133.206992-1-swboyd@chromium.org> <20190925055933.GA2810@tuxbook-pro> <5d8b6b8b.1c69fb81.14b36.c053@mx.google.com> From: David Dai Message-ID: <91d09847-31ad-e238-d84d-f7e0e21c6ef1@codeaurora.org> Date: Fri, 27 Sep 2019 10:16:07 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <5d8b6b8b.1c69fb81.14b36.c053@mx.google.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190927_101614_284285_28231CF5 X-CRM114-Status: GOOD ( 30.56 ) 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, linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Maxime Ripard , Rob Herring , Evan Green , Georgi Djakov , linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 9/25/2019 6:28 AM, Stephen Boyd wrote: > Quoting Bjorn Andersson (2019-09-24 22:59:33) >> On Tue 24 Sep 22:41 PDT 2019, Stephen Boyd wrote: >> >>> The DT binding could also be simplified somewhat. Currently a path needs >>> to be specified in DT for each and every use case that is possible for a >>> device to want. Typically the path is to memory, which looks to be >>> reserved for in the binding with the "dma-mem" named path, but sometimes >>> the path is from a device to the CPU or more generically from a device >>> to another device which could be a CPU, cache, DMA master, or another >>> device if some sort of DMA to DMA scenario is happening. Let's remove >>> the pair part of the binding so that we just list out a device's >>> possible endpoints on the bus or busses that it's connected to. >>> >>> If the kernel wants to figure out what the path is to memory or the CPU >>> or a cache or something else it should be able to do that by finding the >>> node for the "destination" endpoint, extracting that node's >>> "interconnects" property, and deriving the path in software. For >>> example, we shouldn't need to write out each use case path by path in DT >>> for each endpoint node that wants to set a bandwidth to memory. We >>> should just be able to indicate what endpoint(s) a device sits on based >>> on the interconnect provider in the system and then walk the various >>> interconnects to find the path from that source endpoint to the >>> destination endpoint. >>> >> But doesn't this implies that the other end of the path is always some >> specific node, e.g. DDR? With a single node how would you describe >> CPU->LLCC or GPU->OCIMEM? > By only specifying the endpoint the device uses it describes what the > hardware block interfaces with. It doesn't imply that there's only one > other end of the path. It implies that the paths should be discoverable > by walking the interconnect graph given some source device node and > target device node. In most cases the target device node will be a DDR > controller node, but sometimes it could be LLCC or OCIMEM. We may need > to add some sort of "get the DDR controller device" API or work it into > the interconnect API somehow to indicate what target endpoint is > desired. By not listing all those paths in DT we gain flexibility to add > more paths later on without having to update or tweak DT to describe > more paths/routes through the interconnect. I'm unsure that using the target device node or target source device is the correct way to represent the constraints that the consumers apply on the interconnects. While it's true the traffic is intended for the targeted devices, the constraints(QoS or BW) are for the interconnect or specifically the paths that span across the ports of various interconnects(NoC devices in this case). I think having both src and dst properties is still the simplest way to achieve the flexibility that we require to set the constraints for ports(that may not have a target device defined in DT or exists as some intermediate port across multiple interconnects). >>> Obviously this patch doesn't compile but I'm sending it out to start >>> this discussion so we don't get stuck on the binding or the kernel APIs >>> for a long time. It looks like we should be OK in terms of backwards >>> compatibility because we can just ignore the second element in an old >>> binding, but maybe we'll want to describe paths in different directions >>> (e.g. the path from the CPU to the SD controller may be different than >>> the path the SD controller takes to the CPU) and that may require >>> extending interconnect-names to indicate what direction/sort of path it >>> is. I'm basically thinking about master vs. slave ports in AXI land. >>> >>> Cc: Maxime Ripard >>> Cc: >>> Cc: Rob Herring >>> Cc: >>> Cc: Bjorn Andersson >>> Cc: Evan Green >>> Cc: David Dai >>> Signed-off-by: Stephen Boyd >>> --- >>> .../bindings/interconnect/interconnect.txt | 19 ++++--------------- >>> include/linux/interconnect.h | 13 ++----------- >>> 2 files changed, 6 insertions(+), 26 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/interconnect/interconnect.txt b/Documentation/devicetree/bindings/interconnect/interconnect.txt >>> index 6f5d23a605b7..f8979186b8a7 100644 >>> --- a/Documentation/devicetree/bindings/interconnect/interconnect.txt >>> +++ b/Documentation/devicetree/bindings/interconnect/interconnect.txt >>> @@ -11,7 +11,7 @@ The interconnect provider binding is intended to represent the interconnect >>> controllers in the system. Each provider registers a set of interconnect >>> nodes, which expose the interconnect related capabilities of the interconnect >>> to consumer drivers. These capabilities can be throughput, latency, priority >>> -etc. The consumer drivers set constraints on interconnect path (or endpoints) >>> +etc. The consumer drivers set constraints on interconnect paths (or endpoints) >>> depending on the use case. Interconnect providers can also be interconnect >>> consumers, such as in the case where two network-on-chip fabrics interface >>> directly. >>> @@ -42,23 +42,12 @@ multiple paths from different providers depending on use case and the >>> components it has to interact with. >>> >>> Required properties: >>> -interconnects : Pairs of phandles and interconnect provider specifier to denote >>> - the edge source and destination ports of the interconnect path. >>> - >>> -Optional properties: >>> -interconnect-names : List of interconnect path name strings sorted in the same >>> - order as the interconnects property. Consumers drivers will use >>> - interconnect-names to match interconnect paths with interconnect >>> - specifier pairs. >>> - >>> - Reserved interconnect names: >>> - * dma-mem: Path from the device to the main memory of >>> - the system >>> +interconnects : phandle and interconnect provider specifier to denote >>> + the edge source for this node. >>> >>> Example: >>> >>> sdhci@7864000 { >>> ... >>> - interconnects = <&pnoc MASTER_SDCC_1 &bimc SLAVE_EBI_CH0>; >>> - interconnect-names = "sdhc-mem"; >>> + interconnects = <&pnoc MASTER_SDCC_1>; >> This example seems incomplete, as it doesn't describe the path between >> CPU and the config space, with this in place I think you need the >> interconnect-names. >> >> >> But with a single interconnect, the interconnect-names should be >> omitted, as done in other frameworks. >> > Sure, no names makes sense when it's just one path. > -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel