On 28 May 2014 15:47, <srinivas.kandagatla@xxxxxxxxxx> wrote:
> From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
>
> This patch adds specifics of clk and datactrl register on Qualcomm SD
> Card controller. This patch also populates the Qcom variant data with
> these new values specific to Qualcomm SD Card Controller.
>
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
> Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> drivers/mmc/host/mmci.c | 4 ++++
> drivers/mmc/host/mmci.h | 17 +++++++++++++++++
> 2 files changed, 21 insertions(+)
>
> diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c
> index 8deea4a..dbcb952 100644
> --- a/drivers/mmc/host/mmci.c
> +++ b/drivers/mmc/host/mmci.c
> @@ -179,6 +179,10 @@ static struct variant_data variant_qcom = {
> .fifosize = 16 * 4,
> .fifohalfsize = 8 * 4,
> .clkreg = MCI_CLK_ENABLE,
> + .clkreg_enable = MCI_QCOM_CLK_FLOWENA |
> + MCI_QCOM_CLK_SELECT_IN_FBCLK,
> + .clkreg_8bit_bus_enable = MCI_QCOM_CLK_WIDEBUS_8,
> + .datactrl_mask_ddrmode = MCI_QCOM_CLK_SELECT_IN_DDR_MODE,
This stuff should go in patch3.
> .blksz_datactrl4 = true,
> .datalength_bits = 24,
> .pwrreg_powerup = MCI_PWR_UP,
> diff --git a/drivers/mmc/host/mmci.h b/drivers/mmc/host/mmci.h
> index cd83ca3..706eb513 100644
> --- a/drivers/mmc/host/mmci.h
> +++ b/drivers/mmc/host/mmci.h
> @@ -41,6 +41,15 @@
> /* Modified PL180 on Versatile Express platform */
> #define MCI_ARM_HWFCEN BIT(12)
>
> +/* Modified on Qualcomm Integrations */
> +#define MCI_QCOM_CLK_WIDEBUS_8 (BIT(10) | BIT(11))
> +#define MCI_QCOM_CLK_FLOWENA BIT(12)
> +#define MCI_QCOM_CLK_INVERTOUT BIT(13)
> +
> +/* select in latch data and command in */
> +#define MCI_QCOM_CLK_SELECT_IN_FBCLK BIT(15)
> +#define MCI_QCOM_CLK_SELECT_IN_DDR_MODE (BIT(14) | BIT(15))
> +
> #define MMCIARGUMENT 0x008
> #define MMCICOMMAND 0x00c
> #define MCI_CPSM_RESPONSE BIT(6)
> @@ -54,6 +63,14 @@
> #define MCI_ST_NIEN BIT(13)
> #define MCI_ST_CE_ATACMD BIT(14)
>
> +/* Modified on Qualcomm Integrations */
> +#define MCI_QCOM_CSPM_DATCMD BIT(12)
> +#define MCI_QCOM_CSPM_MCIABORT BIT(13)
> +#define MCI_QCOM_CSPM_CCSENABLE BIT(14)
> +#define MCI_QCOM_CSPM_CCSDISABLE BIT(15)
> +#define MCI_QCOM_CSPM_AUTO_CMD19 BIT(16)
> +#define MCI_QCOM_CSPM_AUTO_CMD21 BIT(21)
> +
Maybe you should have one patch in the beginning of the patchset, that
adds all the new QCOM bits in the header file? Instead of splitting
them up?
> #define MMCIRESPCMD 0x010
> #define MMCIRESPONSE0 0x014
> #define MMCIRESPONSE1 0x018
> --
> 1.9.1
>
Kind regards
Ulf Hansson
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
