[PATCH 1/4] ARM: mxs: dts: Add SSP2 SD mux

[PATCH 1/4] ARM: mxs: dts: Add SSP2 SD mux

[Marek Vasut]

Add pinmux configuration for SSP2 port in SD mode, both for
the 4-bit and 8-bit case.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
Cc: Shawn Guo <shawnguo@kernel.org>
---
 arch/arm/boot/dts/imx23.dtsi | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi
index 302d116..c70ebef 100644
--- a/arch/arm/boot/dts/imx23.dtsi
+++ b/arch/arm/boot/dts/imx23.dtsi
@@ -249,6 +249,40 @@
 					fsl,pull-up = <MXS_PULL_DISABLE>;
 				};
 
+				mmc1_4bit_pins_a: mmc1-4bit at 0 {
+					reg = <0>;
+					fsl,pinmux-ids = <
+						MX23_PAD_GPMI_D00__SSP2_DATA0
+						MX23_PAD_GPMI_D01__SSP2_DATA1
+						MX23_PAD_GPMI_D02__SSP2_DATA2
+						MX23_PAD_GPMI_D03__SSP2_DATA3
+						MX23_PAD_GPMI_RDY1__SSP2_CMD
+						MX23_PAD_GPMI_WRN__SSP2_SCK
+					>;
+					fsl,drive-strength = <MXS_DRIVE_8mA>;
+					fsl,voltage = <MXS_VOLTAGE_HIGH>;
+					fsl,pull-up = <MXS_PULL_ENABLE>;
+				};
+
+				mmc1_8bit_pins_a: mmc1-8bit at 0 {
+					reg = <0>;
+					fsl,pinmux-ids = <
+						MX23_PAD_GPMI_D00__SSP2_DATA0
+						MX23_PAD_GPMI_D01__SSP2_DATA1
+						MX23_PAD_GPMI_D02__SSP2_DATA2
+						MX23_PAD_GPMI_D03__SSP2_DATA3
+						MX23_PAD_GPMI_D04__SSP2_DATA4
+						MX23_PAD_GPMI_D05__SSP2_DATA5
+						MX23_PAD_GPMI_D06__SSP2_DATA6
+						MX23_PAD_GPMI_D07__SSP2_DATA7
+						MX23_PAD_GPMI_RDY1__SSP2_CMD
+						MX23_PAD_GPMI_WRN__SSP2_SCK
+					>;
+					fsl,drive-strength = <MXS_DRIVE_8mA>;
+					fsl,voltage = <MXS_VOLTAGE_HIGH>;
+					fsl,pull-up = <MXS_PULL_ENABLE>;
+				};
+
 				pwm2_pins_a: pwm2 at 0 {
 					reg = <0>;
 					fsl,pinmux-ids = <
-- 
2.7.0

[PATCH 2/4] ARM: mxs: dts: Add AUART2 pinmux

[Marek Vasut]

Add 2-pin pinmux settings for AUART2.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
Cc: Shawn Guo <shawnguo@kernel.org>
---
 arch/arm/boot/dts/imx23.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm/boot/dts/imx23.dtsi b/arch/arm/boot/dts/imx23.dtsi
index c70ebef..0c9a357e 100644
--- a/arch/arm/boot/dts/imx23.dtsi
+++ b/arch/arm/boot/dts/imx23.dtsi
@@ -171,6 +171,17 @@
 					fsl,pull-up = <MXS_PULL_DISABLE>;
 				};
 
