From patchwork Tue Jul 26 06:28:39 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Axe Yang X-Patchwork-Id: 594439 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 66803C43334 for ; Tue, 26 Jul 2022 06:28:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232067AbiGZG25 (ORCPT ); Tue, 26 Jul 2022 02:28:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46724 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230027AbiGZG2z (ORCPT ); Tue, 26 Jul 2022 02:28:55 -0400 Received: from mailgw02.mediatek.com (unknown [210.61.82.184]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D0E4ABA; Mon, 25 Jul 2022 23:28:52 -0700 (PDT) X-UUID: 5248d6bea70f45e481e40891ab2af8fc-20220726 X-CID-P-RULE: Release_Ham X-CID-O-INFO: VERSION:1.1.8, REQID:cd3a1308-9178-42cf-9286-fa21fa4a1164, OB:0, LO B:0,IP:0,URL:0,TC:0,Content:0,EDM:0,RT:0,SF:0,FILE:0,RULE:Release_Ham,ACTI ON:release,TS:0 X-CID-META: VersionHash:0f94e32, CLOUDID:1d12f4d3-912a-458b-a623-74f605a77e93, C OID:IGNORED,Recheck:0,SF:nil,TC:nil,Content:0,EDM:-3,IP:nil,URL:0,File:nil ,QS:nil,BEC:nil,COL:0 X-UUID: 5248d6bea70f45e481e40891ab2af8fc-20220726 Received: from mtkexhb02.mediatek.inc [(172.21.101.103)] by mailgw02.mediatek.com (envelope-from ) (Generic MTA with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 792789619; Tue, 26 Jul 2022 14:28:47 +0800 Received: from mtkcas10.mediatek.inc (172.21.101.39) by mtkmbs10n1.mediatek.inc (172.21.101.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.2.792.15; Tue, 26 Jul 2022 14:28:45 +0800 Received: from localhost.localdomain (10.17.3.154) by mtkcas10.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Tue, 26 Jul 2022 14:28:44 +0800 From: Axe Yang To: Ulf Hansson , Rob Herring , Chaotian Jing , Matthias Brugger , Adrian Hunter CC: Yoshihiro Shimoda , Satya Tangirala , Andy Shevchenko , Wolfram Sang , Axe Yang , Lucas Stach , Eric Biggers , Andrew Jeffery , Stephen Boyd , Kiwoong Kim , Yue Hu , Tian Tao , , , , , , , Subject: [PATCH v14 0/3] mmc: mediatek: add support for SDIO async IRQ Date: Tue, 26 Jul 2022 14:28:39 +0800 Message-ID: <20220726062842.18846-1-axe.yang@mediatek.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-MTK: N Precedence: bulk List-ID: X-Mailing-List: linux-mmc@vger.kernel.org Changes in v14: - add sdio_irq_claimed() judgement condition around msdc irq and pinctrl control in suspend|resume callback Changes in v13: - remove state_dat1 related description in mtk-sd.yaml - move device_init_wakeup() to probe stage - remove redundancy SDC_CFG_SDIOIDE bit control in msdc_runtime_suspend() - replace SDC_CFG_SDIOIDE control with __msdc_enable_sdio_irq() function to disable sdio irq when sdio_irq_claimed() return true in msdc_runtime_resume() - restore to use pm_runtime_force_resume|suspend(), to avoid go out directly in force resume, bump up runtime PM usage counter before force suspend. Changes in v12: - assign NULL to pins_eint directly instead of using kfree() Changes in v11: - remove '_irq' suffix in interrupts-names property - fix yaml example build error - refactor msdc_enable_sdio_irq(), free pins_eint if async irq is not supported Changes in v10: - add sample node for SDIO host which support wakeup interrupt in yaml - skip MMC_PM_WAKE_SDIO_IRQ check before enable SDIO async interrupt - add MMC_PM_KEEP_POWER check before SDIO eint pinstate parsing - use dev_pm_set_dedicated_wake_irq_reverse() to correct irq control sequence - set dedicated irq in msdc_enable_sdio_irq() rather than msdc_drv_probe() - remove unnecessary wake irq control, rpm/dpm system shall manage that - move wake irq/msdc irq control back to system suspend phase, use rpm_suspend and rpm_resume to ensure irq control sequence: disable msdc irq -> enable wake irq -> disable wake irq -> enable msdc irq - simplify variables, check pins_eint to know whether wakeup settings are managed Changes in v9: - remove pinctrl "state_dat1" Changes in v8: - remove maxItems property under pinctrl-names property Changes in v7: - add device_init_wakeup() to register SDIO host as wakeup source Changes in v6: - abandon cap-sdio-async-irq flag, use wakeup-source flag instead - extend interrupts and pinctrls in mediatek mmc host controller DT documents - add mmc_card_enable_async_irq() to access enable_async_irq flag - simplify wakeup irq implementation with dedicate wake up irq related interface Changes in v5: - resort variables to reversed xmas tree order - restore old copyright year range and add current year back Changes in v4: - add MMC_CAP2_SDIO_ASYNC_IRQ judge before lookup eint pinctrl - replace spin_lock_irqsave() variant with spin_lock() in eint irq handler Changes in v3: - correct abbreviations with capital letters in commit message - replace copyright year with 2022 in mtk-sd.c - remove unnessary pointer casting - adjust variable order to reversed xmas tree - remove a redundant blank line - refine if statement, following standard pattern Changes in v2: - change flag name from 'cap-sdio-async-int' to 'cap-sdio-async-irq' - change corresponding macro names from xxx_INT to xxx_IRQ - resort new member in msdc_host structure - refine function msdc_request_dat1_eint_irq() - rename msdc_{suspend,resume} function names, add suffix '_noirq' - add MMC_CAP2_NO_SDIO judgement before parse eint related pin setting Axe Yang (3): dt-bindings: mmc: mtk-sd: extend interrupts and pinctrls properties mmc: core: Add support for SDIO wakeup interrupt mmc: mediatek: add support for SDIO eint wakup IRQ .../devicetree/bindings/mmc/mtk-sd.yaml | 50 ++++++++++- drivers/mmc/core/sdio.c | 14 +++ drivers/mmc/host/mtk-sd.c | 86 +++++++++++++++++-- include/linux/mmc/card.h | 8 +- include/linux/mmc/sdio.h | 5 ++ 5 files changed, 155 insertions(+), 8 deletions(-)