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 S932219AbbJMVT2 (ORCPT <rfc822;w@1wt.eu>);
	Tue, 13 Oct 2015 17:19:28 -0400
Received: from mout.kundenserver.de ([212.227.126.187]:54304 "EHLO
	mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S932079AbbJMVT1 (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Tue, 13 Oct 2015 17:19:27 -0400
From: Arnd Bergmann <arnd@arndb.de>
To: Jon Mason <jonmason@broadcom.com>
Cc: 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>,
        Scott Branden <sbranden@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: Tue, 13 Oct 2015 23:18:53 +0200
Message-ID: <5280890.6npshDUNFZ@wuerfel>
User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; )
In-Reply-To: <1444770485-11210-8-git-send-email-jonmason@broadcom.com>
References: <1444770485-11210-1-git-send-email-jonmason@broadcom.com> <1444770485-11210-8-git-send-email-jonmason@broadcom.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:xalZbH6UOi7M5+sR4/DSJQsrYcPgHwzRPV8AyTUC/yaMXchf4oc
 QHJKBav9Z4BrFbG//w+JRSmknWI+fJMqX6mk0NlGJFhUftKc8L6l1mBtonDr/SP9IKXP+Cu
 ACAPEw3r/JCeSXGSoiUy4ViRKyJZSaGW9BYX2QwQOzvvfwC0r97D25mJj/EnFp3k9rzI3ZQ
 o4FOO1YKlRrHE3BVGr/ZA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:WCxKKw/CMEk=:fhXL3KYe/u/iwX2sSYTW2G
 OP1+/6D6dAEnGVnEmH67Qt/hRHzBag3+ENJAog/DXsJivVlPbAk8xyzZuChfdMGUXMPSy7+Vm
 O6lLCK3LcXx8v1gqJTP6ilBpoQqc9ZvMqG9yzCfOtSyb84AOuivISwnwFksDmLB3BbOiWxt7o
 TK9L6u2pi/1LfyWA+jDj3rKjSUffx7N0pUd9kt905p4O7S7vwVW2HmUdtWbTAnRvV2et7NXTA
 I1XlxHsGLz0W9d/2GtZUNuGSY8SnYO+sMAlyqZQ6JG1hB/O6dj4y+vLkosv3yvMclZijQrial
 bGuIqNgJSQ7EjPqjFGEfcXg01fmcf1TX/TFdbldG24g0gBMjGN2hX7hHk13RXED5d05zoaSpM
 J+kQFFCAQxFiMqE/72BYTe3Q9cxdgsbX1MwIOKVBCoWvMVljvDCGffYNezGZX/5a5Puz1NJgT
 E5hqua4+lCOO5SxDl9sKaCGqpcCQ+lSzg2hOcvlkEDGFPQqZPociWVjQq08585xnEaI+lS9kp
 aVsrV3XJKHJHhKfSkvdalDyJ9AeAN1G0INghIViE1Oug8LxvQatpE7av1ebJJ24gZrfXXu0Mz
 hQeEdtk3kEMZcEfzjd7TII59/CAd/o1voSVGI/Cu0QRLOOFLvN3KbZRVY7HVznoA/gYj0w8UI
 4VT4WQJon1W+Yw0YNxN/2VnqDFELuzXE7CWDpOYuR3NqkDSVSK3ovyOBTmEBl87zy6/e/Xmcf
 TrqmdCzp6vQvk6Av
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tuesday 13 October 2015 17:08:05 Jon Mason wrote:
> @@ -3,6 +3,7 @@ obj-$(CONFIG_CLK_BCM_KONA)      += clk-kona-setup.o
>  obj-$(CONFIG_CLK_BCM_KONA)     += clk-bcm281xx.o
>  obj-$(CONFIG_CLK_BCM_KONA)     += clk-bcm21664.o
>  obj-$(CONFIG_COMMON_CLK_IPROC) += clk-iproc-armpll.o clk-iproc-pll.o clk-iproc-asiu.o
> +obj-$(CONFIG_COMMON_CLK_IPROC) += clk-ns2.o
>  obj-$(CONFIG_ARCH_BCM_CYGNUS)  += clk-cygnus.o
>  obj-$(CONFIG_ARCH_BCM_NSP)     += clk-nsp.o
>  obj-$(CONFIG_ARCH_BCM_5301X)   += clk-nsp.o

If I understand this right, both CYGNUS and NS2 are IPROC based, but it
looks like you now require building the clk-ns2 file for both?

On a related note, I'm seeing problems when CONFIG_CYGNUS is set but
CONFIG_COMMON_CLK_IPROC is disabled, as that currently leads to a link
failure.

	Arnd