linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Dong Aisheng <dongas86@gmail.com>
To: Jacky Bai <ping.bai@nxp.com>
Cc: Rob Herring <robh+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
	 Stephen Boyd <sboyd@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	 Linus Walleij <linus.walleij@linaro.org>,
	Dong Aisheng <aisheng.dong@nxp.com>,
	 Fabio Estevam <festevam@gmail.com>,
	Sascha Hauer <kernel@pengutronix.de>,
	 dl-linux-imx <linux-imx@nxp.com>,
	devicetree <devicetree@vger.kernel.org>,
	 "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 11/11] arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board
Date: Fri, 11 Jun 2021 22:43:11 +0800	[thread overview]
Message-ID: <CAA+hA=Qf=a8QKBFJvH2pnVOu6j1DV4tGVLLhed2DOZ3TNHaybg@mail.gmail.com> (raw)
In-Reply-To: <20210607083921.2668568-12-ping.bai@nxp.com>

On Mon, Jun 7, 2021 at 4:39 PM Jacky Bai <ping.bai@nxp.com> wrote:
>
> Add the basic dts file for i.MX8ULP EVK board.
> Only the necessary devices for minimal system boot up are enabled:
> enet, emmc, usb, console uart.
>
> some of the devices' pin status may lost during low power mode,
> so additional sleep pinctrl properties are included by default.
>
> Signed-off-by: Jacky Bai <ping.bai@nxp.com>

Mostly looks good

