From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sudeep Holla Subject: Re: [BUG] dev_pm_opp refcount issue on Arm Juno r0 Date: Thu, 3 Jan 2019 10:41:27 +0000 Message-ID: <20190103104127.GB23511@e107155-lin> References: <20190103070514.o6yaqvpxouazosnn@vireshk-i7> <20190103103318.GA23511@e107155-lin> <20190103103829.2bqxhkpihp4r7x22@vireshk-i7> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20190103103829.2bqxhkpihp4r7x22@vireshk-i7> Sender: linux-kernel-owner@vger.kernel.org To: Viresh Kumar Cc: Valentin Schneider , linux-kernel , Linux PM , LAK , "Rafael J. Wysocki" , nm@ti.com, sboyd@kernel.org, Quentin Perret , Dietmar Eggemann , Sudeep Holla , Douglas Raillard List-Id: linux-pm@vger.kernel.org On Thu, Jan 03, 2019 at 04:08:29PM +0530, Viresh Kumar wrote: > On 03-01-19, 10:33, Sudeep Holla wrote: > > As you already state the checks are unnecessary, if we drop them we don't need > > to add any firmware specific callbacks. I am thinking if it makes sense to > > add a generic helper function to remove the OPPs from a device. If we have > > that any driver needing that can use it. The main reason I think helper is > > useful is that we need exactly same fix for SCMI driver too. > > > > Thoughts ? > > Okay, will do that and fix both the drivers as well. > Thanks, happy to test here. -- Regards, Sudeep