Input: of_touchscreen - always issue warning if axis is not set up
authorDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 6 Jul 2015 21:45:25 +0000 (14:45 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Sun, 12 Jul 2015 00:26:13 +0000 (17:26 -0700)
Do issue warning about axis that is present in device tree but not specified
by the driver even in case of multi-touch axis as callers now tell us if they
expect multi-touch data or not.

Reviewed-by: Roger Quadros <rogerq@ti.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/of_touchscreen.c

index 806cd0ad160f95c9b08e0bc4f9e9a33e2655f8bc..759cf4b8ba096440b2259fedeebba8c454b2aa96 100644 (file)
@@ -39,13 +39,9 @@ static void touchscreen_set_params(struct input_dev *dev,
        struct input_absinfo *absinfo;
 
        if (!test_bit(axis, dev->absbit)) {
-               /*
-                * Emit a warning only if the axis is not a multitouch
-                * axis, which might not be set by the driver.
-                */
-               if (!input_is_mt_axis(axis))
-                       dev_warn(&dev->dev,
-                                "DT specifies parameters but the axis is not set up\n");
+               dev_warn(&dev->dev,
+                        "DT specifies parameters but the axis %lu is not set up\n",
+                        axis);
                return;
        }