From patchwork Mon Jan 30 16:49:21 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Clark X-Patchwork-Id: 721624 Return-Path: X-Original-To: incoming-dt@patchwork.ozlabs.org Delivered-To: patchwork-incoming-dt@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3vBwZQ4hmhz9sdm for ; Tue, 31 Jan 2017 03:58:22 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="SbCIurse"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752278AbdA3Q6U (ORCPT ); Mon, 30 Jan 2017 11:58:20 -0500 Received: from mail-qk0-f195.google.com ([209.85.220.195]:36375 "EHLO mail-qk0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751302AbdA3Q6A (ORCPT ); Mon, 30 Jan 2017 11:58:00 -0500 Received: by mail-qk0-f195.google.com with SMTP id i34so17106182qkh.3; Mon, 30 Jan 2017 08:58:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=OSg2h800Lu8Ddu8dptBAUDn8HrzrEFqyiQ/ZVDCVNDM=; b=SbCIursezZYpIPqLtH0Dvw9CpCJ97iDC/hubjoLqVbXDrykU7USDhIFt8DxAE8gn2E F/Davie6jkEuVGnUzVDMJotNNReoZyvNF0G2d6CzWI7/vITZNjlaxTObWAWMyy144y4o 3hBfTLOOme01GPFGGDgqfNIoXWKX436jefux3ifvC2FzcsmQ3ye2nx29bjc8j4y/gV+h vpzzMn/RizCzSCJ3kb/bVFf+psNw+JDg0CZoez8vz44ri4YaaXFOSQxv+mm48d1qxqIP XNA6Z0xleJ/Bp07OQNDwWMDQGQiN2nZtn6IkuGY/VzJE/kYfH/ZOh0gJHNjd1KOtTXZ6 oujA== 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:in-reply-to :references; bh=OSg2h800Lu8Ddu8dptBAUDn8HrzrEFqyiQ/ZVDCVNDM=; b=dPPon5LK0ZVHH81CQw1Z0TDWuM8rBHA4nDL0xoXKP1PRadsX9anLpbWSiN/nJwh+Xx jcklvFBXZo3wtNOspgtpRrsLPunOCPJPKAyNKPxG1poZpvIsNmQNwm3qS26pfPCo43ni jlBysQKQAyLbZdGZuVhT1TkTovLIHsdQ0lvDC/qrNyYgtBF6Q/QywWeKlLADFHST1R1J 7GBlTLrFEOY2Ypx8H+JcNZE59Kp9wYKoqM/Ejr+m2D3a/VhodwmMx8yZL1oxlKcPx+S+ r6VE0nABibLEMGTgxn9R/9ctt+sH+zGHT66IiKAnoud9CZarX1dvOX95Xc2PJy17oQ/2 xtbQ== X-Gm-Message-State: AIkVDXIR/a7DxQVtCxbzAnyShAVY/BFDzxIiEuZoTxBX5xK5TSs3pYzw7yGspbsvu3wSEw== X-Received: by 10.55.33.136 with SMTP id f8mr23453879qki.132.1485794981754; Mon, 30 Jan 2017 08:49:41 -0800 (PST) Received: from localhost (nat-pool-bos-t.redhat.com. [66.187.233.206]) by smtp.gmail.com with ESMTPSA id 12sm12676732qtv.31.2017.01.30.08.49.41 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 30 Jan 2017 08:49:41 -0800 (PST) From: Rob Clark To: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org Cc: linux-arm-msm@vger.kernel.org, freedreno@lists.freedesktop.org, Rob Herring , Archit Taneja , jcrouse@codeaurora.org, Rob Clark Subject: [PATCH 4/4] drm/msm: drop _clk suffix from clk names Date: Mon, 30 Jan 2017 11:49:21 -0500 Message-Id: <20170130164921.20744-5-robdclark@gmail.com> X-Mailer: git-send-email 2.9.3 In-Reply-To: <20170130164921.20744-1-robdclark@gmail.com> References: <20170130164921.20744-1-robdclark@gmail.com> Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Suggested by Rob Herring. We still support the old names for compatibility with downstream android dt files. Cc: Rob Herring Signed-off-by: Rob Clark Reviewed-by: Eric Anholt Acked-by: Rob Herring --- Documentation/devicetree/bindings/display/msm/gpu.txt | 12 ++++++------ drivers/gpu/drm/msm/msm_drv.c | 19 +++++++++++++++++++ drivers/gpu/drm/msm/msm_drv.h | 1 + drivers/gpu/drm/msm/msm_gpu.c | 7 +++---- 4 files changed, 29 insertions(+), 10 deletions(-) diff --git a/Documentation/devicetree/bindings/display/msm/gpu.txt b/Documentation/devicetree/bindings/display/msm/gpu.txt index 7ac3052..43fac0f 100644 --- a/Documentation/devicetree/bindings/display/msm/gpu.txt +++ b/Documentation/devicetree/bindings/display/msm/gpu.txt @@ -11,9 +11,9 @@ Required properties: - clocks: device clocks See ../clocks/clock-bindings.txt for details. - clock-names: the following clocks are required: - * "core_clk" - * "iface_clk" - * "mem_iface_clk" + * "core" + * "iface" + * "mem_iface" Example: @@ -27,9 +27,9 @@ Example: interrupts = ; interrupt-names = "kgsl_3d0_irq"; clock-names = - "core_clk", - "iface_clk", - "mem_iface_clk"; + "core", + "iface", + "mem_iface"; clocks = <&mmcc GFX3D_CLK>, <&mmcc GFX3D_AHB_CLK>, diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c index 6b85c41..a51d783 100644 --- a/drivers/gpu/drm/msm/msm_drv.c +++ b/drivers/gpu/drm/msm/msm_drv.c @@ -91,6 +91,25 @@ module_param(dumpstate, bool, 0600); * Util/helpers: */ +struct clk *msm_clk_get(struct platform_device *pdev, const char *name) +{ + struct clk *clk; + char name2[32]; + + clk = devm_clk_get(&pdev->dev, name); + if (!IS_ERR(clk) || PTR_ERR(clk) == -EPROBE_DEFER) + return clk; + + snprintf(name2, sizeof(name2), "%s_clk", name); + + clk = devm_clk_get(&pdev->dev, name2); + if (!IS_ERR(clk) || PTR_ERR(clk) == -EPROBE_DEFER) + dev_warn(&pdev->dev, "Using legacy clk name binding. Use " + "\"%s\" instead of \"%s\"\n", name, name2); + + return clk; +} + void __iomem *msm_ioremap(struct platform_device *pdev, const char *name, const char *dbgname) { diff --git a/drivers/gpu/drm/msm/msm_drv.h b/drivers/gpu/drm/msm/msm_drv.h index ed4dad3..5f6f48f 100644 --- a/drivers/gpu/drm/msm/msm_drv.h +++ b/drivers/gpu/drm/msm/msm_drv.h @@ -318,6 +318,7 @@ static inline int msm_debugfs_late_init(struct drm_device *dev) { return 0; } static inline void msm_rd_dump_submit(struct msm_gem_submit *submit) {} #endif +struct clk *msm_clk_get(struct platform_device *pdev, const char *name); void __iomem *msm_ioremap(struct platform_device *pdev, const char *name, const char *dbgname); void msm_writel(u32 data, void __iomem *addr); diff --git a/drivers/gpu/drm/msm/msm_gpu.c b/drivers/gpu/drm/msm/msm_gpu.c index d8420be..7b29843 100644 --- a/drivers/gpu/drm/msm/msm_gpu.c +++ b/drivers/gpu/drm/msm/msm_gpu.c @@ -564,8 +564,7 @@ static irqreturn_t irq_handler(int irq, void *data) } static const char *clk_names[] = { - "core_clk", "iface_clk", "rbbmtimer_clk", "mem_clk", - "mem_iface_clk", "alt_mem_iface_clk", + "core", "iface", "rbbmtimer", "mem", "mem_iface", "alt_mem_iface", }; int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev, @@ -629,13 +628,13 @@ int msm_gpu_init(struct drm_device *drm, struct platform_device *pdev, /* Acquire clocks: */ for (i = 0; i < ARRAY_SIZE(clk_names); i++) { - gpu->grp_clks[i] = devm_clk_get(&pdev->dev, clk_names[i]); + gpu->grp_clks[i] = msm_clk_get(pdev, clk_names[i]); DBG("grp_clks[%s]: %p", clk_names[i], gpu->grp_clks[i]); if (IS_ERR(gpu->grp_clks[i])) gpu->grp_clks[i] = NULL; } - gpu->ebi1_clk = devm_clk_get(&pdev->dev, "bus_clk"); + gpu->ebi1_clk = msm_clk_get(pdev, "bus"); DBG("ebi1_clk: %p", gpu->ebi1_clk); if (IS_ERR(gpu->ebi1_clk)) gpu->ebi1_clk = NULL;