From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Greer Subject: Re: [PATCH 6/6] power: bq24190_charger: Use PM runtime autosuspend Date: Mon, 16 Jan 2017 12:22:01 -0700 Message-ID: <20170116192201.GJ27262@animalcreek.com> References: <20170112004154.31568-1-tony@atomide.com> <20170112004154.31568-7-tony@atomide.com> <20170112154626.GC2630@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:57224 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750863AbdAPTbD (ORCPT ); Mon, 16 Jan 2017 14:31:03 -0500 Content-Disposition: inline In-Reply-To: <20170112154626.GC2630@atomide.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Tony Lindgren Cc: Liam Breck , Sebastian Reichel , linux-pm@vger.kernel.org, linux-omap@vger.kernel.org, Matt Ranostay , Liam Breck On Thu, Jan 12, 2017 at 07:46:26AM -0800, Tony Lindgren wrote: > * Liam Breck [170111 18:03]: > > On Wed, Jan 11, 2017 at 4:41 PM, Tony Lindgren wrote: > > > @@ -1119,7 +1148,9 @@ static int bq24190_battery_set_property(struct power_supply *psy, > > > ret = -EINVAL; > > > } > > > > pm_runtime_get_sync() in this function needs its return value checked. > > OK will check. Tony, if you add the check, feel free to add my Acked-by: Acked-by: Mark Greer