From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757825AbbJVOQY (ORCPT ); Thu, 22 Oct 2015 10:16:24 -0400 Received: from mail-gw2-out.broadcom.com ([216.31.210.63]:60667 "EHLO mail-gw2-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756888AbbJVOQW (ORCPT ); Thu, 22 Oct 2015 10:16:22 -0400 X-IronPort-AV: E=Sophos;i="5.20,182,1444719600"; d="scan'208";a="78421322" Date: Thu, 22 Oct 2015 10:16:19 -0400 From: Jon Mason To: Stephen Boyd CC: Michael Turquette , Florian Fainelli , Hauke Mehrtens , Ray Jui , Scott Branden , , , , , Subject: Re: [PATCH v3 0/10] clk: iproc: add support for BCM NS, NSP, and NS2 Message-ID: <20151022141618.GC651@broadcom.com> References: <1444938513-10758-1-git-send-email-jonmason@broadcom.com> <20151021235144.GQ19782@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20151021235144.GQ19782@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 21, 2015 at 04:51:44PM -0700, Stephen Boyd wrote: > On 10/15, Jon Mason wrote: > > > > v3 - Added a patch to fix cygnus clk link failures, and added back the > > DT documentation patches that were removed from v2. > > > > v2 - Added a patch to change the Cygnus clk macros to uppercase, > > removed the device tree changes from the series (will send out those as > > an RFC), and other minor changes that Stephen Boyd requested. > > > > This patch series adds support for the Broadcom Northstar, Northstar > > Plus, and Northstar 2 clocks. Some slight modifications were necessary > > to clk-iproc-pll to get Northstar and Northstar Plus working, due to > > differences in register layout. This is the reason why the first patch > > is necessary. Some more modifications were necessary to clk-iproc-pll > > to get Northstar 2 working, due to differences in register layout (and > > resulting fallout in Cygnus and NSP). This is the reason why the sixth > > and seventh patches are necessary. The fifth patch is clean-up to > > prevent accidentally forgetting to adjust for the base write errata > > (which happened a few times, but was caught in internal review). > > > > Can you please use git cover-letter format where the diffstat for > the series is shown? Without that information here I started > applying patches until I realized this added dt-bindings headers, > thus probably needing some stable clk branch to pull into arm-soc. My apologies. I will do so in the future. Thanks, Jon > > -- > Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, > a Linux Foundation Collaborative Project