- 03 Oct, 2013 14 commits
-
-
Matthijs Kooijman authored
The hardware offers a 4-bit register containing the number of host channels. However, the values of these register mean 1-16 host channels, not 0-15. Since the dwc2_hw_params struct stores the actual number of host channels supported instead of the raw register value, it should be 5 bits wide instead of 4. Before this commit, hardware with 16 host channels would overflow the field, making it appear as 0 channels. This bug was introduced in commit 9badec2f (staging: dwc2: interpret all hwcfg and related register at init time). Reported-by:
Dinh Nguyen <dinguyen@altera.com> Signed-off-by:
Matthijs Kooijman <matthijs@stdin.nl> Acked-by:
Paul Zimmerman <paulz@synopsys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shalin Mehta authored
Fixed warnings in all of three files where the string was initilized with an integer instead of NULL Signed-off-by:
Shalin Mehta <shalinmehta85@gmail.com> Signed-off-by:
-
Aaro Koskinen authored
Make comments to fit into 80 columns. Signed-off-by:
Aaro Koskinen <aaro.koskinen@iki.fi> Signed-off-by:
-
Aaro Koskinen authored
Make comments to fit into 80 columns. Signed-off-by:
-
Aaro Koskinen authored
Delete unused cvmx_read_csr / cvmx_write_csr macros. Signed-off-by:
-
Aaro Koskinen authored
Eliminate CVMX_PREFETCH_PREFX. Signed-off-by:
-
Aaro Koskinen authored
Eliminate CVMX_PREFETCH_PREF0. Signed-off-by:
-
Aaro Koskinen authored
Delete unused macro. Signed-off-by:
-
Aaro Koskinen authored
Inline CVMX_PREFETCH0 macro. Signed-off-by:
-
Aaro Koskinen authored
Replace CVMX_CLZ macro with __fls(). Signed-off-by:
-
Aaro Koskinen authored
Delete redudant flags parameter. Signed-off-by:
-
Aaro Koskinen authored
Delete redundant flags parameter. Signed-off-by:
-
Aaro Koskinen authored
Delete unused flags. Signed-off-by:
-
Aaro Koskinen authored
The function will always figure out the used clock internally, so delete a redudant parameter and the flag. Signed-off-by:
-
- 02 Oct, 2013 2 commits
-
-
Tugce Sirin authored
Signed-off-by:
Tugce Sirin <ztugcesirin@gmail.com> Reviewed-by:
Josh Triplett <josh@joshtriplett.org> Signed-off-by:
-
Tugce Sirin authored
This patch fixes space after if statement in nvec/nvec.c Signed-off-by:
-
- 01 Oct, 2013 24 commits
-
-
Greg Kroah-Hartman authored
Turns out that LL_MRF_RETURN is "void" so loop_make_request() would fall through to the error path for a successful call. Either this never matters, or the error path really doesn't do anything. Fix it up to do what the code is written to look like it is doing, if that really is what it should be doing is another story... Cc: Peng Tao <tao.peng@emc.com> Cc: Zhao Hongjiang <zhaohongjiang@huawei.com> Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by: -
Greg Kroah-Hartman authored
It's just 'void' and only used in one place, so just remove it, it's useless. Cc: Peng Tao <tao.peng@emc.com> Cc: Zhao Hongjiang <zhaohongjiang@huawei.com> Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by: -
Sachin Kamat authored
Return negative error codes as is followed in the rest of the kernel. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
-
Sachin Kamat authored
Return negative error codes as is followed in the rest of the kernel. Signed-off-by:
-
Sachin Kamat authored
Return negative error codes as is followed in the rest of the kernel. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Functions returning pointer should return NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
Pointers should be assigned NULL instead of 0. Signed-off-by:
-
Sachin Kamat authored
This check is not required. Signed-off-by:
Lidza Louina <lidza.louina@gmail.com> Signed-off-by:
-
Sachin Kamat authored
'imx_ldb_dt_ids' is always compiled in. Hence of_match_ptr() is not needed. Signed-off-by:
-
Martin Berglund authored
Fixed coding style issues. Signed-off-by:
Martin Berglund <martin@rogsta.net> Signed-off-by:
-
Randy Dunlap authored
Fix build errors when GPIO_CS5535=m and FB_OLPC_DCON=y by preventing that kconfig combination. These build errors are caused by having a kconfig bool symbol (FB_OLPC_DCON_1) that depend on a tristate symbol (GPIO_CS5535), but when the tristate symbol is =m, the bool symbol is =y. drivers/built-in.o: In function `dcon_read_status_xo_1': olpc_dcon_xo_1.c:(.text+0x359531): undefined reference to `cs5535_gpio_set' drivers/built-in.o: In function `dcon_wiggle_xo_1': olpc_dcon_xo_1.c:(.text+0x35959f): undefined reference to `cs5535_gpio_set' olpc_dcon_xo_1.c:(.text+0x359610): undefined reference to `cs5535_gpio_clear' drivers/built-in.o:olpc_dcon_xo_1.c:(.text+0x3596a1): more undefined references to `cs5535_gpio_clear' follow drivers/built-in.o: In function `dcon_wiggle_xo_1': olpc_dcon_xo_1.c:(.text+0x359708): undefined reference to `cs5535_gpio_set' drivers/built-in.o: In function `dcon_init_xo_1': olpc_dcon_xo_1.c:(.text+0x35989b): undefined reference to `cs5535_gpio_clear' olpc_dcon_xo_1.c:(.text+0x3598b5): undefined reference to `cs5535_gpio_isset' olpc_dcon_xo_1.c:(.text+0x359963): undefined reference to `cs5535_gpio_setup_event' olpc_dcon_xo_1.c:(.text+0x359980): undefined reference to `cs5535_gpio_set_irq' olpc_dcon_xo_1.c:(.text+0x359a36): undefined reference to `cs5535_gpio_set' Signed-off-by:
Randy Dunlap <rdunlap@infradead.org> Cc: Daniel Drake <dsd@laptop.org> Cc: Jens Frederich <jfrederich@gmail.com> Signed-off-by:
-
Dan Carpenter authored
We knew "peventbuf" was a valid pointer and "peventbuf + 2" is also non-NULL. I have removed the check. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
Larry Finger <Larry.Finger@lwfinger.net> Signed-off-by:
-
Adam Rall authored
Fixed checkpatch.pl errors in r8180_dm.c. Signed-off-by:
Adam Rall <adam.rall4@gmail.com> Signed-off-by:
-
Jesper Juhl authored
Make sure we always free(param); and remove a redundant "goto out;" just before we'll hit the label anyway. Signed-off-by:
Jesper Juhl <jj@chaosbits.net> Signed-off-by:
-
Martin Berglund authored
Cleanup of a few brace coding style issues. Signed-off-by:
-
Malcolm Priestley authored
Tasklet to workqueue. RxMngWorkItem -> rx_mng_work_item Reduce atomic area of driver and dependency on system timer. Signed-off-by:
Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by:
-
Malcolm Priestley authored
Tasklet to workqueue. ReadWorkItem -> read_work_item Reduce atomic area of driver. Signed-off-by:
-
Malcolm Priestley authored
timer to delay workqueue. sTimerSecondCallback -> second_callback_work The delayed work queue is declared in device.h This timer is very heavy on the system. Improves over performance of driver and reduce the atomic area of driver. Signed-off-by:
-
