lkml.org 
[lkml]   [2011]   [Mar]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] input/touchscreen: Synaptics RMI4 Touchscreen Driver
On 03/30/2011 01:02 AM, Wolfram Sang wrote:
> * PGP Signed by an unknown key: 03/30/2011 at 12:02:23 AM
> Hi,
>
>> + retval = rmi_register_sensor(&instancedata->rmiphysdrvr, platformdata->perfunctiondata);
>> + if (retval) {
>> + dev_err(&client->dev, "%s: Failed to Register %s sensor drivers\n",
>> + __func__, instancedata->rmiphysdrvr.name);
>> + i2c_set_clientdata(client, NULL);
>
> I originally just wanted to say that this line can be removed. Then I
> remembered that I already sent a patch for a similar driver in staging
> (dc7b202a4ee6cb686e2bbef80c84443f43ec91bd). The staging driver looks in
> a better shape to me. Do you know about it?

Hi Wolfram,

That call is required in order to register the sensor device on the rmi
bus, so that the rmi functions can be associated with it. Without that
call, the driver will not function.

I'm afraid I can't locate the commit you specified. I do find a driver
in staging/ste_rmi4/synaptics_i2c_rmi4.[ch] - is that the one? If not
which, tree/branch can I find it in?

Thanks,
Chris



\
 
 \ /
  Last update: 2011-03-30 20:39    [W:0.084 / U:2.400 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site