From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH 04/10] PM / OPP: Error out on failing to add static OPPs for v1 bindings Date: Thu, 8 Dec 2016 09:00:59 +0530 Message-ID: <20161208033059.GA24152@vireshk-i7> References: <1f876a6d176784f9ab901aea7ff521d53522682d.1481015522.git.viresh.kumar@linaro.org> <20161207011722.GJ4388@codeaurora.org> <20161207032555.GE31255@vireshk-i7> <20161207211343.GA5423@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pf0-f180.google.com ([209.85.192.180]:35022 "EHLO mail-pf0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753274AbcLHDcT (ORCPT ); Wed, 7 Dec 2016 22:32:19 -0500 Received: by mail-pf0-f180.google.com with SMTP id i88so80557335pfk.2 for ; Wed, 07 Dec 2016 19:31:06 -0800 (PST) Content-Disposition: inline In-Reply-To: <20161207211343.GA5423@codeaurora.org> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Stephen Boyd , shawnguo@kernel.org Cc: Rafael Wysocki , Viresh Kumar , Nishanth Menon , linaro-kernel@lists.linaro.org, linux-pm@vger.kernel.org, Vincent Guittot Hi Shawn, On 07-12-16, 13:13, Stephen Boyd wrote: > On 12/07, Viresh Kumar wrote: > > On 06-12-16, 17:17, Stephen Boyd wrote: > > > On 12/06, Viresh Kumar wrote: > > > > The code adding static OPPs for V2 bindings already does so. Make the V1 > > > > bindings specific code behave the same. > > > > > > > > Signed-off-by: Viresh Kumar > > > > --- > > > > > > Reviewed-by: Stephen Boyd > > > > > > It looks like that goes back to the original DT OPP binding > > > support written by Shawn Guo. Any idea why we kept trying to add > > > more OPPs if it failed to add one? Best effort or something? Can you please answer this question from Stephen? > > Not sure, maybe for handling duplicate OPPs. > > > > So perhaps we should Cc Shawn Guo who wrote the original code > then? I wouldn't mind doing that, but I don't think it needs to be a blocker for now. -- viresh