+				auart1_2pins_a: auart1-2pins at 0 {
+					reg = <0>;
+					fsl,pinmux-ids = <
+						MX23_PAD_GPMI_D14__AUART2_RX
+						MX23_PAD_GPMI_D15__AUART2_TX
+					>;
+					fsl,drive-strength = <MXS_DRIVE_4mA>;
+					fsl,voltage = <MXS_VOLTAGE_HIGH>;
+					fsl,pull-up = <MXS_PULL_DISABLE>;
+				};
+
 				gpmi_pins_a: gpmi-nand at 0 {
 					reg = <0>;
 					fsl,pinmux-ids = <
-- 
2.7.0

[PATCH 3/4] ARM: mxs: dts: Add Creative X-Fi3 support

[Marek Vasut]

Add support for this small MP3 player based on STMP3780 (rev.4).
Currently supported is both external microSD and internal SD-NAND
bridge, PWM and USB gadget.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
Cc: Shawn Guo <shawnguo@kernel.org>
---
 arch/arm/boot/dts/Makefile       |   1 +
 arch/arm/boot/dts/imx23-xfi3.dts | 190 +++++++++++++++++++++++++++++++++++++++
 2 files changed, 191 insertions(+)
 create mode 100644 arch/arm/boot/dts/imx23-xfi3.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 8f9f1c2..7a79d37 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -417,6 +417,7 @@ dtb-$(CONFIG_ARCH_MXS) += \
 	imx23-evk.dtb \
 	imx23-olinuxino.dtb \
 	imx23-stmp378x_devb.dtb \
+	imx23-xfi3.dtb \
 	imx28-apf28.dtb \
 	imx28-apf28dev.dtb \
 	imx28-apx4devkit.dtb \
diff --git a/arch/arm/boot/dts/imx23-xfi3.dts b/arch/arm/boot/dts/imx23-xfi3.dts
new file mode 100644
index 0000000..d7e7cf5
--- /dev/null
+++ b/arch/arm/boot/dts/imx23-xfi3.dts
@@ -0,0 +1,190 @@
+/*
+ * Copyright (C) 2013-2016 Marek Vasut <marex@denx.de>
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ *     modify it under the terms of the GNU General Public License as
+ *     published by the Free Software Foundation; either version 2 of
+ *     the License, or (at your option) any later version.
+ *
+ *     This file is distributed in the hope that it will be useful,
+ *     but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *     GNU General Public License for more details.
+ *
+ *     You should have received a copy of the GNU General Public
+ *     License along with this file; if not, write to the Free
+ *     Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston,
+ *     MA 02110-1301 USA
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ *     obtaining a copy of this software and associated documentation
+ *     files (the "Software"), to deal in the Software without
+ *     restriction, including without limitation the rights to use,
+ *     copy, modify, merge, publish, distribute, sublicense, and/or
+ *     sell copies of the Software, and to permit persons to whom the
+ *     Software is furnished to do so, subject to the following
+ *     conditions:
+ *
+ *     The above copyright notice and this permission notice shall be
+ *     included in all copies or substantial portions of the Software.
+ *
+ *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ *     OTHER DEALINGS IN THE SOFTWARE.
+ *
+ */
+
+/dts-v1/;
+#include "imx23.dtsi"
+
+/ {
+	model = "Creative ZEN X-Fi3";
+	compatible = "creative,x-fi3", "fsl,imx23";
+
+	memory {
+		reg = <0x40000000 0x04000000>;
+	};
+
+	apb at 80000000 {
+		apbh at 80000000 {
+			ssp0: ssp at 80010000 {
+				compatible = "fsl,imx23-mmc";
+				pinctrl-names = "default";
+				pinctrl-0 = <&mmc0_4bit_pins_a &mmc0_pins_fixup>;
+				bus-width = <4>;
+				vmmc-supply = <&reg_vddio_sd0>;
+				cd-inverted;
+				status = "okay";
+			};
+
+			ssp1: ssp at 80034000 {
+				compatible = "fsl,imx23-mmc";
+				pinctrl-names = "default";
+				pinctrl-0 = <&mmc1_4bit_pins_a>;
+				bus-width = <4>;
+				non-removable;
+				status = "okay";
+			};
+
+			pinctrl at 80018000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&hog_pins_a>;
+
+				hog_pins_a: hog at 0 {
+					reg = <0>;
+					fsl,pinmux-ids = <
+						MX23_PAD_GPMI_D07__GPIO_0_7
+					>;
+					fsl,drive-strength = <0>;
+					fsl,voltage = <1>;
+					fsl,pull-up = <0>;
+				};
+
+				key_pins_a: keys at 0 {
+					reg = <0>;
+					fsl,pinmux-ids = <
+						MX23_PAD_ROTARYA__GPIO_2_7
+						MX23_PAD_ROTARYB__GPIO_2_8
+					>;
+					fsl,drive-strength = <0>;
+					fsl,voltage = <1>;
+					fsl,pull-up = <1>;
+				};
+			};
+		};
+
+		apbx at 80040000 {
+			i2c: i2c at 80058000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&i2c_pins_a>;
+				status = "okay";
+			};
+
+			pwm: pwm at 80064000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&pwm2_pins_a>;
+				status = "okay";
+			};
+
+			duart: serial at 80070000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&duart_pins_a>;
+				status = "okay";
+			};
+
+			auart1: serial at 8006e000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&auart1_2pins_a>;
+				status = "okay";
+			};
+
+			usbphy0: usbphy at 8007c000 {
+				status = "okay";
+			};
+
+			lradc at 80050000 {
+				status = "okay";
+			};
+		};
+	};
+
+	ahb at 80080000 {
+		usb0: usb at 80080000 {
+			dr_mode = "peripheral";
+			status = "okay";
+		};
+	};
+
+	regulators {
+		compatible = "simple-bus";
+
+		reg_vddio_sd0: vddio-sd0 {
+			compatible = "regulator-fixed";
+			regulator-name = "vddio-sd0";
+			regulator-min-microvolt = <3300000>;
+			regulator-max-microvolt = <3300000>;
+			gpio = <&gpio0 7 0>;
+		};
+	};
+
+	backlight {
+		compatible = "pwm-backlight";
+		pwms = <&pwm 2 5000000>;
+		brightness-levels = <0 4 8 16 32 64 128 255>;
+		default-brightness-level = <6>;
+	};
+
+	gpio_keys {
+		compatible = "gpio-keys";
+		pinctrl-names = "default";
+		pinctrl-0 = <&key_pins_a>;
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		button at 0 {
+			label = "volume-down";
+			linux,code = <114>;
+			gpios = <&gpio2 7 0>;
+			debounce-interval = <20>;
+		};
+
+		button at 1 {
+			label = "volume-up";
+			linux,code = <115>;
+			gpios = <&gpio2 8 0>;
+			debounce-interval = <20>;
+		};
+	};
+};
-- 
2.7.0

