From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1755086AbcHSIDD (ORCPT <rfc822;w@1wt.eu>);
        Fri, 19 Aug 2016 04:03:03 -0400
Received: from mga03.intel.com ([134.134.136.65]:56445 "EHLO mga03.intel.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1753916AbcHSIDA (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 19 Aug 2016 04:03:00 -0400
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.28,544,1464678000"; 
   d="scan'208";a="158275541"
Message-ID: <1471593773.2691.48.camel@intel.com>
Subject: Re: [PATCH] thermal: hisilicon: Add dependency on the clock driver
 to allow frequency scaling
From: Zhang Rui <rui.zhang@intel.com>
To: Amit Kucheria <amit.kucheria@linaro.org>,
        Wei Xu <xuwei5@hisilicon.com>
Cc: Leo Yan <leo.yan@linaro.org>, Guodong Xu <guodong.xu@linaro.org>,
        Eduardo Valentin <edubezval@gmail.com>,
        LKML <linux-kernel@vger.kernel.org>,
        Linux PM list <linux-pm@vger.kernel.org>,
        kongxinwei <kong.kongxinwei@hisilicon.com>
Date: Fri, 19 Aug 2016 16:02:53 +0800
In-Reply-To: <CAHLCerMiWdh5QaXgrXyLOaEe+9hp5xg9Aj759RLCQxCComY5Vw@mail.gmail.com>
References: <32d418f24a4da44ad4ca5b2b1276ffcea2aa7d6c.1466424352.git.amit.kucheria@linaro.org>
         <f6bbd7ddf978001aa0f1c6fb8fc454a30241d0a8.1466424352.git.amit.kucheria@linaro.org>
         <20160620131616.GA400@leoy-linaro>
         <CAHLCerPjDdQToVnkh6q=xq4=BWjV-tnX3=599yudfVwf_XBnPQ@mail.gmail.com>
         <CAHLCerNp=xbqUmj3yOpRT7pUc-bOkBquSW0Z3MdcTYVPqP+b8g@mail.gmail.com>
         <CAHLCerPRi6JiUNAOAR5ULmX_6m1PbkJbB1tgO3meDJpiq0H74g@mail.gmail.com>
         <579611F3.3040307@hisilicon.com>
         <CAHLCerMiWdh5QaXgrXyLOaEe+9hp5xg9Aj759RLCQxCComY5Vw@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.18.5.2-0ubuntu3 
Mime-Version: 1.0
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 一, 2016-08-01 at 11:43 +0530, Amit Kucheria wrote:
> On Mon, Jul 25, 2016 at 6:49 PM, Wei Xu <xuwei5@hisilicon.com> wrote:
> > 
> > 
> > 
> > On 25/07/2016 13:29, Amit Kucheria wrote:
> > > 
> > > On Thu, Jul 7, 2016 at 12:13 PM, Amit Kucheria <amit.kucheria@lin
> > > aro.org> wrote:
> > > > 
> > > > On Sun, Jun 26, 2016 at 10:02 PM, Amit Kucheria
> > > > <amit.kucheria@linaro.org> wrote:
> > > > > 
> > > > > On Mon, Jun 20, 2016 at 6:46 PM, Leo Yan <leo.yan@linaro.org>
> > > > > wrote:
> > > > > > 
> > > > > > Hi Amit,
> > > > > > 
> > > > > > On Mon, Jun 20, 2016 at 05:46:36PM +0530, Amit Kucheria
> > > > > > wrote:
> > > > > > > 
> > > > > > > The Hisilicon clock stub driver is needed to allow the
> > > > > > > thermal drivers to
> > > > > > > actually scale the frequency. Make it an automatic
> > > > > > > dependency.
> > > > > > > 
> > > > > > > Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>
> > > > > > > ---
> > > > > > >  drivers/thermal/Kconfig | 1 +
> > > > > > >  1 file changed, 1 insertion(+)
> > > > > > > 
> > > > > > > diff --git a/drivers/thermal/Kconfig
> > > > > > > b/drivers/thermal/Kconfig
> > > > > > > index 22ae1f7..4e843f7 100644
> > > > > > > --- a/drivers/thermal/Kconfig
> > > > > > > +++ b/drivers/thermal/Kconfig
> > > > > > > @@ -178,6 +178,7 @@ config THERMAL_EMULATION
> > > > > > >  config HISI_THERMAL
> > > > > > >       tristate "Hisilicon thermal driver"
> > > > > > >       depends on (ARCH_HISI && CPU_THERMAL && OF) ||
> > > > > > > COMPILE_TEST
> > > > > > > +     select STUB_CLK_HI6220
> > > > > > Acked-by: Leo Yan <leo.yan@linaro.org>
> > > > > Thanks Leo. Eduardo, will you take this through your tree or
> > > > > should
> > > > > Wei include this in his tree?
> > > > Ping!
> > Hi Amit
> > 
> > > 
> > > 
> > > Wei, could you pull this into your tree for the next release?
> > I am OK If Eduardo is OK to pick up my pull request.
> > 
> > Hi Eduardo,
> > 
> > Is it OK for me to send a pull request to you about this?
> > Or could you let us know whether you could directly pick it up?
> > Thanks!
> Wei, if you can sign-off on this, perhaps Zhang can pick it up? I
> haven't heard from Eduardo for over a month.

Patch queued for 4.9.

thanks,
rui