提交 80f8594f 编写于 作者: G Greg Kroah-Hartman

USB: input: appletouch.c: fix up dev_err() usage

We should always reference the input device for dev_err(), not the USB
device.  Fix up the places where I got this wrong.
Reported-by: NDmitry Torokhov <dmitry.torokhov@gmail.com>
CC: Alessandro Rubini <rubini@ipvvis.unipv.it>
Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
上级 aa330384
...@@ -253,8 +253,9 @@ MODULE_PARM_DESC(debug, "Activate debugging output"); ...@@ -253,8 +253,9 @@ MODULE_PARM_DESC(debug, "Activate debugging output");
* packets (Report ID 2). This code changes device mode, so it * packets (Report ID 2). This code changes device mode, so it
* sends raw sensor reports (Report ID 5). * sends raw sensor reports (Report ID 5).
*/ */
static int atp_geyser_init(struct usb_device *udev) static int atp_geyser_init(struct atp *dev)
{ {
struct usb_device *udev = dev->udev;
char *data; char *data;
int size; int size;
int i; int i;
...@@ -262,7 +263,7 @@ static int atp_geyser_init(struct usb_device *udev) ...@@ -262,7 +263,7 @@ static int atp_geyser_init(struct usb_device *udev)
data = kmalloc(8, GFP_KERNEL); data = kmalloc(8, GFP_KERNEL);
if (!data) { if (!data) {
dev_err(&udev->dev, "Out of memory\n"); dev_err(&dev->input->dev, "Out of memory\n");
return -ENOMEM; return -ENOMEM;
} }
...@@ -277,7 +278,7 @@ static int atp_geyser_init(struct usb_device *udev) ...@@ -277,7 +278,7 @@ static int atp_geyser_init(struct usb_device *udev)
for (i = 0; i < 8; i++) for (i = 0; i < 8; i++)
dprintk("appletouch[%d]: %d\n", i, data[i]); dprintk("appletouch[%d]: %d\n", i, data[i]);
dev_err(&udev->dev, "Failed to read mode from device.\n"); dev_err(&dev->input->dev, "Failed to read mode from device.\n");
ret = -EIO; ret = -EIO;
goto out_free; goto out_free;
} }
...@@ -296,7 +297,7 @@ static int atp_geyser_init(struct usb_device *udev) ...@@ -296,7 +297,7 @@ static int atp_geyser_init(struct usb_device *udev)
for (i = 0; i < 8; i++) for (i = 0; i < 8; i++)
dprintk("appletouch[%d]: %d\n", i, data[i]); dprintk("appletouch[%d]: %d\n", i, data[i]);
dev_err(&udev->dev, "Failed to request geyser raw mode\n"); dev_err(&dev->input->dev, "Failed to request geyser raw mode\n");
ret = -EIO; ret = -EIO;
goto out_free; goto out_free;
} }
...@@ -313,15 +314,14 @@ static int atp_geyser_init(struct usb_device *udev) ...@@ -313,15 +314,14 @@ static int atp_geyser_init(struct usb_device *udev)
static void atp_reinit(struct work_struct *work) static void atp_reinit(struct work_struct *work)
{ {
struct atp *dev = container_of(work, struct atp, work); struct atp *dev = container_of(work, struct atp, work);
struct usb_device *udev = dev->udev;
int retval; int retval;
dprintk("appletouch: putting appletouch to sleep (reinit)\n"); dprintk("appletouch: putting appletouch to sleep (reinit)\n");
atp_geyser_init(udev); atp_geyser_init(dev);
retval = usb_submit_urb(dev->urb, GFP_ATOMIC); retval = usb_submit_urb(dev->urb, GFP_ATOMIC);
if (retval) if (retval)
dev_err(&udev->dev, dev_err(&dev->input->dev,
"atp_reinit: usb_submit_urb failed with error %d\n", "atp_reinit: usb_submit_urb failed with error %d\n",
retval); retval);
} }
...@@ -589,7 +589,7 @@ static void atp_complete_geyser_1_2(struct urb *urb) ...@@ -589,7 +589,7 @@ static void atp_complete_geyser_1_2(struct urb *urb)
exit: exit:
retval = usb_submit_urb(dev->urb, GFP_ATOMIC); retval = usb_submit_urb(dev->urb, GFP_ATOMIC);
if (retval) if (retval)
dev_err(&dev->udev->dev, dev_err(&dev->input->dev,
"atp_complete: usb_submit_urb failed with result %d\n", "atp_complete: usb_submit_urb failed with result %d\n",
retval); retval);
} }
...@@ -724,7 +724,7 @@ static void atp_complete_geyser_3_4(struct urb *urb) ...@@ -724,7 +724,7 @@ static void atp_complete_geyser_3_4(struct urb *urb)
exit: exit:
retval = usb_submit_urb(dev->urb, GFP_ATOMIC); retval = usb_submit_urb(dev->urb, GFP_ATOMIC);
if (retval) if (retval)
dev_err(&dev->udev->dev, dev_err(&dev->input->dev,
"atp_complete: usb_submit_urb failed with result %d\n", "atp_complete: usb_submit_urb failed with result %d\n",
retval); retval);
} }
...@@ -751,14 +751,12 @@ static void atp_close(struct input_dev *input) ...@@ -751,14 +751,12 @@ static void atp_close(struct input_dev *input)
static int atp_handle_geyser(struct atp *dev) static int atp_handle_geyser(struct atp *dev)
{ {
struct usb_device *udev = dev->udev;
if (dev->info != &fountain_info) { if (dev->info != &fountain_info) {
/* switch to raw sensor mode */ /* switch to raw sensor mode */
if (atp_geyser_init(udev)) if (atp_geyser_init(dev))
return -EIO; return -EIO;
printk(KERN_INFO "appletouch: Geyser mode initialized.\n"); dev_info(&dev->input->dev, "Geyser mode initialized.\n");
} }
return 0; return 0;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册