> ---
>  arch/arm64/boot/dts/freescale/Makefile        |   1 +
>  arch/arm64/boot/dts/freescale/imx8ulp-evk.dts | 143 ++++++++++++++++++
>  2 files changed, 144 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
>
> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
> index 25806c4924cb..8c24a05d55af 100644
> --- a/arch/arm64/boot/dts/freescale/Makefile
> +++ b/arch/arm64/boot/dts/freescale/Makefile
> @@ -65,5 +65,6 @@ dtb-$(CONFIG_ARCH_MXC) += imx8qm-mek.dtb
>  dtb-$(CONFIG_ARCH_MXC) += imx8qxp-ai_ml.dtb
>  dtb-$(CONFIG_ARCH_MXC) += imx8qxp-colibri-eval-v3.dtb
>  dtb-$(CONFIG_ARCH_MXC) += imx8qxp-mek.dtb
> +dtb-$(CONFIG_ARCH_MXC) += imx8ulp-evk.dtb
>
>  dtb-$(CONFIG_ARCH_S32) += s32v234-evb.dtb
> diff --git a/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> new file mode 100644
> index 000000000000..f4db04f357fe
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8ulp-evk.dts
> @@ -0,0 +1,143 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +/*
> + * Copyright 2021 NXP
> + */
> +
> +/dts-v1/;
> +
> +#include "imx8ulp.dtsi"
> +
> +/ {
> +       model = "NXP i.MX8ULP EVK";
> +       compatible = "fsl,imx8ulp-evk", "fsl,imx8ulp";
> +
> +       chosen {
> +               stdout-path = &lpuart5;
> +       };

no memory node?

Regards
Aisheng

> +};
> +
> +&fec {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&pinctrl_enet>;
> +       phy-mode = "rmii";
> +       phy-handle = <&ethphy>;
> +       status = "okay";
> +
> +       mdio {
> +               #address-cells = <1>;
> +               #size-cells = <0>;
> +
> +               ethphy: ethernet-phy {
> +                       reg = <1>;
> +                       micrel,led-mode = <1>;
> +               };
> +       };
> +};
> +
> +&lpuart5 {
> +       /* console */
> +       pinctrl-names = "default", "sleep";
> +       pinctrl-0 = <&pinctrl_lpuart5>;
> +       pinctrl-1 = <&pinctrl_lpuart5>;
> +       status = "okay";
> +};
> +
> +&usbotg1 {
> +       pinctrl-names = "default", "sleep";
> +       pinctrl-0 = <&pinctrl_otgid1>;
> +       pinctrl-1 = <&pinctrl_otgid1>;
> +       dr_mode = "otg";
> +       hnp-disable;
> +       srp-disable;
> +       adp-disable;
> +       status = "okay";
> +};
> +
> +&usbphy1 {
> +       status = "okay";
> +};
> +
> +&usbmisc1 {
> +       status = "okay";
> +};
> +
> +&usbotg2 {
> +       pinctrl-names = "default", "sleep";
> +       pinctrl-0 = <&pinctrl_otgid2>;
> +       pinctrl-1 = <&pinctrl_otgid2>;
> +       dr_mode = "otg";
> +       hnp-disable;
> +       srp-disable;
> +       adp-disable;
> +       status = "okay";
> +};
> +
> +&usbphy2 {
> +       status = "okay";
> +};
> +
> +&usbmisc2 {
> +       status = "okay";
> +};
> +
> +&usdhc0 {
> +       pinctrl-names = "default", "sleep";
> +       pinctrl-0 = <&pinctrl_usdhc0>;
> +       pinctrl-1 = <&pinctrl_usdhc0>;
> +       non-removable;
> +       bus-width = <4>;
> +       status = "okay";
> +};
> +
> +&iomuxc1 {
> +       pinctrl_enet: enetgrp {
> +               fsl,pins = <
> +                       MX8ULP_PAD_PTE15__ENET0_MDC     0x43
> +                       MX8ULP_PAD_PTE14__ENET0_MDIO    0x43
> +                       MX8ULP_PAD_PTE17__ENET0_RXER    0x43
> +                       MX8ULP_PAD_PTE18__ENET0_CRS_DV  0x43
> +                       MX8ULP_PAD_PTF1__ENET0_RXD0     0x43
> +                       MX8ULP_PAD_PTE20__ENET0_RXD1    0x43
> +                       MX8ULP_PAD_PTE16__ENET0_TXEN    0x43
> +                       MX8ULP_PAD_PTE23__ENET0_TXD0    0x43
> +                       MX8ULP_PAD_PTE22__ENET0_TXD1    0x43
> +                       MX8ULP_PAD_PTE19__ENET0_REFCLK  0x43
> +                       MX8ULP_PAD_PTF10__ENET0_1588_CLKIN 0x43
> +               >;
> +       };
> +
> +       pinctrl_lpuart5: lpuart5grp {
> +               fsl,pins = <
> +                       MX8ULP_PAD_PTF14__LPUART5_TX    0x3
> +                       MX8ULP_PAD_PTF15__LPUART5_RX    0x3
> +               >;
> +       };
> +
> +       pinctrl_otgid1: usb1grp {
> +               fsl,pins = <
> +                       MX8ULP_PAD_PTF2__USB0_ID        0x10003
> +               >;
> +       };
> +
> +       pinctrl_otgid2: usb2grp {
> +               fsl,pins = <
> +                       MX8ULP_PAD_PTD23__USB1_ID       0x10003
> +               >;
> +       };
> +
> +       pinctrl_usdhc0: usdhc0grp {
> +               fsl,pins = <
> +                       MX8ULP_PAD_PTD1__SDHC0_CMD      0x43
> +                       MX8ULP_PAD_PTD2__SDHC0_CLK      0x10042
> +                       MX8ULP_PAD_PTD10__SDHC0_D0      0x43
> +                       MX8ULP_PAD_PTD9__SDHC0_D1       0x43
> +                       MX8ULP_PAD_PTD8__SDHC0_D2       0x43
> +                       MX8ULP_PAD_PTD7__SDHC0_D3       0x43
> +                       MX8ULP_PAD_PTD6__SDHC0_D4       0x43
> +                       MX8ULP_PAD_PTD5__SDHC0_D5       0x43
> +                       MX8ULP_PAD_PTD4__SDHC0_D6       0x43
> +                       MX8ULP_PAD_PTD3__SDHC0_D7       0x43
> +                       MX8ULP_PAD_PTD11__SDHC0_DQS     0x10042
> +               >;
> +       };
> +};
> --
> 2.26.2
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2021-06-11 14:47 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-07  8:39 [PATCH 00/11] Add imx8ulp basic dtsi support Jacky Bai
2021-06-07  8:39 ` [PATCH 01/11] dt-bindings: gpio: gpio-vf610: Add imx8ulp compatible string Jacky Bai
2021-06-09 10:56   ` Linus Walleij
2021-06-11 13:45   ` Dong Aisheng
2021-06-16 23:42   ` Rob Herring
2021-06-07  8:39 ` [PATCH 02/11] dt-bindings: i2c: imx-lpi2c: " Jacky Bai
2021-06-11 13:46   ` Dong Aisheng
2021-06-16 23:43   ` Rob Herring
2021-06-07  8:39 ` [PATCH 03/11] dt-bindings: mmc: imx-esdhc: Add imx8ulp compatibe string Jacky Bai
2021-06-11 13:50   ` Dong Aisheng
2021-06-16 23:44   ` Rob Herring
2021-06-07  8:39 ` [PATCH 04/11] dt-bindings: serial: fsl-lpuart: Add imx8ulp compatible string Jacky Bai
2021-06-11 13:52   ` Dong Aisheng
2021-06-16 23:44   ` Rob Herring
2021-06-07  8:39 ` [PATCH 05/11] dt-bindings: spi: fsl-lpspi: " Jacky Bai
2021-06-11 13:57   ` Dong Aisheng
2021-06-16 23:45   ` Rob Herring
2021-06-07  8:39 ` [PATCH 06/11] dt-bindings: timer: tpm-timer: " Jacky Bai
2021-06-11 13:58   ` Dong Aisheng
2021-06-16 23:45   ` Rob Herring
2021-06-07  8:39 ` [PATCH 07/11] dt-bindings: watchdog: imx7ulp-wdt: " Jacky Bai
2021-06-11 13:58   ` Dong Aisheng
2021-06-16 23:45   ` Rob Herring
2021-06-07  8:39 ` [PATCH 08/11] dt-bindings: arm: fsl: Add binding for imx8ulp evk Jacky Bai
2021-06-11 13:59   ` Dong Aisheng
2021-06-16 23:46   ` Rob Herring
2021-06-07  8:39 ` [PATCH 09/11] dt-bindings: clock: Add imx8ulp clock support Jacky Bai
2021-06-11 14:17   ` Dong Aisheng
2021-06-16 23:47   ` Rob Herring
2021-06-17  9:09     ` Jacky Bai
2021-06-07  8:39 ` [PATCH 11/11] arm64: dts: imx8ulp: Add the basic dts for imx8ulp evk board Jacky Bai
2021-06-11 14:43   ` Dong Aisheng [this message]
     [not found] ` <20210607083921.2668568-11-ping.bai@nxp.com>
2021-06-11 14:40   ` [PATCH 10/11] arm64: dts: imx8ulp: Add the basic dtsi file for imx8ulp Aisheng Dong
2021-06-16 23:51   ` Rob Herring
2021-06-17  9:03     ` Jacky Bai
2021-06-18  1:41       ` Jacky Bai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAA+hA=Qf=a8QKBFJvH2pnVOu6j1DV4tGVLLhed2DOZ3TNHaybg@mail.gmail.com' \
    --to=dongas86@gmail.com \
    --cc=aisheng.dong@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=kernel@pengutronix.de \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=ping.bai@nxp.com \
    --cc=robh+dt@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=sboyd@kernel.org \
    --cc=shawnguo@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).