From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 26060C4338F for ; Wed, 25 Aug 2021 14:36:38 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D81AD610CD for ; Wed, 25 Aug 2021 14:36:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org D81AD610CD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=foss.st.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:Date:Message-ID:CC:To:Subject:From: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=InvwM9Cu65aB8+ghGhX/6nj8KJI0NO+q21KOjN+csyE=; b=Y2GCb9JhCE7+sNngNSaGE3ux/G bUXFtPNKZ53eGImZXeJY5A+WnrIESsmnSaimFW/4lFxEgkFGOuhCWlKQJ4heY08WZWuzVpLOabVTC QR8YDr18TFQvTknGIa9Gi8a9tFxiG6OiB9ZGFeLxGApwClx/QAgIUzHdoUeIQ7M6oVL7KBgzt9K7p 9RXRmFc8Yz5+5IG57gUIUK0S5d+V4GzGSECom+de1hmXqNHptsHLl+zkNARvgecSaMaPMs/J0crA6 /KUAhh3xHr6cfVQ7MyVnTIq/XYoJfIoCPfN5EKw544VqNDe7I8453nnH7fJecVVIwGopidszoHwI+ Fk8VxTPQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mItz7-007HSD-Uw; Wed, 25 Aug 2021 14:34:34 +0000 Received: from mx08-00178001.pphosted.com ([91.207.212.93] helo=mx07-00178001.pphosted.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mItz0-007HQ9-Tk for linux-arm-kernel@lists.infradead.org; Wed, 25 Aug 2021 14:34:31 +0000 Received: from pps.filterd (m0046660.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.16.1.2/8.16.0.43) with SMTP id 17PDhOum003655; Wed, 25 Aug 2021 16:34:13 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foss.st.com; h=from : subject : to : cc : message-id : date : mime-version : content-type : content-transfer-encoding; s=selector1; bh=uB5VgDvAvQ72i1SM94ZVMnpWG1uPhE/nxCUf1gUKsDM=; b=J8/nsnlyrDt5jpFg+8ufKUpDe6dVqgJmzghbCCt+7/M/+I3kVGJCgbzgdP3TQZCjIOTe iWbxcWomfrrB10/SNyFZTN1CM6gGvvVUDK5QtgRueMmhZnJIedm+5AVnYPd+Nlv1kono MUbyH1DR3gCN6eP+mU/cbblmM8l5F+Oxg1IUTlE7N8wi+DTA6ewIA+82kIMzO4+KKv76 9dxXrNsvU+WNHO0uqnXDhqyRYxo6MeCE+1ObFqDjvqAleK0tWd86Y5U7APjKMOvX3Xhn C3f7JdvJgVPaALaIESNwEYpJClgEYYakEtGyDlmOAtlDSPv7nhgLs/3HfZnrhOe61voI Hg== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com with ESMTP id 3ana52vvwe-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 25 Aug 2021 16:34:13 +0200 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 00AD910002A; Wed, 25 Aug 2021 16:34:11 +0200 (CEST) Received: from Webmail-eu.st.com (sfhdag2node3.st.com [10.75.127.6]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id C47FC25F3F3; Wed, 25 Aug 2021 16:34:11 +0200 (CEST) Received: from lmecxl0504.lme.st.com (10.75.127.48) by SFHDAG2NODE3.st.com (10.75.127.6) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Wed, 25 Aug 2021 16:34:10 +0200 From: Yann Gautier Subject: Question about MMC_PM_KEEP_POWER in MMCI driver To: Ulf Hansson , "linux-mmc@vger.kernel.org" CC: Linus Walleij , Russell King , Rob Herring , "linux-arm-kernel@lists.infradead.org" , , "linux-stm32@st-md-mailman.stormreply.com" , Message-ID: Date: Wed, 25 Aug 2021 16:34:10 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 Content-Language: en-US X-Originating-IP: [10.75.127.48] X-ClientProxiedBy: SFHDAG1NODE2.st.com (10.75.127.2) To SFHDAG2NODE3.st.com (10.75.127.6) X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.182.1,Aquarius:18.0.790,Hydra:6.0.391,FMLib:17.0.607.475 definitions=2021-08-25_06,2021-08-25_02,2020-04-07_01 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210825_073427_380861_85A721C6 X-CRM114-Status: GOOD ( 18.29 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, In drivers/mmc/host/mmci.c, MMC_PM_KEEP_POWER is unconditionally enabled. This prevents correct low-power sequence on STM32MP157C-DK2 board which embeds a Wifi chip brcm,bcm4329-fmac (this wifi part has not yet been sent upstream). This MMC_PM_KEEP_POWER can be taken from DT with the property keep-power-in-suspend. This is what is done for other MMC drivers. I wonder what should be the best solution for this. 1) Remove MMC_PM_KEEP_POWER from the driver, and modify all SoC device tree files embedding a arm,pl18x with adding keep-power-in-suspend; property (except stm32mp151.dtsi file). This can be easy to do (~10 files to modify). But that could be more board dependent, if an SDIO chip is plugged on this MMC IP. And the name keep-power-in-suspend can be misleading as it only applies to SDIO. 2) Remove MMC_PM_KEEP_POWER from the driver, and modify board DT files with the property. This could be a difficult task to find all those boards. And this should be applied only for SDIO configs. 3) Just modify the driver to apply this capability for all MMCI chips but STM32. This could be done in the dedicated file, in sdmmc_variant_init() function. But some boards based on STM32MP15 chip might want to keep this capability. All advice is welcome. Thanks, Yann _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel