* [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure
@ 2019-11-12 21:01 Sven Van Asbroeck
2019-11-12 21:01 ` [PATCH v2 2/2] Input: ili210x - optionally show calibrate sysfs attribute Sven Van Asbroeck
2019-11-12 23:52 ` [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure Dmitry Torokhov
0 siblings, 2 replies; 4+ messages in thread
From: Sven Van Asbroeck @ 2019-11-12 21:01 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: Marek Vasut, Adam Ford, linux-kernel, linux-input
Optionally allow the touch screen resolution to be set by adding
it to the chip operations structure. If it is omitted (left zero),
the resolution defaults to 64K. Which is the previously hard-coded
value.
Set the ili2117 resolution to 2048, as indicated in its datasheet.
Link: https://lore.kernel.org/lkml/20191111181657.GA57214@dtor-ws/
Cc: Marek Vasut <marex@denx.de>
Cc: Adam Ford <aford173@gmail.com>
Cc: <linux-kernel@vger.kernel.org>
Cc: linux-input@vger.kernel.org
Tree: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/log/?h=next
Signed-off-by: Sven Van Asbroeck <TheSven73@gmail.com>
---
drivers/input/touchscreen/ili210x.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/input/touchscreen/ili210x.c b/drivers/input/touchscreen/ili210x.c
index a6feae5ce887..3b8e24815a1f 100644
--- a/drivers/input/touchscreen/ili210x.c
+++ b/drivers/input/touchscreen/ili210x.c
@@ -31,6 +31,7 @@ struct ili2xxx_chip {
unsigned int *x, unsigned int *y);
bool (*continue_polling)(const u8 *data, bool touch);
unsigned int max_touches;
+ unsigned int resolution;
};
struct ili210x {
@@ -160,6 +161,7 @@ static const struct ili2xxx_chip ili211x_chip = {
.parse_touch_data = ili211x_touchdata_to_coords,
.continue_polling = ili211x_decline_polling,
.max_touches = 10,
+ .resolution = 2048,
};
static int ili251x_read_reg(struct i2c_client *client,
@@ -336,6 +338,7 @@ static int ili210x_i2c_probe(struct i2c_client *client,
struct gpio_desc *reset_gpio;
struct input_dev *input;
int error;
+ unsigned int max_xy;
dev_dbg(dev, "Probing for ILI210X I2C Touschreen driver");
@@ -386,8 +389,9 @@ static int ili210x_i2c_probe(struct i2c_client *client,
input->id.bustype = BUS_I2C;
/* Multi touch */
- input_set_abs_params(input, ABS_MT_POSITION_X, 0, 0xffff, 0, 0);
- input_set_abs_params(input, ABS_MT_POSITION_Y, 0, 0xffff, 0, 0);
+ max_xy = (chip->resolution ?: SZ_64K) - 1;
+ input_set_abs_params(input, ABS_MT_POSITION_X, 0, max_xy, 0, 0);
+ input_set_abs_params(input, ABS_MT_POSITION_Y, 0, max_xy, 0, 0);
touchscreen_parse_properties(input, true, &priv->prop);
error = input_mt_init_slots(input, priv->chip->max_touches,
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH v2 2/2] Input: ili210x - optionally show calibrate sysfs attribute
2019-11-12 21:01 [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure Sven Van Asbroeck
@ 2019-11-12 21:01 ` Sven Van Asbroeck
2019-11-12 23:52 ` Dmitry Torokhov
2019-11-12 23:52 ` [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure Dmitry Torokhov
1 sibling, 1 reply; 4+ messages in thread
From: Sven Van Asbroeck @ 2019-11-12 21:01 UTC (permalink / raw)
To: Dmitry Torokhov; +Cc: Marek Vasut, Adam Ford, linux-kernel, linux-input
Only show the 'calibrate' sysfs attribute on chip flavours
which support calibration by writing to a calibration register.
Do this by adding a flag to the chip operations structure.
Link: https://lore.kernel.org/lkml/20191111181657.GA57214@dtor-ws/
Cc: Marek Vasut <marex@denx.de>
Cc: Adam Ford <aford173@gmail.com>
Cc: <linux-kernel@vger.kernel.org>
Cc: linux-input@vger.kernel.org
Tree: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/log/?h=next
Signed-off-by: Sven Van Asbroeck <TheSven73@gmail.com>
---
drivers/input/touchscreen/ili210x.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/drivers/input/touchscreen/ili210x.c b/drivers/input/touchscreen/ili210x.c
index 3b8e24815a1f..b0454cdacf38 100644
--- a/drivers/input/touchscreen/ili210x.c
+++ b/drivers/input/touchscreen/ili210x.c
@@ -32,6 +32,7 @@ struct ili2xxx_chip {
bool (*continue_polling)(const u8 *data, bool touch);
unsigned int max_touches;
unsigned int resolution;
+ bool has_calibrate_reg;
};
struct ili210x {
@@ -102,6 +103,7 @@ static const struct ili2xxx_chip ili210x_chip = {
.parse_touch_data = ili210x_touchdata_to_coords,
.continue_polling = ili210x_check_continue_polling,
.max_touches = 2,
+ .has_calibrate_reg = true,
};
static int ili211x_read_touch_data(struct i2c_client *client, u8 *data)
@@ -227,6 +229,7 @@ static const struct ili2xxx_chip ili251x_chip = {
.parse_touch_data = ili251x_touchdata_to_coords,
.continue_polling = ili251x_check_continue_polling,
.max_touches = 10,
+ .has_calibrate_reg = true,
};
static bool ili210x_report_events(struct ili210x *priv, u8 *touchdata)
@@ -310,8 +313,19 @@ static struct attribute *ili210x_attributes[] = {
NULL,
};
+static umode_t ili210x_calibrate_visible(struct kobject *kobj,
+ struct attribute *attr, int index)
+{
+ struct device *dev = kobj_to_dev(kobj);
+ struct i2c_client *client = to_i2c_client(dev);
+ struct ili210x *priv = i2c_get_clientdata(client);
+
+ return priv->chip->has_calibrate_reg;
+}
+
static const struct attribute_group ili210x_attr_group = {
.attrs = ili210x_attributes,
+ .is_visible = ili210x_calibrate_visible,
};
static void ili210x_power_down(void *data)
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH v2 2/2] Input: ili210x - optionally show calibrate sysfs attribute
2019-11-12 21:01 ` [PATCH v2 2/2] Input: ili210x - optionally show calibrate sysfs attribute Sven Van Asbroeck
@ 2019-11-12 23:52 ` Dmitry Torokhov
0 siblings, 0 replies; 4+ messages in thread
From: Dmitry Torokhov @ 2019-11-12 23:52 UTC (permalink / raw)
To: Sven Van Asbroeck; +Cc: Marek Vasut, Adam Ford, linux-kernel, linux-input
On Tue, Nov 12, 2019 at 04:01:48PM -0500, Sven Van Asbroeck wrote:
> Only show the 'calibrate' sysfs attribute on chip flavours
> which support calibration by writing to a calibration register.
>
> Do this by adding a flag to the chip operations structure.
>
> Link: https://lore.kernel.org/lkml/20191111181657.GA57214@dtor-ws/
> Cc: Marek Vasut <marex@denx.de>
> Cc: Adam Ford <aford173@gmail.com>
> Cc: <linux-kernel@vger.kernel.org>
> Cc: linux-input@vger.kernel.org
> Tree: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/log/?h=next
> Signed-off-by: Sven Van Asbroeck <TheSven73@gmail.com>
Applied, thank you.
> ---
> drivers/input/touchscreen/ili210x.c | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/drivers/input/touchscreen/ili210x.c b/drivers/input/touchscreen/ili210x.c
> index 3b8e24815a1f..b0454cdacf38 100644
> --- a/drivers/input/touchscreen/ili210x.c
> +++ b/drivers/input/touchscreen/ili210x.c
> @@ -32,6 +32,7 @@ struct ili2xxx_chip {
> bool (*continue_polling)(const u8 *data, bool touch);
> unsigned int max_touches;
> unsigned int resolution;
> + bool has_calibrate_reg;
> };
>
> struct ili210x {
> @@ -102,6 +103,7 @@ static const struct ili2xxx_chip ili210x_chip = {
> .parse_touch_data = ili210x_touchdata_to_coords,
> .continue_polling = ili210x_check_continue_polling,
> .max_touches = 2,
> + .has_calibrate_reg = true,
> };
>
> static int ili211x_read_touch_data(struct i2c_client *client, u8 *data)
> @@ -227,6 +229,7 @@ static const struct ili2xxx_chip ili251x_chip = {
> .parse_touch_data = ili251x_touchdata_to_coords,
> .continue_polling = ili251x_check_continue_polling,
> .max_touches = 10,
> + .has_calibrate_reg = true,
> };
>
> static bool ili210x_report_events(struct ili210x *priv, u8 *touchdata)
> @@ -310,8 +313,19 @@ static struct attribute *ili210x_attributes[] = {
> NULL,
> };
>
> +static umode_t ili210x_calibrate_visible(struct kobject *kobj,
> + struct attribute *attr, int index)
> +{
> + struct device *dev = kobj_to_dev(kobj);
> + struct i2c_client *client = to_i2c_client(dev);
> + struct ili210x *priv = i2c_get_clientdata(client);
> +
> + return priv->chip->has_calibrate_reg;
> +}
> +
> static const struct attribute_group ili210x_attr_group = {
> .attrs = ili210x_attributes,
> + .is_visible = ili210x_calibrate_visible,
> };
>
> static void ili210x_power_down(void *data)
> --
> 2.17.1
>
--
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure
2019-11-12 21:01 [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure Sven Van Asbroeck
2019-11-12 21:01 ` [PATCH v2 2/2] Input: ili210x - optionally show calibrate sysfs attribute Sven Van Asbroeck
@ 2019-11-12 23:52 ` Dmitry Torokhov
1 sibling, 0 replies; 4+ messages in thread
From: Dmitry Torokhov @ 2019-11-12 23:52 UTC (permalink / raw)
To: Sven Van Asbroeck; +Cc: Marek Vasut, Adam Ford, linux-kernel, linux-input
On Tue, Nov 12, 2019 at 04:01:47PM -0500, Sven Van Asbroeck wrote:
> Optionally allow the touch screen resolution to be set by adding
> it to the chip operations structure. If it is omitted (left zero),
> the resolution defaults to 64K. Which is the previously hard-coded
> value.
>
> Set the ili2117 resolution to 2048, as indicated in its datasheet.
>
> Link: https://lore.kernel.org/lkml/20191111181657.GA57214@dtor-ws/
> Cc: Marek Vasut <marex@denx.de>
> Cc: Adam Ford <aford173@gmail.com>
> Cc: <linux-kernel@vger.kernel.org>
> Cc: linux-input@vger.kernel.org
> Tree: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git/log/?h=next
> Signed-off-by: Sven Van Asbroeck <TheSven73@gmail.com>
> ---
> drivers/input/touchscreen/ili210x.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/input/touchscreen/ili210x.c b/drivers/input/touchscreen/ili210x.c
> index a6feae5ce887..3b8e24815a1f 100644
> --- a/drivers/input/touchscreen/ili210x.c
> +++ b/drivers/input/touchscreen/ili210x.c
> @@ -31,6 +31,7 @@ struct ili2xxx_chip {
> unsigned int *x, unsigned int *y);
> bool (*continue_polling)(const u8 *data, bool touch);
> unsigned int max_touches;
> + unsigned int resolution;
> };
>
> struct ili210x {
> @@ -160,6 +161,7 @@ static const struct ili2xxx_chip ili211x_chip = {
> .parse_touch_data = ili211x_touchdata_to_coords,
> .continue_polling = ili211x_decline_polling,
> .max_touches = 10,
> + .resolution = 2048,
> };
>
> static int ili251x_read_reg(struct i2c_client *client,
> @@ -336,6 +338,7 @@ static int ili210x_i2c_probe(struct i2c_client *client,
> struct gpio_desc *reset_gpio;
> struct input_dev *input;
> int error;
> + unsigned int max_xy;
>
> dev_dbg(dev, "Probing for ILI210X I2C Touschreen driver");
>
> @@ -386,8 +389,9 @@ static int ili210x_i2c_probe(struct i2c_client *client,
> input->id.bustype = BUS_I2C;
>
> /* Multi touch */
> - input_set_abs_params(input, ABS_MT_POSITION_X, 0, 0xffff, 0, 0);
> - input_set_abs_params(input, ABS_MT_POSITION_Y, 0, 0xffff, 0, 0);
> + max_xy = (chip->resolution ?: SZ_64K) - 1;
I had to add linux/sizes.h for this.
Applied, thank you.
--
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-12 23:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-12 21:01 [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure Sven Van Asbroeck
2019-11-12 21:01 ` [PATCH v2 2/2] Input: ili210x - optionally show calibrate sysfs attribute Sven Van Asbroeck
2019-11-12 23:52 ` Dmitry Torokhov
2019-11-12 23:52 ` [PATCH v2 1/2] Input: ili210x - add resolution to chip operations structure Dmitry Torokhov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).