- Sep 14, 2013
-
-
Sachin Kamat authored
'break' after return is redundant. Remove it. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
Jonathan Cameron <jic23@kernel.org>
-
Sachin Kamat authored
'break' after return is redundant. Remove it. Signed-off-by:
-
Sachin Kamat authored
'break' after return is redundant. Remove it. Signed-off-by:
-
Sachin Kamat authored
'break' after return is redundant. Remove it. Signed-off-by:
-
- Sep 12, 2013
-
-
Peter Meerwald authored
Signed-off-by:
Peter Meerwald <pmeerw@pmeerw.net> Signed-off-by:
-
Sachin Kamat authored
Updated with the current one. Signed-off-by:
-
- Sep 10, 2013
-
-
Xie XiuQi authored
Remove unnecessary work pending test before calling schedule_work(). It has been tested in queue_work_on() already. No functional changed. Signed-off-by:
Xie XiuQi <xiexiuqi@huawei.com> Reviewed-by:
Tejun Heo <tj@kernel.org> Acked-by:
Michael Hennerich <michael.hennerich@analog.com> Signed-off-by:
-
- Sep 08, 2013
-
-
Fabio Estevam authored
Add MODULE_ALIAS, so that auto module loading can work. Signed-off-by:
Fabio Estevam <fabio.estevam@freescale.com> Acked-by:
Marek Vasut <marex@denx.de> Signed-off-by:
-
Fabio Estevam authored
Fix the order of resources removal in the error path of mxs_lradc_probe() and also in mxs_lradc_remove(). Signed-off-by:
-
Jingoo Han authored
The usage of strict_strto*() is not preferred, because strict_strto*() is obsolete. Thus, kstrto*() should be used. Previously, there were only strict_strtol(), strict_strtoul(), strict_strtoull(), and strict_strtoll(). Thus, when converting to the variables, only long, unsigned long, unsigned long long, and long long can be used. However, kstrto*() provides various functions handling all types of variables. Therefore, the types of variables can be changed properly. Signed-off-by:
Jingoo Han <jg1.han@samsung.com> Signed-off-by:
-
Angelo Compagnucci authored
Sysfs access to all channels. Signed-off-by:
Angelo Compagnucci <angelo.compagnucci@gmail.com> Signed-off-by:
-
Sachin Kamat authored
Add a missing closing brace. Signed-off-by:
-
Sachin Kamat authored
Silences the following warning: drivers/staging/iio/light/tsl2x7x_core.c:553:70: warning: Using plain integer as NULL pointer Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes the code simple. While at it also fixed an uninitialized return with -EINVAL. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. This also fixes an error in return type during clk_get failure. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_iio_device_alloc makes code simpler. Signed-off-by:
-
Sachin Kamat authored
devm_* APIs are device managed and make code simpler. The memory leak was a lack of freeing the iio_dev structure in the remove function. Signed-off-by:
-
- Sep 01, 2013
-
-
Peter Meerwald authored
Signed-off-by:
-
- Aug 31, 2013
-
-
Matthijs Kooijman authored
Some of the defaults were missing or unclear. In particular, I suspect the defaults were documented assuming there were still module parameters and taking the default module parameters into account. Now, the defaults are the values that will get chosen when the params passed to dwc2_hcd_init are all -1. Signed-off-by:
Matthijs Kooijman <matthijs@stdin.nl> Acked-by:
Paul Zimmerman <paulz@synopsys.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Matthijs Kooijman authored
The HWCFG4 register stores the supported utmi width values (8, 16 or both). This commit reads that value and validates the configured value against that. If no (valid) value is given, the parameter defaulted to 8 bits previously. However, the documentation for dwc2_core_params_struct suggests that the default should have been 16. Also, the pci bindings explicitely set the value to 16, so this commit changes the default to 16 bits (if supported, 8 bits otherwise). With the default changed, the value set in pci.c is changed to -1 to make it autodetected as well. Signed-off-by:
-
Matthijs Kooijman authored
Before, the hwcfg registers were read at device init time, but interpreted at various parts in the code. This commit unpacks the hwcfg register values into a struct with properly labeled variables at init time, which makes all the other code using these values more consise and easier to read. Some values that were previously stored in the hsotg struct are now moved into this new struct as well. In addition to the hwcfg registers, the contents of some fifo size registers are also unpacked. The hwcfg registers are read-only, so they can be safely stored. The fifo size registers are read-write registers, but their power-on values are significant: they give the maximum depth of the fifo they describe. This commit mostly moves code, but also attempts to simplify some expressions from (val >> shift) & (mask >> shift) to (val & mask) >> shift. Finally, all of the parameters read from the hardware are debug printed after unpacking them, so a bunch of debug prints can be removed from other places. Signed-off-by:
-
Matthijs Kooijman authored
Bits 16-31 are reserved, so the old code just reads the whole register to get bits 0-15, assuming the reserved bits would be 0 (which seems true on current hardware, but who knows...). This commit properly masks out the reserved bits when reading and doesn't touch the reserved bits while writing. Signed-off-by:
-
Matthijs Kooijman authored
For calculating FIFO offsets, the sizes of preceding fifos need to be known. For filling the GDFIFOCFG register, these fifo sizes were read from hardware registers. However, these values were written to these registers just a few lines before, so we can just use the values written instead. Signed-off-by:
-
Matthijs Kooijman authored
For some reason, the value of the HPTXFSIZ register was built in the ptxfsiz variable, while there was also a hptxfsiz variable availble. Better just use that and remove the (now unused) ptxfsiz variable. Signed-off-by:
-
Matthijs Kooijman authored
The value of the hcchar register is built from individual values by shifting and masking. Before, the debug output extracted the individual values out of the complete hcchar register again by doing the reverse. This commit makes the debug output use the original values instead. One debug message got removed, since it would always print a fixed value of zero. Signed-off-by:
-
