From patchwork Tue Oct 5 13:44:43 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adrian Hunter X-Patchwork-Id: 12536857 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5472AC433F5 for ; Tue, 5 Oct 2021 14:07:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3F6E5611F2 for ; Tue, 5 Oct 2021 14:07:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235185AbhJEOJY (ORCPT ); Tue, 5 Oct 2021 10:09:24 -0400 Received: from mga18.intel.com ([134.134.136.126]:47631 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236603AbhJEOI2 (ORCPT ); Tue, 5 Oct 2021 10:08:28 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10127"; a="212679094" X-IronPort-AV: E=Sophos;i="5.85,349,1624345200"; d="scan'208";a="212679094" Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Oct 2021 06:45:01 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.85,348,1624345200"; d="scan'208";a="477679475" Received: from ahunter-desktop.fi.intel.com ([10.237.72.76]) by orsmga007.jf.intel.com with ESMTP; 05 Oct 2021 06:44:56 -0700 From: Adrian Hunter To: "Martin K . Petersen" Cc: "James E . J . Bottomley" , Bean Huo , Avri Altman , Alim Akhtar , Can Guo , Asutosh Das , Bart Van Assche , Kiwoong Kim , Manivannan Sadhasivam , Wei Li , linux-scsi@vger.kernel.org Subject: [PATCH V7 0/2] scsi: ufs: Let devices remain runtime suspended during system suspend Date: Tue, 5 Oct 2021 16:44:43 +0300 Message-Id: <20211005134445.234671-1-adrian.hunter@intel.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki, Business Identity Code: 0357606 - 4, Domiciled in Helsinki Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org Hi UFS devices can remain runtime suspended at system suspend time, if the conditions are right. Add support for that, first fixing the impediments. Changes in V7: scsi: ufs: Fix error handler clear ua deadlock Dropped because superseded by "scsi: ufs: core: Stop clearing UNIT ATTENTIONS" scsi: ufs: Let devices remain runtime suspended during system suspend Re-based Changes in V6: scsi: ufs: Fix error handler clear ua deadlock Ensure data byte count bits 1:0 are 11b Use ufshcd_compose_dev_cmd() to set up command Changes in V5: scsi: ufs: Fix error handler clear ua deadlock Update commit message Try to abort REQUEST SENSE if it times out Changes in V4: scsi: ufs: Fix error handler clear ua deadlock Do request-sense directly Changes in V3: scsi: ufs: Fix error handler clear ua deadlock Correct commit message. Amend stable tags to add dependent cherry picks Changes in V2: scsi: ufs: Let devices remain runtime suspended during system suspend The ufs-hisi driver uses different RPM and SPM, but it is made explicit by a new parameter to suspend prepare. Adrian Hunter (2): scsi: ufs: Fix runtime PM dependencies getting broken scsi: ufs: Let devices remain runtime suspended during system suspend drivers/scsi/scsi_pm.c | 16 +++++++++++----- drivers/scsi/ufs/ufs-hisi.c | 8 +++++++- drivers/scsi/ufs/ufshcd.c | 46 ++++++++++++++++++++++++++++++++++++++++----- drivers/scsi/ufs/ufshcd.h | 11 +++++++++++ include/scsi/scsi_device.h | 1 + 5 files changed, 71 insertions(+), 11 deletions(-) Regards Adrian