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=-2.2 required=3.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A,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 3D90FC47094 for ; Thu, 10 Jun 2021 16:02:06 +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 0E8B960FEA for ; Thu, 10 Jun 2021 16:02:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 0E8B960FEA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:Cc:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=/lNZrP6kLvXwXZfKqWjPCBaaw72jzRoMJ0AjJ/rXGC4=; b=Tvez9CrfTq3+g0w0kkX0UNNRln DgP747hk1PZZHko7uvBIQkuPDJdwvJvwpHzZa9eT/ddiD5W51JgfL48G0BrK+h/xlLLK6kkqkLU+W ecpZJnoGaUi6I0rxNBydr3X8ehYBJPaP/aF2VlX9BTfTWCG/HDcGEBxVzI6j8/lkm/JK9TUyhi0iF S3x5W3eY59kb/e975KRWegzbFHHhNHXbntsm/XS8cb6tz5Megvd2qE+29xWu32uDYgRjOuoDGHCtn zLkmSPMMVDQmrJODT3JeejLdQQpnc3keuqiWkq7eVcnv1MktZ7I59ltjQOFrxyGCrm3Fn/KUzUNFH gGKJsA5w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lrN5w-001XFy-3Q; Thu, 10 Jun 2021 15:59:48 +0000 Received: from mail-pg1-x52c.google.com ([2607:f8b0:4864:20::52c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lrN5S-001X2q-Tl for linux-arm-kernel@lists.infradead.org; Thu, 10 Jun 2021 15:59:20 +0000 Received: by mail-pg1-x52c.google.com with SMTP id i34so114820pgl.9 for ; Thu, 10 Jun 2021 08:59:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=gEOhLn04ufSH/EYWeHJiDResBxu7Zjy3LEUOA404M9M=; b=MvNErbbuJgsYARWhegkk6WNOEMgQbvVcbR6iy+mrAf8t7qZEQVAxp85okAny4keJwX ktOiDfhx4YSPeOjaSjn/FAnDn1o1k0B1UQP3Pmrgpo6e9oiLdScB6ih72n/LORqPLHTx 2VUoOZ01+qKFp/xpD7TNsWYLe9AQEWeB7x1mZurgDS0W90mzURpQcKdKnd5lc7XTjStO Y1TRsa3iOSUUbArQiTztEZzJ2UUUuih/Q2gM5x0OVoL9iH8CKLa2xlxm8qr0AgsOuW9B K0DFR3SSjqIVPPZ7TDp8Ok8JzMstH0xwBLoUeElcE+B7PNX1GB4APwxDOjjLyKrdtljF 44xA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=gEOhLn04ufSH/EYWeHJiDResBxu7Zjy3LEUOA404M9M=; b=Qsrk/4KbjEHX00xiyV+8E7v2po6gWXsuElP+ErJs744JRpJ0Hpt/FVuJQWjV4jDDa0 gSR0FyXJd59cu+dT5CEHv7hlrGKV3VZ8C8Du1c6o4y6y5+RJYVRgJuXldZ6NyzSQdKF7 kO8bNUHGcUTm/P5gvw7zQDekDTDLww6vu1n17PgSs5HhiNjam/Vf0BrITVrcMNUtIPfo hpkLNCeNcZkOXV8qWGli0CvRMtS4WNOIgAPU4cMzb7jcb9SdUI1CD3ZZAfipx+TBiaU6 dTWQaiuGLiMoSrCfXSppYo/96yDgNTJHIjrlWH8Dw1hIc5ey5dT2vp13qGlPZomV4m9R y9AA== X-Gm-Message-State: AOAM533sZ0OylJbrvQzv5NeM9yEbk1WHFyArlWY1d4BI6C11p5W8tbw1 v1s5OJe81i8lLUCZjjfKLlM= X-Google-Smtp-Source: ABdhPJx9vmsDtDTL4nbg98zKMRj7QHMyQ/zFV0vq4rlqsoMu8CSPycYkvg5HeWMzsT0A8Xewb8Gt3A== X-Received: by 2002:a63:ff1f:: with SMTP id k31mr5482804pgi.315.1623340757631; Thu, 10 Jun 2021 08:59:17 -0700 (PDT) Received: from [10.230.29.202] ([192.19.223.252]) by smtp.gmail.com with ESMTPSA id p20sm2708534pff.204.2021.06.10.08.59.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 10 Jun 2021 08:59:17 -0700 (PDT) Subject: Re: [PATCH 2/2] mmc: sdhci-iproc: Add support for the legacy sdhci controller on the BCM7211 To: Ulf Hansson , "Rafael J. Wysocki" , Linux PM Cc: Al Cooper , Linux Kernel Mailing List , Adrian Hunter , BCM Kernel Feedback , DTML , Linux ARM , linux-mmc , Nicolas Saenz Julienne , Ray Jui , Rob Herring , Scott Branden References: <20210602192758.38735-1-alcooperx@gmail.com> <20210602192758.38735-2-alcooperx@gmail.com> <6acd480a-8928-89bb-0f40-d278294973a1@gmail.com> From: Florian Fainelli Message-ID: Date: Thu, 10 Jun 2021 08:59:07 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Firefox/78.0 Thunderbird/78.10.2 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210610_085919_024725_49E000CA X-CRM114-Status: GOOD ( 34.61 ) 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-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 6/10/2021 1:49 AM, Ulf Hansson wrote: > On Thu, 10 Jun 2021 at 01:59, Florian Fainelli wrote: >> >> >> >> On 6/9/2021 2:22 AM, Ulf Hansson wrote: >>> On Wed, 9 Jun 2021 at 05:07, Florian Fainelli wrote: >>>> >>>> >>>> >>>> On 6/8/2021 5:40 AM, Ulf Hansson wrote: >>>>> On Wed, 2 Jun 2021 at 21:28, Al Cooper wrote: >>>>>> >>>>>> Add support for the legacy Arasan sdhci controller on the BCM7211 and >>>>>> related SoC's. This includes adding a .shutdown callback to increase >>>>>> the power savings during S5. >>>>> >>>>> Please split this into two separate changes. >>>>> >>>>> May I also ask about the ->shutdown() callback and in relation to S5. >>>>> What makes the ->shutdown callback only being invoked for S5? >>>> >>>> It is not only called for S5 (entered via poweroff on a prompt) but also >>>> during kexec or reboot. The poweroff path is via: >>>> >>>> kernel_power_off() -> kernel_shutdown_prepare() -> device_shutdown() -> >>>> .shutdown() >>>> >>>> For kexec or reboot we do not really care about power savings since we >>>> are about to load a new image anyway, however for S5/poweroff we do care >>>> about quiescing the eMMC controller in a way that its clocks and the >>>> eMMC device can be put into low power mode since we will stay in that >>>> mode for seconds/hours/days until someone presses a button on their >>>> remote (or other wake-up sources). >>> >>> Hmm, I am not sure I understand correctly. At shutdown we don't care >>> about wake-up sources from the kernel point of view, instead we treat >>> everything as if it will be powered off. >> >> The same .shutdown() path is used whether you kexec, reboot or poweroff, >> but for poweroff we do care about allowing specific wake-up sources >> configured as such to wake-up the system at a later time, like GPIOs, >> RTC, etc. > > That's true, but using the ->shutdown() callbacks in this way would > certainly be a new use case. > > Most subsystems/drivers don't care about power management in those > callbacks, but rather just about managing a graceful shutdown. > > It sounds to me like you should have a look at the hibernation > path/callbacks instead - or perhaps even the system suspend > path/callback. Normally, that's where we care about power management. The platforms we use do not support hibernation, keep in mind that these are embedded SoCs that support the S2 (standby), S3 (mem) and poweroff suspend states, hibernation is not something that we can support. > > I have looped in Rafael, to allow him to share his opinion on this. > >> >>> >>> We put devices into low power state at system suspend and potentially >>> also during some of the hibernation phases. >>> >>> Graceful shutdown of the eMMC is also managed by the mmc core. >> >> AFAICT that calls mmc_blk_shutdown() but that is pretty much it, the >> SDHCI platform_driver still needs to do something in order to conserve >> power including disabling host->clk, otherwise we would not have done >> that for sdhci-brcmstb.c. > > That's not entirely correct. When mmc_bus_shutdown() is called for the > struct device* that belongs to an eMMC card, two actions are taken. > > *) We call mmc_blk_shutdown(), to suspend the block device queue from > receiving new I/O requests. > **) We call host->bus_ops->shutdown(), which is an eMMC specific > callback set to mmc_shutdown(). In this step, we do a graceful > shutdown/power-off of the eMMC card. > > When it comes to controller specific resources, like clocks and PM > domains, for example, those may very well stay turned on. Do deal with > these, then yes, you would need to implement the ->shutdown() > callback. But as I said above, I am not sure it's the right thing to > do. As explained before, we can enter S5 for an indefinite amount of time until a wake-up source wakes us up so we must conserve power, even if we happen to wake up the next second, we don't know that ahead of time. The point of calling sdhci_pltfm_suspend() here is to ensure that host->clk is turned off which cuts the eMMC controller digital clock, I forgot how much power we save by doing so, but every 10s of mW counts for us. -- Florian _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel