On 03/23/2012 03:07 AM, Viresh Kumar wrote: > On 3/23/2012 1:30 PM, Stefan Roese wrote: >> There could be differences between the IP cores integrated into the SoCs. I >> only tested on spear600, so thats what I integrated into the drivers. While >> supporting more SoCs (SPEAr300 etc) to those drivers, you should add the >> specific compatible properties to the lists. > > Ya. That's what i was expecting. > >> If the IP core doesn't differ between those SoCs, then it might make sense >> to >> change this property to something more general, like "st,spear-ohci" (etc). >> Not 100% sure though. Other DT experts might have more insight here. > > Ok. FSMC version is same for 3xx and 6xx. Is different for 13xx. > But driver is capable of detecting the version itself and so we might > go ahead with "st,spear-fsmc there" > > SMI, EHCI and OHCI have exactly the same version for all three families of > SoC's. > So, we must better name them "st,spear-***"
No, it's better to be safe and be more specific. You have to think of it as if you created the DT entries as the h/w came out. Define the compatible string in terms of the 1st version. Then if the 2nd SOC happens to have the same version, you can re-use the compatible string and not change the driver. Rob _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
