From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1754117AbbJNURY (ORCPT <rfc822;w@1wt.eu>);
	Wed, 14 Oct 2015 16:17:24 -0400
Received: from mout.kundenserver.de ([212.227.17.10]:56116 "EHLO
	mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1753385AbbJNURW (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 14 Oct 2015 16:17:22 -0400
From: Arnd Bergmann <arnd@arndb.de>
To: Scott Branden <sbranden@broadcom.com>
Cc: Jon Mason <jonmason@broadcom.com>,
        Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@codeaurora.org>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Hauke Mehrtens <hauke@hauke-m.de>, Ray Jui <rjui@broadcom.com>,
        linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        bcm-kernel-feedback-list@broadcom.com
Subject: Re: [PATCH v2 7/7] clk: ns2: add clock support for Broadcom Northstar 2 SoC
Date: Wed, 14 Oct 2015 22:16:45 +0200
Message-ID: <6398606.WW44dcgQDV@wuerfel>
User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; )
In-Reply-To: <561EB10B.40502@broadcom.com>
References: <1444770485-11210-1-git-send-email-jonmason@broadcom.com> <6427519.eIBEvCNlle@wuerfel> <561EB10B.40502@broadcom.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:qn3rHt6VbtsfeX5KwKoELZdDNDqPNsBXRmO2FguGQQ27jXhEF5z
 c5Q0qI41RJsSlnObLzUNvUPJ36uzXaQemfaACAIHHPmR2KiUCapE0vo+mQtB3Wyey72oYO2
 Kfh4RBWSpT81TzQRvTnmEGX+0+7amx9zBUjIAEZQbNBbR+4uhURMnYHDgjWCQOWawpp0Plj
 RRei9z+dxzr1+hIO/LSkA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:6Rw+MHxyk3g=:rjn15viByO1OPKz/99IB+K
 pVLPv0yIOLlAarPQNCCFnw1iKWwqxKmhyXDibWAjw2ltfEb4J+Z2VCiUGn/1gyt3DG67H504G
 q/ncKwXjpkJUFZzSNerTPvuVoZ4u2bPdlxaDfldB65e7k43X1N+7sY+2psM8X49Ycz4BbAmmx
 DOoxF0jdTD5wC3DBqvBoBDl1ludFtoopn8RcXqJcV90+RGtM5oBTnbYl6G4y1+dLouWJYPt0+
 UqlFFOI+kAjk2TvquUOgkKbwySr46DCBZlpDVQkqMPBt+0QKsTXovWlW2YqmGJmSlgMGaktp0
 tcg6P99qRHpUQ3bZ8sDapl61GjGvBjqEHUWT7jEt45vTHtgXQnsZ+O5IylTMW7PSOjPBfnXI8
 3/TmM6BSURwXIz8WL1GitsMFT+pT8olz76iZIy/9YIJr8GtQEUbnvYTmnkcyB95JfIuMrCjSq
 YxQ6PK77lLrhAay9SbxYR92RW/d0wBnmybjssqviEZsW6XbHlfO1dsuObe+uE7073/VNFT5BP
 eKRe5eByhQgsK0UAAz44fe4HBNvcGTy07HmkKqWD3iXl9cdXaeuu6uoOoeIs/v2sgSkSAS8t5
 9jfoDOcm0pAJzXxY7TSbQM2RM4Gmi0GJPD5iDh8rpl487JezWrvjI9fMDzWCHi9C3xsfuatwV
 ZO6QxZ933TGHmPMALooJFZOEkP2KoHtACZ2D4m1c/zJDsKZaY6CO68sU2aRmCvg7klPI5WPcM
 /GZKkkhi6kll0Osh
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wednesday 14 October 2015 12:46:19 Scott Branden wrote:
> >
> > My patch fixes it by always selecting COMMON_CLK_IPROC from
> > ARCH_BCM_CYGNUS. I wasn't sure whether you want COMMON_CLK_IPROC
> > to still be user-selectable, so I left that in place. Normally
> > I'd expect it to be a silent option though, that just gets
> > implicitly enabled whenever a platform that needs it is built
> > into the kernel.
> COMMON_CLK_IPROC can be a silent option and selected as Arnd has done below.

Ok, please do that then. If you want to apply my patch directly,
you can remove that last paragraph from the description.

> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Acked-by: Scott Branden <sbranden@broadcom.com>

Thanks for taking a look!

	Arnd