From b03a7413a463ec4f332a71227a7e48983bb963a9 Mon Sep 17 00:00:00 2001 From: Narendra Muppalla Date: Mon, 3 Feb 2020 18:19:46 -0800 Subject: [PATCH] disp: msm: sde: remove an uninitialized index variable Avoid exposing any uninitialized data when setting up the interrupt tables. Change-Id: I00d3dc8b5acc843401148585a129fde58e326beb Signed-off-by: Narendra Muppalla Signed-off-by: Steve Cohen --- msm/sde/sde_hw_interrupts.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/msm/sde/sde_hw_interrupts.c b/msm/sde/sde_hw_interrupts.c index a2e870cafc91..81fddec0dce7 100644 --- a/msm/sde/sde_hw_interrupts.c +++ b/msm/sde/sde_hw_interrupts.c @@ -1,6 +1,6 @@ // SPDX-License-Identifier: GPL-2.0-only /* - * Copyright (c) 2016-2019, The Linux Foundation. All rights reserved. + * Copyright (c) 2016-2020, The Linux Foundation. All rights reserved. */ #include @@ -1119,7 +1119,7 @@ static int _sde_hw_intr_init_irq_tables(struct sde_hw_intr *intr, struct sde_mdss_cfg *m) { struct sde_intr_irq_offsets *item; - int i, idx, sde_irq_tbl_idx = 0, ret = 0; + int i, sde_irq_tbl_idx = 0, ret = 0; u32 low_idx, high_idx; u32 sde_irq_map_idx = 0; @@ -1128,9 +1128,6 @@ static int _sde_hw_intr_init_irq_tables(struct sde_hw_intr *intr, low_idx = sde_irq_map_idx; high_idx = low_idx + _get_irq_map_size(item); - pr_debug("init[%d]=%d low:%d high:%d\n", - sde_irq_tbl_idx, idx, low_idx, high_idx); - if (sde_irq_tbl_idx >= intr->sde_irq_size || sde_irq_tbl_idx < 0) { ret = -EINVAL;