From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Gross Subject: Re: [RFC PATCH 1/2] soc: qcom: do not disable the iface clock in probe Date: Tue, 10 Jun 2014 12:57:13 -0500 Message-ID: <20140610175713.GB10275@qualcomm.com> References: <1402410678-12931-1-git-send-email-srinivas.kandagatla@linaro.org> <1402410717-12977-1-git-send-email-srinivas.kandagatla@linaro.org> <6D685866-4AE5-458B-BB8C-EFEBAE221A9E@codeaurora.org> <539742B6.7030303@codeaurora.org> <539744B1.9090503@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.codeaurora.org ([198.145.11.231]:56884 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751888AbaFJR5P (ORCPT ); Tue, 10 Jun 2014 13:57:15 -0400 Content-Disposition: inline In-Reply-To: <539744B1.9090503@linaro.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Srinivas Kandagatla Cc: Stephen Boyd , Kumar Gala , linux-arm-msm , Mike Turquette On Tue, Jun 10, 2014 at 06:47:29PM +0100, Srinivas Kandagatla wrote: > >What does mode and crci have to do with this patch? Can't we just put > >the clock into the platform data? > It has nothing to do with this but, for completeness and we might > need this if we are doing PM in future. for example pm resume might > want to reconfigure the gsbi. Yes, setting idle mode would save some power. -- sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation