One a Synaptics S7817, this patch works correctly.
Tested-by: Chris Healy <cphealy@xxxxxxxxx>
On Fri, Mar 24, 2017 at 5:58 PM, Nick Dyer <nick@xxxxxxxxxxxxx> wrote:
> The data in F12_2D_Ctrl8 corresponds to the inactive border width used by the
> RMI device. It is not in pixel units and should not be treated as a clip value.
>
> Signed-off-by: Nick Dyer <nick@xxxxxxxxxxxxx>
> ---
> drivers/input/rmi4/rmi_f12.c | 18 +++++++-----------
> 1 file changed, 7 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/input/rmi4/rmi_f12.c b/drivers/input/rmi4/rmi_f12.c
> index 07aff43..8b0db08 100644
> --- a/drivers/input/rmi4/rmi_f12.c
> +++ b/drivers/input/rmi4/rmi_f12.c
> @@ -113,20 +113,16 @@ static int rmi_f12_read_sensor_tuning(struct f12_data *f12)
> }
>
> if (rmi_register_desc_has_subpacket(item, 2)) {
> - sensor->axis_align.clip_x_low = buf[offset];
> - sensor->axis_align.clip_x_high = sensor->max_x
> - - buf[offset + 1];
> - sensor->axis_align.clip_y_low = buf[offset + 2];
> - sensor->axis_align.clip_y_high = sensor->max_y
> - - buf[offset + 3];
> + /* Units 1/128 sensor pitch */
> + rmi_dbg(RMI_DEBUG_FN, &fn->dev,
> + "%s: Inactive Border xlo:%d xhi:%d ylo:%d yhi:%d\n",
> + __func__,
> + buf[offset], buf[offset + 1],
> + buf[offset + 2], buf[offset + 3]);
> +
> offset += 4;
> }
>
> - rmi_dbg(RMI_DEBUG_FN, &fn->dev, "%s: x low: %d x high: %d y low: %d y high: %d\n",
> - __func__,
> - sensor->axis_align.clip_x_low, sensor->axis_align.clip_x_high,
> - sensor->axis_align.clip_y_low, sensor->axis_align.clip_y_high);
> -
> if (rmi_register_desc_has_subpacket(item, 3)) {
> rx_receivers = buf[offset];
> tx_receivers = buf[offset + 1];
> --
> 2.7.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
