Re: [PATCH 1/3] input/touchscreen: Synaptics RMI4 Touchscreen Driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
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


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux