From patchwork Thu Sep 3 13:31:27 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Georgi Djakov X-Patchwork-Id: 11753823 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 1672413B1 for ; Thu, 3 Sep 2020 14:42:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E99F1206EB for ; Thu, 3 Sep 2020 14:42:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="d72daTpV" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729386AbgICOmW (ORCPT ); Thu, 3 Sep 2020 10:42:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37270 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729378AbgICOgy (ORCPT ); Thu, 3 Sep 2020 10:36:54 -0400 Received: from mail-wr1-x444.google.com (mail-wr1-x444.google.com [IPv6:2a00:1450:4864:20::444]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C2BB3C0619D1 for ; Thu, 3 Sep 2020 06:31:38 -0700 (PDT) Received: by mail-wr1-x444.google.com with SMTP id z1so3284797wrt.3 for ; Thu, 03 Sep 2020 06:31:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=1ZdEfEgmL9xMVW+zYcpKfDQO7HkBHnE+KMZRuI8Lf9k=; b=d72daTpVXoFFKFMcoZerr4SN73VKG7KBahyo81SxAWwlnUIXnP4Hiu0wEqMahwIn2o LGa/L5PaDhSKkW6A2aKmCvXUc1Y7CQlo1vIRokt4mjGZ/E8C0NkC8DFHdRcyR6zzGStq jD2svGqpzvApSdfm4Yd9u3HDuCaTuayg11hPq4rk7boRhViVFGOwYD1DCSGWHHEQPuyv Umy4g1qdPh5N+ALxnFjOayg/Yw5EsjlS2OOxZcPUTLLLIom3LVSgdN+akv4yZOHb/LFw B4/tzmmeCZek/9PDbgTIoQnidxfXI7T4MJGG1/VXEKZXqpOHHbbuMB+oiIF9CfadmXbU KNRw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=1ZdEfEgmL9xMVW+zYcpKfDQO7HkBHnE+KMZRuI8Lf9k=; b=gTE4hEZx50rR5nrFQtYCsyUsN/pjWic8mQegXsqD8i4xjEVNBWKvIpNkAJWs+hufuO PP9C21jlBqVaTBxuMopRT8d8YhY+3L0/mA4Y1COeKkm9pP6zByPcqjaIvfXWwINIl/3r 38aEa3uLRX+dhVuRAT4yzbRN+Te+0yidOElSH9pi789MFSJHCBMlFPHCuC0WvAfrdqcZ S9kGiGACW1IYMs8hQfxeqosO3TAlzwFiy82cAkRwBFivmROPeweWe68QNqEudDzE//EH eCPAs6/m/uljGd3FVlqaZjCesiWI7quFD3p05aA4umWB4ONhiDE4CSfAlZWVWHioe1Bm rEIw== X-Gm-Message-State: AOAM530lw8y73GbljLaq5TttBrFw68VZcTn/G/X50MmmFMGcOupc1jG/ 7QGPclwdLs5vUuA9QO+Jx3XHHwdM4Is1xA== X-Google-Smtp-Source: ABdhPJwhjR2MLxPCogbMWCND1xPIuqqI6Vt+eUGTlk9kaW1cSI5oFWJ5vZON7txdbDjoS+wkK1t6og== X-Received: by 2002:a5d:5602:: with SMTP id l2mr2677121wrv.410.1599139896777; Thu, 03 Sep 2020 06:31:36 -0700 (PDT) Received: from localhost.localdomain ([212.45.67.2]) by smtp.googlemail.com with ESMTPSA id b2sm4310768wmh.47.2020.09.03.06.31.35 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 03 Sep 2020 06:31:36 -0700 (PDT) From: Georgi Djakov To: linux-pm@vger.kernel.org, bjorn.andersson@linaro.org Cc: devicetree@vger.kernel.org, robh+dt@kernel.org, sibis@codeaurora.org, mka@chromium.org, dianders@chromium.org, georgi.djakov@linaro.org, linux-kernel@vger.kernel.org Subject: [PATCH v3 0/7] interconnect: Introduce xlate_extended() Date: Thu, 3 Sep 2020 16:31:27 +0300 Message-Id: <20200903133134.17201-1-georgi.djakov@linaro.org> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org Currently the interconnect framework provides the xlate() callback for mapping the endpoints from phandle arguments in DT. This is implemented by the provider drivers. But the endpoints (or the path between the endpoints) might have an additional attributes, which should be parsed and populated. During previous discussions [1], it was suggested that adding more and more DT properties will not scale very well. So instead of adding DT properties, such data should be put into the 'interconnects' arguments. This patchset is introducing a new xlate_extended() callback, which would allow the interconnect provider drivers to parse additional data (such as path tags) and allow the framework to make use of them. Thanks to Sibi for the preliminary feedback and testing this patchset on the sc7180 platform. The patchset is based on linux-next (next-20200903). Changelog: v3: * Rebase and update both dts patches as Doug reported that i missed to update the cells for the GPU and also other stuff changed in DT recently. * Pick Rob's Reviewed-by v2: http://lore.kernel.org/r/20200806163126.22667-1-georgi.djakov@linaro.org * Add a patch to update the DT documentation, which i missed initially. * Minor changes in commit text of patch 1/7. (Sibi) * Remove a useless null check in patch 4/7. (Sibi) * Print a warning if the number of DT args is higher than what we expect in patch 3/7. (Matthias) * Pick Reviewed-by/Tested-by tags. * TODO: Replace tag IDs with macros in patches 5/7 and 7/7 when [2] lands. v1: http://lore.kernel.org/r/20200723130942.28491-1-georgi.djakov@linaro.org [1] https://lore.kernel.org/linux-devicetree/20200519185836.GA469006@bogus/ [2] https://lore.kernel.org/r/20200801033215.1440-2-mdtipton@codeaurora.org Georgi Djakov (5): interconnect: Introduce xlate_extended() callback dt-bindings: interconnect: Document the support of optional path tag interconnect: qcom: Implement xlate_extended() to parse tags interconnect: qcom: sdm845: Replace xlate with xlate_extended arm64: dts: qcom: sdm845: Increase the number of interconnect cells Sibi Sankar (2): interconnect: qcom: sc7180: Replace xlate with xlate_extended arm64: dts: qcom: sc7180: Increase the number of interconnect cells .../bindings/interconnect/interconnect.txt | 24 +- arch/arm64/boot/dts/qcom/sc7180.dtsi | 218 +++++++++--------- arch/arm64/boot/dts/qcom/sdm845.dtsi | 48 ++-- drivers/interconnect/core.c | 73 ++++-- drivers/interconnect/qcom/icc-rpmh.c | 27 +++ drivers/interconnect/qcom/icc-rpmh.h | 1 + drivers/interconnect/qcom/sc7180.c | 2 +- drivers/interconnect/qcom/sdm845.c | 2 +- include/linux/interconnect-provider.h | 17 +- 9 files changed, 251 insertions(+), 161 deletions(-)