- 09 6月, 2015 40 次提交
-
-
由 Mateusz Kulikowski 提交于
Remove cmpk_message_handle_rx() and static functions used by it. This function was never called. Signed-off-by: NMateusz Kulikowski <mateusz.kulikowski@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Mateusz Kulikowski 提交于
Remove cmpk_handle_query_config_rx function. It is called once, does some calculation, but generates no output - it has neither return value, nor modify its parameters. Signed-off-by: NMateusz Kulikowski <mateusz.kulikowski@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Mateusz Kulikowski 提交于
Delete unused code. Removed some macros, enums, unions and several structures (tx_fwinfo, wmm_tspec, qos_tstream, sta_qos, bss_qos). Signed-off-by: NMateusz Kulikowski <mateusz.kulikowski@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Pedro Marzo Perez 提交于
Remove two lines at ieee80211_wep_null which checkpatch.pl reported as errors. The first one because it has a C99 comment style and the second one because it is a void return which is useless. The function ieee80211_wep_null cannot be completely removed because it is exported and used to autoload the module. Signed-off-by: NPedro Marzo Perez <marzo.pedro@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Pedro Marzo Perez 提交于
Simplify prism2_wep_init error check code employing goto when a failure is detected. Removed pr_debug which was given a checkpatch.pl error because of literal string splitted across two lines of code, it was seldom going to be printed anyway. Signed-off-by: NPedro Marzo Perez <marzo.pedro@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Gaston Gonzalez 提交于
Fix the following sparse warnings: drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: warning: incorrect type in assignment (different base types) drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: expected restricted __le16 [usertype] frame_ctl drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:663:32: got int drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: warning: invalid assignment: |= drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: left side has type restricted __le16 drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:664:50: right side has type int Signed-off-by: NGaston Gonzalez <gascoar@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Tolga Ceylan 提交于
auth_parse() return result is in incorrect le16 byte order. Currently this still works since the user code merely checks if error code is equal to 0. However debug statement in ieee80211_check_auth_response() prints the error code in the incorrect le16 byte order. This fix corrects the byte order as cpu order. Signed-off-by: NTolga Ceylan <tolga.ceylan@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Dan Carpenter 提交于
The underflow in OZ_DATA_F_ISOC_FIXED seems not harmful, but this patch is a clean up and makes my static checker a bit happier. The underflow in OZ_VENDOR_CLASS_RSP seems like it could result in memory corruption. Signed-off-by: NDan Carpenter <dan.carpenter@oracle.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Buţiu Alexandru Octavian 提交于
Fixed coding style issue line "over 80 characters" detected by checkpatch.pl in r819xU_firmware.h Signed-off-by: NBuţiu Alexandru Octavian <predator5047@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Buţiu Alexandru Octavian 提交于
Removed commented macro that was no longer needed Signed-off-by: NBuţiu Alexandru Octavian <predator5047@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Use a table to get a channel plan from a given country code.This was a TODO mentioned as a comment in the driver. Signed-off-by: NHari Prasath Gujulan Elango <hgujulan@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Yijing Wang 提交于
We already have a inline pci_get_bus_and_slot() in include/linux/pci.h, Use it instead of local one. Signed-off-by: NYijing Wang <wangyijing@huawei.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Prasanna Karthik 提交于
Fix reported by coccinelle compressing last two lines with single return call Signed-off-by: NPrasanna Karthik <mkarthi3@visteon.com> Acked-by: NLarry Finger <Larry.Finger@lwfinger.net> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 David Kershner 提交于
When the bus device was created the list_all variables were not being initialized. When the CONTROLVM_BUS_CONFIGURE message was being sent, it was failing to find the bus and produced a panic. Initialize the bus_info->list_all variable by doing a INIT_LIST_HEAD. Signed-off-by: NDavid Kershner <david.kershner@unisys.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
When unloading a module, we need to cleanup the platform registration. However, unregistering the platform uncovered a couple of quirks, namely a missing device_release function. Fix things up so module unload works and allows us to reload the module. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NDavid Kershner <david.kershner@unisys.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
When we combined visorchipset and visorbus into one driver we negelected to strip out some of the MODULE_ stuff from one of the files. When building the drivers in, it causes a WARN that we try to create /proc/modules/visorbus/version when it is already created. visorchipset.c is the driver entry point, remove the cruft from visorbus_main.c. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NDavid Kershner <david.kershner@unisys.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 David Kershner 提交于
Get rid of unused values in the enum. Signed-off-by: NDavid Kershner <david.kershner@unisys.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 David Kershner 提交于
Diagchannel.h is used primarily for the diagnostics channel. The diagnostics channel is not being used by linux guests currently, so the majority of the file is not needed. What is left is what is needed to perform postcode vmcalls. Those postcodes will eventually end up in the diag channel. Signed-off-by: NDavid Kershner <david.kershner@unisys.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 David Kershner 提交于
Give the enum the correct values instead of based on other values. Signed-off-by: NDavid Kershner <david.kershner@unisys.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Jes Sorensen 提交于
min_size/max_size aren't used anywhere, and they were just causing headaches in the drivers being ported over to the new interfaces. Signed-off-by: NJes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
This patch removes the legacy dev_info struct and instead creates and passes around a traditional struct device. This allows us to remove a lot of the various look up code and removes the doubt if the struct exists or not. Half of the churn is just the conversion of visorchipset_device_info to visor_device. Various cleanups include re-arranging the failure paths to make more sense. Pay attention to the create_visor_device function. This had a lot of churn to simplify everything. Lots of functions disappeared because they are not needed any more. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
This patch removes the legacy bus_info struct and instead creates and passes around a traditional struct device. This allows us to remove a lot of the various look up code and removes the doubt if the struct exists or not. Half of the churn is just the conversion of visorchipset_bus_info to visor_device. Various cleanups include re-arranging the failure paths to make more sense. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
I used 0 as the device id for the bus root, neglecting the fact that device 0 is a valid id in Unisys's configuration. Modify this to use UINT_MAX instead as a unique number. As fallout from this change it was noticed the bus_no and dev_no was not defined the same way consistently. Fix visorbus.h to use u32 there. Fix the resulting printk warning too. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
The bus driver doesn't work in server mode, just remove the left over pieces. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
Removing stuff that isn't being used. Another prepartion patch to allow us to use visor_device everywhere without the baggage of bus/dev_info. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Don Zickus 提交于
The bus device and regular device were using two different structs. Let's combine them as they are not entirely different from one another. This allows us to move this creation up the stack later and actually remove bus/dev_info easily. Most of the churn is just renaming devdata -> dev and 'struct visorbus_devdata' to 'struct visor_device'. Signed-off-by: NDon Zickus <dzickus@redhat.com> Signed-off-by: NBenjamin Romer <benjamin.romer@unisys.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Chaitanya Dhere 提交于
This change was detected with help of checkpatch.pl script. Signed-off-by: NChaitanya Dhere <cvijaydh@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Chaitanya Dhere 提交于
In this patch, the if, else conditions are modified to remove the unnecessary equality checks. This change was detected with help of coccinelle tool. Signed-off-by: NChaitanya Dhere <cvijaydh@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Style (line over 80 chars) in drivers/staging/wlan-ng/prism2fw.c Signed-off-by: NDouglas Barbonaglia Sathler Figueiredo <eng.douglasfigueiredo@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Madhusudhanan Ravindran 提交于
removed the commented INFO lines. Signed-off-by: NMadhusudhanan Ravindran <mravindr@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Fixed indentation issue in few lines Signed-off-by: NHari Prasath Gujulan Elango <hgujulan@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
remove an unwanted assignment to a variable which is overwritten in the very next line.The first value assigned is not used. Signed-off-by: NHari Prasath Gujulan Elango <hgujulan@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Remove the ununsed Macro Signed-off-by: NHari Prasath Gujulan Elango <hgujulan@visteon.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_display.c: ERROR: space required after that ',' Signed-off-by: NIsaac Assegai <isaac.a.travers@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_help.c and ddk750_mode.c: ERROR: space required after that ',' Signed-off-by: NIsaac Assegai <isaac.a.travers@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_power.c and ddk750_swi2c.c: ERROR: space required after that ',' Signed-off-by: Isaac Assegai <isaac.a.travers@gmail.com>: Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in sm750_cursor.c: ERROR: space required after that ',' Signed-off-by: NIsaac Assegai <isaac.a.travers@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in sm750_accel.c: ERROR: space required after that ',' Signed-off-by: NIsaac Assegai <isaac.a.travers@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in sm750_hw.c: ERROR: space required after that ',' Signed-off-by: NIsaac Assegai <isaac.a.travers@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-
由 Isaac Assegai 提交于
Insert Spaces after commas to rectify the following checkpatch errors in ddk750_hwi2c.h, ddk750_help.h and ddk750_chip.h: ERROR: space required after that ',' Signed-off-by: NIsaac Assegai <isaac.a.travers@gmail.com> Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
-