Am 09.01.19 um 09:21 schrieb Horatiu Vultur:
Hi Daniel,
First, thank you for all the feedback.
I have created a new patch series that will fix the issue that you addressed bellow but the fix it would not use compatible string, because that presume that I need to change the other pinctrl-drivers(ocelot, luton). I was thinking if it is OK the way I have done in v3 and then create a new patch series specific for fixing pinctrl. Similar with the issue that I have for sysreset driver.
I didn't expect that you select the registers via compatible strings in the pinctrl driver because you effectively have three drivers and a common driver core. Your v3 variant is perfect. The compatible string method makes sense for drivers that support more than one SoC. The sysreset driver could be an example.