浏览代码

Revert "Input: atmel_mxt_ts - warn if sysfs could not be created"

Dmitry: I understand that I am a bit late to the party :) but I do not
agree with this change. Failure to create attributes is not sometihng
that user could cause (at least not easily) and thus would not be a
setup issue but something more severe. I believe we should fail
loading the driver so sysfs attribute breakage will be noticed as soon
as possible, instead of discovering it much much later in the process.

This reverts commit 639900380062ecd78ee8b265ea23929c565469b4.

Requested-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Henrik Rydberg <rydberg@euromail.se>
Henrik Rydberg 13 年之前
父节点
当前提交
c45361a128
共有 1 个文件被更改,包括 4 次插入1 次删除
  1. 4 1
      drivers/input/touchscreen/atmel_mxt_ts.c

+ 4 - 1
drivers/input/touchscreen/atmel_mxt_ts.c

@@ -1184,10 +1184,13 @@ static int __devinit mxt_probe(struct i2c_client *client,
 
 	error = sysfs_create_group(&client->dev.kobj, &mxt_attr_group);
 	if (error)
-		dev_warn(&client->dev, "error creating sysfs entries.\n");
+		goto err_unregister_device;
 
 	return 0;
 
+err_unregister_device:
+	input_unregister_device(input_dev);
+	input_dev = NULL;
 err_free_irq:
 	free_irq(client->irq, data);
 err_free_object: