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))
 void mmmc_trace_before_send(struct mmc *mmc, struct mmc_cmd *cmd)
 {
        printf("CMD_SEND:%d\n", cmd->cmdidx);
@@ -3190,7 +3189,7 @@ static int mmc_probe(struct bd_info *bis)
 
 int mmc_initialize(struct bd_info *bis)
 {
-       static int initialized = 0;
+       static int initialized;
        int ret;
        if (initialized)        /* Avoid initializing mmc multiple times */
                return 0;
-- 
2.43.0

Reply via email to