Comment 28 for bug 365952

Revision history for this message
In , Henrik Rydberg (rydberg) wrote :

(In reply to comment #23)
> Created an attachment (id=27726) [details]
> Third draft of the patch
>
> The attached patch contains the latest changes described in the previous
> comment.
>
> Let me know what you think.
>

The code is clear and constitutes a good patch. I did not read all the details of the discussion, so I might have missed some important point, but the area parameters constitutes the physical area of the device and could be filled with default values based on the reported driver dimensions?