- 21 Jan, 2014 40 commits
-
-
Sachin Kamat authored
'lp3943_gpio_of_match' is always compiled in. Hence the helper macro is not needed. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Acked-by:
Linus Walleij <linus.walleij@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Julia Lawall authored
If nothing more than to improve code readability. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression pdev; @@ pci_set_power_state(pdev, - 0 + PCI_D0 ) // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Sachin Kamat authored
LDO indices start from 1. Fix the example appropriately. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Sachin Kamat authored
Commit a1fd844c ("ARM: sa1100: move platform_data definitions") moved the file to the current location but forgot to remove the pointer to its previous location. Clean it up. While at it also change the header file protection macros appropriately. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Manish Badarkhe authored
Instead of "#if define CONFIG_OF" use "IS_ENABLED(CONFIG_OF)" option for DT code to avoid if-deffery in code. Signed-off-by:
Manish Badarkhe <badarkhe.manish@gmail.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Danke Xie authored
The current TWL 6030 IRQ handler assumes little endianness. This change makes it endian-neutral. Signed-off-by:
Danke Xie <d.xie@sta.samsung.com> Signed-off-by:
Taras Kondratiuk <taras.kondratiuk@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Tushar Behera authored
S5M8767 chip has 3 crystal oscillators running at 32KHz. These are supported by s2mps11-clk driver. Signed-off-by:
Tushar Behera <tushar.behera@linaro.org> Reviewed-by:
Tomasz Figa <t.figa@samsung.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Sachin Kamat authored
'max14577_dt_match' is always compiled in. Hence the helper macro is not needed. Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Wei Yongjun authored
Fixes the following sparse warning: drivers/mfd/twl6040.c:89:20: warning: symbol 'twl6040_patch' was not declared. Should it be static? Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Lee Jones authored
This reverts commit 68044bee13770918e0b28dd44aa98c889ec7558f. We've had confirmed reports of this patch causing unforeseen issues with existing MFD users. It has been agreed by the original author and myself that reversion is the best solution. Acked-by:
Stephen Warren <swarren@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Wei Yongjun authored
Fixes the following sparse warning: drivers/mfd/rtl8411.c:473:6: warning: symbol 'rtl8411_init_common_params' was not declared. Should it be static? Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Krzysztof Kozlowski authored
The MUIC block in max77693 has different I2C address than PMIC. The driver allocated two regmaps: for PMIC and MUIC. However it used the same regmap_config (with max_register field) for both regmaps. Actual maximum address of register for MUIC is different than for PMIC. Define another regmap_config for MUIC with proper max_register value. Signed-off-by:
Krzysztof Kozlowski <k.kozlowski@samsung.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Krzysztof Kozlowski authored
The regmap used by max77686 MFD driver was not freed with regmap_exit() on driver exit. This lead to leak of resources. Replace regmap_init_i2c() call in driver probe with initialization of managed register map so the regmap will be properly freed by the device management code. Cc: stable@vger.kernel.org Signed-off-by:
Krzysztof Kozlowski <k.kozlowski@samsung.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Laszlo Papp authored
The original author(s) probably copy/pasted these headers from the existing public header files. Signed-off-by:
Laszlo Papp <lpapp@kde.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Micky Ching authored
rtl8402 is much like rtl8411, so just add it to rtl8411.c Signed-off-by:
Micky Ching <micky_ching@realsil.com.cn> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Micky Ching authored
Add set pull control macro to reduce code for setting pull control, and use a common init function to reduce code for rtl8411.c. So this patch is used to just simplify code. Signed-off-by:
Micky Ching <micky_ching@realsil.com.cn> Reviewed-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Laszlo Papp authored
The original author provided a random return value check which is redundant and seemingly floating. This patch not only relocates the check so it is more clearly associated with the invokation of mfd_add_devices(), but provides a store for the error value. We also print a meaningful message on error before returning. Signed-off-by:
Laszlo Papp <lpapp@kde.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Alexander Shiyan authored
This patch simplifies probe() and remove() functions by moving some initialisation code out from the I2C/SPI init() and exit() functions and into the core driver. Signed-off-by:
Alexander Shiyan <shc_work@mail.ru> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Alexander Shiyan authored
Symbol MFD_MC13783 always selected by MFD_MC13XXX, so no need to keep additional symbol. Signed-off-by:
Alexander Shiyan <shc_work@mail.ru> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Alexander Shiyan authored
mc13xxx_get_flags() declaration given twice. This patch removes this duplicate. Signed-off-by:
Alexander Shiyan <shc_work@mail.ru> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Stephen Boyd authored
This is a read-only data structure. Signed-off-by:
Stephen Boyd <sboyd@codeaurora.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Stephen Boyd authored
In preparation for passing a const pointer directly to ssbi_write() from the regmap APIs. Signed-off-by:
Stephen Boyd <sboyd@codeaurora.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Stephen Boyd authored
The ssbi driver assumes that the device is DT based. Remove the platform data structs that will never be used and hide the enum in the only C file that uses it. Signed-off-by:
Stephen Boyd <sboyd@codeaurora.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Stephen Warren authored
mfd_add_device() assigns .of_node in the device objects it creates only if the mfd_cell for the device has the .of_compatible field set and the DT node for the top-level MFD device contains a child whose compatible property matches the cell's .of_compatible field. This leaves .of_node unset in many cases. When this happens, entries in the DT /aliases property which refer to the top-level MFD DT node will never match the MFD child devices, hence causing the requested alias not to be honored. Solve this by setting each MFD child device's .of_node equal to the top- level MFD device's .of_node field in the cases where it would otherwise remain unset. The first use-case for this will be aliases for the TPS6586x's RTC device. Signed-off-by:
Stephen Warren <swarren@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Rhyland Klein authored
The EC has specific timing it requires. Add support for an optional delay after raising CS to fix timing issues. This is configurable based on a DT property "google,cros-ec-spi-msg-delay". If this property isn't set, then no delay will be added. However, if set it will cause a delay equal to the value passed to it to be inserted at the end of a transaction. Signed-off-by:
Rhyland Klein <rklein@nvidia.com> Reviewed-by:
Bernie Thompson <bhthompson@chromium.org> Reviewed-by:
Andrew Bresticker <abrestic@chromium.org> Acked-by:
Mark Rutland <mark.rutland@arm.com> Signed-off-by:
Thierry Reding <treding@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Thierry Reding authored
memset() was being called with the second parameter set to '\0', which is equivalent but longer than the more canonical 0. Update the code to use the latter variant consistently across the driver. Signed-off-by:
Thierry Reding <treding@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Thierry Reding authored
The driver is not used on any non-DT platform, so it can depend on the OF Kconfig symbol to make that explicit. This is in preparation of a subsequent patch which parses some parameters from the device tree and would otherwise have to conditionalize that code. Signed-off-by:
Thierry Reding <treding@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Thierry Reding authored
According to the header comment in the source file the driver is licensed under GPL v2, so update MODULE_LICENSE() to match that. Signed-off-by:
Thierry Reding <treding@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Wei Yongjun authored
Fixes the following sparse warning: drivers/mfd/sec-core.c:202:16: warning: Using plain integer as NULL pointer Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Wei Yongjun authored
In case of error, the function devm_regmap_init_mmio() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR(). Signed-off-by:
Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Alexander Shiyan authored
Checking for maximal register is already provided by regmap API, so remove redundant checks. Signed-off-by:
Alexander Shiyan <shc_work@mail.ru> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Stefan Agner authored
Depending on the regulator version, the voltage table might be different. Use version specific regulator tables in order to select correct voltage table. For the following regulator versions different voltage tables are now used: * TPS658623: Use correct voltage table for SM2 * TPS658643: New voltage table for SM2 Both versions are in use on the Colibri T20 module. Make use of the correct tables by requesting the correct SM2 voltage of 1.8V. This change is not backward compatible since an old driver is not able to correctly set that value. The value 1.8V is out of range for the old driver and will refuse to probe the device. The regulator starts with default settings and the driver shows appropriate error messages. On Colibri T20, the old value used to work with TPS658623 since the driver applied a wrong voltage table too. However, the TPS658643 used on V1.2 devices uses yet another voltage table and those broke that pseudo-compatibility. The regulator driver now has the correct voltage table for both regulator versions and those the correct voltage can be used in the device tree. Signed-off-by:
Stefan Agner <stefan@agner.ch> Reviewed-by:
Thierry Reding <treding@nvidia.com> Acked-by:
Mark Brown <broonie@linaro.org> Acked-by:
Stephen Warren <swarren@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Stefan Agner authored
Use the VERSIONCRC to determine the exact device version. According to the datasheet this register can be used as device identifier. The identification is needed since some tps6586x regulators use a different voltage table. Signed-off-by:
Stefan Agner <stefan@agner.ch> Reviewed-by:
Thierry Reding <treding@nvidia.com> Acked-by:
Stephen Warren <swarren@nvidia.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Milo Kim authored
This is the other of the LP3943 MFD driver. LP3943 can be used as a PWM generator, up to 2 channels. * Two PWM generators supported * Supported PWM operations request, free, config, enable and disable * Pin assignment A driver data, 'pin_used' is checked when a PWM is requested. If the output pin is already assigned, then returns as failure. If the pin is available, 'pin_used' is set. When the PWM is not used anymore, then it is cleared. It is defined as unsigned long type for atomic bit operation APIs, but only LSB 16bits are used because LP3943 has 16 outputs. Signed-off-by:
Milo Kim <milo.kim@ti.com> Acked-by:
Thierry Reding <thierry.reding@gmail.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Milo Kim authored
LP3943 has 16 output pins which can be used as GPIO expander and PWM generator. * Regmap I2C interface for R/W LP3943 registers * Atomic operations for output pin assignment The driver should check whether requested pin is available or not. If the pin is already used, pin request returns as a failure. A driver data, 'pin_used' is checked when gpio_request() and pwm_request() are called. If the pin is available, then pin_used is set. And it is cleared when gpio_free() and pwm_free(). * Device tree support Compatible strings for GPIO and PWM driver. LP3943 platform data is PWM related, so parsing the device tree is implemented in the PWM driver. Signed-off-by:
Milo Kim <milo.kim@ti.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Milo Kim authored
This is one of LP3943 MFD driver. LP3943 is configurable as a GPIO expander, up to 16 GPIOs. * Application note: how to configure LP3943 as a GPIO expander http://www.ti.com/lit/an/snva287a/snva287a.pdf * Supported GPIO controller operations request, free, direction_input, direction_output, get and set * GPIO direction register not supported LP3943 doesn't have the GPIO direction register. It only provides input and output status registers. So, private data for the direction should be handled manually. This variable is updated whenever the direction is changed and used in 'get' operation. * Pin assignment A driver data, 'pin_used' is checked when a GPIO is requested. If the GPIO is already assigned, then returns as failure. If the GPIO is available, 'pin_used' is set. When the GPIO is not used anymore, then it is cleared. It is defined as unsigned long type for atomic bit operation APIs, but only LSB 16bits are used because LP3943 has 16 outputs. Signed-off-by:
Milo Kim <milo.kim@ti.com> Reviewed-by:
Linus Walleij <linus.walleij@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Milo Kim authored
Bindings for LP3943 MFD, GPIO and PWM controller are added. Signed-off-by:
Milo Kim <milo.kim@ti.com> Acked-by:
Thierry Reding <thierry.reding@gmail.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Dan Carpenter authored
"pdata" is a NULL not an ERR_PTR so there is no use printing it. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Mark Brown authored
This helps move us towards removing the bus custom operations. Acked-by:
Charles Keepax <ckeepax@opensource.wolfsonmicro.com> Signed-off-by:
Mark Brown <broonie@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-
Linus Walleij authored
All subdrivers use the irqdomain to demux AB8500 IRQs but here in the hierarchical path we find a leftover instance using the hard-coded IRQ base. Convert it to use irqdomain with a oneliner. Signed-off-by:
Linus Walleij <linus.walleij@linaro.org> Signed-off-by:
Lee Jones <lee.jones@linaro.org>
-