On Fri, Mar 08, 2013 at 05:15:06PM +0100, Markus Pargmann wrote:

> +             if (
> +                             abs_x[0] > (data.x & 0xfff)
> +                             || abs_x[1] < (data.x & 0xfff)
> +                             || abs_y[0] > (data.y & 0xfff)
> +                             || abs_y[1] < (data.y & 0xfff)) {
> +                     dev_dbg(wm->dev, "Measurement out of range, dropping 
> it\n");
> +                     rc = RC_AGAIN;
> +                     goto out;

The change is good but not a fan of the coding style here.  Otherwise

Acked-by: Mark Brown <[email protected]>

Attachment: signature.asc
Description: Digital signature

Reply via email to