On 2/24/2012 5:00 AM, Rajendra Nayak wrote:
On Friday 24 February 2012 12:27 AM, Tony Lindgren wrote:
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -113,5 +113,31 @@
#size-cells =<0>;
ti,hwmods = "i2c3";
};
+
+ mmc1: mmc@1 {
+ compatible = "ti,omap2-hsmmc";
+ ti,hwmods = "mmc1";
+ ti,dual-volt;
+ };
+
+ mmc2: mmc@2 {
+ compatible = "ti,omap2-hsmmc";
+ ti,hwmods = "mmc2";
+ };
+
+ mmc3: mmc@3 {
+ compatible = "ti,omap2-hsmmc";
+ ti,hwmods = "mmc3";
+ };
+
+ mmc4: mmc@4 {
+ compatible = "ti,omap2-hsmmc";
+ ti,hwmods = "mmc4";
+ };
+
+ mmc5: mmc@5 {
+ compatible = "ti,omap2-hsmmc";
+ ti,hwmods = "mmc5";
+ };
};
};
These all should all be "ti,omap3-hsmmc" I guess?
Well, I defined the binding such that both omap2 and omap3
can use the same compatible "ti,omap2-hsmmc" since there is
no difference in the way they are defined or handled. If thats
confusing, I can have separate compatibles.
Btw, I guess we do the same with a few other re-used IPs as well,
I just checked and mcpsi does the same.
Yes, the goal is mainly to identify the IP version, so since OMAP3 is
using OMAP2 IP, I do not think there is a need to add a new compatible
string for OMAP3. The should only be documented in the binding potentially.
The advantage is that that will avoid adding an extra compatible string
in the driver as well whereas there is no change in the IP.
Regards,
Benoit
--
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
