-----Original Message-----
From: Fabio Estevam [mailto:festevam at gmail.com]
Sent: Tuesday, October 23, 2018 8:29 PM

[...] 

On Tue, Oct 23, 2018 at 8:56 AM A.s. Dong [off-list ref] wrote:

quoted

+/ {
+       model = "NXP i.MX7ULP EVK";
+       compatible = "fsl,imx7ulp-evk", "fsl,imx7ulp", "Generic DT
+based system";

This "Generic DT based system" should be removed.

Got it

quoted

+       chosen {
+               stdout-path = &lpuart4;
+       };
+
+       memory at 60000000 {

       device_type = "memory";

Got it

quoted

+               reg = <0x60000000 0x40000000>;
+       };

quoted

+&iomuxc1 {
+       pinctrl_lpuart4: lpuart4grp {
+               pinmux = <
+                       IMX7ULP_PAD_PTC3__LPUART4_RX
+                       IMX7ULP_PAD_PTC2__LPUART4_TX
+               >;
+               bias-pull-up;

Please change the mx7ulp pinctrl driver to accept the "old" notation and
change the dts to use:

IMX7ULP_PAD_PTC3__LPUART4_RX 0x3

This way we use the same method for all i.MX devices.

I'm a bit hesitate to do that as the driver already supports it. There's no extra effort
to use it. And we probably could take ULP as a special case to test generic binding
rather then simply drop it and drop driver features. Once we get objection from users
later, we still can simply fallback as there's still only official boards using it.

Last, it's not correct that there's not only one method for all i.MX devices.
MX23/28 are different ones and ULP is more like MX23/28.
And I saw no objections from users for MX23/28.

Regards
Dong Aisheng