From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753874AbcBZIWe (ORCPT ); Fri, 26 Feb 2016 03:22:34 -0500 Received: from mail-wm0-f68.google.com ([74.125.82.68]:34323 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753598AbcBZIWc (ORCPT ); Fri, 26 Feb 2016 03:22:32 -0500 From: Holger Schurig To: Pankaj DEV Cc: "linux-arm-kernel\@lists.infradead.org" , "linux-kernel\@vger.kernel.org" , "devicetree\@vger.kernel.org" , "mturquette\@linaro.org" , "kernel\@stlinux.com" , "sboyd\@codeaurora.org" , Laurent MEUNIER , "lee.jones\@linaro.org" , Maxime COQUELIN Subject: Re: [RFC v1] clk: Add debugfs nodes for enable/disable/set-rate/set-parent In-Reply-To: <8B5B4FB66A982549A83D90C7791B761007F362D2E1@EAPEX1MAIL2.st.com> (Pankaj DEV's message of "Fri, 26 Feb 2016 16:06:48 +0800") References: <1456454309-25037-1-git-send-email-pankaj.dev@st.com> <87fuwg9bog.fsf@gmail.com> <8B5B4FB66A982549A83D90C7791B761007F362D2E1@EAPEX1MAIL2.st.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.4 (gnu/linux) Date: Fri, 26 Feb 2016 09:22:29 +0100 Message-ID: <87twkvj2yi.fsf@gmail.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Pankaj DEV writes: > The prime motive for clk_set_rate is to set new rate for a clock, > since the 'clk_rate' currently available, allows only reading. > To provide reading rate from 'clk_set_rate' is just additional feature. Sure. But my point is to combine things. After your patch: - read via clk_rate - read via clk_set_rate - write via clk_set_rate My proposal: - read via clk_rate - write via clk_rate