From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann 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> References: <1444770485-11210-1-git-send-email-jonmason@broadcom.com> <6427519.eIBEvCNlle@wuerfel> <561EB10B.40502@broadcom.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <561EB10B.40502-dY08KVG/lbpWk0Htik3J/w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Scott Branden Cc: Jon Mason , Michael Turquette , Stephen Boyd , Florian Fainelli , Hauke Mehrtens , Ray Jui , linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org List-Id: devicetree@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 > Acked-by: Scott Branden Thanks for taking a look! Arnd -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html