- 31 Aug, 2010 40 commits
-
-
Roberto Rodriguez Alkala authored
In today linux-next I got a compile warning in staging/otus driver. This patch solves the issue and also improves the coding style. Signed-off-by:
Roberto Rodriguez Alcala <rralcala@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Denis Kirjanov authored
Remove explicit arch dependencies Signed-off-by:
Denis Kirjanov <dkirjanov@kernel.org> Signed-off-by:
-
Manuel Stahl authored
this one completes my last patch set to compile fine for all drivers. Signed-off-by:
Manuel Stahl <manuel.stahl@iis.fraunhofer.de> Signed-off-by:
Jonathan Cameron <jic23@cam.ac.uk> Signed-off-by:
Datta, Shubhrajyoti <shubhrajyoti@ti.com> Signed-off-by:
-
Manuel Stahl authored
g -> ms/s^2 deg -> rad deg/s -> rad/s some temperatures still broken Signed-off-by:
-
Manuel Stahl authored
Signed-off-by:
-
Manuel Stahl authored
Signed-off-by:
-
Manuel Stahl authored
tested with sca3000, adis16400 Signed-off-by:
-
Manuel Stahl authored
Now fits other driver's structure for easier modification. Signed-off-by:
-
Manuel Stahl authored
bps -> bytes_per_datum ring_enable -> enable Signed-off-by:
-
Dan Carpenter authored
i2c_smbus_read_byte_data() returns an s32 type. We need to change "rate" to signed for the error handling to work. Also I changed it to propogate the error code instead of just returning -EINVAL. Other error codes could be -EAGAIN for example. Signed-off-by:
Dan Carpenter <error27@gmail.com> Acked-by:
-
Julia Lawall authored
Adjust the error handling code so that it benefits from the call to mutex_unlock at the end of the function. The semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @rcu exists@ position p1; expression E; @@ mutex_lock@p1(E); ... mutex_unlock(E); @exists@ position rcu.p1; expression E; @@ *mutex_lock@p1(E); ... when != mutex_unlock(E); ?*return ...; // </smpl> Signed-off-by:
Julia Lawall <julia@diku.dk> Acked-by:
-
Dan Carpenter authored
The first kfree(pDevice) is pointless because pDevice is NULL. The second kfree(pDevice) is a double free because pDevice is the driver's private data and that is already freed by free_netdev(netdev). Also the free_netdev() error path doesn't call usb_put_dev(). Signed-off-by:
-
Luis Bosch authored
Added spaces in for loop arguments. Signed-off-by:
Luis Bosch <luis.bosch@gmail.com> Signed-off-by:
-
Victor Rosales authored
Removed spaces between functions name and parenthesis. Signed-off-by:
Victor Rosales <victorhrosales@gmail.com> Signed-off-by:
-
Alejandro Emanuel Paredes authored
resolved checkpatch finding Signed-off-by:
Alejandro Emanuel Paredes <aleparedes@gmail.com> Signed-off-by:
-
Andres More authored
Removed custom macro used to test bits. Signed-off-by:
Andres More <more.andres@gmail.com> Signed-off-by:
-
Andres More authored
Removed int redefinition for function results. Signed-off-by:
-
Andres More authored
Removed some unused definitions of Ethernet packet types, also replaced two of them with in-kernel counterparts from include/linux/if_ether.h Signed-off-by:
-
Ivan Gomez Castellanos authored
The services_init() and services_exit() functions don't do anything, so they are removed, and as these are the only two functions defined in services.c and services.h, then these files are also removed. Signed-off-by:
Ivan Gomez Castellanos <ivan.gomez@ti.com> Signed-off-by:
-
Ivan Gomez Castellanos authored
So far, the cfg.c file is empty, and the function prototypes in cfg.h are not used in any place. So they can be removed. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, the cfg_set_object function has also to be removed. Since the driver object handle is retrieved from the drv_data structure, then the word "Registry" is replaced by "driver data" in the comments. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, the cfg_set_dev_object function has also to be removed. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, the cfg_get_object function has also to be removed. Since the driver object handle is retrieved from the drv_data structure, then the word "Registry" is replaced by "driver data" in the comments. Also, the hdrv_obj is not used in function omap34_xx_bridge_remove(), so it is removed. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, the cfg_get_exec_file function has also to be removed. This patch also avoids a possible NULL pointer dereference in function cfg_get_exec_file(), when drv_datap is checked for NULL and then pass drv_datap->base_img as argument to strlen(). Signed-off-by:
-
Ivan Gomez Castellanos authored
The cfg_get_dev_object function is only used in one place and because of its simplicity, it can be removed. The parameter *value can be left uninitialized if the strcmp() returns a nonzero value, so in the function dev_remove_device(), the hdev_obj could be used uninitialized and could be dereferenced in dev_destroy_device(). This patch fixes this issue, and also removes the dev_obj pointer which is not used. Signed-off-by:
-
Ivan Gomez Castellanos authored
The functions cfg_init() and cfg_exit() do nothing, so they are removed. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, this patch is needed. The function cfg_get_auto_start() only assigns a value to the flag tmp, and it is only called by the function api_init_complete2(). So the function cfg_get_auto_start() is not required. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, this patch is needed. ntfy.c only contained the function dsp_notifier_event(), and since this function calls sync_set_event(), then the dsp_notifier_event() is moved to the sync.c file. Signed-off-by:
-
Ivan Gomez Castellanos authored
As the services directory is going to be removed, the file sync.c is moved from services to core. Signed-off-by:
-
Andrea Gelmini authored
Signed-off-by:
Andrea Gelmini <andrea.gelmini@gelma.net> Signed-off-by:
-
Andres More authored
Removed comments about who changed/added lines, they do not seem useful. Signed-off-by:
-
Kulikov Vasiliy authored
IRQ and resource[] may not have correct values until after PCI hotplug setup occurs at pci_enable_device() time. The semantic match that finds this problem is as follows: // <smpl> @@ identifier x; identifier request ~= "pci_request.*|pci_resource.*"; @@ ( * x->irq | * x->resource | * request(x, ...) ) ... *pci_enable_device(x) // </smpl> Signed-off-by:
Kulikov Vasiliy <segooon@gmail.com> Signed-off-by:
-
Andres More authored
Cleared checkpatch ERROR: spaces required around that '||' Signed-off-by:
-
Julia Lawall authored
list_for_each_entry binds its first argument to a non-null value, and thus any null test on the value of that argument is superfluous. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ iterator I; expression x; statement S; @@ I(x,...) { <... - if (x != NULL || ...) S ...> } // </smpl> Signed-off-by:
-
Jacob Smith authored
This patch fixes a whitespace issue detected by checkpatch.pl Signed-off-by:
Jacob Smith <tarinaky@gmail.com> Signed-off-by:
-
Larry Finger authored
Signed-off-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Larry Finger authored
Signed-off-by:
Albert Wang <albert_wang@realtek.com.tw> Signed-off-by:
-
Larry Finger authored
When the driver received the Addba request frame from AP, a crash could occur. Signed-off-by:
-
Larry Finger authored
Signed-off-by:
-
Larry Finger authored
There are some IOT issues when 802.11bgn mode is used to make a WEP connection. Signed-off-by:
-
