From: Rickard Strandqvist <rickard.strandqvist@xxxxxxxxxxxxxx> Variable ar assigned a value that is never used. I have also removed all the code that thereby serves no purpose. This was found using a static code analysis program called cppcheck Signed-off-by: Rickard Strandqvist <rickard_strandqvist@xxxxxxxxxxxxxxxxxx> --- drivers/video/fbdev/msm/mddi.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/video/fbdev/msm/mddi.c b/drivers/video/fbdev/msm/mddi.c index e0f8011..d65ff75a 100644 --- a/drivers/video/fbdev/msm/mddi.c +++ b/drivers/video/fbdev/msm/mddi.c @@ -257,12 +257,12 @@ static irqreturn_t mddi_isr(int irq, void *data) struct msm_mddi_client_data *cdata = data; struct mddi_info *mddi = container_of(cdata, struct mddi_info, client_data); - uint32_t active, status; + uint32_t active; spin_lock(&mddi->int_lock); active = mddi_readl(INT); - status = mddi_readl(STAT); + mddi_readl(STAT); mddi_writel(active, INT); @@ -548,7 +548,6 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg) struct mddi_llentry *ll; struct mddi_register_access *ra; struct reg_read_info ri; - unsigned s; int retry_count = 2; unsigned long irq_flags; @@ -572,7 +571,7 @@ uint32_t mddi_remote_read(struct msm_mddi_client_data *cdata, uint32_t reg) ll->next = 0; ll->reserved = 0; - s = mddi_readl(STAT); + mddi_readl(STAT); ri.reg = reg; ri.status = -1; -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html