Dear Dong Aisheng,

On Wed, Mar 14, 2012 at 01:58:25PM +0800, Marek Vasut wrote:

quoted

Dear Dong Aisheng,

quoted

Signed-off-by: Dong Aisheng <redacted>

........

quoted

quoted

+static const struct of_device_id mxs_mmc_dt_ids[] = {
+	{ .compatible = "fsl,imx23-mmc", .data = NULL, },
+	{ .compatible = "fsl,imx28-mmc", .data = NULL, },

Do you really need two distinct ones here?

Hmm, my original purpose is to put soc difference data in .data
to remove cpu_is_* function calls in the driver later.
Do you think if any issue?

Well, what's the difference between the interfaces on mx233 and mx28? Is it 
something that can't be encoded otherwise? I think they're not so different.

Best regards,
Marek Vasut