From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753343AbdKWSNC (ORCPT ); Thu, 23 Nov 2017 13:13:02 -0500 Received: from mail-it0-f65.google.com ([209.85.214.65]:41462 "EHLO mail-it0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753312AbdKWSM7 (ORCPT ); Thu, 23 Nov 2017 13:12:59 -0500 X-Google-Smtp-Source: AGs4zMZAph9SVu+lhWEV5S15IfzcwIyH85DOyzMPNvTHk9QRCaTPpXW/4+5LE8Zkote+OCPGts/HuU69xdwZ2B3XNTQ= MIME-Version: 1.0 In-Reply-To: <1511271770-3444-4-git-send-email-adrian.hunter@intel.com> References: <1511271770-3444-1-git-send-email-adrian.hunter@intel.com> <1511271770-3444-4-git-send-email-adrian.hunter@intel.com> From: Ulf Hansson Date: Thu, 23 Nov 2017 19:12:58 +0100 Message-ID: Subject: Re: [PATCH V14 03/24] mmc: core: Do not leave the block driver in a suspended state To: Adrian Hunter Cc: linux-mmc , linux-block , linux-kernel , Bough Chen , Alex Lemberg , Mateusz Nowak , Yuliy Izrailov , Jaehoon Chung , Dong Aisheng , Das Asutosh , Zhangfei Gao , Sahitya Tummala , Harjani Ritesh , Venu Byravarasu , Linus Walleij , Shawn Lin , Bartlomiej Zolnierkiewicz , Christoph Hellwig Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 21 November 2017 at 14:42, Adrian Hunter wrote: > The block driver must be resumed if the mmc bus fails to suspend the card. > > Signed-off-by: Adrian Hunter Thanks, applied for fixes and added a stable tag (I think v3.19+ is the first one we can pick, else some other manual back porting is needed). Kind regards Uffe > --- > drivers/mmc/core/bus.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/mmc/core/bus.c b/drivers/mmc/core/bus.c > index a4b49e25fe96..7586ff2ad1f1 100644 > --- a/drivers/mmc/core/bus.c > +++ b/drivers/mmc/core/bus.c > @@ -157,6 +157,9 @@ static int mmc_bus_suspend(struct device *dev) > return ret; > > ret = host->bus_ops->suspend(host); > + if (ret) > + pm_generic_resume(dev); > + > return ret; > } > > -- > 1.9.1 >