aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2015-08-07staging: rtl8188eu: remove unused defineSudip Mukherjee1-2/+0
_RECV_OSDEP_C_ was only defined here but never checked anywhere. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8188eu: rearrange codeSudip Mukherjee1-6/+4
Re-arrange the code to directly return success or failure, thus removing the variable used in the function. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8188eu: make function voidSudip Mukherjee3-4/+3
The return value of rtw_os_recv_resource_alloc() is never checked, so make it as void. Moreover as of now the function can not fail. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8188eu: remove unused argumentSudip Mukherjee3-5/+3
The function rtw_os_recv_resource_alloc() only uses the argument struct recv_frame *. So remove the other unused argument. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: wilc1000: remove WILC_memcmp functionChaehyun Lim2-22/+0
Remove WILC_memcmp function because it is changed to memcmp. Signed-off-by: Chaehyun Lim <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: wilc1000: use memcmp instead of WILC_memcmpChaehyun Lim2-26/+26
Use memcmp instead of WILC_memcmp. Signed-off-by: Chaehyun Lim <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: wilc1000: remove WILC_memset functionChaehyun Lim2-21/+0
Remove WILC_memset function because it is changed to memset. Signed-off-by: Chaehyun Lim <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: wilc1000: use memset instead of WILC_memsetChaehyun Lim3-83/+83
Use memset instead of WILC_memset. Signed-off-by: Chaehyun Lim <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging/lustre: use kmemdup rather than duplicating its implementationAndrzej Hajda1-4/+2
The patch was generated using fixed coccinelle semantic patch scripts/coccinelle/api/memdup.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2014320 Signed-off-by: Andrzej Hajda <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: change SupportedRates to ratesJoshua Clayton3-25/+20
Change the value to a name that conforms to Linux coding style. "rates" is equally expressive in this context, and I have left alone a comment and function name that describe the rates as supported rates. Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: remove typedefsJoshua Clayton2-12/+7
Coding style fix. Get rid of typedefs NDIS_802_11_RATES and NDIS_802_11_RATES_EX Undo any casting that was done as a result of the typedef. Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: rename functionJoshua Clayton4-14/+14
Rename r8712_get_ndis_wlan_bssid_ex_sz() to r8712_get_wlan_bssid_ex_sz(), which corresponds to the struct whose size it measures. Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: remove duplicate structJoshua Clayton7-49/+26
struct ndis_wlan_bssid_ex is a doppelganger of struct wlan_bssid_ex, and is used about a third as often. Switch all instances to wlan_bssid_ex, and remove ndis_wlan_bssid_ex This also gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: removed unused wrapper structsJoshua Clayton1-18/+0
Remove wrapper structs that just wrap struct ndis_wlan_bssid_ex, and are unused. Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: fix commentJoshua Clayton1-11/+1
The old comment refers to a typedef name which is being removed, and to a style of calculation which is no longer being used. It falsely states that IELength is variable length, instead of IEs. Change comment to simply state that the IEs field is a buffer of variable size and that IELength refers to the current size. Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: simplify size calculationJoshua Clayton1-10/+1
replace item-by-item size calculation of a struct with the size of the struct. This gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: sm750fb: simplify returnSudip Mukherjee1-5/+1
Lets return the return value directly instead of using a variable to store the result. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: sm750fb: rearrange codeSudip Mukherjee1-14/+13
Rearrange the code to remove one exit label which also makes the code less indented and more readable. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: sm750fb: fix multiline commentSudip Mukherjee1-24/+39
Multiline comments are edited to be in the kernel coding style. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: sm750fb: space around operatorSudip Mukherjee1-13/+17
Kernel coding style says to have a space around the operators. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: sm750fb: remove multiple blank lineSudip Mukherjee1-26/+0
Multiple blank lines are not kernel coding style and so checkpatch complains. Remove them. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8712: fix buggy size calculationJoshua Clayton1-11/+1
r8712_get_ndis_wlan_bssid_ex_sz has a "6 * sizeof(unsigned long)" where the underlying struct has a 6 * unsigned char. Simplify the calculation by just subtracting the variable part from the size of the struct. This also gets rid of a use of typedef NDIS_802_11_RATES_EX Signed-off-by: Joshua Clayton <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07Staging: lustre: lnet: Remove unnecessary parentheses on returnMiguel Bernabeu Diaz1-3/+3
Removed three instances of parentheses in return calls that are unnecessary and do not contribute to readability. Signed-off-by: Miguel Bernabeu Diaz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: lustre: Fix space before '[' errorMiguel Bernabeu Diaz1-1/+1
Fix checkpatch.pl error: ERROR: space prohibited before open square bracket '[' Signed-off-by: Miguel Bernabeu Diaz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: lustre: Fix space before '(' warningsMiguel Bernabeu Diaz1-50/+50
Fix several instances of checkpatch.pl warning: WARNING: space prohibited between function name and open parenthesis '(' Signed-off-by: Miguel Bernabeu Diaz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: lustre: Fix space before '++' errorMiguel Bernabeu Diaz1-1/+1
Fix checkpatch.pl error: ERROR: space prohibited before that '++' (ctx:WxO) Signed-off-by: Miguel Bernabeu Diaz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: lustre: Fix code indent errorMiguel Bernabeu Diaz1-1/+1
Fix checkpatch.pl error: ERROR: code indent should use tabs where possible Signed-off-by: Miguel Bernabeu Diaz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: lustre: Unnecessary line continuationMiguel Bernabeu Diaz1-2/+1
Fix checkpatch.pl warning: WARNING: Avoid unnecessary line continuations Signed-off-by: Miguel Bernabeu Diaz <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: fsl-mc: add DPAA2 overview readmeStuart Yoder2-4/+364
add README file providing an overview of the DPAA2 architecture and how it is integrated in Linux Signed-off-by: Stuart Yoder <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07Staging: vme: remove an unnecessary and wrong warning messageNavy Cheng1-3/+1
The wrong warning message in vme_user_probe() will mislead developers and users. As the error message which prompt cdev_add() error already exists, just remove the unnecessary and wrong message. Signed-off-by: Navy Cheng <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: octeon: fix coding style warnings for block commentsOkash Khawaja1-3/+4
The Linux kernel coding style suggests starting every line in a block comment with an asterisk and finishing the block comment with */ on a separate line. This patch fixes those warnings, clearing all warnings and errors in this file, as reported by the checkpatch script. Signed-off-by: Okash Khawaja <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07Staging: fbtft: Use a struct to describe each LCD controllerFabio Falzoi1-96/+136
Use a struct flexfb_lcd_controller to holds chip properties, instead of relying on a long 'if - else if' chain. This allows to: - use a simple linear search to verify if a certain LCD controller model is supported or not. - add support for a new LCD chip controller simply defining a new flexfb_lcd_controller struct. Signed-off-by: Fabio Falzoi <[email protected]> Acked-by: Noralf Trønnes <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8723au: rtl8723a_hal_init: Improve code readabilityJohannes Postma1-1/+1
This patch improves code readability in the function rtl8723a_cal_txdesc_chksum. It improves the readability of the argument of the function le16_to_cpu. Signed-off-by: Johannes Postma <[email protected]> Acked-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: rtl8723au: Fix sparse warning: cast to restricted __le16Johannes Postma1-1/+1
usPtr is used as __le16 *, but was defined as u16 *. This was reported by sparse as: drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:1850:29: warning: cast to restricted __le16 This patch fixes the type of usPtr. Signed-off-by: Johannes Postma <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07MAINTAINERS: update maintainers listSudip Mukherjee1-8/+9
Now since sm712fb has moved out of staging update the maintainers list accordingly. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07Documentation/fb: add documentation for sm712fbSudip Mukherjee1-0/+31
Create the documentation for SM712. Mention all the supported modes and how to use. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: sm7xxfb: merge sm712fb with fbdevSudip Mukherjee9-30/+16
Now since all cleanups are done and the code is ready to be merged lets move it out of staging into fbdev location. Signed-off-by: Sudip Mukherjee <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: update MODULE_DESCRIPTIONH Hartley Sweeten1-1/+1
Change the MODULE_DESCRIPTION to something more useful than "Comedi low- level driver" Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: updata driver status in comedi commentH Hartley Sweeten1-4/+2
Firmware loading was fixed by: Commit: ac584af5 "staging: comedi: me4000: fix firmware downloading" Change the driver status to "untested" and remove the comments about the driver being broken, Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: cleanup multi-line commentsH Hartley Sweeten1-37/+37
Format the multi-line comments in the kernel CodingStyle. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: usleep_range is preferred over udelayH Hartley Sweeten1-3/+3
Fix checkpatch issue: "CHECK: usleep_range is preferred over udelay; see Documentation/timers/timers-howto.txt". `udelay()` is only used in the firmware upload process. Replace them with `usleep_range()` with a reasonable upper limit. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: introduce me4000_ai_get_sample()H Hartley Sweeten1-10/+13
The hardware returns two's complement values for the analog input samples. These need to be converted to the unsigned binary format that the comedi core expects. Introduce a helper function to handle this. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: comedi_handle_events() will stop conversionsH Hartley Sweeten1-47/+7
The irq handler does not need to manually stop conversions and disable interrupts when "end-of-acquisition", "error", or "overflow" events are detected. The comedi_handle_events() will call the subdevice (*cancel) when these are detected and stop the acquisition. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: tidy up analog output subdevice initH Hartley Sweeten1-12/+8
For aesthetics, add some white space to the analog output subdevice initialization. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: fix me4000_detach()H Hartley Sweeten1-2/+6
There is no real reason to reset the board when detaching. The comedi core will ensure that any commands are canceled before the detach. But the PLX interrupts should be disabled. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: only enable PLX interrupt if we have and irqH Hartley Sweeten1-7/+11
Currently me4000_reset() always enables the PLX interrupt. Move the enable of the interrupt into me4000_auto_attach() and only do the enable if we actually have and irq. Make sure the PLX interrupt is disabled in me4000_reset() before we try to request the irq. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: introduce me4000_ai_reset()H Hartley Sweeten1-33/+23
Introduce a helper function to stop any ai conversions and reset the ai control register. This consolidates the common code in me4000_reset() and me4000_ai_cancel(). Use the new helper in the ai (*insn_read) to ensure that the ai control register is set to a known state after reading the samples. The ai control register will now always be '0' after the (*cancel) of a command or doing an (*insn_read). Knowing this the programming of the register for single acquisition mode in the (*insn_read) can be simplified. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: clear the ME4000_AI_CTRL_REG in me4000_reset()H Hartley Sweeten1-0/+3
Reset the analog input control register after ensuring that any active conversions have been stopped. This mimics what the ai subdevice (*cancel) does. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: remove unnecessary me4000_ai_cancel()H Hartley Sweeten1-6/+0
The comedi core ensures that the subdevice is not busy before it allows starting a new command. The subdevice (*cancel) is called when the subdevice is set to not busy. In this driver the me4000_ai_cancel() is the ai (*cancel) so the extra call in the ai (*do_cmd) is not necessary. Remove it. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2015-08-07staging: comedi: me4000: remove unnecessary ai control register resetH Hartley Sweeten1-3/+0
The me4000_ai_cancel() already reset this register. Signed-off-by: H Hartley Sweeten <[email protected]> Reviewed-by: Ian Abbott <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
Powered by cgit, Gruvboxed by Blaster4385.