On Mon, Sep 01, 2025 at 07:14:10PM +0530, Bhimeswararao Matsa wrote: >Fix a couple of style issues reported by checkpatch.pl: > >- Replace `#ifdef CONFIG_MMC_TRACE` with `if (IS_ENABLED(CONFIG_MMC_TRACE))` > to follow the preferred kernel style for config-dependent branches. >- Drop explicit zero initialization of a static variable. > >No functional change intended. > >Signed-off-by: Bhimeswararao Matsa <[email protected]> >--- > drivers/mmc/mmc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > >diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c >index 5f2efbe6df9..54ee6d40a75 100644 >--- a/drivers/mmc/mmc.c >+++ b/drivers/mmc/mmc.c >@@ -104,8 +104,7 @@ __weak int board_mmc_getcd(struct mmc *mmc) > return -1; > } > #endif >- >-#ifdef CONFIG_MMC_TRACE >+#if (IS_ENABLED(CONFIG_MMC_TRACE))
No need the outmost (). I will fix when apply. Thanks, Peng
