Re: [PATCH v1 08/11] mmc: mmci: Qcom fix MCICLK register settings.

From: Srinivas Kandagatla
Date: Tue May 13 2014 - 05:36:51 EST


Thanks Linus W.

On 13/05/14 09:19, Linus Walleij wrote:
Again follow the pattern of storing register templates in the vendor_data
struct. I think you will quickly realize how this can be cut down with
new fields like .clk_4bitmode etc.

> /* Modified PL180 on Versatile Express platform */
> #define MCI_ARM_HWFCEN (1 << 12)
>
>+/* Modified on Qualcomm Integrations */
First: follow the convention set for the ST-specific registers that
look e.g. like this:
I agree, Will fix these in next version.

MCI_ST_8BIT_BUS i.e. MCI_<vendor>_SPECIFIER
So the below becomes MCI_QCOM_CLK_WIDEBUS... etc.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/