[PATCH 4/4] ARM: mxs: dts: Add SanDisk Sansa Fuze+ support

[Marek Vasut]

Add support for this small MP3 player based on STMP3780 (rev.3).
Currently supported are both external microSD and internal SD-NAND
bridge, PWM and USB gadget.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>
Cc: Shawn Guo <shawnguo@kernel.org>
---
 arch/arm/boot/dts/Makefile        |   1 +
 arch/arm/boot/dts/imx23-sansa.dts | 216 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 217 insertions(+)
 create mode 100644 arch/arm/boot/dts/imx23-sansa.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 7a79d37..eaa8048 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -416,6 +416,7 @@ dtb-$(CONFIG_SOC_VF610) += \
 dtb-$(CONFIG_ARCH_MXS) += \
 	imx23-evk.dtb \
 	imx23-olinuxino.dtb \
+	imx23-sansa.dtb \
 	imx23-stmp378x_devb.dtb \
 	imx23-xfi3.dtb \
 	imx28-apf28.dtb \
diff --git a/arch/arm/boot/dts/imx23-sansa.dts b/arch/arm/boot/dts/imx23-sansa.dts
new file mode 100644
index 0000000..60a578c
--- /dev/null
+++ b/arch/arm/boot/dts/imx23-sansa.dts
@@ -0,0 +1,216 @@
+/*
+ * Copyright (C) 2013-2016 Marek Vasut <marex@denx.de>
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ *     modify it under the terms of the GNU General Public License as
+ *     published by the Free Software Foundation; either version 2 of
+ *     the License, or (at your option) any later version.
+ *
+ *     This file is distributed in the hope that it will be useful,
+ *     but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *     GNU General Public License for more details.
+ *
+ *     You should have received a copy of the GNU General Public
+ *     License along with this file; if not, write to the Free
+ *     Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston,
+ *     MA 02110-1301 USA
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ *     obtaining a copy of this software and associated documentation
+ *     files (the "Software"), to deal in the Software without
+ *     restriction, including without limitation the rights to use,
+ *     copy, modify, merge, publish, distribute, sublicense, and/or
+ *     sell copies of the Software, and to permit persons to whom the
+ *     Software is furnished to do so, subject to the following
+ *     conditions:
+ *
+ *     The above copyright notice and this permission notice shall be
+ *     included in all copies or substantial portions of the Software.
+ *
+ *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ *     OTHER DEALINGS IN THE SOFTWARE.
+ *
+ */
+
+/dts-v1/;
+#include "imx23.dtsi"
+
+/ {
+	model = "SanDisk Sansa Fuze+";
+	compatible = "sandisk,sansa_fuze_plus", "fsl,imx23";
+
+	memory {
+		reg = <0x40000000 0x04000000>;
+	};
+
+	apb at 80000000 {
+		apbh at 80000000 {
+			ssp0: ssp at 80010000 {
+				compatible = "fsl,imx23-mmc";
+				pinctrl-names = "default";
+				pinctrl-0 = <&mmc0_4bit_pins_a &mmc0_pins_fixup>;
+				bus-width = <4>;
+				vmmc-supply = <&reg_vddio_sd0>;
+				cd-inverted;
+				status = "okay";
+			};
+
+			ssp1: ssp at 80034000 {
+				compatible = "fsl,imx23-mmc";
+				pinctrl-names = "default";
+				pinctrl-0 = <&mmc1_8bit_pins_a>;
+				bus-width = <8>;
+				vmmc-supply = <&reg_vddio_sd1>;
+				non-removable;
+				status = "okay";
+			};
+
+			pinctrl at 80018000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&hog_pins_a>;
+
+				hog_pins_a: hog at 0 {
+					reg = <0>;
+					fsl,pinmux-ids = <
+						MX23_PAD_GPMI_D08__GPIO_0_8
+						MX23_PAD_PWM3__GPIO_1_29
+						MX23_PAD_AUART1_RTS__GPIO_0_27
+						MX23_PAD_AUART1_CTS__GPIO_0_26
+						MX23_PAD_I2C_SCL__I2C_SCL
+						MX23_PAD_I2C_SDA__I2C_SDA
+						MX23_PAD_LCD_DOTCK__GPIO_1_22
+						MX23_PAD_LCD_HSYNC__GPIO_1_24
+						MX23_PAD_PWM3__GPIO_1_29
+					>;
+					fsl,drive-strength = <0>;
+					fsl,voltage = <1>;
+					fsl,pull-up = <0>;
+				};
+			};
+		};
+
+		apbx at 80040000 {
+			pwm: pwm at 80064000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&pwm2_pins_a>;
+				status = "okay";
+			};
+
+			duart: serial at 80070000 {
+				pinctrl-names = "default";
+				pinctrl-0 = <&duart_pins_a>;
+				status = "okay";
+			};
+
+			usbphy0: usbphy at 8007c000 {
+				status = "okay";
+			};
+
+			lradc at 80050000 {
+				status = "okay";
+			};
+		};
+	};
+
+	ahb at 80080000 {
+		usb0: usb at 80080000 {
+			dr_mode = "peripheral";
+			status = "okay";
+		};
+	};
+
+	regulators {
+		compatible = "simple-bus";
+
+		reg_vddio_sd0: vddio-sd0 {
+			compatible = "regulator-fixed";
+			regulator-name = "vddio-sd0";
+			regulator-min-microvolt = <3300000>;
+			regulator-max-microvolt = <3300000>;
+			gpio = <&gpio0 8 0>;
+		};
+
+		reg_vddio_sd1: vddio-sd1 {
+			compatible = "regulator-fixed";
+			regulator-name = "vddio-sd1";
+			regulator-min-microvolt = <3300000>;
+			regulator-max-microvolt = <3300000>;
+			gpio = <&gpio1 29 0>;
+		};
+
+		reg_vdd_touchpad: vdd-touchpad0 {
+			compatible = "regulator-fixed";
+			regulator-name = "vdd-touchpad0";
+			regulator-min-microvolt = <3300000>;
+			regulator-max-microvolt = <3300000>;
+			gpio = <&gpio0 26 0>;
+			regulator-always-on;
+			enable-active-low;
+		};
+
+		reg_vdd_tuner: vdd-tuner0 {
+			compatible = "regulator-fixed";
+			regulator-name = "vdd-tuner0";
+			regulator-min-microvolt = <3300000>;
+			regulator-max-microvolt = <3300000>;
+			gpio = <&gpio0 29 0>;
+			regulator-always-on;
+			enable-active-low;
+		};
+	};
+
+	backlight {
+		compatible = "pwm-backlight";
+		pwms = <&pwm 2 5000000>;
+		brightness-levels = <0 4 8 16 32 64 128 255>;
+		default-brightness-level = <6>;
+	};
+
+	i2c at 0 {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		compatible = "i2c-gpio";
+		gpios = <
+			&gpio1 24 0		/* SDA */
+			&gpio1 22 0		/* SCL */
+		>;
+		i2c-gpio,delay-us = <2>;	/* ~100 kHz */
+	};
+
+	i2c at 1 {
+		#address-cells = <1>;
+		#size-cells = <0>;
+		compatible = "i2c-gpio";
+		gpios = <
+			&gpio0 31 0		/* SDA */
+			&gpio0 30 0		/* SCL */
+		>;
+		i2c-gpio,delay-us = <2>;	/* ~100 kHz */
+
+		touch: touch at 20 {
+			compatible = "synaptics,synaptics_i2c";
+			reg = <0x20>;
+		};
+
+		eeprom: eeprom at 50 {
+			compatible = "atmel,24c64";
+			reg = <0x50>;
+			pagesize = <32>;
+		};
+	};
+
+};
-- 
2.7.0

