Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753019AbcKNNUB (ORCPT ); Mon, 14 Nov 2016 08:20:01 -0500 Received: from mail-lf0-f67.google.com ([209.85.215.67]:34871 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751564AbcKNNT6 (ORCPT ); Mon, 14 Nov 2016 08:19:58 -0500 MIME-Version: 1.0 In-Reply-To: <3587281.4kOCcW9Ryd@wuerfel> References: <1479099731-28108-1-git-send-email-pankaj.dubey@samsung.com> <1479099731-28108-5-git-send-email-pankaj.dubey@samsung.com> <3587281.4kOCcW9Ryd@wuerfel> From: Pankaj Dubey Date: Mon, 14 Nov 2016 18:49:36 +0530 X-Google-Sender-Auth: 5MIeNKZmRzg1iakAomXhnbCL3XM Message-ID: Subject: Re: [PATCH 04/16] ARM: realview: use generic API for enabling SCU To: Arnd Bergmann Cc: geert+renesas@glider.be, vireshk@kernel.org, magnus.damm@gmail.com, linux-kernel@vger.kernel.org, Krzysztof Kozlowski , Russell King , rmk+kernel@armlinux.org.uk, horms@verge.net.au, "thomas.ab@samsung.com" , shiraz.linux.kernel@gmail.com, "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1801 Lines: 52 Hi Arnd, On 14 November 2016 at 17:26, Arnd Bergmann wrote: > On Monday, November 14, 2016 10:31:59 AM CET Pankaj Dubey wrote: >> static const struct of_device_id realview_scu_match[] = { >> { .compatible = "arm,arm11mp-scu", }, >> - { .compatible = "arm,cortex-a9-scu", }, >> - { .compatible = "arm,cortex-a5-scu", }, >> { } >> }; >> >> @@ -41,27 +39,18 @@ static void __init realview_smp_prepare_cpus(unsigned int max_cpus) >> struct device_node *np; >> void __iomem *scu_base; >> struct regmap *map; >> - unsigned int ncores; >> int i; >> >> - np = of_find_matching_node(NULL, realview_scu_match); >> - if (!np) { >> - pr_err("PLATSMP: No SCU base address\n"); >> - return; >> + if (of_scu_enable()) { >> + np = of_find_matching_node(NULL, realview_scu_match); >> + scu_base = of_iomap(np, 0); >> + of_node_put(np); >> + if (!scu_base) { >> + pr_err("PLATSMP: No SCU remap\n"); >> + return; >> + } >> + scu_enable(scu_base); >> } >> > > The only difference here seems to be that realview also needs to handle > "arm,arm11mp-scu". Why not move that into the generic implementation? > Just checked scu binding documentation for "arm,arm11mp-scu" and came to know its for ARM11 MPCore based SoC's SCU. So as you said, it can surely be moved to genenric imeplemenation. Will do this change and resubmit again. Thanks, Pankaj Dubey > Arnd > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel