- 27 Mar, 2014 3 commits
-
-
Georgi Djakov authored
This patch adds implementation for platform specific tuning in order to support HS200 bus speed mode on Qualcomm SDHCI controller. Signed-off-by:
Asutosh Das <asutoshd@codeaurora.org> Signed-off-by:
Venkat Gopalakrishnan <venkatg@codeaurora.org> Signed-off-by:
Georgi Djakov <gdjakov@mm-sol.com> Acked-by:
Ulf Hansson <ulf.hansson@linaro.org> Signed-off-by:
Chris Ball <chris@printf.net>
-
Georgi Djakov authored
This platform driver adds the initial support of Secure Digital Host Controller Interface compliant controller found in Qualcomm chipsets. Signed-off-by:
Asutosh Das <asutoshd@codeaurora.org> Signed-off-by:
Venkat Gopalakrishnan <venkatg@codeaurora.org> Tested-by:
Ivan T. Ivanov <iivanov@mm-sol.com> Signed-off-by:
Georgi Djakov <gdjakov@mm-sol.com> Acked-by:
Ulf Hansson <ulf.hansson@linaro.org> Signed-off-by:
Chris Ball <chris@printf.net>
-
Georgi Djakov authored
This patch adds the device-tree binding documentation for Qualcomm SDHCI driver. It contains the differences between the core properties in mmc.txt and the properties used by the sdhci-msm driver. Signed-off-by:
Georgi Djakov <gdjakov@mm-sol.com> Acked-by:
Ulf Hansson <ulf.hansson@linaro.org> Signed-off-by:
Chris Ball <chris@printf.net>
-
- 25 Mar, 2014 1 commit
-
-
Arend van Spriel authored
When the host->tuning_count is zero it means that the retuning is disabled. This is checked on the first run of sdhci_execute_tuning() by the if statement below: if (!(host->flags & SDHCI_NEEDS_RETUNING) && host->tuning_count && (host->tuning_mode == SDHCI_TUNING_MODE_1)) { So only when tuning_count is non-zero it will set the host flag SDHCI_USING_RETUNING_TIMER. The else statement is only for re-programming the timer, which means that flag must be set. Because that is not checked the else statement is executed in the first run when tuning_count is zero. This was seen on a host controller which indicated SDHCI_TUNING_MODE_1 (0) and tuning_count being zero. Suspect that (one of) these registers is not properly set. Signed-off-by:
Arend van Spriel <arend@broadcom.com> Acked-by:
Ulf Hansson <ulf.hansson@linaro.org> Reviewed-by:
Aaron Lu <aaron.lu@intel.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
- 24 Mar, 2014 1 commit
-
-
Christian Daudt authored
Currently ARCH_BCM has been used for Broadcom Mobile V7 based SoCs. In order to allow other Broadcom SoCs to also use mach-bcm directory and files, this patch renames the original ARCH_BCM to ARCH_BCM_MOBILE, and uses ARCH_BCM to define any Broadcom chip residing in mach-bcm directory. Signed-off-by:
Christian Daudt <bcm@fixthebug.org> Acked-by:
Olof Johansson <olof@lixom.net> Signed-off-by:
Chris Ball <chris@printf.net>
-
- 17 Mar, 2014 7 commits
-
-
Adrian Hunter authored
If the SDHCI irq is shared with another device then the interrupt handler can get called while SDHCI is runtime suspended. That is harmless but the warning message is not useful so remove it. Also returning IRQ_NONE is more appropriate. Signed-off-by:
Adrian Hunter <adrian.hunter@intel.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Adrian Hunter authored
Add ACPI HID 80860F16 as a host controller for a SD card. Signed-off-by:
Adrian Hunter <adrian.hunter@intel.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Adrian Hunter authored
Some 80860F14 devices do not support card detect and must rely completely on GPIO. Presently the card detect GPIO is used only to wake-up from runtime suspend. Change to using mmc_gpioid_request_cd() which will cause the SDHCI driver to prefer the GPIO to the host controller's native card detect. Signed-off-by:
Adrian Hunter <adrian.hunter@intel.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Adrian Hunter authored
Add functions to request a CD GPIO using the GPIO descriptor API. Note that the new request function is paired with mmc_gpiod_free_cd() not mmc_gpio_free_cd(). Note also that it must be called prior to mmc_add_host() otherwise the caller must also call mmc_gpiod_request_cd_irq(). Signed-off-by:
Adrian Hunter <adrian.hunter@intel.com> Reviewed-by:
Linus Walleij <linus.walleij@linaro.org> Signed-off-by:
Chris Ball <chris@printf.net>
-
Adrian Hunter authored
In preparation for adding a descriptor-based CD GPIO API, split out CD IRQ request into a separate function. Signed-off-by:
Adrian Hunter <adrian.hunter@intel.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Adrian Hunter authored
In preparation for adding a descriptor-based CD GPIO API, switch from recording GPIO numbers to recording GPIO descriptors. Signed-off-by:
Adrian Hunter <adrian.hunter@intel.com> Tested-by:
Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Chris Ball authored
This reverts commit d9c3f5df, which should not have been merged via mmc-next. It's in arm-soc instead now.
-
- 04 Mar, 2014 24 commits
-
-
Russell King authored
sdhci has support for using GPIOs for card detection. If we have a GPIO specified, we can use that directly, without needing our own interrupt handler. Signed-off-by:
Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by:
Chris Ball <chris@printf.net>
-
Russell King authored
None of this code is currently used: there are no definitions of struct sdhci_plat_data in arch/arm, neither are there any DT properties which use card_power_gpio/power_active_high/power_always_enb. In any case, slot power control should be rigged up via vmmc and the regulator subsystem in the DT case. Signed-off-by:
Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by:
Chris Ball <chris@printf.net>
-
Russell King authored
Use devm_ioremap_resource() to simplify iomem resource handling in the probe path. Signed-off-by:
Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by:
Chris Ball <chris@printf.net>
-
Russell King authored
sdhci-spear is unsafe should a probe fail or defer, since it overwrites the platform_data with its own driver-private data. It's trivial to fix as SDHCI allows for driver-private data to be appended to its own structure - we just need to arrange the code to allow this. Signed-off-by:
Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by:
Chris Ball <chris@printf.net>
-
Russell King authored
Fix the error handling paths for DT and simplify using the devm_* API for clk_get(). Signed-off-by:
Russell King <rmk+kernel@arm.linux.org.uk> Signed-off-by:
Chris Ball <chris@printf.net>
-
Russell King authored
`sdhci_bcm_kona_remove' referenced in section `.data' of drivers/built-in.o: defined in discarded section `.exit.text' of drivers/built-in.o Fixes: 058feb53 ("mmc: sdhci-bcm-kona: make linker-section warning go away") Signed-off-by:
Russell King <rmk+kernel@arm.linux.org.uk> Tested-by:
Markus Mayer <markus.mayer@linaro.org> Acked-by:
Matt Porter <mporter@linaro.org> Cc: <stable@vger.kernel.org> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
Add support for autocmd23 support Signed-off-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
Add set block count command support for close ended multiblock read/write. Signed-off-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
split start dma function into setup and start dma to keep track of host_cookie when cmd23 support is enabled along with async request. Signed-off-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
mrq is not populated for set block count(cmd23) command. Use block read/write mmc_commond pointer for request done and avoid NULL pointer access in error case for sbc (cmd23). Signed-off-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
clk_get_rate throws DEBUG_LOCKS_WARN_ON(in_interrupt()) warning if called from interrupt context. use cached clock rate in set_data_timeout, so that set_data_timeout can be called from interrupt context. Signed-off-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
Remove redundant reset done check since omap hwmod layer ensures IP reset. Signed-off-by:
Balaji T K <balajitk@ti.com> Reported-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
of_get_hsmmc_pdata returns a pointer, returning NULL is invalid, return ERR_PTR for error case. Signed-off-by:
Balaji T K <balajitk@ti.com> Reported-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
remove pbias workaround Signed-off-by:
Balaji T K <balajitk@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
Enable REGULATOR_PBIAS needed for SD card on most OMAPs. Signed-off-by:
Balaji T K <balajitk@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
Add pbias regulator node as a child of system control module - syscon. Signed-off-by:
Balaji T K <balajitk@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
In DT case, PBAIS registers are programmed via regulator, use regulator APIs to control PBIAS. Signed-off-by:
Balaji T K <balajitk@ti.com> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
pbias register controls internal power supply to sd card i/o pads in most OMAPs (OMAP2-5, DRA7). Control bits for selecting voltage level and enabling/disabling are in the same PBIAS register. Signed-off-by:
Balaji T K <balajitk@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Acked-by:
Mark Brown <broonie@linaro.org> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
handle vcc and vcc_aux independently to reduce indent. Signed-off-by:
Balaji T K <balajitk@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Balaji T K authored
Use devm_regulator API, while at it use devm_regulator_get_optional for optional vmmc_aux supply Signed-off-by:
Balaji T K <balajitk@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Tested-by:
Florian Vaussard <florian.vaussard@epfl.ch> Tested-by:
Stefan Roese <sr@denx.de> Signed-off-by:
Chris Ball <chris@printf.net>
-
Daniel Mack authored
This should probably be done implicitly through mmc_of_parse(), but that doesn't play well along with the multi-slot model the hsmmc driver features. Hence, for now, do it manually. The properties are already documented in Documentation/devicetree/bindings/mmc/mmc.txt. Signed-off-by:
Daniel Mack <zonque@gmail.com> Acked-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Nishanth Menon authored
MMC1 is the only MMC interface available on the platform. Further, since the platform is based on older revision of SoC which is not capable of doing multi-block reads, mark it with compatibility for the same and add pinmux to ensure that all relevant pins are configured for non-MMC boot mode. Signed-off-by:
Nishanth Menon <nm@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Acked-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Nishanth Menon authored
When device is booted using devicetree, platforms impacted by Erratum 2.1.1.128 is not detected easily in the mmc driver. This erratum indicates that the module cannot do multi-block transfers. Platforms such as LDP which use OMAP3 ES revision prior to ES3.0 are impacted by this. Provide a new compatible property "ti,omap3-pre-es3-hsmmc" to allow driver to determine if driver needs to implement quirks associated with the specific module version (primarily because the IP revision information is not sufficient for the same). Signed-off-by:
Nishanth Menon <nm@ti.com> Acked-by:
Tony Lindgren <tony@atomide.com> Acked-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Dan Carpenter authored
Static checkers complain that testing for both "next" and "!next" is duplicative. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Acked-by:
Balaji T K <balajitk@ti.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
- 03 Mar, 2014 4 commits
-
-
Felipe Balbi authored
Fix the following build errors: drivers/mmc/host/dw_mmc-k3.c: In function ‘dw_mci_k3_suspend’: drivers/mmc/host/dw_mmc-k3.c:58:2: error: implicit declaration of function ‘dw_mci_suspend’ [-Werror=implicit-function-declaration] ret = dw_mci_suspend(host); ^ drivers/mmc/host/dw_mmc-k3.c: In function ‘dw_mci_k3_resume’: drivers/mmc/host/dw_mmc-k3.c:76:2: error: implicit declaration of function ‘dw_mci_resume’ [-Werror=implicit-function-declaration] return dw_mci_resume(host); ^ drivers/mmc/host/dw_mmc-k3.c: At top level: drivers/mmc/host/dw_mmc-k3.c:53:12: warning: ‘dw_mci_k3_suspend’ defined but not used [-Wunused-function] static int dw_mci_k3_suspend(struct device *dev) ^ drivers/mmc/host/dw_mmc-k3.c:65:12: warning: ‘dw_mci_k3_resume’ defined but not used [-Wunused-function] static int dw_mci_k3_resume(struct device *dev) ^ Signed-off-by:
Felipe Balbi <balbi@ti.com> Acked-by:
Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Tomasz Figa authored
This patch modifies sdhci_s3c_consider_clock() to fail if bus clock being considered can not provide frequency lower or equal requested, instead of returning the lowest supported. Signed-off-by:
Tomasz Figa <tomasz.figa@gmail.com> Tested-by:
Heiko Stuebner <heiko@sntech.de> Acked-by:
Heiko Stuebner <heiko@sntech.de> Tested-by:
Jaehoon Chung <jh80.chung@samsung.com> Acked-by; Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Tomasz Figa authored
Currently the driver assumes at probe that controller is configured for last valid enumerated bus clock. This assumption is completely wrong, as there is no way to ensure such configuration until the hardware gets first configured (by calling sdhci_s3c_set_clock()). This patch modifies the driver to set current clock at probe to unknown state (represented by negative value) and make sure that the hardware gets actually configured to selected clock in sdhci_s3c_set_clock(). Signed-off-by:
Tomasz Figa <tomasz.figa@gmail.com> Tested-by:
Heiko Stuebner <heiko@sntech.de> Acked-by:
Heiko Stuebner <heiko@sntech.de> Tested-by:
Jaehoon Chung <jh80.chung@samsung.com> Acked-by; Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by:
Chris Ball <chris@printf.net>
-
Tomasz Figa authored
This patch reimplements functions calculating minimum and maximum clock rates to leverage clock rate cache introduced by previous patches. In addition, the calculation is simplified to just comparing input clock rates (max case) or input clock rates divided by maximum divisor (min case), which is basically what the original code did, but with much more unnecessary work. Signed-off-by:
Tomasz Figa <tomasz.figa@gmail.com> Tested-by:
Heiko Stuebner <heiko@sntech.de> Acked-by:
Heiko Stuebner <heiko@sntech.de> Tested-by:
Jaehoon Chung <jh80.chung@samsung.com> Acked-by; Jaehoon Chung <jh80.chung@samsung.com> Signed-off-by:
Chris Ball <chris@printf.net>
-