[PATCH 3/4] ARM: mxs: dts: Add Creative X-Fi3 support

[Stefan Wahren]

Hi Marek,

sorry for the reply format.

> Marek Vasut <marex@denx.de> hat am 5. Mai 2016 um 22:26 geschrieben:
> 
> Add support for this small MP3 player based on STMP3780 (rev.4).
> Currently supported is both external microSD and internal SD-NAND
> bridge, PWM and USB gadget.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Fabio Estevam <fabio.estevam@nxp.com>
> Cc: Shawn Guo <shawnguo@kernel.org>
> ---
>  arch/arm/boot/dts/Makefile | 1 +
>  arch/arm/boot/dts/imx23-xfi3.dts | 190
> +++++++++++++++++++++++++++++++++++++++
>  2 files changed, 191 insertions(+)
>  create mode 100644 arch/arm/boot/dts/imx23-xfi3.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 8f9f1c2..7a79d37 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -417,6 +417,7 @@ dtb-$(CONFIG_ARCH_MXS) += \
>  imx23-evk.dtb \
>  imx23-olinuxino.dtb \
>  imx23-stmp378x_devb.dtb \
> + imx23-xfi3.dtb \
>  imx28-apf28.dtb \
>  imx28-apf28dev.dtb \
>  imx28-apx4devkit.dtb \
> diff --git a/arch/arm/boot/dts/imx23-xfi3.dts
> b/arch/arm/boot/dts/imx23-xfi3.dts
> new file mode 100644
> index 0000000..d7e7cf5
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx23-xfi3.dts
> @@ -0,0 +1,190 @@
> +/*
> + * Copyright (C) 2013-2016 Marek Vasut <marex@denx.de>
> + *
> + * This file is dual-licensed: you can use it either under the terms
> + * of the GPL or the X11 license, at your option. Note that this dual
> + * licensing only applies to this file, and not this project as a
> + * whole.
> + *
> + * a) This file is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This file is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public
> + * License along with this file; if not, write to the Free
> + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston,
> + * MA 02110-1301 USA

AFAIK this address shouldn't be included.

> + *
> + * Or, alternatively,
> + *
> + * b) Permission is hereby granted, free of charge, to any person
> + * obtaining a copy of this software and associated documentation
> + * files (the "Software"), to deal in the Software without
> + * restriction, including without limitation the rights to use,
> + * copy, modify, merge, publish, distribute, sublicense, and/or
> + * sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following
> + * conditions:
> + *
> + * The above copyright notice and this permission notice shall be
> + * included in all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
> + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
> + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
> + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
> + * OTHER DEALINGS IN THE SOFTWARE.
> + *
> + */
> +
> +/dts-v1/;
> +#include "imx23.dtsi"
> +
> +/ {
> + model = "Creative ZEN X-Fi3";
> + compatible = "creative,x-fi3", "fsl,imx23";

Is there a pending patch to add "creative" to the vendor-prefixes.txt?

> +
> + memory {
> + reg = ;
> + };
> +
> + apb at 80000000 {
> + apbh at 80000000 {
> + ssp0: ssp at 80010000 {
> + compatible = "fsl,imx23-mmc";
> + pinctrl-names = "default";
> + pinctrl-0 = <&mmc0_4bit_pins_a &mmc0_pins_fixup>;
> + bus-width = ;
> + vmmc-supply = <&reg_vddio_sd0>;
> + cd-inverted;
> + status = "okay";
> + };
> +
> + ssp1: ssp at 80034000 {
> + compatible = "fsl,imx23-mmc";
> + pinctrl-names = "default";
> + pinctrl-0 = <&mmc1_4bit_pins_a>;
> + bus-width = ;
> + non-removable;
> + status = "okay";
> + };
> +
> + pinctrl at 80018000 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&hog_pins_a>;
> +
> + hog_pins_a: hog at 0 {
> + reg = ;
> + fsl,pinmux-ids = <
> + MX23_PAD_GPMI_D07__GPIO_0_7
> + >;
> + fsl,drive-strength = ;
> + fsl,voltage = ;
> + fsl,pull-up = ;
> + };
> +
> + key_pins_a: keys at 0 {
> + reg = ;
> + fsl,pinmux-ids = <
> + MX23_PAD_ROTARYA__GPIO_2_7
> + MX23_PAD_ROTARYB__GPIO_2_8
> + >;
> + fsl,drive-strength = ;
> + fsl,voltage = ;
> + fsl,pull-up = ;
> + };
> + };
> + };
> +
> + apbx at 80040000 {
> + i2c: i2c at 80058000 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&i2c_pins_a>;
> + status = "okay";
> + };
> +
> + pwm: pwm at 80064000 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pwm2_pins_a>;
> + status = "okay";
> + };
> +
> + duart: serial at 80070000 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&duart_pins_a>;
> + status = "okay";
> + };
> +
> + auart1: serial at 8006e000 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&auart1_2pins_a>;
> + status = "okay";
> + };
> +
> + usbphy0: usbphy at 8007c000 {
> + status = "okay";
> + };
> +
> + lradc at 80050000 {
> + status = "okay";
> + };
> + };
> + };
> +
> + ahb at 80080000 {
> + usb0: usb at 80080000 {
> + dr_mode = "peripheral";
> + status = "okay";
> + };
> + };
> +
> + regulators {
> + compatible = "simple-bus";
> +
> + reg_vddio_sd0: vddio-sd0 {
> + compatible = "regulator-fixed";
> + regulator-name = "vddio-sd0";
> + regulator-min-microvolt = ;
> + regulator-max-microvolt = ;
> + gpio = <&gpio0 7 0>;
> + };
> + };
> +
> + backlight {
> + compatible = "pwm-backlight";
> + pwms = <&pwm 2 5000000>;
> + brightness-levels = ;
> + default-brightness-level = ;
> + };
> +
> + gpio_keys {
> + compatible = "gpio-keys";
> + pinctrl-names = "default";
> + pinctrl-0 = <&key_pins_a>;
> + #address-cells = ;
> + #size-cells = ;
> +
> + button at 0 {
> + label = "volume-down";
> + linux,code = ;
> + gpios = <&gpio2 7 0>;
> + debounce-interval = ;
> + };

I think this would introduce new dtc warnings.

Please look at
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=f2953a461042721e9a2aca58d2b1cfc67c70d371

> +
> + button at 1 {
> + label = "volume-up";
> + linux,code = ;
> + gpios = <&gpio2 8 0>;
> + debounce-interval = ;
> + };
> + };
> +};
> -- 
> 2.7.0
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

