From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srinivas Kandagatla Subject: Re: [PATCH v2 04/15] clk: divider: Make generic for usage elsewhere Date: Wed, 08 Oct 2014 16:04:57 +0100 Message-ID: <54355299.4000501@linaro.org> References: <1409957256-23729-1-git-send-email-sboyd@codeaurora.org> <1409957256-23729-5-git-send-email-sboyd@codeaurora.org> <542EE5FE.9080806@codeaurora.org> <54342287.7020005@linaro.org> <54343038.2060007@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f44.google.com ([74.125.82.44]:54383 "EHLO mail-wg0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750858AbaJHPFE (ORCPT ); Wed, 8 Oct 2014 11:05:04 -0400 Received: by mail-wg0-f44.google.com with SMTP id y10so11701991wgg.15 for ; Wed, 08 Oct 2014 08:05:03 -0700 (PDT) In-Reply-To: <54343038.2060007@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Stephen Boyd Cc: Mike Turquette , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Viresh Kumar , linux-pm@vger.kernel.org On 07/10/14 19:26, Stephen Boyd wrote: > On 10/07/2014 10:27 AM, Srinivas Kandagatla wrote: >> Hi Stephen, >> >> Just noticed this regression while testing the patch on Arndale board. >> >> https://bugs.linaro.org/show_bug.cgi?id=740 > > If you return value it works correctly right? > I think you are right, .. From last message from bugs.linaro.org: " That seems to be it! This change: --- a/drivers/clk/clk-divider.c +++ b/drivers/clk/clk-divider.c @@ -366,7 +366,7 @@ int divider_get_val(unsigned long rate, unsigned long parent min_t(unsigned int, value, div_mask(width)); - return 0; + return value; } EXPORT_SYMBOL_GPL(divider_get_val); - makes Arndale to boot; rootfs on mmc mounts OK. " thanks, srini