From patchwork Tue Jun 20 00:18:53 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Boyd X-Patchwork-Id: 9798267 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 4D34660328 for ; Tue, 20 Jun 2017 00:19:23 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 3D59D23E64 for ; Tue, 20 Jun 2017 00:19:23 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 3199A26E46; Tue, 20 Jun 2017 00:19:23 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id B054523E64 for ; Tue, 20 Jun 2017 00:19:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=chZdZTOY//7jvqi+u/MaHuTOP2tu4Y+DTzH5fisudSE=; b=rGGBoZu36PrAKN oLPYCIPXsaE3C2wFQtLxbGlnN40VRO5gRzG9okvECHOTMuf5zbZR19rn35WCG4cDre0svmyAvWKCT YfoE3DqF1w4APXY/h3aZc5OTYX7KDR8lf+yy4CggcmGfm43i65SCGlDaaF2Q7syccqWmsZg8hNC6L u7X7pWHI8HVnPC1QCknHtPUslEV+VAuaihR5+ksO0dUrVgZjIEHgqh9RMxvlpH2FQ0CRlCJlAMBcG +8/uk2okt4cH1KNkszUQRz3K0IPO2gkt9hRmvgdv9VbnJm4GuyadDou35Y9CHgewCt1dTD5wukaMM G6X7uVT5qaou0D2V0bsQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dN6t5-0001s1-Ao; Tue, 20 Jun 2017 00:19:19 +0000 Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dN6t1-0001em-N3 for linux-arm-kernel@lists.infradead.org; Tue, 20 Jun 2017 00:19:17 +0000 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 07F6C609C1; Tue, 20 Jun 2017 00:18:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1497917935; bh=w1uoldutl1/r4zJJpJv0LKRDNEk3ueyYSnzKRHTxm5E=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UkBH55YUZBa4ZlSZhhKMdT6oH8QKI1klGVimbcg3fh9ElStpSqbEVIg1NcXvbQdJt wyykbCIyFRtAg+yCuMFtwznxCWOkRX5L7fBbS+D9MVsSFkCAWyoeWx7ywagBBlpCmi 58iXY1/WLnjZiLoTY6dHocQn0Lp2JJMgqUZBE/cc= Received: from localhost (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: sboyd@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id E85A860730; Tue, 20 Jun 2017 00:18:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1497917934; bh=w1uoldutl1/r4zJJpJv0LKRDNEk3ueyYSnzKRHTxm5E=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=SaZuQd32a1rjiAIgQAvtI+Dr54V+qi7yXuOIkG9pZXvy0uIr33Iar2+v9QEL72niG BbHL5pqguYAkzZ2S3/AbE1POkxQtGGof62FZqHJeWKjOkPEMMthu1Ug0TzUDVyLCKl p/Zc4l/114XvgBQOlnPpqYjtnG7RTrRikThamago= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org E85A860730 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=sboyd@codeaurora.org Date: Mon, 19 Jun 2017 17:18:53 -0700 From: Stephen Boyd To: Gregory CLEMENT Subject: Re: [GIT PULL] Improve cp110 clk support on Marvell Armada 7K/8K Message-ID: <20170620001853.GT20170@codeaurora.org> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170619_171915_840125_407794C5 X-CRM114-Status: GOOD ( 20.83 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Thomas Petazzoni , Andrew Lunn , Jason Cooper , devicetree@vger.kernel.org, Mike Turquette , Omri Itach , linux-kernel@vger.kernel.org, Nadav Haklai , Rob Herring , Neta Zur Hershkovits , Kostya Porotchkin , Shadi Ammouri , Marcin Wojtas , linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP On 06/19, Gregory CLEMENT wrote: > > Hi Mike, Stephen, > > This time I turned the series in a pull request so I removed the > device tree binding part which will be in an other series for Rob > Herring and I also removed the dt part that I will take care of once > you will have pulled this one. > > I hope this will be easier for you to apply this way. > Ok. It wasn't clear what was going on and I forgot to ask in the lastest patch series. Pulled into clk-next, but I had to apply this patch on top: ---8<---- From: Stephen Boyd Subject: [PATCH] clk: mvebu: cp110: Minor cleanups Mark an array of strings static const and remove the dereference of a function pointer when assigning to the platform driver probe struct member. drivers/clk/mvebu/cp110-system-controller.c:89:12: warning: symbol 'gate_base_names' was not declared. Should it be static? drivers/clk/mvebu/cp110-system-controller.c:447:18: error: cannot dereference this type Signed-off-by: Stephen Boyd --- drivers/clk/mvebu/cp110-system-controller.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/clk/mvebu/cp110-system-controller.c b/drivers/clk/mvebu/cp110-system-controller.c index b034b79345ec..ca9a0a536174 100644 --- a/drivers/clk/mvebu/cp110-system-controller.c +++ b/drivers/clk/mvebu/cp110-system-controller.c @@ -86,7 +86,7 @@ enum { #define CP110_GATE_EIP150 25 #define CP110_GATE_EIP197 26 -const char *gate_base_names[] = { +static const char * const gate_base_names[] = { [CP110_GATE_AUDIO] = "audio", [CP110_GATE_COMM_UNIT] = "communit", [CP110_GATE_NAND] = "nand", @@ -437,14 +437,13 @@ static int cp110_clk_probe(struct platform_device *pdev) return cp110_syscon_common_probe(pdev, pdev->dev.of_node->parent); } - static const struct of_device_id cp110_syscon_legacy_of_match[] = { { .compatible = "marvell,cp110-system-controller0", }, { } }; static struct platform_driver cp110_syscon_legacy_driver = { - .probe = *cp110_syscon_legacy_clk_probe, + .probe = cp110_syscon_legacy_clk_probe, .driver = { .name = "marvell-cp110-system-controller0", .of_match_table = cp110_syscon_legacy_of_match,