[PATCH] crypto: sahara - Fix misuse of IS_ENABLED
From: Axel Lin <hidden>
Date: 2015-08-10 07:18:41
Subsystem:
crypto api, the rest · Maintainers:
Herbert Xu, "David S. Miller", Linus Torvalds
IS_ENABLED should only be used for CONFIG_* symbols. So use #ifdef DEBUG instead. Signed-off-by: Axel Lin <redacted> --- drivers/crypto/sahara.c | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/drivers/crypto/sahara.c b/drivers/crypto/sahara.c
index 397a500b..b2661a5 100644
--- a/drivers/crypto/sahara.c
+++ b/drivers/crypto/sahara.c@@ -384,11 +384,9 @@ static char *sahara_state[4] = { "Idle", "Busy", "Error", "HW Fault" }; static void sahara_decode_status(struct sahara_dev *dev, unsigned int status) { +#ifdef DEBUG u8 state; - if (!IS_ENABLED(DEBUG)) - return; - state = SAHARA_STATUS_GET_STATE(status); dev_dbg(dev->device, "%s: Status Register = 0x%08x\n",
@@ -432,15 +430,14 @@ static void sahara_decode_status(struct sahara_dev *dev, unsigned int status) sahara_read(dev, SAHARA_REG_CDAR)); dev_dbg(dev->device, "Initial DAR: 0x%08x\n\n", sahara_read(dev, SAHARA_REG_IDAR)); +#endif } static void sahara_dump_descriptors(struct sahara_dev *dev) { +#ifdef DEBUG int i; - if (!IS_ENABLED(DEBUG)) - return; - for (i = 0; i < SAHARA_MAX_HW_DESC; i++) { dev_dbg(dev->device, "Descriptor (%d) (0x%08x):\n", i, dev->hw_phys_desc[i]);
@@ -453,15 +450,14 @@ static void sahara_dump_descriptors(struct sahara_dev *dev) dev->hw_desc[i]->next); } dev_dbg(dev->device, "\n"); +#endif } static void sahara_dump_links(struct sahara_dev *dev) { +#ifdef DEBUG int i; - if (!IS_ENABLED(DEBUG)) - return; - for (i = 0; i < SAHARA_MAX_HW_LINK; i++) { dev_dbg(dev->device, "Link (%d) (0x%08x):\n", i, dev->hw_phys_link[i]);
@@ -471,6 +467,7 @@ static void sahara_dump_links(struct sahara_dev *dev) dev->hw_link[i]->next); } dev_dbg(dev->device, "\n"); +#endif } static int sahara_hw_descriptor_create(struct sahara_dev *dev)
--
2.1.0