From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753341AbdLIAQo (ORCPT ); Fri, 8 Dec 2017 19:16:44 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:50230 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751146AbdLIAQl (ORCPT ); Fri, 8 Dec 2017 19:16:41 -0500 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 0B1666029D Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=sboyd@codeaurora.org Date: Fri, 8 Dec 2017 16:16:39 -0800 From: Stephen Boyd To: Abhishek Sahu Cc: Michael Turquette , Andy Gross , David Brown , Rajendra Nayak , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 00/13] Updates for QCOM Alpha PLL Message-ID: <20171209001639.GA7997@codeaurora.org> References: <1506621050-10129-1-git-send-email-absahu@codeaurora.org> <20171207062331.GI4283@codeaurora.org> <0fc3e5cdf9d5778655f302868154aff6@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <0fc3e5cdf9d5778655f302868154aff6@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 12/08, Abhishek Sahu wrote: > On 2017-12-07 11:53, Stephen Boyd wrote: > >On 09/28, Abhishek Sahu wrote: > >>This patch series does the miscellaneous changes in QCOM Alpha PLL > >>operation and structure to support other types of Alpha PLL’s. > >> > >>1. It adds the pll_type which will be used for determining all > >> the properties of Alpha PLL. > >>2. It adds the support for Brammo and Huayra PLL’s for which > >> the support is not available in existing alpha PLL code. > >>3. There won’t be any change in existing users of Alpha PLL’s > >> since all the newly added code will be under flag for the default > >> PLL operations. > >> > > > >Ok. I took a long look at this today. I rewrote a bunch of stuff. > > Thanks Stephen for reviewing the changes and making the code > cleaner. I checked all the code changes and everything looks > good. It will work for all of our requirement. I will check > with other PLL users also once and then update the patch > series with all your suggested code changes after complete > testing. Ok. Please don't resend. I've pushed out the branch to clk-next, and it can be adjusted if needed, or dropped and rewritten if something is totally busted. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project