Input: goodix - use "inverted_[xy]" flags instead of "rotated_screen"

The goodix touchscreen driver uses a "rotated_screen" flag for
systems on which the touchscreen is mounted rotated by 180
degrees with respect to the display.  With the addition of
support for the dt properties "touchscreen-inverted-x" and
"touchscreen-inverted-y", a separate "rotated_screen" flag
is not necessary anymore. This patch replaces it by setting
the inverted_x and inverted_y flags instead.

Signed-off-by: Karsten Merker <merker@debian.org>
Reviewed-by: Irina Tirdea <irina.tirdea@intel.com>
Tested-by: Bastien Nocera <hadess@hadess.net>
Acked-by: Bastien Nocera <hadess@hadess.net>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>

diff --git a/drivers/input/touchscreen/goodix.c b/drivers/input/touchscreen/goodix.c
index b5e910a44cdf0ae03ab5c5cab7ebc8fbb5fa4a23..240b16f3ee9797c4961709ef8f9a50b874febb3f 100644 (file)
--- a/drivers/input/touchscreen/goodix.c
+++ b/drivers/input/touchscreen/goodix.c
@@ -41,7 +41,6 @@ struct goodix_ts_data {
        bool inverted_y;
        unsigned int max_touch_num;
        unsigned int int_trigger_type;
-       bool rotated_screen;
        int cfg_len;
        struct gpio_desc *gpiod_int;
        struct gpio_desc *gpiod_rst;
@@ -234,11 +233,6 @@ static void goodix_ts_report_touch(struct goodix_ts_data *ts, u8 *coor_data)
        int input_y = get_unaligned_le16(&coor_data[3]);
        int input_w = get_unaligned_le16(&coor_data[5]);
 
-       if (ts->rotated_screen) {
-               input_x = ts->abs_x_max - input_x;
-               input_y = ts->abs_y_max - input_y;
-       }
-
        /* Inversions have to happen before axis swapping */
        if (ts->inverted_x)
                input_x = ts->abs_x_max - input_x;
@@ -521,10 +515,12 @@ static void goodix_read_config(struct goodix_ts_data *ts)
                ts->max_touch_num = GOODIX_MAX_CONTACTS;
        }
 
-       ts->rotated_screen = dmi_check_system(rotated_screen);
-       if (ts->rotated_screen)
+       if (dmi_check_system(rotated_screen)) {
+               ts->inverted_x = true;
+               ts->inverted_y = true;
                dev_dbg(&ts->client->dev,
                         "Applying '180 degrees rotated screen' quirk\n");
+       }
 }
 
 /**