Re: [PATCH 1/1] drivers/staging/vt6656/main_usb.c: usb_device_reset

2017-04-18 Thread Joe Perches
On Tue, 2017-04-18 at 10:33 +0200, Greg KH wrote: > On Tue, Apr 18, 2017 at 01:24:37AM -0700, Chewie Lin wrote: > > Removed the usb_device_reset(), replace with call to usb_reset_device() > > directly. Plus it removes the confusing function name and addressed > > the checkpatch warning as well

Re: [PATCH 1/1] drivers/staging/vt6656/main_usb.c: usb_device_reset

2017-04-18 Thread Joe Perches
On Tue, 2017-04-18 at 10:33 +0200, Greg KH wrote: > On Tue, Apr 18, 2017 at 01:24:37AM -0700, Chewie Lin wrote: > > Removed the usb_device_reset(), replace with call to usb_reset_device() > > directly. Plus it removes the confusing function name and addressed > > the checkpatch warning as well

Re: [PATCH 1/1] drivers/staging/vt6656/main_usb.c: usb_device_reset

2017-04-18 Thread Greg KH
On Tue, Apr 18, 2017 at 01:24:37AM -0700, Chewie Lin wrote: > Removed the usb_device_reset(), replace with call to usb_reset_device() > directly. Plus it removes the confusing function name and addressed > the checkpatch warning as well by swap string in the dev_warn() call with > __func__

Re: [PATCH 1/1] drivers/staging/vt6656/main_usb.c: usb_device_reset

2017-04-18 Thread Greg KH
On Tue, Apr 18, 2017 at 01:24:37AM -0700, Chewie Lin wrote: > Removed the usb_device_reset(), replace with call to usb_reset_device() > directly. Plus it removes the confusing function name and addressed > the checkpatch warning as well by swap string in the dev_warn() call with > __func__

[PATCH 1/1] drivers/staging/vt6656/main_usb.c: usb_device_reset

2017-04-18 Thread Chewie Lin
Removed the usb_device_reset(), replace with call to usb_reset_device() directly. Plus it removes the confusing function name and addressed the checkpatch warning as well by swap string in the dev_warn() call with __func__ argument, instead of explicitly calling the function name in the

[PATCH 1/1] drivers/staging/vt6656/main_usb.c: usb_device_reset

2017-04-18 Thread Chewie Lin
Removed the usb_device_reset(), replace with call to usb_reset_device() directly. Plus it removes the confusing function name and addressed the checkpatch warning as well by swap string in the dev_warn() call with __func__ argument, instead of explicitly calling the function name in the