Hi,
On Tue, Aug 02, 2011 at 02:28:42PM +0300, Peter Ujfalusi wrote:
> @@ -145,7 +143,7 @@ static int vibra_play(struct input_dev *input, void *data,
>
> ret = queue_work(info->workqueue, &info->play_work);
> if (!ret) {
> - dev_err(&input->dev, "work is already on queue\n");
> + dev_info(&input->dev, "work is already on queue\n");why ??? > return ret; > } > > @@ -266,12 +264,10 @@ static int __devinit twl6040_vibra_probe(struct > platform_device *pdev) > goto err_irq; > } > > - printk(KERN_ERR "%s:powering twl6040\n", __func__); > ret = twl6040_power(info->twl6040, 1); > if (ret < 0) > goto err_pwr; > > - printk(KERN_ERR "%s:powered\n", __func__); this hunk is not part of this patch. > return 0; > > err_pwr: > @@ -297,7 +293,6 @@ static int __devexit twl6040_vibra_remove(struct > platform_device *pdev) > twl6040_free_irq(info->twl6040, TWL6040_IRQ_VIB, info); > input_unregister_device(info->input_dev); > kfree(info); > - platform_set_drvdata(pdev, NULL); neither is this. -- balbi
signature.asc
Description: Digital signature
