linux-kernel - Re: [PATCH] mmc: core: expose MMC_CAP_AGGRESSIVE

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAPDyKFqUnp_rDcckBSW386webSgjAhfdONN_AKYCS-kY3XwKPg@mail.gmail.com>
Date:   Tue, 24 Jan 2023 12:06:53 +0100
From:   Ulf Hansson <ulf.hansson@...aro.org>
To:     Hermes Zhang <chenhuiz@...s.com>
Cc:     kernel@...s.com, linux-mmc@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mmc: core: expose MMC_CAP_AGGRESSIVE_PM to DT

On Wed, 18 Jan 2023 at 05:57, Hermes Zhang <chenhuiz@...s.com> wrote:
>
> This patch expose the MMC_CAP_AGGRESSIVE_PM flag to DT which
> allows the host to enable it from DT.
>
> Signed-off-by: Hermes Zhang <chenhuiz@...s.com>
> ---
>  drivers/mmc/core/host.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
> index d17eda753b7e..1d98a301515d 100644
> --- a/drivers/mmc/core/host.c
> +++ b/drivers/mmc/core/host.c
> @@ -376,6 +376,8 @@ int mmc_of_parse(struct mmc_host *host)
>                 host->caps |= MMC_CAP_HW_RESET;
>         if (device_property_read_bool(dev, "cap-sdio-irq"))
>                 host->caps |= MMC_CAP_SDIO_IRQ;
> +       if (device_property_read_bool(dev, "cap-aggressive-pm"))
> +               host->caps |= MMC_CAP_AGGRESSIVE_PM;

I think this can be debated whether this is a description of some
characteristics of the HW.

That said, please add the DT list and maintainers and resend, to let
them share their view too.

>         if (device_property_read_bool(dev, "full-pwr-cycle"))
>                 host->caps2 |= MMC_CAP2_FULL_PWR_CYCLE;
>         if (device_property_read_bool(dev, "full-pwr-cycle-in-suspend"))
> --

Kind regards
Uffe