linux-toradex.git/drivers/mfd/tps65090.c, branch v4.19-rc3 Linux kernel for Apalis and Colibri modules mfd: tps65090: Delete an error message for a failed memory allocation in tps65090_i2c_probe() 2018-05-16T08:21:48+00:00 Markus Elfring elfring@users.sourceforge.net 2018-03-08T14:25:58+00:00 8a232f0b0d26725e2e57a1fc6fca250089f6597d Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
Omit an extra message for a memory allocation failure in this function.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: tps65090: Set regmap config reg counts properly 2016-03-16T08:50:36+00:00 Maciej S. Szmigiero mail@maciej.szmigiero.name 2016-01-31T22:00:06+00:00 5c1488906f852473b4d7837c3e33e411e0eb6910 Regmap config max_register field should contain number of device last register, however num_reg_defaults_raw field should be set to register count instead (usually one register more than max_register). tps65090 driver had both of these fields set to the same value, fix this by introducing separate defines for max register number and total count of registers. Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
Regmap config max_register field should contain number of
device last register, however num_reg_defaults_raw field
should be set to register count instead
(usually one register more than max_register).

tps65090 driver had both of these fields set to the same value,
fix this by introducing separate defines for max register
number and total count of registers.

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: Drop owner assignment from i2c_drivers 2015-08-11T14:08:48+00:00 Krzysztof Kozlowski k.kozlowski@samsung.com 2015-07-10T05:05:40+00:00 0e777366fb0eba6facc44f0604fe6adbaf21d604 i2c_driver does not need to set an owner because i2c_register_driver() will set it. Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
i2c_driver does not need to set an owner because i2c_register_driver()
will set it.

Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: tps65090: Fix bonkers indenting strategy 2014-11-25T16:18:53+00:00 Lee Jones lee.jones@linaro.org 2014-11-10T12:28:36+00:00 12849b63a4e9e22fb63d0fc967726e8cdf2a19c2 First spotted pointless (incorrect) indent of 'if (ret)', then double indentations of a struct attribute 'mask'. Decided to go through the whole file and make amendments instead and this is the result. Signed-off-by: Lee Jones <lee.jones@linaro.org> 
First spotted pointless (incorrect) indent of 'if (ret)', then double
indentations of a struct attribute 'mask'.  Decided to go through the
whole file and make amendments instead and this is the result.

Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: tps65090: Stop caching most registers 2014-04-23T11:32:19+00:00 Doug Anderson dianders@chromium.org 2014-04-16T23:12:27+00:00 c42ba72ec3a7a1b6aa30122931f1f4b91b601c31 Nearly all of the registers in tps65090 combine control bits and status bits. Turn off caching of all registers except the select few that can be cached. In order to avoid adding more duplicate #defines, we also move some register offset definitions to the mfd driver (and resolve inconsistent names). Signed-off-by: Doug Anderson <dianders@chromium.org> Acked-by: Mark Brown <broonie@kernel.org> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
Nearly all of the registers in tps65090 combine control bits and
status bits.  Turn off caching of all registers except the select few
that can be cached.

In order to avoid adding more duplicate #defines, we also move some
register offset definitions to the mfd driver (and resolve
inconsistent names).

Signed-off-by: Doug Anderson <dianders@chromium.org>
Acked-by: Mark Brown <broonie@kernel.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: tps65090: Don't tell child devices we have an IRQ if we don't 2014-04-23T11:23:25+00:00 Doug Anderson dianders@chromium.org 2014-04-16T23:12:25+00:00 7d811771c95ebab358eca8e68b53efe09e3a6a96 If we weren't given an interrupt we shouldn't tell child devices (like the tps65090 charger) that they have an interrupt. This is needed so that we can support polling mode in the tps65090 charger driver. See also (charger: tps65090: Allow charger module to be used when no irq). Signed-off-by: Doug Anderson <dianders@chromium.org> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
If we weren't given an interrupt we shouldn't tell child devices (like
the tps65090 charger) that they have an interrupt.  This is needed so
that we can support polling mode in the tps65090 charger driver.

See also (charger: tps65090: Allow charger module to be used when no
irq).

Signed-off-by: Doug Anderson <dianders@chromium.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: ti: Constify struct mfd_cell where possible 2014-01-06T09:13:29+00:00 Geert Uytterhoeven geert@linux-m68k.org 2013-11-18T13:33:02+00:00 30fe2b5bd5cd732a329ef31062d9d96eee7d18e4 As of commit 03e361b25ee8dfb1fd9b890072c23c4aae01c6c7 ("mfd: Stop setting refcounting pointers in original mfd_cell arrays"), the "cell" parameter of mfd_add_devices() is "const" again. Hence make all cell data passed to mfd_add_devices() const where possible. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
As of commit 03e361b25ee8dfb1fd9b890072c23c4aae01c6c7 ("mfd: Stop setting
refcounting pointers in original mfd_cell arrays"), the "cell" parameter of
mfd_add_devices() is "const" again. Hence make all cell data passed to
mfd_add_devices() const where possible.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: Use dev_get_platdata() 2013-07-31T12:01:29+00:00 Jingoo Han jg1.han@samsung.com 2013-07-30T08:10:05+00:00 334a41ce9b753ec615e8c6c50ee07d6197190610 Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han <jg1.han@samsung.com> Signed-off-by: Lee Jones <lee.jones@linaro.org> 
Use the wrapper function for retrieving the platform data instead of
accessing dev->platform_data directly.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
mfd: tps65090: Add compatible string for charger subnode 2013-04-10T22:16:29+00:00 Rhyland Klein rklein@nvidia.com 2013-04-10T17:14:55+00:00 b50cf35bf41f8aa451667d8ff01e13b9ab17dc86 This patch adds the of_compatible value for the tps65090 charger subnode used when loading the driver to find its properties and of_node. Signed-off-by: Rhyland Klein <rklein@nvidia.com> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com> 
This patch adds the of_compatible value for the tps65090 charger
subnode used when loading the driver to find its properties
and of_node.

Signed-off-by: Rhyland Klein <rklein@nvidia.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
mfd: tps65090: Add resources for charger 2013-04-08T16:00:27+00:00 Rhyland Klein rklein@nvidia.com 2013-03-12T22:08:07+00:00 36c772e31fa72006e80d12d1a94db2a326ba00a5 Add irq resources to pass to the charger mfd sub dev so the charger can listen for interrupts. Signed-off-by: Rhyland Klein <rklein@nvidia.com> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com> 
Add irq resources to pass to the charger mfd sub dev so
the charger can listen for interrupts.

Signed-off-by: Rhyland Klein <rklein@nvidia.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>