Re: [PATCH v2 12/15] nfc: trf7970a: drop unneeded debug prints

From: Mark Greer
Date: Mon Sep 13 2021 - 12:58:16 EST


On Mon, Sep 13, 2021 at 03:20:32PM +0200, Krzysztof Kozlowski wrote:
> ftrace is a preferred and standard way to debug entering and exiting
> functions so drop useless debug prints.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>
> ---
> drivers/nfc/trf7970a.c | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/drivers/nfc/trf7970a.c b/drivers/nfc/trf7970a.c
> index 8890fcd59c39..29ca9c328df2 100644
> --- a/drivers/nfc/trf7970a.c
> +++ b/drivers/nfc/trf7970a.c
> @@ -2170,8 +2170,6 @@ static int trf7970a_suspend(struct device *dev)
> struct spi_device *spi = to_spi_device(dev);
> struct trf7970a *trf = spi_get_drvdata(spi);
>
> - dev_dbg(dev, "Suspend\n");
> -
> mutex_lock(&trf->lock);
>
> trf7970a_shutdown(trf);
> @@ -2187,8 +2185,6 @@ static int trf7970a_resume(struct device *dev)
> struct trf7970a *trf = spi_get_drvdata(spi);
> int ret;
>
> - dev_dbg(dev, "Resume\n");
> -
> mutex_lock(&trf->lock);
>
> ret = trf7970a_startup(trf);
> @@ -2206,8 +2202,6 @@ static int trf7970a_pm_runtime_suspend(struct device *dev)
> struct trf7970a *trf = spi_get_drvdata(spi);
> int ret;
>
> - dev_dbg(dev, "Runtime suspend\n");
> -
> mutex_lock(&trf->lock);
>
> ret = trf7970a_power_down(trf);
> @@ -2223,8 +2217,6 @@ static int trf7970a_pm_runtime_resume(struct device *dev)
> struct trf7970a *trf = spi_get_drvdata(spi);
> int ret;
>
> - dev_dbg(dev, "Runtime resume\n");
> -
> ret = trf7970a_power_up(trf);
> if (!ret)
> pm_runtime_mark_last_busy(dev);
> --
> 2.30.2

Acked-by: Mark Greer <mgreer@xxxxxxxxxxxxxxx>