[...] >> 2) The clock register will be written to before the regulator >> operations have been done. Not sure if that works fine!? > > > you are right, it's probably better if the clock is done after power. > do you agree if I change the order? If the new ST variant can cope with the existing order in the current code, then let's stick to that to avoid breaking legacy variants. > >> >> An overall comment, I think we should create a mmci_host_ops structure >> and put the needed callbacks in there (kept to a minimum of course), >> rather than putting them as a part of the variant struct. More >> importantly, also the legacy mmci variants should get a mmci_host_ops >> structure assigned during probe. >> >> The point is, I think it makes the code above (and future wise) more >> flexible. It should also allow us to better share functions between >> the variants. In principle, I expect that we end up with a bunch of >> "library" mmci functions that can be invoked from variant's >> mmci_host_ops (if not assigned directly). > > > After "Linaro connect" we have exchanged some emails (subject: "STM32MP1 > SDMMC driver review") and the start point was > "Goal is not to redesign mmci.c like sdhci.c." > This it's why I'm surprise by "library" and "mmci_host_ops" > and comment of patch 01. Sometimes it's easier to look at patches/code, instead of giving qualified guesses what is the best solution. Try something, realize what is better, then try again... The point is, we need callbacks to cope with the variants as we can't have quirks for everything, and you are demonstrating that in $subject patch. Since this is the case, then I think it's better to make it more flexible, already from the beginning and thus put the callbacks in a new mmci host ops structure instead. > > So, it's not clear for me on where we wish to go. > Could you clarify the way, if it's with a mmci_ops like sdhci_ops? Yep, something along those lines. However, we don't want to introduce unnecessary callbacks and complexity. We should strive towards sharing common functionality through library functions in mmci.c, rather than having callbacks for every single difference. Does it makes sense? [...] 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
