From: Fabio Estevam <festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Bai Ping <ping.bai-3arQi8VN3Tc@public.gmane.org>
Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>,
Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
NXP Linux Team <linux-imx-3arQi8VN3Tc@public.gmane.org>,
Dong Aisheng <aisheng.dong-3arQi8VN3Tc@public.gmane.org>,
jacky.baip-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
Subject: Re: [PATCH v3 2/2] ARM: dts: imx: Add basic dts support for imx6sll EVK board
Date: Wed, 7 Feb 2018 00:44:42 -0200 [thread overview]
Message-ID: <CAOMZO5DEvVsGbbG_w116OX625DJos0GHsmjUUBkgwVk7NLONMw@mail.gmail.com> (raw)
In-Reply-To: <1517969481-13411-2-git-send-email-ping.bai-3arQi8VN3Tc@public.gmane.org>
On Wed, Feb 7, 2018 at 12:11 AM, Bai Ping <ping.bai-3arQi8VN3Tc@public.gmane.org> wrote:
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx6sll-evk.dts
> @@ -0,0 +1,374 @@
> +/*
> + * Copyright 2016 Freescale Semiconductor, Inc.
> + * Copyright 2017-2018 NXP.
> + *
> + * SPDX-License-Identifier: (GPL-2.0 OR MIT)
The SPDX line should be the first one and it should start with //
> + */
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include "imx6sll.dtsi"
> +
> +/ {
> + model = "Freescale i.MX6SLL EVK Board";
> + compatible = "fsl,imx6sll-evk", "fsl,imx6sll";
> +
> + memory {
memory@80000000
> + reg = <0x80000000 0x80000000>;
> + reg_aud4v: reg-aud4v {
> + compatible = "regulator-fixed";
> + regulator-name = "wm8962-supply-4v2";
> + regulator-min-microvolt = <4325000>;
> + regulator-max-microvolt = <4325000>;
> + regulator-boot-on;
> + };
> +
> + reg_lcd: reg-lcd {
> + compatible = "regulator-fixed";
> + regulator-name = "lcd-pwr";
> + gpio = <&gpio4 8 0>;
Please use GPIO_ACTIVE_HIGH here.
> +&i2c1 {
> + clock-frequency = <100000>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_i2c1>;
> + status = "okay";
> +
> + pmic: pfuze100@08 {
Please remove the leading 0, so this should be @8.
Please build it with W=1 and make sure it generates no warnings.
> +&iomuxc {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_hog0>, <&pinctrl_hog1>;
> +
> + pinctrl_hog0: hog0grp {
> + pinmux = <
> + MX6SLL_PAD_KEY_ROW7__GPIO4_IO07
> + MX6SLL_PAD_GPIO4_IO22__GPIO4_IO22
> + MX6SLL_PAD_KEY_COL3__GPIO3_IO30
> + MX6SLL_PAD_KEY_COL4__GPIO4_IO00
> + MX6SLL_PAD_REF_CLK_32K__GPIO3_IO22 /* SD3 CD */
I see no SD3 pinctrl entry. If this is related to SD3 card detection,
then it should be part of SD3 pinctrl instead of hog.
> + MX6SLL_PAD_KEY_COL6__GPIO4_IO04 /*SD3 RESET */
Same here.
> + MX6SLL_PAD_KEY_COL5__GPIO4_IO02
> + MX6SLL_PAD_GPIO4_IO24__GPIO4_IO24 /* HP DETECT */
Does HP mean head phone? If so, this should be part of audio pinctrl
instead of hog.
> +&usdhc1 {
> + pinctrl-names = "default", "state_100mhz", "state_200mhz";
> + pinctrl-0 = <&pinctrl_usdhc1_data>, <&pinctrl_usdhc1_clk>;
> + pinctrl-1 = <&pinctrl_usdhc1_data_100mhz>, <&pinctrl_usdhc1_clk_100mhz>;
> + pinctrl-2 = <&pinctrl_usdhc1_data_200mhz>, <&pinctrl_usdhc1_clk_200mhz>;
> + cd-gpios = <&gpio4 7 GPIO_ACTIVE_LOW>;
> + wp-gpios = <&gpio4 22 GPIO_ACTIVE_HIGH>;
> + keep-power-in-suspend;
> + enable-sdio-wakeup;
This property is deprecated as described in
Documentation/devicetree/bindings/mmc/mmc.txt.
Please use 'wakeup-source' instead.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2018-02-07 2:44 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-07 2:11 [PATCH v3 1/2] ARM: dts: imx: Add basic dtsi file for imx6sll Bai Ping
[not found] ` <1517969481-13411-1-git-send-email-ping.bai-3arQi8VN3Tc@public.gmane.org>
2018-02-07 2:11 ` [PATCH v3 2/2] ARM: dts: imx: Add basic dts support for imx6sll EVK board Bai Ping
[not found] ` <1517969481-13411-2-git-send-email-ping.bai-3arQi8VN3Tc@public.gmane.org>
2018-02-07 2:44 ` Fabio Estevam [this message]
[not found] ` <CAOMZO5DEvVsGbbG_w116OX625DJos0GHsmjUUBkgwVk7NLONMw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-07 2:52 ` Jacky Bai
2018-02-07 2:52 ` [PATCH v3 1/2] ARM: dts: imx: Add basic dtsi file for imx6sll Fabio Estevam
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=CAOMZO5DEvVsGbbG_w116OX625DJos0GHsmjUUBkgwVk7NLONMw@mail.gmail.com \
--to=festevam-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=aisheng.dong-3arQi8VN3Tc@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=fabio.estevam-3arQi8VN3Tc@public.gmane.org \
--cc=jacky.baip-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-imx-3arQi8VN3Tc@public.gmane.org \
--cc=ping.bai-3arQi8VN3Tc@public.gmane.org \
--cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.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).