- 16 Sep, 2011 10 commits
-
-
K. Y. Srinivasan authored
Now, eliminate the usage of ext field in struct hv_device for netvsc driver. We do this by registering pointer to struct netvsc_device as the driver specific data and eliminating the current usage of driver specific data to save and retrieve the pointer to struct net_device. Additionally, all access to the driver specific data is through the vmbus wrapper functions. As part of this cleanup, we also get rid of some unnecessary debug print statements. Signed-off-by:
K. Y. Srinivasan <kys@microsoft.com> Signed-off-by:
Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
K. Y. Srinivasan authored
Now, eliminate the usage of ext field in struct hv_device for storvsc driver. We do this by registering pointer to struct storvsc_device as the driver specific data and eliminating the current usage of driver specific data to save and retrieve the pointer to struct Scsi_Host. Additionally, all access to the driver specific data is through the vmbus wrapper functions. Note that function to allocate the host gives us a reference on the host object. Signed-off-by:
-
K. Y. Srinivasan authored
Now, get rid of some unnecessary state and definitions. Signed-off-by:
-
K. Y. Srinivasan authored
Get rid of alloc_stor_device() by inlining the code. Signed-off-by:
-
K. Y. Srinivasan authored
Get rid of storvsc_dev_add() by inlining the code. Signed-off-by:
-
K. Y. Srinivasan authored
vmbus_remove() cannot fail; clean it up accordingly. Signed-off-by:
-
K. Y. Srinivasan authored
Introduce vmbus specific wrapper functions to set/get driver specific data. Signed-off-by:
-
K. Y. Srinivasan authored
Use the driver_data to identify ide devices in storvsc_probe(). Signed-off-by:
-
K. Y. Srinivasan authored
In preparation to leveraging the driver_data field in struct hv_vmbus_device_id, change the signature of struct hv_driver probe function. Signed-off-by:
-
K. Y. Srinivasan authored
Introduce a utility function to match hv_vmbus_device_id. Signed-off-by:
-
- 12 Sep, 2011 30 commits
-
-
Mark Brown authored
Signed-off-by:
Lu Guanqun <guanqun.lu@intel.com> Cc: Vinod Koul <vinod.koul@intel.com> Acked-by:
Vinod Koul <vinod.koul@intel.com> Acked-by:
Liam Girdwood <lrg@ti.com> Acked-by:
Mark Brown <broonie@opensource.wolfsonmicro.com> Signed-off-by:
-
Franky Lin authored
Reshuffle function order in wl_cfg80211 of fullmac to get rid of static function declaration Reported-by:
Johannes Berg <johannes@sipsolutions.net> Reviewed-by:
Arend van Spriel <arend@broadcom.com> Reviewed-by:
Roland Vossen <rvossen@broadcom.com> Signed-off-by:
-
Roland Vossen authored
Alloc.c does not contain enough functionality anymore to warrant its own file. Moved functionality into main.c Reported-by:
-
Roland Vossen authored
Members were always set to zero, or set but never read. Reviewed-by:
-
Roland Vossen authored
Struct member variables were only set to '0'. As a result, function brcms_c_local_constraint_qdbm() always returned BRCMS_TXPWR_MAX and thus could be removed. Reviewed-by:
-
Roland Vossen authored
Members were always set to zero, or set but never read. Signed-off-by:
-
Roland Vossen authored
Nothing useful was done with this struct. Signed-off-by:
-
Roland Vossen authored
Since the macro's IS_MCS(0), RSPEC_STF(0) and PHY_TXC1_MODE_SISO all expand to 0, inactive code could be removed. Signed-off-by:
-
Roland Vossen authored
This was unused. Signed-off-by:
-
Roland Vossen authored
Struct members were unused or only written but never read. Signed-off-by:
-
Roland Vossen authored
Removed unused mechanism as part of code cleanup. Signed-off-by:
-
Roland Vossen authored
Softmac relies on Mac80211 for this. Signed-off-by:
-
Arend van Spriel authored
The source file contained two static variable which were used by the driver instance. This makes having multiple instances of the driver on one system impossible. This patch removes the use of these static variables allowing multiple driver instances. Reported-by:
Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by:
-
Arend van Spriel authored
The static local variable save_ms in brcmf_sdbrcm_wd_timer() has been removed and placed in the structure brcmf_bus definition. Reported-by:
-
Roland Vossen authored
brcmf_sdcard_iovar_op was only called with iovar "sd_rxchain", and always returns 'false' as the queried value. Thus, the entire function, related iovar tables and related functions could be removed. Reported-by:
-
Roland Vossen authored
Linux will never call brcmf_netdev_ioctl_entry() with the value BRCMF_IOCTL_MAGIC in a user space buffer. Thus, unused code could be removed. Reported-by:
-
Roland Vossen authored
Substituted/moved/removed macro's. Reported-by:
-
Roland Vossen authored
Substituted/removed macro's. Reported-by:
-
Roland Vossen authored
Substituted/removed macro's. Reported-by:
-
Roland Vossen authored
Substituted/removed macro's. Reported-by:
-
Franky Lin authored
Use work queue instead. Reported-by:
Howard Harte <hharte@broadcom.com> Signed-off-by:
-
Aaro Koskinen authored
Delete redundant casts from mtrr_add() call. Signed-off-by:
Aaro Koskinen <aaro.koskinen@iki.fi> Signed-off-by:
-
Aaro Koskinen authored
The condition for printk() is wrong. Also change it to dev_info(). Signed-off-by:
-
Aaro Koskinen authored
MTRR region should be deleted on probe failure and driver removal. Signed-off-by:
-
Arvydas Sidorenko authored
crystalhd_fw_if.h: indentation fix (spaces to tabs) The rest are brackets. NOTE: there are quite some 80 character warnings, but they look in place, comments mostly on the right next to the constants and stuff like that. I haven't touched them, since this rule is 'going away', but in case you would like it to be fixed, let me know. I'm sending 2 patches, but it they aren't connected in any way, so doesn't matter the apply order. It's just a cleanup. P.S sorry for the first lame patches a couple of days ago Signed-off-by:
Arvydas Sidorenko <asido4@gmail.com> Signed-off-by:
-
Leonid V. Fedorenchik authored
Fix long lines in cx25821-video-upstream.h Signed-off-by:
Leonid V. Fedorenchik <leonidsbox@gmail.com> Reviewed-By:
Valdis Kletnieks <valdis.kletnieks@vt.edu> Reviewed-by:
Dan Carpenter <error27@gmail.com> Signed-off-by:
-
Leonid V. Fedorenchik authored
Fix long lines in cx25821-video-upstream.c Signed-off-by:
-
Leonid V. Fedorenchik authored
Fix long lines in cx25821-video-upstream-ch2.h Signed-off-by:
-
Leonid V. Fedorenchik authored
Fix long lines in cx25821-video-upstream-ch2.c Signed-off-by:
-
Leonid V. Fedorenchik authored
Change obscure line endings to less obscure ones. Rationale: improve readability. Signed-off-by:
-
