Re: [PATCH 6/8] mmc: mxc: Make dev_pm_ops struct static

From: Jonathan Cameron
Date: Wed Jan 05 2022 - 05:12:12 EST


On Tue, 4 Jan 2022 21:42:12 +0000
Paul Cercueil <paul@xxxxxxxxxxxxxxx> wrote:

> The new DEFINE_SIMPLE_DEV_PM_OPS() macro does not set the "static"
> qualifier anymore, so we can add it here, as the underlying dev_pm_ops
> struct is only used in this file.
>
> Signed-off-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
FWIW on this trivial patch

Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>

> ---
> drivers/mmc/host/mxcmmc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mmc/host/mxcmmc.c b/drivers/mmc/host/mxcmmc.c
> index 98c218bd6669..40b6878bea6c 100644
> --- a/drivers/mmc/host/mxcmmc.c
> +++ b/drivers/mmc/host/mxcmmc.c
> @@ -1210,7 +1210,7 @@ static int mxcmci_resume(struct device *dev)
> return ret;
> }
>
> -DEFINE_SIMPLE_DEV_PM_OPS(mxcmci_pm_ops, mxcmci_suspend, mxcmci_resume);
> +static DEFINE_SIMPLE_DEV_PM_OPS(mxcmci_pm_ops, mxcmci_suspend, mxcmci_resume);
>
> static struct platform_driver mxcmci_driver = {
> .probe = mxcmci_probe,