On 9/10/14, 6:00 AM, Pavel Machek wrote: > Hi! > >>>> 2) On the host level, the support for handle multiple slots in DT for >>>> dw-mmc has been broken. While dw-mmc parsed the DT nodes for slots, it >>>> screwed up configurations. Thus the support for slots have never >>>> worked as expected from DT point of view. >>> >>> Well, DT is supposed to describe the hardware. From your description, >>> it seems that linux does not support two slots on one controller and >>> DT parsing code basically ignores the slots. (Logical, if it can't >>> support two slots). >>> >>> So now we are breaking DT description due to Linux limitations. Which >>> >>> a) makes it hard for any other os not having same limitation >>> >>> b) makes it hard for people to fix the limitation >>> >>> c) does not really solve anything >> >> Yes it does, the problem in 2) gets fixed. >> >>> >>> d) breaks backward compatibility with old dts >> >> According to 2), it has never worked - so we don't break anything. > > Umm? u-SD worked for me in 3.14, 3.15 and 3.16 with the device tree, > and with single-slot described in the device tree. Now I have to > change dtb-s to keep working configuration, which is something device > tree should never ever do. I tested the DTB with and without the patch, and it all works fine. Dinh > > Is it so hard to just fix the multiple slot parsing? > > Pavel > -- 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
