Re: [PATCH v4 1/6] Input: synaptics - Split synaptics_resolution(), query first

From: Dmitry Torokhov
Date: Mon Mar 09 2015 - 02:41:47 EST


On Fri, Feb 06, 2015 at 10:44:54AM -0500, Benjamin Tissoires wrote:
> From: Daniel Martin <consume.noise@xxxxxxxxx>
>
> Split the function synaptics_resolution() into
> synaptics_resolution() and synaptics_quirks().
>
> synaptics_resolution() will be called before synaptics_quirks() to query
> dimensions and resolutions before overwriting them with quirks.
>
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Daniel Martin <consume.noise@xxxxxxxxx>
> ---
>
> v2: Removed SYN_ID_MAJOR() check from synaptics_quirks().
>
> drivers/input/mouse/synaptics.c | 37 +++++++++++++++++++++++++------------
> 1 file changed, 25 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/input/mouse/synaptics.c b/drivers/input/mouse/synaptics.c
> index 7e705ee..8c2b343 100644
> --- a/drivers/input/mouse/synaptics.c
> +++ b/drivers/input/mouse/synaptics.c
> @@ -346,7 +346,6 @@ static int synaptics_resolution(struct psmouse *psmouse)
> {
> struct synaptics_data *priv = psmouse->private;
> unsigned char resp[3];
> - int i;
>
> if (SYN_ID_MAJOR(priv->identity) < 4)
> return 0;
> @@ -358,17 +357,6 @@ static int synaptics_resolution(struct psmouse *psmouse)
> }
> }
>
> - for (i = 0; min_max_pnpid_table[i].pnp_ids; i++) {
> - if (psmouse_matches_pnp_id(psmouse,
> - min_max_pnpid_table[i].pnp_ids)) {
> - priv->x_min = min_max_pnpid_table[i].x_min;
> - priv->x_max = min_max_pnpid_table[i].x_max;
> - priv->y_min = min_max_pnpid_table[i].y_min;
> - priv->y_max = min_max_pnpid_table[i].y_max;
> - return 0;
> - }
> - }
> -
> if (SYN_EXT_CAP_REQUESTS(priv->capabilities) >= 5 &&
> SYN_CAP_MAX_DIMENSIONS(priv->ext_cap_0c)) {
> if (synaptics_send_cmd(psmouse, SYN_QUE_EXT_MAX_COORDS, resp)) {
> @@ -394,6 +382,29 @@ static int synaptics_resolution(struct psmouse *psmouse)
> return 0;
> }
>
> +/*
> + * Apply quirk(s) if the hardware matches
> + */
> +
> +static int synaptics_quirks(struct psmouse *psmouse)

This function does not seem to ever failing and does not access the
hardware so I switched it to be void.

Thanks.

> +{
> + struct synaptics_data *priv = psmouse->private;
> + int i;
> +
> + for (i = 0; min_max_pnpid_table[i].pnp_ids; i++) {
> + if (psmouse_matches_pnp_id(psmouse,
> + min_max_pnpid_table[i].pnp_ids)) {
> + priv->x_min = min_max_pnpid_table[i].x_min;
> + priv->x_max = min_max_pnpid_table[i].x_max;
> + priv->y_min = min_max_pnpid_table[i].y_min;
> + priv->y_max = min_max_pnpid_table[i].y_max;
> + break;
> + }
> + }
> +
> + return 0;
> +}
> +
> static int synaptics_query_hardware(struct psmouse *psmouse)
> {
> if (synaptics_identify(psmouse))
> @@ -408,6 +419,8 @@ static int synaptics_query_hardware(struct psmouse *psmouse)
> return -1;
> if (synaptics_resolution(psmouse))
> return -1;
> + if (synaptics_quirks(psmouse))
> + return -1;
>
> return 0;
> }
> --
> 2.1.0
>

--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/