Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751118AbdFTBzu (ORCPT ); Mon, 19 Jun 2017 21:55:50 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:44536 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750905AbdFTBzt (ORCPT ); Mon, 19 Jun 2017 21:55:49 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org B5DC060316 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: Mon, 19 Jun 2017 18:55:47 -0700 From: Stephen Boyd To: Dong Aisheng Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, mturquette@baylibre.com, shawnguo@kernel.org, Anson.Huang@nxp.com, ping.bai@nxp.com Subject: Re: [PATCH 3/9] clk: fractional-divider: add CLK_FRAC_DIVIDER_ZERO_BASED flag support Message-ID: <20170620015547.GN4493@codeaurora.org> References: <1494856763-6543-1-git-send-email-aisheng.dong@nxp.com> <1494856763-6543-4-git-send-email-aisheng.dong@nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1494856763-6543-4-git-send-email-aisheng.dong@nxp.com> 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 Content-Length: 970 Lines: 26 On 05/15, Dong Aisheng wrote: > diff --git a/include/linux/clk-provider.h b/include/linux/clk-provider.h > index a6efbb9..4466cae 100644 > --- a/include/linux/clk-provider.h > +++ b/include/linux/clk-provider.h > @@ -557,6 +557,11 @@ void clk_hw_unregister_fixed_factor(struct clk_hw *hw); > * @lock: register lock > * > * Clock with adjustable fractional divider affecting its output frequency. > + * > + * Flags: > + * CLK_FRAC_DIVIDER_ZERO_BASED - by default the divisor is the value read > + * from the register. If CLK_FRAC_DIVIDER_ZERO_BASED is set then the > + * divider is the raw value read from the register plus one. This should say the numerator and denominator are both the value read plus one. It isn't clear if it applies to the numerator, or the denominator, or both. > */ > struct clk_fractional_divider { > struct clk_hw hw; -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project