[PATCH 3/4] ARM: mxs: dts: Add Creative X-Fi3 support

[Marek Vasut]

On 05/06/2016 09:40 AM, Stefan Wahren wrote:
> Hi Marek,

Hi!

> sorry for the reply format.

replying from a phone ? :)

>> Marek Vasut <marex@denx.de> hat am 5. Mai 2016 um 22:26 geschrieben:
>>
>> Add support for this small MP3 player based on STMP3780 (rev.4).
>> Currently supported is both external microSD and internal SD-NAND
>> bridge, PWM and USB gadget.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> Cc: Fabio Estevam <fabio.estevam@nxp.com>
>> Cc: Shawn Guo <shawnguo@kernel.org>
>> ---
>>  arch/arm/boot/dts/Makefile | 1 +
>>  arch/arm/boot/dts/imx23-xfi3.dts | 190
>> +++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 191 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/imx23-xfi3.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 8f9f1c2..7a79d37 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -417,6 +417,7 @@ dtb-$(CONFIG_ARCH_MXS) += \
>>  imx23-evk.dtb \
>>  imx23-olinuxino.dtb \
>>  imx23-stmp378x_devb.dtb \
>> + imx23-xfi3.dtb \
>>  imx28-apf28.dtb \
>>  imx28-apf28dev.dtb \
>>  imx28-apx4devkit.dtb \
>> diff --git a/arch/arm/boot/dts/imx23-xfi3.dts
>> b/arch/arm/boot/dts/imx23-xfi3.dts
>> new file mode 100644
>> index 0000000..d7e7cf5
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/imx23-xfi3.dts
>> @@ -0,0 +1,190 @@
>> +/*
>> + * Copyright (C) 2013-2016 Marek Vasut <marex@denx.de>
>> + *
>> + * This file is dual-licensed: you can use it either under the terms
>> + * of the GPL or the X11 license, at your option. Note that this dual
>> + * licensing only applies to this file, and not this project as a
>> + * whole.
>> + *
>> + * a) This file is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License as
>> + * published by the Free Software Foundation; either version 2 of
>> + * the License, or (at your option) any later version.
>> + *
>> + * This file is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>> + * GNU General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU General Public
>> + * License along with this file; if not, write to the Free
>> + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston,
>> + * MA 02110-1301 USA
> 
> AFAIK this address shouldn't be included.

Hmmm, some boards do include it. I'll remove it from this submission and
also for the xfi.

Shall I send patch(es) removing the address from the other boards too ?

>> + *
>> + * Or, alternatively,
>> + *
>> + * b) Permission is hereby granted, free of charge, to any person
>> + * obtaining a copy of this software and associated documentation
>> + * files (the "Software"), to deal in the Software without
>> + * restriction, including without limitation the rights to use,
>> + * copy, modify, merge, publish, distribute, sublicense, and/or
>> + * sell copies of the Software, and to permit persons to whom the
>> + * Software is furnished to do so, subject to the following
>> + * conditions:
>> + *
>> + * The above copyright notice and this permission notice shall be
>> + * included in all copies or substantial portions of the Software.
>> + *
>> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>> + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
>> + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
>> + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
>> + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
>> + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
>> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
>> + * OTHER DEALINGS IN THE SOFTWARE.
>> + *
>> + */
>> +
>> +/dts-v1/;
>> +#include "imx23.dtsi"
>> +
>> +/ {
>> + model = "Creative ZEN X-Fi3";
>> + compatible = "creative,x-fi3", "fsl,imx23";
> 
> Is there a pending patch to add "creative" to the vendor-prefixes.txt?

There isn't. I'll send one.

>> +
>> + memory {
>> + reg = ;
>> + };
>> +
>> + apb at 80000000 {
>> + apbh at 80000000 {
>> + ssp0: ssp at 80010000 {
>> + compatible = "fsl,imx23-mmc";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&mmc0_4bit_pins_a &mmc0_pins_fixup>;
>> + bus-width = ;
>> + vmmc-supply = <&reg_vddio_sd0>;
>> + cd-inverted;
>> + status = "okay";
>> + };
>> +
>> + ssp1: ssp at 80034000 {
>> + compatible = "fsl,imx23-mmc";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&mmc1_4bit_pins_a>;
>> + bus-width = ;
>> + non-removable;
>> + status = "okay";
>> + };
>> +
>> + pinctrl at 80018000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&hog_pins_a>;
>> +
>> + hog_pins_a: hog at 0 {
>> + reg = ;
>> + fsl,pinmux-ids = <
>> + MX23_PAD_GPMI_D07__GPIO_0_7
>> + >;
>> + fsl,drive-strength = ;
>> + fsl,voltage = ;
>> + fsl,pull-up = ;
>> + };
>> +
>> + key_pins_a: keys at 0 {
>> + reg = ;
>> + fsl,pinmux-ids = <
>> + MX23_PAD_ROTARYA__GPIO_2_7
>> + MX23_PAD_ROTARYB__GPIO_2_8
>> + >;
>> + fsl,drive-strength = ;
>> + fsl,voltage = ;
>> + fsl,pull-up = ;
>> + };
>> + };
>> + };
>> +
>> + apbx at 80040000 {
>> + i2c: i2c at 80058000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&i2c_pins_a>;
>> + status = "okay";
>> + };
>> +
>> + pwm: pwm at 80064000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pwm2_pins_a>;
>> + status = "okay";
>> + };
>> +
>> + duart: serial at 80070000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&duart_pins_a>;
>> + status = "okay";
>> + };
>> +
>> + auart1: serial at 8006e000 {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&auart1_2pins_a>;
>> + status = "okay";
>> + };
>> +
>> + usbphy0: usbphy at 8007c000 {
>> + status = "okay";
>> + };
>> +
>> + lradc at 80050000 {
>> + status = "okay";
>> + };
>> + };
>> + };
>> +
>> + ahb at 80080000 {
>> + usb0: usb at 80080000 {
>> + dr_mode = "peripheral";
>> + status = "okay";
>> + };
>> + };
>> +
>> + regulators {
>> + compatible = "simple-bus";
>> +
>> + reg_vddio_sd0: vddio-sd0 {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vddio-sd0";
>> + regulator-min-microvolt = ;
>> + regulator-max-microvolt = ;
>> + gpio = <&gpio0 7 0>;
>> + };
>> + };
>> +
>> + backlight {
>> + compatible = "pwm-backlight";
>> + pwms = <&pwm 2 5000000>;
>> + brightness-levels = ;
>> + default-brightness-level = ;
>> + };
>> +
>> + gpio_keys {
>> + compatible = "gpio-keys";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&key_pins_a>;
>> + #address-cells = ;
>> + #size-cells = ;
>> +
>> + button at 0 {
>> + label = "volume-down";
>> + linux,code = ;
>> + gpios = <&gpio2 7 0>;
>> + debounce-interval = ;
>> + };
> 
> I think this would introduce new dtc warnings.
> 
> Please look at
> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=f2953a461042721e9a2aca58d2b1cfc67c70d371

Ha, fixed, thanks.

>> +
>> + button at 1 {
>> + label = "volume-up";
>> + linux,code = ;
>> + gpios = <&gpio2 8 0>;
>> + debounce-interval = ;
>> + };
>> + };
>> +};
>> -- 
>> 2.7.0
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


