summaryrefslogtreecommitdiffstats
path: root/drivers/staging/vt6655
Commit message (Collapse)AuthorAgeFilesLines
* staging: Add missing newline at end of fileGeert Uytterhoeven2019-06-181-1/+1
| | | | | | | | | | | "git diff" says: \ No newline at end of file after modifying the files. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Change return type of function and remove variableNishka Dasgupta2019-06-032-7/+3Star
| | | | | | | | | | Remove return variable bResult from function CARDbRadioPowerOff and change the return type of the function to void as it always returns true and the return value is never stored nor checked when called. Issue found with Coccinelle. Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt665*: use help instead of ---help--- in KconfigMoses Christopher2019-05-201-3/+2Star
| | | | | | | | - Resolve the following warning from the Kconfig, "WARNING: prefer 'help' over '---help---' for new help texts" Signed-off-by: Moses Christopher <moseschristopherb@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: upc: remove double blank linesCesar Santos2019-04-161-2/+0Star
| | | | | | | | Fix checkpatch warning "CHECK: Please don't use multiple blank lines" on upc.h Signed-off-by: Cesar Santos <dev.cmsantos@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: add missing SPDX lines to Kconfig filesGreg Kroah-Hartman2019-04-031-0/+1
| | | | | | | | There are a few remaining drivers/staging/*/Kconfig files that do not have SPDX identifiers in them. Add the correct GPL-2.0 identifier to them to make scanning tools happy. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: rxtx: Add spaces around '*'Briana Oursler2019-04-011-8/+8
| | | | | | | Add spaces around *. Issue found with checkpatch. Signed-off-by: Briana Oursler <briana.oursler@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Merge 5.1-rc3 into staging-nextGreg Kroah-Hartman2019-04-011-6/+5Star
|\ | | | | | | | | | | We want those fixes and this resolves an erofs merge conflict. Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: vt6655: Remove vif check from vnt_interruptMalcolm Priestley2019-03-291-2/+1Star
| | | | | | | | | | | | | | | | | | | | | | A check for vif is made in vnt_interrupt_work. There is a small chance of leaving interrupt disabled while vif is NULL and the work hasn't been scheduled. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> CC: stable@vger.kernel.org # v4.2+ Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| * staging: vt6655: Fix interrupt race condition on device start up.Malcolm Priestley2019-03-251-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It appears on some slower systems that the driver can find its way out of the workqueue while the interrupt is disabled by continuous polling by it. Move MACvIntEnable to vnt_interrupt_work so that it is always enabled on all routes out of vnt_interrupt_process. Move MACvIntDisable so that the device doesn't keep polling the system while the workqueue is being processed. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> CC: stable@vger.kernel.org # v4.2+ Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: lines should not end with a parenthesesVatsala Narang2019-03-251-10/+2Star
| | | | | | | | | | | | | | | | Adjust code to fix the style issue-lines should not end with a parentheses Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Modify comment style of SPDX License IdentifierGanesh Biradar2019-03-257-7/+7
| | | | | | | | | | | | | | | | Fix the comment style of SPDX license identifier based on header file type Signed-off-by: Ganesh Biradar <grbesd1@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Add spaces around operatorVatsala Narang2019-03-221-5/+5
| | | | | | | | | | | | | | Add spaces around '-' operator to get rid of checkpatch warning. Signed-off-by: Vatsala Narang <vatsalanarang@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: lines should not end with a parenthesisPayal Kshirsagar2019-03-211-5/+2Star
| | | | | | | | | | | | | | | | Adjust code to fix the style issue – lines should not end with a parenthesis. Signed-off-by: Payal Kshirsagar <payal.s.kshirsagar.98@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: correction in spellingPayal Kshirsagar2019-03-211-1/+1
| | | | | | | | | | | | | | Correct misspelled word. Signed-off-by: Payal Kshirsagar <payal.s.kshirsagar.98@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: rxtx.c: Remove unnecessary space after a castSanjana Sanikommu2019-03-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | Issue found by checkpatch.pl semantic patch results for rxtx.c. Remove unnecessary space after the cast. CHECK: No necessary space is required after a cast. Signed-off-by: Sanjana Sanikommu <sanjana99reddy99@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: line over 80 charactersBranden Bonaby2019-03-181-2/+4
| | | | | | | | | | | | | | | | Move function arguments that go over the 80 characters a line limit to a new line. "WARNING: line over 80 characters". Signed-off-by: Branden Bonaby <brandonbonaby94@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Remove unused function prototype and definitionMadhumitha Prabakaran2019-03-182-26/+0Star
| | | | | | | | | | | | | | Remove the MACbIsRegBitsOn function, as it is never called in kernel tree. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Remove an unused function prototypeMadhumitha Prabakaran2019-03-181-2/+0Star
| | | | | | | | | | | | | | | | Remove an unused function prototype as there is no such function in the kernel tree. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Fix malformed SPDX-License-Identifier tagDaniela Mormocea2019-03-181-1/+1
| | | | | | | | | | | | | | Replace "//" comment with "/**/" comment Signed-off-by: Daniela Mormocea <daniela.mormocea@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Align function declarationMadhumitha Prabakaran2019-03-181-5/+2Star
| | | | | | | | | | | | | | Fix function return type and function arguments to improve readbility. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Fix the warning line over 80 charactersMadhumitha Prabakaran2019-03-181-1/+2
| | | | | | | | | | | | | | | | Fix the checkpatch.pl warning WARNING: line over 80 characters Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: mac.h: Fix comment style for SPDX-License IdentifierMadhumitha Prabakaran2019-03-181-1/+1
| | | | | | | | | | | | | | | | Modify comment style for header type file based on license identifier syntax. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Remove unnecessary blank lineMadhumitha Prabakaran2019-03-181-1/+0Star
| | | | | | | | | | | | | | | | Fix the check issue provided by checkpatch.pl CHECK: Blank lines aren't necessary before a close brace '}' Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: key.h: Fix comment style for SPDX-License IdentifierMadhumitha Prabakaran2019-03-181-1/+1
| | | | | | | | | | | | | | | | Modify comment style for SPDX-License Identifier based on license identifier syntax. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: dpc.h: Fix comment style of SPDX License IdentifierMadhumitha Prabakaran2019-03-181-1/+1
| | | | | | | | | | | | | | | | Modify comment style of SPDX License Identifier for header file type, as per the license identifier syntax. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Fix space around binary operatorMadhumitha Prabakaran2019-03-181-1/+1
| | | | | | | | | | | | | | | | Fix the check as per Linux kernel style for use of spaces CHECK: spaces preferred around that '-' Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: Fix spelling mistakeMadhumitha Prabakaran2019-03-181-1/+1
| | | | | | | | | | | | | | | | Fix the check for spelling mistake CHECK: 'Regsiter' may be misspelled - perhaps 'Register' Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: desc.h: Fix comment style of SPDX-License IdentifierMadhumitha Prabakaran2019-03-181-1/+1
| | | | | | | | | | | | | | | | Fix comment style of SPDX License Identifier for header file based on License Identifier syntax. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* | staging: vt6655: channel.h: Fix comment style of SPDX License IdentifierMadhumitha Prabakaran2019-03-181-1/+1
|/ | | | | | | | Fix the comment style of SPDX License Identifier for header file based on License Identifier syntax. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Align a static function declarationMadhumitha Prabakaran2019-03-011-8/+3Star
| | | | | | | | Align function arguments and function return type to improve readability. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Modify comment style of SPDX License IdentifierMadhumitha Prabakaran2019-02-281-1/+1
| | | | | | | | Fix the comment style of SPDX license identifier based on header file type Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Modify comment style for SPDX-License-IdentifierMadhumitha Prabakaran2019-02-281-1/+1
| | | | | | | | Fix the comment style for SPDX-License-Identifier, as the file type is header. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Align a function declarationMadhumitha Prabakaran2019-02-281-7/+3Star
| | | | | | | | Align function return type and function arguments to improve readability. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Alignment of function declarationMadhumitha Prabakaran2019-02-281-4/+1Star
| | | | | | | Align function arguments and function type to improve readability. Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Alignment should match open parenthesisMadhumitha Prabakaran2019-02-271-1/+1
| | | | | | | | Fix the check to improve readibility CHECK: Alignment should match open parenthesis Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* Staging: vt6655: Alignment should match open parenthesisMadhumitha Prabakaran2019-02-261-1/+1
| | | | | | | | Fix the check to improve readibility. CHECK: Alignment should match open parenthesis Signed-off-by: Madhumitha Prabakaran <madhumithabiw@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* cross-tree: phase out dma_zalloc_coherent()Luis Chamberlain2019-01-081-13/+6Star
| | | | | | | | | | | | | | | | | | | We already need to zero out memory for dma_alloc_coherent(), as such using dma_zalloc_coherent() is superflous. Phase it out. This change was generated with the following Coccinelle SmPL patch: @ replace_dma_zalloc_coherent @ expression dev, size, data, handle, flags; @@ -dma_zalloc_coherent(dev, size, handle, flags) +dma_alloc_coherent(dev, size, handle, flags) Suggested-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Luis Chamberlain <mcgrof@kernel.org> [hch: re-ran the script on the latest tree] Signed-off-by: Christoph Hellwig <hch@lst.de>
* staging: vt6655: fix small typoYangtao Li2018-11-071-1/+1
| | | | | | | Correct spelling mistakes of "Calculate" Signed-off-by: Yangtao Li <tiny.windzz@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Remove space after castMamta Shukla2018-10-091-5/+5
| | | | | | | | | Remove space after cast to fix checkpatch issue. CHECK: No space is necessary after a cast Signed-off-by: Mamta Shukla <mamtashukla555@gmail.com> Reviewed-by: Himanshu Jha <himanshujha199640@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Add spaces around operatorsMamta Shukla2018-10-091-9/+9
| | | | | | | | Add spaces around '*' and '-' to fix checkpatch issue. Signed-off-by: Mamta Shukla <mamtashukla555@gmail.com> Reviewed-by: Himanshu Jha <himanshujha199640@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Add blank line after function/struct/union/enum declarationsMamta Shukla2018-10-091-0/+1
| | | | | | | | | | | Add blank line after function/struct/union/enum declarations to fix checkpatch issue. CHECK: Please use a blank line after function/struct/union/enum declarations Signed-off-by: Mamta Shukla <mamtashukla555@gmail.com> Reviewed-by: Himanshu Jha <himanshujha199640@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: remove some redundant variablesColin Ian King2018-07-111-3/+1Star
| | | | | | | | | | | | | | Variables rx_sts, sq, frame and is_pspoll are being assigned but are never used hence they are redundant and can be removed. Cleans up clang warnings: warning: variable 'sq' set but not used [-Wunused-but-set-variable] warning: variable 'rx_sts' set but not used [-Wunused-but-set-variable] warning: variable 'frame' set but not used [-Wunused-but-set-variable] warning: variable 'is_pspoll' set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: remove unnecessary line breaks in function definition.Danilo Alves2018-04-231-4/+1Star
| | | | | | | | | | This patch corrects the function definition style of CARDvSafeResetRx. Issue found by checkpatch. CHECK: Lines should not end with a '(' Signed-off-by: Danilo Alves <daniloalves@riseup.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: add handling memory leak on vnt_start()Ji-Hun Kim2018-04-231-2/+6
| | | | | | | | | | | There was no code for handling memory leaks of device_init_rings() and request_irq(). It needs to free allocated memory in the device_init_rings() , when request_irq() would be failed. Add freeing sequences of irq and device init rings. Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: check for memory allocation failuresJi-Hun Kim2018-04-231-26/+118
| | | | | | | | | | | There are no null pointer checking on rd_info and td_info values which are allocated by kzalloc. It has potential null pointer dereferencing issues. Implement error handling code on device_init_rd*, device_init_td* and vnt_start for the allocation failures. Signed-off-by: Ji-Hun Kim <ji_hun.kim@samsung.com> Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Delete unused typedef enum _chip_typeNishka Dasgupta2018-03-281-4/+0Star
| | | | | | | | Delete unused typedef enum _chip_type. Signed-off-by: Nishka Dasgupta <nishka.dasgupta_ug18@ashoka.edu.in> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Delete unused typedef struct _versionNishka Dasgupta2018-03-281-7/+0Star
| | | | | | | | Delete unused typedef struct _version. Signed-off-by: Nishka Dasgupta <nishka.dasgupta_ug18@ashoka.edu.in> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Delete unused typedef enum _CARD_STATUS_TYPENishka Dasgupta2018-03-281-6/+0Star
| | | | | | | | Delete unused typedef enum _CARD_STATUS_TYPE. Signed-off-by: Nishka Dasgupta <nishka.dasgupta_ug18@ashoka.edu.in> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Delete unused enum CARD_PKT_TYPENishka Dasgupta2018-03-281-7/+0Star
| | | | | | | | Delete unused enum CARD_PKT_TYPE. Signed-off-by: Nishka Dasgupta <nishka.dasgupta_ug18@ashoka.edu.in> Acked-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
* staging: vt6655: Change typedef enum to enumNishka Dasgupta2018-03-221-2/+2
| | | | | | | Change typedef enum to enum. Issue found with checkpatch. Signed-off-by: Nishka Dasgupta <nishka.dasgupta_ug18@ashoka.edu.in> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>