Re: [PATCH v4 03/13] mmc: fsl: Use brackets around if()

[Sean Anderson]

On 1/16/22 3:19 PM, Simon Glass wrote:

At present the IS_ENABLED() macro has extra brackets, making it possible
to write:

    if IS_ENABLED(CONFIG_XXX)

but it is a bit confusing. Add the missing brackets.

Signed-off-by: Simon Glass <s...@chromium.org>
---

(no changes since v1)

  drivers/mmc/fsl_esdhc_imx.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mmc/fsl_esdhc_imx.c b/drivers/mmc/fsl_esdhc_imx.c
index 9299635f509..08ea98fe81a 100644
--- a/drivers/mmc/fsl_esdhc_imx.c
+++ b/drivers/mmc/fsl_esdhc_imx.c
@@ -453,7 +453,7 @@ static int esdhc_send_cmd_common(struct fsl_esdhc_priv 
*priv, struct mmc *mmc,

/* Send the command */

        esdhc_write32(&regs->cmdarg, cmd->cmdarg);
-       if IS_ENABLED(CONFIG_FSL_USDHC) {
+       if (IS_ENABLED(CONFIG_FSL_USDHC)) {
                u32 mixctrl = esdhc_read32(&regs->mixctrl);

esdhc_write32(&regs->mixctrl,

Reviewed-by: Sean Anderson <sean...@gmail.com>