Re: [PATCH v2] staging: rtl8188eu: use %pM specifier instead of passing direct values

2015-03-08 Thread Mike Krinkin
On Sun, Mar 08, 2015 at 10:00:52AM -0700, Joe Perches wrote: > On Sun, 2015-03-08 at 19:42 +0300, Mike Krinkin wrote: > > The patch converts code to use %pM specifier instead of pushing > > each byte via stack. > > Hello Mike. > > > diff --git a/drivers/staging/rtl8

[PATCH v3] staging: rtl8188eu: use %pM specifier instead of passing direct values

2015-03-08 Thread Mike Krinkin
The patch converts code to use %pM specifier instead of pushing each byte via stack. Signed-off-by: Mike Krinkin --- Changes v2 -> v3: - actual use of %pM instead %*ph as suggested by Larry Finger. drivers/staging/rtl8188eu/hal/usb_halinit.c | 6 ++ 1 file changed, 2 insertions(+)

[PATCH v2] staging: rtl8188eu: use %pM specifier instead of passing direct values

2015-03-08 Thread Mike Krinkin
The patch converts code to use %pM specifier instead of pushing each byte via stack. Signed-off-by: Mike Krinkin --- Changes in v2: - use %pM instead %*ph as suggested by Larry Finger. drivers/staging/rtl8188eu/hal/usb_halinit.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions

Re: [PATCH] staging: rtl8188eu: use %*ph specifier instead of passing direct values

2015-03-08 Thread Mike Krinkin
On Sun, Mar 08, 2015 at 11:23:05AM -0500, Larry Finger wrote: > On 03/08/2015 10:54 AM, Mike Krinkin wrote: > >The %*ph specifier allows to pass a pointer and length instead of > >pushing each byte via stack. The patch converts code to use it. > > > >Signed-off-by: Mik

[PATCH] staging: rtl8188eu: use %*ph specifier instead of passing direct values

2015-03-08 Thread Mike Krinkin
The %*ph specifier allows to pass a pointer and length instead of pushing each byte via stack. The patch converts code to use it. Signed-off-by: Mike Krinkin --- drivers/staging/rtl8188eu/hal/usb_halinit.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging

[PATCH] staging: vt6655: fix sparse warnings: incorrect argument type

2014-12-21 Thread Mike Krinkin
type in argument 2 (different address spaces) drivers/staging/vt6655/device_main.c:1505:25:expected void [noderef] * drivers/staging/vt6655/device_main.c:1505:25:got struct vnt_private * Signed-off-by: Mike Krinkin --- drivers/staging/vt6655/device_main.c | 6 -- 1 file changed, 4

[PATCH] staging: rtl8712: remove unnecessary else after return

2014-12-02 Thread Mike Krinkin
Fixes checkpatch warning: WARNING: else is not generally useful after a break or return Signed-off-by: Mike Krinkin --- drivers/staging/rtl8712/rtl8712_recv.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging

[PATCH] staging: rtl8712: remove unnecessary else after return

2014-12-02 Thread Mike Krinkin
Fixes checkpatch warning: WARNING: else is not generally useful after a break or return Signed-off-by: Mike Krinkin --- drivers/staging/rtl8712/rtl8712_recv.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging