Thread (4 messages) 4 messages, 4 authors, 2014-03-19

RE: [patch] Input: remove a duplicative NULL test

From: fugang.duan@freescale.com <hidden>
Date: 2014-02-21 09:14:38
Also in: kernel-janitors

From: Dan Carpenter <redacted>
Data: Friday, February 21, 2014 4:55 PM
quoted hunk ↗ jump to hunk
To: Dmitry Torokhov
Cc: Paul Gortmaker; Jingoo Han; Duan Fugang-B38611; Benson Leung; Daniel Kurtz;
linux-input@vger.kernel.org; kernel-janitors@vger.kernel.org
Subject: [patch] Input: remove a duplicative NULL test

"pdata" is non-NULL here.  We verified that at the start of the function.

Signed-off-by: Dan Carpenter <redacted>
diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c
b/drivers/input/touchscreen/atmel_mxt_ts.c
index a70400754e92..40abe90cc924 100644
--- a/drivers/input/touchscreen/atmel_mxt_ts.c
+++ b/drivers/input/touchscreen/atmel_mxt_ts.c
@@ -1146,7 +1146,7 @@ static int mxt_probe(struct i2c_client *client,
		goto err_free_mem;
	}

-	data->is_tp = pdata && pdata->is_tp;
+	data->is_tp = pdata->is_tp;

	input_dev->name = (data->is_tp) ? "Atmel maXTouch Touchpad" :
					  "Atmel maXTouch Touchscreen";
Agree, it is redundant. And if you have free time, you can convert the driver to support devicetree.

Acked-by: Fugang Duan <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help