Hello Andrew, On Fri, Oct 23, 2015 at 4:36 PM, Andrew F. Davis <afd@xxxxxx> wrote: > On 10/23/2015 09:29 AM, Javier Martinez Canillas wrote: >> >> Hello Andrew, >> >> On Fri, Oct 23, 2015 at 3:08 PM, Andrew F. Davis <afd@xxxxxx> wrote: >>> >>> On 10/22/2015 04:23 PM, Tony Lindgren wrote: >>>> >>>> >>>> * Andrew F. Davis <afd@xxxxxx> [151022 09:21]: >>>>> >>>>> >>>>> Using constants for pinctrl allows better readability and removes >>>>> redundancy with comments. >>>> >>>> >>>> >>>> You should use the include/dt-bindings/pinctrl/omap.h macro >>>> AM33XX_IOPAD(pa, val) while at it. Otherwise we'll end up patching >>>> the same things again later on. >>>> >>> >>> Hmm, I haven't really been following this change, it kind of seems to add >>> some unnecessary abstraction by using physical hardware addresses instead >>> of offsets, then just converting them back to offsets. The offset style >>> DT >>> nodes are already auto-generated with existing tools anyway: >>> https://dev.ti.com/pinmux >>> >>> I'm sure this has been discussed already so if this is a blocker I'll >>> re-spin this. >>> >> >> The good thing about the IOPAD pinmux macros is that matches what is >> in the TRM so it is easier to read the DTS and verify that is correct. >> >> I've on my queue to migrate all the remaining DTS for TI SoCs to use >> the IOPAD macros but is just that I didn't have time to do it this >> week. Probably I'll do it at the end of the next week. >> > > Then this patch can probably be dropped, hopefully the converted constants > in this patch can be of some help for you though. > Yes they are but feel free to re-spin your patch if you wish, it will be one less DTS to convert :-) But yes, if you don't then I'll do it when changing all DTS. > Andrew > > Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html