- 27 Feb, 2014 32 commits
-
-
Malcolm Priestley authored
Remove unnecessary cast to void * urb->context Reported-by:
Joe Perches <joe@perches.com> Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Malcolm Priestley authored
Remove camel case and white space. Camel case changes pDevice -> priv pRCB -> rcb ntStatus -> status pUrb -> urb Signed-off-by:
-
Malcolm Priestley authored
Clean up white space and camel case. Camel case changes pDevice -> priv pRCB -> rcb bReAllocSkb -> re_alloc_skb Signed-off-by:
-
Malcolm Priestley authored
Use switch on urb->status Signed-off-by:
-
Malcolm Priestley authored
Replace with urb->actual_length. Signed-off-by:
-
Malcolm Priestley authored
Just use urb->status directly. Signed-off-by:
-
Malcolm Priestley authored
Remove commented out code, white space and camel case. camel case changes pDevice -> priv pContext -> context pUrb -> urb Signed-off-by:
-
Malcolm Priestley authored
Signed-off-by:
-
Malcolm Priestley authored
Get rid of unnessary else statement by !(MP_IS_READY(pDevice) && pDevice->Flags & fMP_POST_WRITES) Signed-off-by:
-
Mark Hounschell authored
This patch addresses the follow error message followed by a kernel oops: dgap: driver does not set tty->port. This will crash the kernel later. Fix the driver It also renames the main function this patch addresses because its name is misleading. Signed-off-by:
Mark Hounschell <markh@compro.net> Signed-off-by:
-
Masanari Iida authored
This patch fix C99 style comments in pci.c and pci.h Signed-off-by:
Masanari Iida <standby24x7@gmail.com> Signed-off-by:
-
Masanari Iida authored
Fix C99 style comments in efuse.c. Signed-off-by:
-
Masanari Iida authored
Fix spelling typo in rtl8821ae/btcoexist. Signed-off-by:
-
Masanari Iida authored
Fix spelling typo in comment and printk within rtl8821ae/rtl8821ae. Signed-off-by:
-
Masanari Iida authored
Fix spelling typo in comment and printk within rtl8821ae. Signed-off-by:
-
Philipp Zabel authored
This patch allows to optionally attach the parallel-display to a panel supported by a drm_panel driver instead of supplying the modes via device tree. Before: parallel-display { compatible = "fsl,imx-parallel-display"; ... display-timings { native-timing = <&timing1>; timing1: etm0700g0dh6 { hactive = <800>; vactive = <480>; clock-frequency = <33260000>; hsync-len = <128>; hback-porch = <88>; hfront-porch = <40>; vsync-len = <2>; vback-porch = <33>; vfront-porch = <10>; hsync-active = <0>; vsync-active = <0>; ... }; }; ... }; After: parallel-display { compatible = "fsl,imx-parallel-display"; fsl,panel = <&panel>; ... }; panel: panel { compatible = "edt,etm0700g0dh6", "simple-panel"; }; Signed-off-by:Philipp Zabel <p.zabel@pengutronix.de> Signed-off-by:
-
Philipp Zabel authored
Since msleep(2) can sleep up to 20ms anyway, make this explicit by using usleep_range(2000, 20000). Signed-off-by:
-
Philipp Zabel authored
The IPU display controller supports a non-standard green-red-blue ordered format that is used on the connection between IPU display interface 1 and the TV encoder on i.MX53. In preparation for the move of IPU core code out of staging, place the IPU_PIX_FMT_GBR24 definition in imx-ipu-v3.h, so that both the IPU display interface driver and the TVE encoder driver can access it. Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in panel.c: WARNING: quoted string split across lines Signed-off-by:
Monam Agarwal <monamagarwal123@gmail.com> Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl issues in panel.c: WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: -
Ana Rey authored
This patch clean-up comment-line style in r8180_core.c as the CodingStyle recommends. Signed-off-by:
Ana Rey <anarey@gmail.com> Signed-off-by:
-
Silvan Jegen authored
Instead of an explicit cast the min_t macro should be used. Signed-off-by:
Silvan Jegen <s.jegen@gmail.com> Signed-off-by:
-
Ana Rey authored
The Documentation/CodingStyle doesn't recommend the use of typedef, convert this to structure. While at it, I have also renamed the variable names that were used in this typedef not to use Hungarian notation. Signed-off-by:
-
Ana Rey authored
After the next warning messages by sparse tools: drivers/staging/rtl8187se/r8180_core.c:1621:5: warning: symbol 'rtl8180_len2duration' was not declared. Should it be static? drivers/staging/rtl8187se/r8180_core.c:1621:12: warning: ‘rtl8180_len2duration’ defined but not used [-Wunused-function] static u16 rtl8180_len2duration(u32 len, short rate, short *ext) ^ And the next search: $ grep -ir rtl8180_len2duration ./* ./r8180_core.c:u16 rtl8180_len2duration(u32 len, short rate, short *ext) Binary file ./r8187se.ko matches Binary file ./r8187se.o matches I decided to delete rtl8180_len2duration function. Signed-off-by: -
John Stultz authored
The new 64bit binder API causes build issues on 32bit ARM due to the lack of 64bit __get_user_asm_* implementation. Until that implementation is done, remove the choice for 32bit ARM, automatically enabling the old 32bit binder protocol. This can be reverted once a 64bit __get_user_asm_* implementation is merged. Cc: Colin Cross <ccross@android.com> Cc: Arve Hjønnevåg <arve@android.com> Cc: Serban Constantinescu <serban.constantinescu@arm.com> Cc: Android Kernel Team <kernel-team@android.com> Reported-by:
Arnd Bergmann <arnd.bergmann@linaro.org> Signed-off-by:
John Stultz <john.stultz@linaro.org> Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in functions.c WARNING: Line length over 80 characters Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in functions.c WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: -
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in functions.c WARNING: please no spaces at the start of a line in Signed-off-by:
-
Daeseok Youn authored
After spin_lock() is called, all of if-else conditions in this brace should reach the end of else and spin_unlock() must be called. So It doesn't need to call spin_unlock() without a return statement for handling an error. Also sparse says: drivers/staging/ced1401/usb1401.c:1080:28: warning: context imbalance in 'Handle1401Esc' - unexpected unlock Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Daeseok Youn <daeseok.youn@gmail.com> Signed-off-by:
-
Chase Southwood authored
The variable ui_Command is as of right now being cleared to a value of zero between everytime that it writes to a port and then takes a new value from a port. Seems like this zeroing is unnecessary, so we can just remove these lines. Signed-off-by:
Chase Southwood <chase.southwood@yahoo.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
-
Kumar Amit Mehta authored
Return -ENOMEM in ni_E_init if ni_gpct_device_construct returns NULL Signed-off-by:
Kumar Amit Mehta <gmate.amit@gmail.com> Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following checkpatch.pl warning in comedidev.h WARNING: Fix unnecessary space after function pointer Signed-off-by:
-
- 25 Feb, 2014 8 commits
-
-
Mark Hounschell authored
The last patch series exposed some dead code causing compile warnings. This patch removes that dead code and fixes the warnings Signed-off-by:
-
Chase Southwood authored
There are a couple of cases where a comment being on the same line as a statement is causing the line to be over 80 characters long. This is an easy fix, move these comments to the previous line. Signed-off-by:
-
Chase Southwood authored
There were some conditional blocks that had an unnecessary level of indentation in them. We can remove this to improve code clarity. Signed-off-by:
-
Chase Southwood authored
This patch further cleans up the comments in hwdrv_apci035.c, converting them to kernel style and removing some commented conditional statements that are unused. Signed-off-by:
-
Monam Agarwal authored
Signed-off-by:
-
Malcolm Priestley authored
Replace with enum assign as u8 type. Signed-off-by:
-
Malcolm Priestley authored
Remove commented out, white space and camel case. Camel case changes pContext -> context pDevice -> priv ContextType -> context_type ulBufLen -> buf_len Signed-off-by:
-
Malcolm Priestley authored
change pContext->bBoolInUse to false on connection errors. Signed-off-by:
-
