Dear Wolfram Sang, [...] > > That'd be cool. Some months ago, you promised to take a look. I tried it > > recently again with not much luck. > > Yes, I wanted to. I couldn't (see above). I do imagine that it really > might not be possible to switch modes at runtime. This is why I was > trying to get the patch into the next release without the > mode-switching. But for that to happen, there was the binding issue. I > came up with the module parameter as the easiest way to fix it. I am > open to other solutions. Simply dropping the binding might be another > idea if we pay attention that there are no regressions, going from > PIOQUEUE to DMA. > > I am also still interested to check the runtime switching, but it might > take another month until I can really hack on it. Good, there is some bit that probably needs to be flipped to allow this switching. I managed to get this working with SPI, not with i2c though. With i2c, if I restarted the controller inbetween each transaction, it worked ... which is not what I'd like to see there. > > > Deprecated > > > properties are also troublesome. Third, we don't really need this per > > > instance, if somebody really has problems with DMA, it will apply to > > > all i2c busses. Makes sense? > > > > No, it doesn't. See above about small transfers. Consider the easy > > situation where you have sensor on one bus (so you do PIO because you > > transfer small data) and you have EEPROM on other bus, where you use DMA > > because you transfer large data. And the mixed mode isn't there yet. > > I fully understand what you want to configure. I did before. Yet, > devicetree bindings are not platform_data and shouldn't be used like > them. But then, how would you configure this detail on a per-bus basis? Well all right, screw this, let's make it impotent and go for the module parameter. This patch actually fixes a real issue, I'd like to have it in ASAP and it's been aboue three months already, which sucks. > Regards, > > Wolfram Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
