* [PATCH 2/6] ARM: dts: imx28-evk: Convert to the new display bindings
2018-07-20 20:39 [PATCH 1/6] ARM: dts: imx28-evk: Move regulators outside simple-bus Fabio Estevam
@ 2018-07-20 20:39 ` Fabio Estevam
2018-07-20 20:39 ` [PATCH 3/6] ARM: dts: imx23-evk: Move regulators outside simple-bus Fabio Estevam
` (3 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2018-07-20 20:39 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
imx28-evk board has a Seiko 43WVF1G parallel display.
Instead of hardcoding the display timings in the device tree, use
the "sii,43wvf1g" compatible instead.
This aligns with the new mxsfb bindings scheme documented at:
Documentation/devicetree/bindings/display/mxsfb.txt
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/boot/dts/imx28-evk.dts | 48 ++++++++++++++++++++---------------------
1 file changed, 24 insertions(+), 24 deletions(-)
diff --git a/arch/arm/boot/dts/imx28-evk.dts b/arch/arm/boot/dts/imx28-evk.dts
index 210aee0..93ab5bd 100644
--- a/arch/arm/boot/dts/imx28-evk.dts
+++ b/arch/arm/boot/dts/imx28-evk.dts
@@ -74,6 +74,26 @@
enable-active-high;
};
+ reg_lcd_5v: regulator-lcd-5v {
+ compatible = "regulator-fixed";
+ regulator-name = "lcd-5v";
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5000000>;
+ };
+
+ panel {
+ compatible = "sii,43wvf1g";
+ backlight = <&backlight_display>;
+ dvdd-supply = <®_lcd_3v3>;
+ avdd-supply = <®_lcd_5v>;
+
+ port {
+ panel_in: endpoint {
+ remote-endpoint = <&display_out>;
+ };
+ };
+ };
+
apb at 80000000 {
apbh at 80000000 {
gpmi-nand at 8000c000 {
@@ -177,31 +197,11 @@
pinctrl-names = "default";
pinctrl-0 = <&lcdif_24bit_pins_a
&lcdif_pins_evk>;
- lcd-supply = <®_lcd_3v3>;
- display = <&display0>;
status = "okay";
- display0: display0 {
- bits-per-pixel = <32>;
- bus-width = <24>;
-
- display-timings {
- native-mode = <&timing0>;
- timing0: timing0 {
- clock-frequency = <33500000>;
- hactive = <800>;
- vactive = <480>;
- hback-porch = <89>;
- hfront-porch = <164>;
- vback-porch = <23>;
- vfront-porch = <10>;
- hsync-len = <10>;
- vsync-len = <10>;
- hsync-active = <0>;
- vsync-active = <0>;
- de-active = <1>;
- pixelclk-active = <0>;
- };
+ port {
+ display_out: endpoint {
+ remote-endpoint = <&panel_in>;
};
};
};
@@ -350,7 +350,7 @@
};
};
- backlight {
+ backlight_display: backlight {
compatible = "pwm-backlight";
pwms = <&pwm 2 5000000>;
brightness-levels = <0 4 8 16 32 64 128 255>;
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 3/6] ARM: dts: imx23-evk: Move regulators outside simple-bus
2018-07-20 20:39 [PATCH 1/6] ARM: dts: imx28-evk: Move regulators outside simple-bus Fabio Estevam
2018-07-20 20:39 ` [PATCH 2/6] ARM: dts: imx28-evk: Convert to the new display bindings Fabio Estevam
@ 2018-07-20 20:39 ` Fabio Estevam
2018-07-20 20:39 ` [PATCH 4/6] ARM: dts: imx23-evk: Convert to the new display bindings Fabio Estevam
` (2 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2018-07-20 20:39 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
It is recommended to place regulators outside simple-bus, so move them
accordingly.
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/boot/dts/imx23-evk.dts | 42 +++++++++++++++++------------------------
1 file changed, 17 insertions(+), 25 deletions(-)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index 9fb4772..494095d 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -13,6 +13,23 @@
reg = <0x40000000 0x08000000>;
};
+ reg_vddio_sd0: regulator-vddio-sd0 {
+ compatible = "regulator-fixed";
+ regulator-name = "vddio-sd0";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ gpio = <&gpio1 29 0>;
+ };
+
+ reg_lcd_3v3: regulator-lcd-3v3 {
+ compatible = "regulator-fixed";
+ regulator-name = "lcd-3v3";
+ regulator-min-microvolt = <3300000>;
+ regulator-max-microvolt = <3300000>;
+ gpio = <&gpio1 18 0>;
+ enable-active-high;
+ };
+
apb at 80000000 {
apbh at 80000000 {
gpmi-nand at 8000c000 {
@@ -118,31 +135,6 @@
};
};
- regulators {
- compatible = "simple-bus";
- #address-cells = <1>;
- #size-cells = <0>;
-
- reg_vddio_sd0: regulator at 0 {
- compatible = "regulator-fixed";
- reg = <0>;
- regulator-name = "vddio-sd0";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- gpio = <&gpio1 29 0>;
- };
-
- reg_lcd_3v3: regulator at 1 {
- compatible = "regulator-fixed";
- reg = <1>;
- regulator-name = "lcd-3v3";
- regulator-min-microvolt = <3300000>;
- regulator-max-microvolt = <3300000>;
- gpio = <&gpio1 18 0>;
- enable-active-high;
- };
- };
-
backlight {
compatible = "pwm-backlight";
pwms = <&pwm 2 5000000>;
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 4/6] ARM: dts: imx23-evk: Convert to the new display bindings
2018-07-20 20:39 [PATCH 1/6] ARM: dts: imx28-evk: Move regulators outside simple-bus Fabio Estevam
2018-07-20 20:39 ` [PATCH 2/6] ARM: dts: imx28-evk: Convert to the new display bindings Fabio Estevam
2018-07-20 20:39 ` [PATCH 3/6] ARM: dts: imx23-evk: Move regulators outside simple-bus Fabio Estevam
@ 2018-07-20 20:39 ` Fabio Estevam
2018-07-20 20:39 ` [PATCH 5/6] ARM: mxs_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G Fabio Estevam
2018-07-20 20:39 ` [PATCH 6/6] ARM: imx_v6_v7_defconfig: " Fabio Estevam
4 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2018-07-20 20:39 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
imx23-evk board has a Seiko 43WVF1G parallel display.
Instead of hardcoding the display timings in the device tree, use
the "sii,43wvf1g" compatible instead.
This aligns with the new mxsfb bindings scheme documented at:
Documentation/devicetree/bindings/display/mxsfb.txt
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/boot/dts/imx23-evk.dts | 48 ++++++++++++++++++++---------------------
1 file changed, 24 insertions(+), 24 deletions(-)
diff --git a/arch/arm/boot/dts/imx23-evk.dts b/arch/arm/boot/dts/imx23-evk.dts
index 494095d..ad2ae25 100644
--- a/arch/arm/boot/dts/imx23-evk.dts
+++ b/arch/arm/boot/dts/imx23-evk.dts
@@ -30,6 +30,26 @@
enable-active-high;
};
+ reg_lcd_5v: regulator-lcd-5v {
+ compatible = "regulator-fixed";
+ regulator-name = "lcd-5v";
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5000000>;
+ };
+
+ panel {
+ compatible = "sii,43wvf1g";
+ backlight = <&backlight_display>;
+ dvdd-supply = <®_lcd_3v3>;
+ avdd-supply = <®_lcd_5v>;
+
+ port {
+ panel_in: endpoint {
+ remote-endpoint = <&display_out>;
+ };
+ };
+ };
+
apb at 80000000 {
apbh at 80000000 {
gpmi-nand at 8000c000 {
@@ -69,31 +89,11 @@
lcdif at 80030000 {
pinctrl-names = "default";
pinctrl-0 = <&lcdif_24bit_pins_a>;
- lcd-supply = <®_lcd_3v3>;
- display = <&display0>;
status = "okay";
- display0: display0 {
- bits-per-pixel = <32>;
- bus-width = <24>;
-
- display-timings {
- native-mode = <&timing0>;
- timing0: timing0 {
- clock-frequency = <9200000>;
- hactive = <480>;
- vactive = <272>;
- hback-porch = <15>;
- hfront-porch = <8>;
- vback-porch = <12>;
- vfront-porch = <4>;
- hsync-len = <1>;
- vsync-len = <1>;
- hsync-active = <0>;
- vsync-active = <0>;
- de-active = <1>;
- pixelclk-active = <0>;
- };
+ port {
+ display_out: endpoint {
+ remote-endpoint = <&panel_in>;
};
};
};
@@ -135,7 +135,7 @@
};
};
- backlight {
+ backlight_display: backlight {
compatible = "pwm-backlight";
pwms = <&pwm 2 5000000>;
brightness-levels = <0 4 8 16 32 64 128 255>;
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 5/6] ARM: mxs_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G
2018-07-20 20:39 [PATCH 1/6] ARM: dts: imx28-evk: Move regulators outside simple-bus Fabio Estevam
` (2 preceding siblings ...)
2018-07-20 20:39 ` [PATCH 4/6] ARM: dts: imx23-evk: Convert to the new display bindings Fabio Estevam
@ 2018-07-20 20:39 ` Fabio Estevam
2018-07-20 20:39 ` [PATCH 6/6] ARM: imx_v6_v7_defconfig: " Fabio Estevam
4 siblings, 0 replies; 10+ messages in thread
From: Fabio Estevam @ 2018-07-20 20:39 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
imx23-evk and imx28-evk boards use a Seiko 43WVF1G panel.
Now that the DRM mxsfb driver is the one selected by default, let's
also select CONFIG_DRM_PANEL_SEIKO_43WVF1G so that these boards continue
to have a working display by default.
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/configs/mxs_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig
index 148226e..7b82128 100644
--- a/arch/arm/configs/mxs_defconfig
+++ b/arch/arm/configs/mxs_defconfig
@@ -95,6 +95,7 @@ CONFIG_MFD_MXS_LRADC=y
CONFIG_REGULATOR=y
CONFIG_REGULATOR_FIXED_VOLTAGE=y
CONFIG_DRM=y
+CONFIG_DRM_PANEL_SEIKO_43WVF1G=y
CONFIG_DRM_MXSFB=y
CONFIG_FB_MODE_HELPERS=y
CONFIG_BACKLIGHT_LCD_SUPPORT=y
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 6/6] ARM: imx_v6_v7_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G
2018-07-20 20:39 [PATCH 1/6] ARM: dts: imx28-evk: Move regulators outside simple-bus Fabio Estevam
` (3 preceding siblings ...)
2018-07-20 20:39 ` [PATCH 5/6] ARM: mxs_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G Fabio Estevam
@ 2018-07-20 20:39 ` Fabio Estevam
2018-07-23 13:57 ` Fabio Estevam
4 siblings, 1 reply; 10+ messages in thread
From: Fabio Estevam @ 2018-07-20 20:39 UTC (permalink / raw)
To: linux-arm-kernel
From: Fabio Estevam <fabio.estevam@nxp.com>
imx6sl-evk, imx6sll-evk and imx6sx-sdb boards use a Seiko 43WVF1G panel.
Now that the DRM mxsfb driver is the one selected by default, let's
also select CONFIG_DRM_PANEL_SEIKO_43WVF1G so that these boards continue
to have a working display by default.
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
arch/arm/configs/imx_v6_v7_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index e2c1276..7eca43f 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -257,6 +257,7 @@ CONFIG_IMX_IPUV3_CORE=y
CONFIG_DRM=y
CONFIG_DRM_PANEL_LVDS=y
CONFIG_DRM_PANEL_SIMPLE=y
+CONFIG_DRM_PANEL_SEIKO_43WVF1G=y
CONFIG_DRM_DW_HDMI_AHB_AUDIO=m
CONFIG_DRM_DW_HDMI_CEC=y
CONFIG_DRM_IMX=y
--
2.7.4
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 6/6] ARM: imx_v6_v7_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G
2018-07-20 20:39 ` [PATCH 6/6] ARM: imx_v6_v7_defconfig: " Fabio Estevam
@ 2018-07-23 13:57 ` Fabio Estevam
2018-08-21 9:08 ` Shawn Guo
0 siblings, 1 reply; 10+ messages in thread
From: Fabio Estevam @ 2018-07-23 13:57 UTC (permalink / raw)
To: linux-arm-kernel
Hi Shawn,
On Fri, Jul 20, 2018 at 5:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> imx6sl-evk, imx6sll-evk and imx6sx-sdb boards use a Seiko 43WVF1G panel.
>
> Now that the DRM mxsfb driver is the one selected by default, let's
> also select CONFIG_DRM_PANEL_SEIKO_43WVF1G so that these boards continue
> to have a working display by default.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
I understand you have already submitted the imx 4.19 material to Olof
and he has already merged it.
Without this series we will have many NXP development boards without
functional display in 4.19 by default due to the drm mxsfb conversion
in defconfig, which would not be a very good user experience.
I would like to kindly ask you to consider applying this series to 4.19 as well.
Thanks
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 6/6] ARM: imx_v6_v7_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G
2018-07-23 13:57 ` Fabio Estevam
@ 2018-08-21 9:08 ` Shawn Guo
2018-08-21 9:23 ` Shawn Guo
0 siblings, 1 reply; 10+ messages in thread
From: Shawn Guo @ 2018-08-21 9:08 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jul 23, 2018 at 10:57:22AM -0300, Fabio Estevam wrote:
> Hi Shawn,
>
> On Fri, Jul 20, 2018 at 5:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> > From: Fabio Estevam <fabio.estevam@nxp.com>
> >
> > imx6sl-evk, imx6sll-evk and imx6sx-sdb boards use a Seiko 43WVF1G panel.
> >
> > Now that the DRM mxsfb driver is the one selected by default, let's
> > also select CONFIG_DRM_PANEL_SEIKO_43WVF1G so that these boards continue
> > to have a working display by default.
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
>
> I understand you have already submitted the imx 4.19 material to Olof
> and he has already merged it.
>
> Without this series we will have many NXP development boards without
> functional display in 4.19 by default due to the drm mxsfb conversion
> in defconfig, which would not be a very good user experience.
>
> I would like to kindly ask you to consider applying this series to 4.19 as well.
I guess only the last two defconfig patches are necessary to keep those
boards getting functional display for 4.19, no?
Shawn
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 6/6] ARM: imx_v6_v7_defconfig: Select CONFIG_DRM_PANEL_SEIKO_43WVF1G
2018-08-21 9:08 ` Shawn Guo
@ 2018-08-21 9:23 ` Shawn Guo
2018-08-21 18:31 ` Fabio Estevam
0 siblings, 1 reply; 10+ messages in thread
From: Shawn Guo @ 2018-08-21 9:23 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 21, 2018 at 05:08:50PM +0800, Shawn Guo wrote:
> On Mon, Jul 23, 2018 at 10:57:22AM -0300, Fabio Estevam wrote:
> > Hi Shawn,
> >
> > On Fri, Jul 20, 2018 at 5:39 PM, Fabio Estevam <festevam@gmail.com> wrote:
> > > From: Fabio Estevam <fabio.estevam@nxp.com>
> > >
> > > imx6sl-evk, imx6sll-evk and imx6sx-sdb boards use a Seiko 43WVF1G panel.
> > >
> > > Now that the DRM mxsfb driver is the one selected by default, let's
> > > also select CONFIG_DRM_PANEL_SEIKO_43WVF1G so that these boards continue
> > > to have a working display by default.
> > >
> > > Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
> >
> > I understand you have already submitted the imx 4.19 material to Olof
> > and he has already merged it.
> >
> > Without this series we will have many NXP development boards without
> > functional display in 4.19 by default due to the drm mxsfb conversion
> > in defconfig, which would not be a very good user experience.
> >
> > I would like to kindly ask you to consider applying this series to 4.19 as well.
>
> I guess only the last two defconfig patches are necessary to keep those
> boards getting functional display for 4.19, no?
Okay, just found that only the generic panel-lvds driver will parse
those timing parameters from DT.
I will try to send the series for 4.19 inclusion.
Shawn
^ permalink raw reply [flat|nested] 10+ messages in thread