From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v2] clk: fixed-factor: add optional dt-binding clock-flags Date: Tue, 28 Jun 2016 15:55:18 -0500 Message-ID: <20160628205518.GF3737@rob-hp-laptop> References: <20160621005910.GN1521@codeaurora.org> <1466741572-58802-1-git-send-email-neidhard.kim@lge.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1466741572-58802-1-git-send-email-neidhard.kim@lge.com> Sender: linux-kernel-owner@vger.kernel.org To: Jongsung Kim Cc: Maxime Ripard , Mike Turquette , Stephen Boyd , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Chanho Min List-Id: devicetree@vger.kernel.org On Fri, Jun 24, 2016 at 01:12:52PM +0900, Jongsung Kim wrote: > There is no way to set additional flags for a DT-initialized fixed- > factor-clock, and it can be problematic i.e., when the clock rate > needs to be changed. [1][2] > > This patch introduces an optional dt-binding named "clock-flags" to > be used for passing any needed flags from dts. I don't think we want this in DT. If we did, the flags would need some documentation about what the flags mean. Rob