From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754580Ab2CDRmt (ORCPT ); Sun, 4 Mar 2012 12:42:49 -0500 Received: from londo.lunn.ch ([80.238.139.98]:52855 "EHLO londo.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754457Ab2CDRms (ORCPT ); Sun, 4 Mar 2012 12:42:48 -0500 Date: Sun, 4 Mar 2012 18:42:29 +0100 From: Andrew Lunn To: Mike Turquette Cc: Russell King , patches@linaro.org, linaro-dev@lists.linaro.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Mike Turquette , Jeremy Kerr , Thomas Gleixner , Arnd Bergman , Paul Walmsley , Shawn Guo , Sascha Hauer , Jamie Iles , Richard Zhao , Saravana Kannan , Magnus Damm , Rob Herring , Mark Brown , Linus Walleij , Stephen Boyd , Amit Kucheria , Deepak Saxena , Grant Likely , Andrew Lunn Subject: Re: [PATCH v5 4/4] clk: basic clock hardware types Message-ID: <20120304174229.GD11546@lunn.ch> References: <1330763341-3437-1-git-send-email-mturquette@linaro.org> <1330763341-3437-5-git-send-email-mturquette@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1330763341-3437-5-git-send-email-mturquette@linaro.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 Sat, Mar 03, 2012 at 12:29:01AM -0800, Mike Turquette wrote: > Many platforms support simple gateable clocks, fixed-rate clocks, > adjustable divider clocks and multi-parent multiplexer clocks. > > This patch introduces basic clock types for the above-mentioned hardware > which share some common characteristics. Hi Mike These basic clocks don't allow the use of prepare/unprepare, from the side of the clock provider. I think i need this. The Orion Kirkwood SoC has clock gating for most of its on chip peripherals, which the other older Orion devices don't have. The SATA and PCIe also allows the PHY to be shut down, again which older Orion devices don't have. The current code links the clock and the PHY together, shutting both down are the same time. So i would like to perform the PHY shutdown in the unprepare() function of the clk driver. Is allowing to pass prepare/unprepare functions to basic clocks something you want to support? If i prepare a patch would you consider it? Thanks Andrew