[...] >> Thanks for taking the time to reply and your detailed answer. I see why the bindings currently don't fit into the mmc core model. However, we still have the issue that these bindings are shipping in hardware and we can't change them. As far as I can see, ther major difference between these bindings and what is upstream, is the interpretation of the reg property, and the compatible string of what is connected to the controller. The Octeon MMC controller supports connections to up to 4 devices, either eMMC devices or SD card slots. I think it will be difficult to impossible to programatically interpret the device tree supplied by the bootloader to classify the slots as SD card / eMMC such that the necessary reg / compatible strings can be added. > >> >> If we were to remove the slots entirely from the device tree, how would the core deal with having one controller with multiple devices attached? Obviously we need some locking of the shared controller between the child devices. > > Currently the core doesn't deal with multiple card slots and I doubt > we ever will be adding that. The reason is simply because, in practice > there don't exist such configurations, even if some HWs supports it. I > assume that's the case here as well, right!? > > Kind regards > Uffe > > Hi Ulf, > Unfortunately not, for example the Rhinolabs UTM8 board which we have several of here (http://www.rhinolabsinc.com/rhino-sdna7130-networking-appliance/) contains an eMMC device attached to "slot 1" and a microSD card slot on "slot 2". The driver as it stands supports this board by registering an mmc host for each slot, and it is possible to use both devices while shaing the controller resources. I see! Are you sure you don't need any additional out of tree patch for the mmc core/block as well? Moreover, of course these configurations suffers from poor performance, but I guess that's irrelevant for these systems!? Perhaps we can add specific DT property which tells about whether the childnode is a slot? Then you will have to patch your DTB during boot and add that information. Or you have another suggestion? Kind regards Uffe -- 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
