From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753115AbcI3I4k (ORCPT ); Fri, 30 Sep 2016 04:56:40 -0400 Received: from mailgw01.mediatek.com ([210.61.82.183]:41618 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752763AbcI3I4R (ORCPT ); Fri, 30 Sep 2016 04:56:17 -0400 Message-ID: <1475225766.25044.33.camel@mtksdaap41> Subject: Re: [PATCH v14 4/4] CMDQ: save more energy in idle From: Horng-Shyang Liao To: Jassi Brar CC: Rob Herring , Matthias Brugger , Daniel Kurtz , "Sascha Hauer" , Devicetree List , Linux Kernel Mailing List , "linux-arm-kernel@lists.infradead.org" , , , Sascha Hauer , "Philipp Zabel" , Nicolas Boichat , "CK HU" , cawa cheng , Bibby Hsieh , YT Shen , Daoyuan Huang , Damon Chu , "Josh-YC Liu" , Glory Hung , Jiaguang Zhang , Dennis-YC Hsieh , Monica Wang , Date: Fri, 30 Sep 2016 16:56:06 +0800 In-Reply-To: <1474622885.21723.25.camel@mtksdaap41> References: <1473039885-24009-1-git-send-email-hs.liao@mediatek.com> <1473039885-24009-5-git-send-email-hs.liao@mediatek.com> <1474622885.21723.25.camel@mtksdaap41> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2016-09-23 at 17:28 +0800, Horng-Shyang Liao wrote: > On Thu, 2016-09-22 at 13:22 +0530, Jassi Brar wrote: > > On Mon, Sep 5, 2016 at 7:14 AM, HS Liao wrote: > > > Use clk_disable_unprepare instead of clk_disable to save more energy > > > when CMDQ is idle. > > > > > > Signed-off-by: HS Liao > > > --- > > > drivers/mailbox/mtk-cmdq.c | 54 +++++++++++++++++++++++++++++++++++++++------- > > > > The driver is introduced by second patch of the set, so it makes sense > > to merge this patch into patch 2/4. > > Hi Jassi, > > Could you take a look at previous discussion between Matthias and me? > http://lkml.iu.edu/hypermail/linux/kernel/1606.2/05239.html > His basic idea is to simplify first working version. > Therefore, I move some code to this patch. > > Thanks, > HS > Hi Jassi, What do you think about our previous discussion? Thanks, HS From mboxrd@z Thu Jan 1 00:00:00 1970 From: Horng-Shyang Liao Subject: Re: [PATCH v14 4/4] CMDQ: save more energy in idle Date: Fri, 30 Sep 2016 16:56:06 +0800 Message-ID: <1475225766.25044.33.camel@mtksdaap41> References: <1473039885-24009-1-git-send-email-hs.liao@mediatek.com> <1473039885-24009-5-git-send-email-hs.liao@mediatek.com> <1474622885.21723.25.camel@mtksdaap41> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1474622885.21723.25.camel@mtksdaap41> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jassi Brar Cc: Rob Herring , Matthias Brugger , Daniel Kurtz , Sascha Hauer , Devicetree List , Linux Kernel Mailing List , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, srv_heupstream-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org, Sascha Hauer , Philipp Zabel , Nicolas Boichat , CK HU , cawa cheng , Bibby Hsieh , YT Shen , Daoyuan Huang , Damon Chu , Jo List-Id: devicetree@vger.kernel.org On Fri, 2016-09-23 at 17:28 +0800, Horng-Shyang Liao wrote: > On Thu, 2016-09-22 at 13:22 +0530, Jassi Brar wrote: > > On Mon, Sep 5, 2016 at 7:14 AM, HS Liao wrote: > > > Use clk_disable_unprepare instead of clk_disable to save more energy > > > when CMDQ is idle. > > > > > > Signed-off-by: HS Liao > > > --- > > > drivers/mailbox/mtk-cmdq.c | 54 +++++++++++++++++++++++++++++++++++++++------- > > > > The driver is introduced by second patch of the set, so it makes sense > > to merge this patch into patch 2/4. > > Hi Jassi, > > Could you take a look at previous discussion between Matthias and me? > http://lkml.iu.edu/hypermail/linux/kernel/1606.2/05239.html > His basic idea is to simplify first working version. > Therefore, I move some code to this patch. > > Thanks, > HS > Hi Jassi, What do you think about our previous discussion? Thanks, HS -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: hs.liao@mediatek.com (Horng-Shyang Liao) Date: Fri, 30 Sep 2016 16:56:06 +0800 Subject: [PATCH v14 4/4] CMDQ: save more energy in idle In-Reply-To: <1474622885.21723.25.camel@mtksdaap41> References: <1473039885-24009-1-git-send-email-hs.liao@mediatek.com> <1473039885-24009-5-git-send-email-hs.liao@mediatek.com> <1474622885.21723.25.camel@mtksdaap41> Message-ID: <1475225766.25044.33.camel@mtksdaap41> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, 2016-09-23 at 17:28 +0800, Horng-Shyang Liao wrote: > On Thu, 2016-09-22 at 13:22 +0530, Jassi Brar wrote: > > On Mon, Sep 5, 2016 at 7:14 AM, HS Liao wrote: > > > Use clk_disable_unprepare instead of clk_disable to save more energy > > > when CMDQ is idle. > > > > > > Signed-off-by: HS Liao > > > --- > > > drivers/mailbox/mtk-cmdq.c | 54 +++++++++++++++++++++++++++++++++++++++------- > > > > The driver is introduced by second patch of the set, so it makes sense > > to merge this patch into patch 2/4. > > Hi Jassi, > > Could you take a look at previous discussion between Matthias and me? > http://lkml.iu.edu/hypermail/linux/kernel/1606.2/05239.html > His basic idea is to simplify first working version. > Therefore, I move some code to this patch. > > Thanks, > HS > Hi Jassi, What do you think about our previous discussion? Thanks, HS