- 23 May, 2014 40 commits
-
-
Malcolm Priestley authored
camel case changes pDevice -> priv byRegOfs -> reg_ofs wData -> word pbyData -> data Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Malcolm Priestley authored
camel case changes pDevice -> priv byRegOfs -> reg_ofs byBits -> bits pbyData -> data Signed-off-by:
-
Malcolm Priestley authored
camel case changes pDevice -> priv byRegOfs -> reg_ofs byBits -> bits pbyData -> data Signed-off-by:
-
Malcolm Priestley authored
Camel case changes pDevice -> priv uEntryIdx -> entry_idx Signed-off-by:
-
Malcolm Priestley authored
Remove byData and change all callers to u8. Signed-off-by:
-
Malcolm Priestley authored
Camel case changes pDevice -> priv byType -> type pbyData -> data Signed-off-by:
-
Malcolm Priestley authored
Camel case changes pDevice -> priv signed-off-by:
-
Malcolm Priestley authored
Camel case changes. pDevice -> priv Signed-off-by:
-
Chaitanya Hazarey authored
Added a blank line after declarations to fix the following warning issued by checkpatch.pl: drivers/staging/xgifb/vb_init.c:800: WARNING: Missing a blank line after declarations Signed-off-by:
Chaitanya Hazarey <c@24.io> Signed-off-by:
-
Joel Pelaez Jorge authored
Remove support for building against ancient WIRELESS_EXT versions, only leaving support for the current version: 22 Signed-off-by:
Joel Pelaez Jorge <joelpelaez@gmail.com> Signed-off-by:
-
Mark Hounschell authored
We don't need to use GFP_ATOMIC at driver load time so use GFP_KERNEL instead. Signed-off-by:
Mark Hounschell <markh@compro.net> Tested-by:
-
Mark Hounschell authored
Simplify dgap_config_get_useintr and dgap_config_get_altpin changing the case statement to an if statement. Signed-off-by:
-
Mark Hounschell authored
This patch replaces double negativeconditionals Signed-off-by:
-
Mark Hounschell authored
Removes more extra white space and tabs Signed-off-by:
-
Mark Hounschell authored
Removes unnecessary initialization of variables in declarations Signed-off-by:
-
Djordje Zekovic authored
The advantage of kcalloc is, that will prevent integer overflows which could result from the multiplication of number of elements and size and it is also a bit nicer to read. Signed-off-by:
Djordje Zekovic <dj@zeko.me> Signed-off-by:
-
Eli Billauer authored
devm_get_free_pages() and devm_free_pages() are the managed counterparts for __get_free_pages() and free_pages(). Signed-off-by:
Eli Billauer <eli.billauer@gmail.com> Acked-by:
Tejun Heo <tj@kernel.org> Signed-off-by:
-
Peter Senna Tschudin authored
This patch remove variables that are initialized with a constant, are never updated, and are only used as parameter of return. Return the constant instead of using a variable. Verified by compilation only. The coccinelle script that find and fixes this issue is: // <smpl> @@ type T; constant C; identifier ret; @@ - T ret = C; ... when != ret - return ret; + return C; // </smpl> Signed-off-by:
Peter Senna Tschudin <peter.senna@gmail.com> Signed-off-by:
-
Gengis Dave authored
Return values cleanup Signed-off-by:
Davide Gianforte <davide@gengisdave.org> Signed-off-by:
-
Peter Senna Tschudin authored
This patch remove variables that are initialized with a constant, are never updated, and are only used as parameter of return. Return the constant instead of using a variable. Verified by compilation only. The coccinelle script that find and fixes this issue is: // <smpl> @@ type T; constant C; identifier ret; @@ - T ret = C; ... when != ret - return ret; + return C; // </smpl> Signed-off-by:
-
Benoit Taine authored
This patch enhances kernel style usage for the rts5208 driver. We should prefer `struct pci_device_id` over `DEFINE_PCI_DEVICE_TABLE` to meet kernel coding style guidelines. This issue was reported by checkpatch. Signed-off-by:
Benoit Taine <benoit.taine@lip6.fr> Signed-off-by:
-
Julia Lawall authored
Delete unnecessary use of a local variable to immediately return 0. A simplified version of the semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r exists@ local idexpression ret; expression e; position p; @@ -ret = 0; ... when != ret = e return - ret + 0 ; // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Masaru Nomura authored
Remove a prohibited space before a closed parenthesis of if statement to meet kernel coding style. Signed-off-by:
Masaru Nomura <massa.nomura@gmail.com> Signed-off-by:
-
Masaru Nomura authored
Add a required space before an open parenthesis of if statement to meet kernel coding style. Signed-off-by:
-
Eli Billauer authored
Suggested-by:
Baruch Siach <baruch@tkos.co.il> Signed-off-by:
-
Masanari Iida authored
This patch converts printk to dev_err in ft1000_cs.c Signed-off-by:
Masanari Iida <standby24x7@gmail.com> Signed-off-by:
-
Mark Hounschell authored
Removes dgap_savestring function to use kstrdup instead. Signed-off-by:
-
Mark Hounschell authored
This patch removes an unnecessary test. We've already tested it by the time we get here with no possiblity of it being changed. Signed-off-by:
-
Mark Hounschell authored
This patch removes a false comment Signed-off-by:
-
Mark Hounschell authored
The sniff buffer was a /proc filesystem file. This module no longer uses /proc. This patch removes the deaed code associated with the sniff buffer. Signed-off-by:
-
Mark Hounschell authored
This patch fixes a smatch warning about an unsigned integer being tested for less than zero. Signed-off-by:
-
Daeseok Youn authored
- alloc_tty_driver() is deprecated so it is changed to tty_alloc_driver() - Pointers which are allocated by alloc_tty_driver() and kzalloc() can be NULL so it need to check NULL for them. - If one of those is failed, it need to add proper handler for avoiding memory leak. - If both of drivers are registered normally, and then set TRUE to dgap_major_serial{print}_registered. If one of drivers is failed to register, leave a default value as FALSE. Signed-off-by:Daeseok Youn <daeseok.youn@gmail.com> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Vitaly Osipov authored
Replaced all uses of printk() in wlan-ng with netdev_err / _warn where a netdev exists. If a few cases where a netdev does not yet exist, dev_ or pr_ was used. Checkpatch complains about lines over 80 chars or split string constants - the messages are just too long, keeping it completely happy would make the code less readable. Signed-off-by:
Vitaly Osipov <vitaly.osipov@gmail.com> Signed-off-by:
-
Denis Pithon authored
Wherever possible, replaced printk() and pr_xxx() calls with netdev_xxx() / dev_xxx() calls; used pr_xxx() otherwise. Signed-off-by:
Denis Pithon <denis.pithon@gmail.com> Signed-off-by:
-
Denis Pithon authored
Gathered together comments in front of multicast filtering block. Signed-off-by:
-
Denis Pithon authored
Used is_multicast_ether_addr() to perform the checking. Signed-off-by:
-
Denis Pithon authored
Replaced generic memcmp() with dedicated ether_addr_equal_unaligned() call. I did not find any clue of u16 alignment for both addresses. Signed-off-by:
-
Denis Pithon authored
Renamed convert_frame_to_ether() to p80211_convert_to_ether(). Signed-off-by:
-
Paul Davies C authored
Some lines with more than 80 characters are converted in to multiple lines. Signed-off-by:
Paul Davies C <pauldaviesc@gmail.com> Signed-off-by:
-
Paul Davies C authored
Add blank lines after each declarations in drivers/staging/octeon-usb/octeon-hcd.c. Signed-off-by:
-
