[{"id":2224491,"web_url":"http://patchwork.ozlabs.org/comment/2224491/","msgid":"","list_archive_url":null,"date":"2019-07-26T17:54:20","subject":"Re: [PATCH v2 2/2] ARM: dts: imx6ul-kontron-n6310: Add Kontron\n\ti.MX6UL N6310 SoM and boards","submitter":{"id":6978,"url":"http://patchwork.ozlabs.org/api/people/6978/","name":"Fabio Estevam","email":"festevam@gmail.com"},"content":"Hi Krzysztof,\n\nOn Fri, Jul 26, 2019 at 3:17 AM Krzysztof Kozlowski wrote:\n\n> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml\n> index 7294ac36f4c0..afb61a55e26f 100644\n> --- a/Documentation/devicetree/bindings/arm/fsl.yaml\n> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml\n> @@ -161,6 +161,10 @@ properties:\n> items:\n> - enum:\n> - fsl,imx6ul-14x14-evk # i.MX6 UltraLite 14x14 EVK Board\n> + - kontron,n6310-som # Kontron N6310 SOM\n> + - kontron,n6310-s # Kontron N6310 S Board\n> + - kontron,n6310-s-43 # Kontron N6310 S 43 Board\n> + - kontron,n6310-s-50 # Kontron N6310 S 50 Board\n\nThese entries should be:\n imx6ul-kontron-n6310-s.dtb\n imx6ul-kontron-n6310-s-43.dtb\n imx6ul-kontron-n6310-s-50.dtb\n\n> + panel {\n> + compatible = \"admatec,t043c004800272t2a\";\n\nI do not find this binding documented.\n\n> +&i2c4 {\n> + gt911@5d {\n\nNode names should be generic according to the devicetree spec, so:\n\ntouchscreen@5d\n\n> + compatible = \"goodix,gt928\";\n> + reg = <0x5d>;\n> + pinctrl-names = \"default\";\n> + pinctrl-0 = <&pinctrl_cap_touch>;\n> + interrupt-parent = <&gpio5>;\n> + interrupts = <6 8>;\n\nIt would be better to use a laber to indicate the irq type:\n\ninterrupts = <6 IRQ_TYPE_LEVEL_LOW>;\n\n> + reset-gpios = <&gpio5 8 GPIO_ACTIVE_HIGH>;\n> + irq-gpios = <&gpio5 6 GPIO_ACTIVE_HIGH>;\n\nActive high?\n\nAbove you use \"interrupts = <6 8>;\" which means IRQ_TYPE_LEVEL_LOW.\n\n> + };\n> +};\n> +\n> +&iomuxc {\n\nWe tend to prefer putting iomuxc as the last node.\n\n> + pinctrl_lcdif_dat: lcdifdatgrp {\n> + fsl,pins = <\n> + MX6UL_PAD_LCD_DATA00__LCDIF_DATA00 0x79\n> + MX6UL_PAD_LCD_DATA01__LCDIF_DATA01 0x79\n> + MX6UL_PAD_LCD_DATA02__LCDIF_DATA02 0x79\n> + MX6UL_PAD_LCD_DATA03__LCDIF_DATA03 0x79\n> + MX6UL_PAD_LCD_DATA04__LCDIF_DATA04 0x79\n> + MX6UL_PAD_LCD_DATA05__LCDIF_DATA05 0x79\n> + MX6UL_PAD_LCD_DATA06__LCDIF_DATA06 0x79\n> + MX6UL_PAD_LCD_DATA07__LCDIF_DATA07 0x79\n> + MX6UL_PAD_LCD_DATA08__LCDIF_DATA08 0x79\n> + MX6UL_PAD_LCD_DATA09__LCDIF_DATA09 0x79\n> + MX6UL_PAD_LCD_DATA10__LCDIF_DATA10 0x79\n> + MX6UL_PAD_LCD_DATA11__LCDIF_DATA11 0x79\n> + MX6UL_PAD_LCD_DATA12__LCDIF_DATA12 0x79\n> + MX6UL_PAD_LCD_DATA13__LCDIF_DATA13 0x79\n> + MX6UL_PAD_LCD_DATA14__LCDIF_DATA14 0x79\n> + MX6UL_PAD_LCD_DATA15__LCDIF_DATA15 0x79\n> + MX6UL_PAD_LCD_DATA16__LCDIF_DATA16 0x79\n> + MX6UL_PAD_LCD_DATA17__LCDIF_DATA17 0x79\n> + MX6UL_PAD_LCD_DATA18__LCDIF_DATA18 0x79\n> + MX6UL_PAD_LCD_DATA19__LCDIF_DATA19 0x79\n> + MX6UL_PAD_LCD_DATA20__LCDIF_DATA20 0x79\n> + MX6UL_PAD_LCD_DATA21__LCDIF_DATA21 0x79\n> + MX6UL_PAD_LCD_DATA22__LCDIF_DATA22 0x79\n> + MX6UL_PAD_LCD_DATA23__LCDIF_DATA23 0x79\n> + >;\n> + };\n> +\n> + pinctrl_lcdif_ctrl: lcdifctrlgrp {\n> + fsl,pins = <\n> + MX6UL_PAD_LCD_CLK__LCDIF_CLK 0x79\n> + MX6UL_PAD_LCD_ENABLE__LCDIF_ENABLE 0x79\n> + MX6UL_PAD_LCD_HSYNC__LCDIF_HSYNC 0x79\n> + MX6UL_PAD_LCD_VSYNC__LCDIF_VSYNC 0x79\n> + MX6UL_PAD_LCD_RESET__LCDIF_RESET 0x79\n> + >;\n> + };\n> +\n> + pinctrl_cap_touch: captouchgrp {\n> + fsl,pins = <\n> + MX6UL_PAD_SNVS_TAMPER6__GPIO5_IO06 0x1b0b0 /* Touch Interrupt */\n> + MX6UL_PAD_SNVS_TAMPER7__GPIO5_IO07 0x1b0b0 /* Touch Reset */\n> + MX6UL_PAD_SNVS_TAMPER8__GPIO5_IO08 0x1b0b0 /* Touch Wake */\n> + >;\n> + };\n> +\n> + pinctrl_pwm7: pwm7grp {\n> + fsl,pins = <\n> + MX6UL_PAD_CSI_VSYNC__PWM7_OUT 0x110b0\n> + >;\n> + };\n> +};\n> +\n> +&lcdif {\n> + pinctrl-names = \"default\";\n> + pinctrl-0 = <&pinctrl_lcdif_dat\n> + &pinctrl_lcdif_ctrl>;\n\nCould fit into a single line.\n\n> + panel {\n> + compatible = \"admatec,t070p133t0s301\";\n\nSame here. Undocumented binding.\n\n> + backlight = <&backlight>;\n> +\n> + port {\n> + panel_in: endpoint {\n> + remote-endpoint = <&display_out>;\n> + };\n> + };\n> + };\n> +};\n> +\n> +&i2c4 {\n> + gt911@5d {\n\nSame comments as previously apply.\n\n> +\n> + regulators {\n\nNo need to have this regulators indent level.\n\n> + reg_3v3: regulator1 {\n\nYou can place this one directly. The preferred format is:\n\nreg_3v3: regulator-reg-3v3 {\n\n> +&ecspi1 {\n> + fsl,spi-num-chipselects = <1>;\n\nThis property is obsoleted. Please remove it.\n\n> + cs-gpios = <&gpio4 26 GPIO_ACTIVE_HIGH>;\n> + pinctrl-names = \"default\";\n> + pinctrl-0 = <&pinctrl_ecspi1>;\n> + status = \"okay\";\n> +\n> + fram@0 {\n\nGeneric name please. eeprom@0\n\n> + compatible = \"atmel,at25\";\n\nPlease use the recommended compatible scheme as per\nDocumentation/devicetree/bindings/eeprom/at25.txt.\n\n> + reg = <0>;\n> + spi-max-frequency = <20000000>;\n> + spi-cpha;\n> + spi-cpol;\n> + pagesize = <1>;\n> + size = <8192>;\n> + address-width = <16>;\n> + };\n> +};\n\n\n> +&usbotg1 {\n> + pinctrl-names = \"default\";\n> + pinctrl-0 = <&pinctrl_usbotg1>;\n> + dr_mode = \"otg\";\n> + status = \"okay\";\n\nWe prefer to put the 'status' property as the last one.\n\n> + srp-disable;\n> + hnp-disable;\n> + adp-disable;\n> + vbus-supply = <®_usb_otg1_vbus>;\n> +};\n\n> +/ {\n> + model = \"Kontron N6310 SOM\";\n> + compatible = \"kontron,n6310-som\", \"fsl,imx6ul\";\n> +\n> + memory@80000000 {\n\ndevice_type = \"memory\"; is missing here.\n\n> + reg = <0x80000000 0x10000000>;\n> + };\n> +};\n> +\n> +&cpu0 {\n> + clock-frequency = <528000000>;\n\nIs this one really needed?\n\n> +&ecspi2 {\n> + cs-gpios = <&gpio4 22 GPIO_ACTIVE_HIGH>;\n> + pinctrl-names = \"default\";\n> + pinctrl-0 = <&pinctrl_ecspi2>;\n> + status = \"okay\";\n> +\n> + flash: mx25v80@0 {\n\nspi-flash@0\n\n> +&qspi {\n> + pinctrl-names = \"default\";\n> + pinctrl-0 = <&pinctrl_qspi>;\n> + status = \"okay\";\n> +\n> + flash0: w25m02gv@0 {\n\ngeneric name, please.","headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org;\n\treceiver=)","ozlabs.org;\n\tdmarc=pass (p=none dis=none) header.from=gmail.com","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"rQOj9fPr\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 45wGtP32Qtz9s8m\n\tfor ;\n\tSat, 27 Jul 2019 03:54:21 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1726902AbfGZRyT (ORCPT\n\t);\n\tFri, 26 Jul 2019 13:54:19 -0400","from mail-lj1-f196.google.com ([209.85.208.196]:35564 \"EHLO\n\tmail-lj1-f196.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1726279AbfGZRyT (ORCPT\n\t); Fri, 26 Jul 2019 13:54:19 -0400","by mail-lj1-f196.google.com with SMTP id x25so52350138ljh.2;\n\tFri, 26 Jul 2019 10:54:17 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=gmail.com; s=20161025;\n\th=mime-version:references:in-reply-to:from:date:message-id:subject:to\n\t:cc; bh=7F9P+gn3gvNT3AC9joEJMhO0myCfMHXJbTDH35bFWhE=;\n\tb=rQOj9fPrQ6OxOwysGYtnZSht0cogKvvqwoKtRAK9Q3ll0Mb2vJBhqNa2GetwrSiJtu\n\tDLuwBTT0zBg+jhizQaFYFBxntaLJdCyrj5TapxUTjiH0ssoaClcMkJLy3cdsmlLQCYO1\n\tAcsuOWG/g3WzFEHs5oEMfsQfh4dn6JpDrr1sXWllzyQiBg8B7P0jtaHigPSu6I+KAqig\n\tNigr9zw3TxXArpI811Jfspl0snFEI4Yt9Ofw23tIKP1MRRCzsQh/7NxIK62ZqkjH4Ack\n\txY7wIaB3E79U/VMyypR1HEyAfTX7LnOlCepGQCUIeGUJsrInca22QDnuqtNtVUguHIvN\n\txeig==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:references:in-reply-to:from:date\n\t:message-id:subject:to:cc;\n\tbh=7F9P+gn3gvNT3AC9joEJMhO0myCfMHXJbTDH35bFWhE=;\n\tb=INB/ap3je8SxrgazYfhGyajy/bjHk/cvFRz/tATR5mZWTvl/ZpbS9atKwfN86po8/2\n\tdn2124yxOmrhrf/7XCU0QOSn/bQl6yIvA5phvBHsWdUFqdp104PyGUGLpdf7QsCRoK9/\n\tEJTo7wiVw5iHl10Z8FGsC81uroNB+91QzPfcA34GOZGY7ET8A5QypLJLojTVqAQVuAkr\n\t2JyI4TC6tbvbPimaXrcMErq4asA6r3FXBD96Dz3Q+pfD03qTir4RicZKrUmRWzvQ/4o1\n\tWQE42xu6rYAlupjKLdW2H5vGp5eZtExckjGJgfEDtAckQSkVJSoP5McbXh4JapQJLknp\n\tGvFg==","X-Gm-Message-State":"APjAAAUgdbEkWAD6HYAB8nvP6XdaC2c9qZEWRpHH4+0WXiwhY+cIsBik\n\theAIOKiS+7hRZpzq4m1Blfxxj9YnxJ+a88W2Ny2P5lg98Ac=","X-Google-Smtp-Source":"APXvYqzvc9w8orflrBxbRsQ40VAYsSe1QT+d7UCX8s/a99rpkhrp3detgHyHVNrS7Q7AR7oCajpJhOJv/mi3+vXSxeI=","X-Received":"by 2002:a2e:8650:: with SMTP id\n\ti16mr50725830ljj.178.1564163656259; \n\tFri, 26 Jul 2019 10:54:16 -0700 (PDT)","MIME-Version":"1.0","References":"<20190726061705.14764-1-krzk@kernel.org>\n\t<20190726061705.14764-2-krzk@kernel.org>","In-Reply-To":"<20190726061705.14764-2-krzk@kernel.org>","From":"Fabio Estevam ","Date":"Fri, 26 Jul 2019 14:54:20 -0300","Message-ID":"","Subject":"Re: [PATCH v2 2/2] ARM: dts: imx6ul-kontron-n6310: Add Kontron\n\ti.MX6UL N6310 SoM and boards","To":"Krzysztof Kozlowski ","Cc":"Rob Herring , Mark Rutland ,\n\tShawn Guo , Sascha Hauer ,\n\tPengutronix Kernel Team ,\n\tNXP Linux Team ,\n\t\"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS\" \n\t,\n\tlinux-kernel ,\n\t\"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE\" \n\t,\n\tSchrempf Frieder ","Content-Type":"text/plain; charset=\"UTF-8\"","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"}},{"id":2224780,"web_url":"http://patchwork.ozlabs.org/comment/2224780/","msgid":"<20190727125758.GA7674@kozik-lap>","list_archive_url":null,"date":"2019-07-27T12:57:58","subject":"Re: [PATCH v2 2/2] ARM: dts: imx6ul-kontron-n6310: Add Kontron\n\ti.MX6UL N6310 SoM and boards","submitter":{"id":68952,"url":"http://patchwork.ozlabs.org/api/people/68952/","name":"Krzysztof Kozlowski","email":"krzk@kernel.org"},"content":"On Fri, Jul 26, 2019 at 02:54:20PM -0300, Fabio Estevam wrote:\n> Hi Krzysztof,\n> \n> On Fri, Jul 26, 2019 at 3:17 AM Krzysztof Kozlowski wrote:\n> \n> > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml\n> > index 7294ac36f4c0..afb61a55e26f 100644\n> > --- a/Documentation/devicetree/bindings/arm/fsl.yaml\n> > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml\n> > @@ -161,6 +161,10 @@ properties:\n> > items:\n> > - enum:\n> > - fsl,imx6ul-14x14-evk # i.MX6 UltraLite 14x14 EVK Board\n> > + - kontron,n6310-som # Kontron N6310 SOM\n> > + - kontron,n6310-s # Kontron N6310 S Board\n> > + - kontron,n6310-s-43 # Kontron N6310 S 43 Board\n> > + - kontron,n6310-s-50 # Kontron N6310 S 50 Board\n> \n> These entries should be:\n> imx6ul-kontron-n6310-s.dtb\n> imx6ul-kontron-n6310-s-43.dtb\n> imx6ul-kontron-n6310-s-50.dtb\n\nOK\n\n(I'll apply all your suggestions without and just reply here where there\nis some discussion)\n\n> \n> > + panel {\n> > + compatible = \"admatec,t043c004800272t2a\";\n> \n> I do not find this binding documented.\n\nBecause they are not... I mentioned it in commit msg - there are no\ndriver and bindings for them. I put them for completness of HW\ndescription.\n\n> \n> > +&i2c4 {\n> > + gt911@5d {\n> \n> Node names should be generic according to the devicetree spec, so:\n> \n> touchscreen@5d\n> \n> > + compatible = \"goodix,gt928\";\n> > + reg = <0x5d>;\n> > + pinctrl-names = \"default\";\n> > + pinctrl-0 = <&pinctrl_cap_touch>;\n> > + interrupt-parent = <&gpio5>;\n> > + interrupts = <6 8>;\n> \n> It would be better to use a laber to indicate the irq type:\n> \n> interrupts = <6 IRQ_TYPE_LEVEL_LOW>;\n\nIndeed.\n\n> \n> > + reset-gpios = <&gpio5 8 GPIO_ACTIVE_HIGH>;\n> > + irq-gpios = <&gpio5 6 GPIO_ACTIVE_HIGH>;\n> \n> Active high?\n> \n> Above you use \"interrupts = <6 8>;\" which means IRQ_TYPE_LEVEL_LOW.\n\nYes, it is confusing but it looks correct. The driver does not use the\nGPIO flag so ACTIVE_HIGH or any other setting does not have effect.\nDriver uses this pin (as active high) after disabling the interrupts as\nan additional reset pin during resume. After this additional reset, it\nserves back as interrupt pin.\n\n> \n> > + };\n> > +};\n> > +\n> > +&iomuxc {\n> \n> We tend to prefer putting iomuxc as the last node.\n> \n> > + pinctrl_lcdif_dat: lcdifdatgrp {\n> > + fsl,pins = <\n> > + MX6UL_PAD_LCD_DATA00__LCDIF_DATA00 0x79\n> > + MX6UL_PAD_LCD_DATA01__LCDIF_DATA01 0x79\n> > + MX6UL_PAD_LCD_DATA02__LCDIF_DATA02 0x79\n> > + MX6UL_PAD_LCD_DATA03__LCDIF_DATA03 0x79\n> > + MX6UL_PAD_LCD_DATA04__LCDIF_DATA04 0x79\n> > + MX6UL_PAD_LCD_DATA05__LCDIF_DATA05 0x79\n> > + MX6UL_PAD_LCD_DATA06__LCDIF_DATA06 0x79\n> > + MX6UL_PAD_LCD_DATA07__LCDIF_DATA07 0x79\n> > + MX6UL_PAD_LCD_DATA08__LCDIF_DATA08 0x79\n> > + MX6UL_PAD_LCD_DATA09__LCDIF_DATA09 0x79\n> > + MX6UL_PAD_LCD_DATA10__LCDIF_DATA10 0x79\n> > + MX6UL_PAD_LCD_DATA11__LCDIF_DATA11 0x79\n> > + MX6UL_PAD_LCD_DATA12__LCDIF_DATA12 0x79\n> > + MX6UL_PAD_LCD_DATA13__LCDIF_DATA13 0x79\n> > + MX6UL_PAD_LCD_DATA14__LCDIF_DATA14 0x79\n> > + MX6UL_PAD_LCD_DATA15__LCDIF_DATA15 0x79\n> > + MX6UL_PAD_LCD_DATA16__LCDIF_DATA16 0x79\n> > + MX6UL_PAD_LCD_DATA17__LCDIF_DATA17 0x79\n> > + MX6UL_PAD_LCD_DATA18__LCDIF_DATA18 0x79\n> > + MX6UL_PAD_LCD_DATA19__LCDIF_DATA19 0x79\n> > + MX6UL_PAD_LCD_DATA20__LCDIF_DATA20 0x79\n> > + MX6UL_PAD_LCD_DATA21__LCDIF_DATA21 0x79\n> > + MX6UL_PAD_LCD_DATA22__LCDIF_DATA22 0x79\n> > + MX6UL_PAD_LCD_DATA23__LCDIF_DATA23 0x79\n> > + >;\n> > + };\n> > +\n> > + pinctrl_lcdif_ctrl: lcdifctrlgrp {\n> > + fsl,pins = <\n> > + MX6UL_PAD_LCD_CLK__LCDIF_CLK 0x79\n> > + MX6UL_PAD_LCD_ENABLE__LCDIF_ENABLE 0x79\n> > + MX6UL_PAD_LCD_HSYNC__LCDIF_HSYNC 0x79\n> > + MX6UL_PAD_LCD_VSYNC__LCDIF_VSYNC 0x79\n> > + MX6UL_PAD_LCD_RESET__LCDIF_RESET 0x79\n> > + >;\n> > + };\n> > +\n> > + pinctrl_cap_touch: captouchgrp {\n> > + fsl,pins = <\n> > + MX6UL_PAD_SNVS_TAMPER6__GPIO5_IO06 0x1b0b0 /* Touch Interrupt */\n> > + MX6UL_PAD_SNVS_TAMPER7__GPIO5_IO07 0x1b0b0 /* Touch Reset */\n> > + MX6UL_PAD_SNVS_TAMPER8__GPIO5_IO08 0x1b0b0 /* Touch Wake */\n> > + >;\n> > + };\n> > +\n> > + pinctrl_pwm7: pwm7grp {\n> > + fsl,pins = <\n> > + MX6UL_PAD_CSI_VSYNC__PWM7_OUT 0x110b0\n> > + >;\n> > + };\n> > +};\n> > +\n> > +&lcdif {\n> > + pinctrl-names = \"default\";\n> > + pinctrl-0 = <&pinctrl_lcdif_dat\n> > + &pinctrl_lcdif_ctrl>;\n> \n> Could fit into a single line.\n> \n> > + panel {\n> > + compatible = \"admatec,t070p133t0s301\";\n> \n> Same here. Undocumented binding.\n> \n> > + backlight = <&backlight>;\n> > +\n> > + port {\n> > + panel_in: endpoint {\n> > + remote-endpoint = <&display_out>;\n> > + };\n> > + };\n> > + };\n> > +};\n> > +\n> > +&i2c4 {\n> > + gt911@5d {\n> \n> Same comments as previously apply.\n> \n> > +\n> > + regulators {\n> \n> No need to have this regulators indent level.\n> \n> > + reg_3v3: regulator1 {\n> \n> You can place this one directly. The preferred format is:\n> \n> reg_3v3: regulator-reg-3v3 {\n> \n> > +&ecspi1 {\n> > + fsl,spi-num-chipselects = <1>;\n> \n> This property is obsoleted. Please remove it.\n> \n> > + cs-gpios = <&gpio4 26 GPIO_ACTIVE_HIGH>;\n> > + pinctrl-names = \"default\";\n> > + pinctrl-0 = <&pinctrl_ecspi1>;\n> > + status = \"okay\";\n> > +\n> > + fram@0 {\n> \n> Generic name please. eeprom@0\n> \n> > + compatible = \"atmel,at25\";\n> \n> Please use the recommended compatible scheme as per\n> Documentation/devicetree/bindings/eeprom/at25.txt.\n> \n> > + reg = <0>;\n> > + spi-max-frequency = <20000000>;\n> > + spi-cpha;\n> > + spi-cpol;\n> > + pagesize = <1>;\n> > + size = <8192>;\n> > + address-width = <16>;\n> > + };\n> > +};\n> \n> \n> > +&usbotg1 {\n> > + pinctrl-names = \"default\";\n> > + pinctrl-0 = <&pinctrl_usbotg1>;\n> > + dr_mode = \"otg\";\n> > + status = \"okay\";\n> \n> We prefer to put the 'status' property as the last one.\n> \n> > + srp-disable;\n> > + hnp-disable;\n> > + adp-disable;\n> > + vbus-supply = <®_usb_otg1_vbus>;\n> > +};\n> \n> > +/ {\n> > + model = \"Kontron N6310 SOM\";\n> > + compatible = \"kontron,n6310-som\", \"fsl,imx6ul\";\n> > +\n> > + memory@80000000 {\n> \n> device_type = \"memory\"; is missing here.\n> \n> > + reg = <0x80000000 0x10000000>;\n> > + };\n> > +};\n> > +\n> > +&cpu0 {\n> > + clock-frequency = <528000000>;\n> \n> Is this one really needed?\n\nI'll check.\n\n> \n> > +&ecspi2 {\n> > + cs-gpios = <&gpio4 22 GPIO_ACTIVE_HIGH>;\n> > + pinctrl-names = \"default\";\n> > + pinctrl-0 = <&pinctrl_ecspi2>;\n> > + status = \"okay\";\n> > +\n> > + flash: mx25v80@0 {\n> \n> spi-flash@0\n> \n> > +&qspi {\n> > + pinctrl-names = \"default\";\n> > + pinctrl-0 = <&pinctrl_qspi>;\n> > + status = \"okay\";\n> > +\n> > + flash0: w25m02gv@0 {\n> \n> generic name, please.\n\nThanks for review!\n\nBest regards,\nKrzysztof","headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org;\n\treceiver=)","ozlabs.org;\n\tdmarc=fail (p=none dis=none) header.from=kernel.org"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 45wmG66skWz9sBF\n\tfor ;\n\tSat, 27 Jul 2019 22:58:06 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1728824AbfG0M6F (ORCPT\n\t);\n\tSat, 27 Jul 2019 08:58:05 -0400","from mail-wm1-f65.google.com ([209.85.128.65]:34455 \"EHLO\n\tmail-wm1-f65.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1728431AbfG0M6E (ORCPT\n\t); Sat, 27 Jul 2019 08:58:04 -0400","by mail-wm1-f65.google.com with SMTP id w9so39776635wmd.1;\n\tSat, 27 Jul 2019 05:58:02 -0700 (PDT)","from kozik-lap ([194.230.155.239])\n\tby smtp.googlemail.com with ESMTPSA id\n\tt6sm61711574wmb.29.2019.07.27.05.58.00\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tSat, 27 Jul 2019 05:58:00 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=yuT/hglJeQK1J4nRZbzbYMdV0GZGrTOn1crG/aogOjo=;\n\tb=KinBUzVk4Ol62qB6bOd3MLWEIg/sogi0VfBK0LpkXL+c56gVeTaQ1sd6fi0ULjpH+U\n\tWl8siMbPHGiW+Ks2E4gLamP05rXHBdnYzPyWuCk8v59PFEz8xyoHO+nWs+AIRKamr2un\n\tEej32QtFVa+8baONRmG2QyOsH7gBEn5WIeORWk622vz+CyVglV1pFv4Vqt7wW6myyIop\n\tsmOLM3YLb3MvXVMWVjTNbyo0wDINYjCNWouIEfxJOK032A0E1McxNINjjxRhavOMT61V\n\thRdi2CtFt7TDJXRCdYsakNuiGdZ2yr9DgW2ZTNv2JZ5q2nOgPQYx7uXnto6eHhePoQRV\n\tQi6g==","X-Gm-Message-State":"APjAAAWTuDuDcCSeJWJaGpTKSZMNafak1BIONAkfb2uQu65MJe1oFWnV\n\ti6Y7JqpcCSJH0qKUy4+SQ3o=","X-Google-Smtp-Source":"APXvYqz38Nf2fk0FktWCXHAAwXHsasNBsHe0MvvFBhth9CAgwqVF0lpDciMlG1FTvAkbpg1pBL0fmQ==","X-Received":"by 2002:a1c:a481:: with SMTP id\n\tn123mr84460454wme.123.1564232281278; \n\tSat, 27 Jul 2019 05:58:01 -0700 (PDT)","Date":"Sat, 27 Jul 2019 14:57:58 +0200","From":"Krzysztof Kozlowski ","To":"Fabio Estevam ","Cc":"Rob Herring , Mark Rutland ,\n\tShawn Guo , Sascha Hauer ,\n\tPengutronix Kernel Team ,\n\tNXP Linux Team ,\n\t\"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS\" \n\t,\n\tlinux-kernel ,\n\t\"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE\" \n\t,\n\tSchrempf Frieder ","Subject":"Re: [PATCH v2 2/2] ARM: dts: imx6ul-kontron-n6310: Add Kontron\n\ti.MX6UL N6310 SoM and boards","Message-ID":"<20190727125758.GA7674@kozik-lap>","References":"<20190726061705.14764-1-krzk@kernel.org>\n\t<20190726061705.14764-2-krzk@kernel.org>\n\t","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"","User-Agent":"Mutt/1.9.4 (2018-02-28)","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"}}]