-- 
Best regards,
Marek Vasut

[PATCH 3/4] ARM: mxs: dts: Add Creative X-Fi3 support

[Stefan Wahren]

Hi,

Am 08.05.2016 um 22:23 schrieb Marek Vasut:
> On 05/06/2016 09:40 AM, Stefan Wahren wrote:
>> Hi Marek,
> Hi!
>
>> sorry for the reply format.
> replying from a phone ? :)

no an ugly webmailer :(

>
>>> Marek Vasut <marex@denx.de> hat am 5. Mai 2016 um 22:26 geschrieben:
>>>
>>> Add support for this small MP3 player based on STMP3780 (rev.4).
>>> Currently supported is both external microSD and internal SD-NAND
>>> bridge, PWM and USB gadget.
>>>
>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>> Cc: Fabio Estevam <fabio.estevam@nxp.com>
>>> Cc: Shawn Guo <shawnguo@kernel.org>
>>> ---
>>>  arch/arm/boot/dts/Makefile | 1 +
>>>  arch/arm/boot/dts/imx23-xfi3.dts | 190
>>> +++++++++++++++++++++++++++++++++++++++
>>>  2 files changed, 191 insertions(+)
>>>  create mode 100644 arch/arm/boot/dts/imx23-xfi3.dts
>>>
>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>>> index 8f9f1c2..7a79d37 100644
>>> --- a/arch/arm/boot/dts/Makefile
>>> +++ b/arch/arm/boot/dts/Makefile
>>> @@ -417,6 +417,7 @@ dtb-$(CONFIG_ARCH_MXS) += \
>>>  imx23-evk.dtb \
>>>  imx23-olinuxino.dtb \
>>>  imx23-stmp378x_devb.dtb \
>>> + imx23-xfi3.dtb \
>>>  imx28-apf28.dtb \
>>>  imx28-apf28dev.dtb \
>>>  imx28-apx4devkit.dtb \
>>> diff --git a/arch/arm/boot/dts/imx23-xfi3.dts
>>> b/arch/arm/boot/dts/imx23-xfi3.dts
>>> new file mode 100644
>>> index 0000000..d7e7cf5
>>> --- /dev/null
>>> +++ b/arch/arm/boot/dts/imx23-xfi3.dts
>>> @@ -0,0 +1,190 @@
>>> +/*
>>> + * Copyright (C) 2013-2016 Marek Vasut <marex@denx.de>
>>> + *
>>> + * This file is dual-licensed: you can use it either under the terms
>>> + * of the GPL or the X11 license, at your option. Note that this dual
>>> + * licensing only applies to this file, and not this project as a
>>> + * whole.
>>> + *
>>> + * a) This file is free software; you can redistribute it and/or
>>> + * modify it under the terms of the GNU General Public License as
>>> + * published by the Free Software Foundation; either version 2 of
>>> + * the License, or (at your option) any later version.
>>> + *
>>> + * This file is distributed in the hope that it will be useful,
>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>> + * GNU General Public License for more details.
>>> + *
>>> + * You should have received a copy of the GNU General Public
>>> + * License along with this file; if not, write to the Free
>>> + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston,
>>> + * MA 02110-1301 USA
>> AFAIK this address shouldn't be included.
> Hmmm, some boards do include it. I'll remove it from this submission and
> also for the xfi.
>
> Shall I send patch(es) removing the address from the other boards too ?

i'm not sure. Maybe Shawn has an opinion about it.

Thanks
Stefan

[PATCH 3/4] ARM: mxs: dts: Add Creative X-Fi3 support

[Marek Vasut]

On 05/09/2016 10:30 AM, Stefan Wahren wrote:
> Hi,
> 
> Am 08.05.2016 um 22:23 schrieb Marek Vasut:
>> On 05/06/2016 09:40 AM, Stefan Wahren wrote:
>>> Hi Marek,
>> Hi!
>>
>>> sorry for the reply format.
>> replying from a phone ? :)
> 
> no an ugly webmailer :(
> 
>>
>>>> Marek Vasut <marex@denx.de> hat am 5. Mai 2016 um 22:26 geschrieben:
>>>>
>>>> Add support for this small MP3 player based on STMP3780 (rev.4).
>>>> Currently supported is both external microSD and internal SD-NAND
>>>> bridge, PWM and USB gadget.
>>>>
>>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>>> Cc: Fabio Estevam <fabio.estevam@nxp.com>
>>>> Cc: Shawn Guo <shawnguo@kernel.org>
>>>> ---
>>>>  arch/arm/boot/dts/Makefile | 1 +
>>>>  arch/arm/boot/dts/imx23-xfi3.dts | 190
>>>> +++++++++++++++++++++++++++++++++++++++
>>>>  2 files changed, 191 insertions(+)
>>>>  create mode 100644 arch/arm/boot/dts/imx23-xfi3.dts
>>>>
>>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>>>> index 8f9f1c2..7a79d37 100644
>>>> --- a/arch/arm/boot/dts/Makefile
>>>> +++ b/arch/arm/boot/dts/Makefile
>>>> @@ -417,6 +417,7 @@ dtb-$(CONFIG_ARCH_MXS) += \
>>>>  imx23-evk.dtb \
>>>>  imx23-olinuxino.dtb \
>>>>  imx23-stmp378x_devb.dtb \
>>>> + imx23-xfi3.dtb \
>>>>  imx28-apf28.dtb \
>>>>  imx28-apf28dev.dtb \
>>>>  imx28-apx4devkit.dtb \
>>>> diff --git a/arch/arm/boot/dts/imx23-xfi3.dts
>>>> b/arch/arm/boot/dts/imx23-xfi3.dts
>>>> new file mode 100644
>>>> index 0000000..d7e7cf5
>>>> --- /dev/null
>>>> +++ b/arch/arm/boot/dts/imx23-xfi3.dts
>>>> @@ -0,0 +1,190 @@
>>>> +/*
>>>> + * Copyright (C) 2013-2016 Marek Vasut <marex@denx.de>
>>>> + *
>>>> + * This file is dual-licensed: you can use it either under the terms
>>>> + * of the GPL or the X11 license, at your option. Note that this dual
>>>> + * licensing only applies to this file, and not this project as a
>>>> + * whole.
>>>> + *
>>>> + * a) This file is free software; you can redistribute it and/or
>>>> + * modify it under the terms of the GNU General Public License as
>>>> + * published by the Free Software Foundation; either version 2 of
>>>> + * the License, or (at your option) any later version.
>>>> + *
>>>> + * This file is distributed in the hope that it will be useful,
>>>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>>>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
>>>> + * GNU General Public License for more details.
>>>> + *
>>>> + * You should have received a copy of the GNU General Public
>>>> + * License along with this file; if not, write to the Free
>>>> + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston,
>>>> + * MA 02110-1301 USA
>>> AFAIK this address shouldn't be included.
>> Hmmm, some boards do include it. I'll remove it from this submission and
>> also for the xfi.
>>
>> Shall I send patch(es) removing the address from the other boards too ?
> 
> i'm not sure. Maybe Shawn has an opinion about it.

I am hoping for that.

btw V2 of patches is out, feel free to review :)

-- 
Best regards,
Marek Vasut