Thread (8 messages) 8 messages, 3 authors, 2011-12-28

Re: [PATCH 2/3] input: mpu3050: Configure the sampling method

flat view

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2011-12-24 08:08:02

Hi Alan,

On Thu, Dec 15, 2011 at 10:18:47PM +0000, Alan Cox wrote:
From: Heikki Krogerus <heikki.krogerus@linux.intel.com>

This will improve the output of the sensor.
...
 
+/* Configures the sampling method */
+static int mpu3050_hw_init(struct mpu3050_sensor *sensor)
This should be __devinit.

...
quoted hunk ↗ jump to hunk
@@ -336,6 +423,10 @@ static int __devinit mpu3050_probe(struct i2c_client *client,
 
 	pm_runtime_set_active(&client->dev);
 
+	error = mpu3050_hw_init(sensor);
+	if (error)
+		goto err_free_mem;
We should go to err_pm_set_suspended, not err_free_mem.

I fixed it up locally.

Thanks.

-- 
Dmitry
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help