* [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow @ 2014-08-27 14:48 Ajay Kumar 2014-08-27 14:48 ` [PATCH 2/2] ARM: dts: Add DT changes for display on peach_pit Ajay Kumar 2014-08-27 15:01 ` [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Mark Rutland 0 siblings, 2 replies; 9+ messages in thread From: Ajay Kumar @ 2014-08-27 14:48 UTC (permalink / raw) To: linux-arm-kernel, linux-samsung-soc, devicetree Cc: kgene.kim, kgene.kim, inki.dae, thierry.reding, seanpaul, ajaynumb, jg1.han, joshi, prashanth.g, Ajay Kumar Add DT nodes for ptn3460 bridge chip and panel. Add backlight enable pin and backlight power supply for pwm-backlight. Also add bridge phandle needed by dp to enable display on snow. Note that, snow doesn't support "chunghwa,claa101wb01" panel, but still we choose to reuse the binding since "chunghwa,claa101wb01" has similar LCD timings. Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> --- Changes since V1: -- Remove "simple-panel" compatible string. -- Use GPIO_ACTIVE_HIGH instead of 0. -- Change panel node naming from "panel-simple" to "panel". arch/arm/boot/dts/exynos5250-snow.dts | 41 ++++++++++++++++++--------------- 1 file changed, 22 insertions(+), 19 deletions(-) diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts index f2b8c41..1ac9709 100644 --- a/arch/arm/boot/dts/exynos5250-snow.dts +++ b/arch/arm/boot/dts/exynos5250-snow.dts @@ -9,6 +9,7 @@ */ /dts-v1/; +#include <dt-bindings/gpio/gpio.h> #include "exynos5250.dtsi" #include "exynos5250-cros-common.dtsi" @@ -181,7 +182,7 @@ dcdc3 { ti,enable-ext-control; }; - fet1 { + fet1: fet1 { regulator-name = "vcd_led"; ti,overcurrent-wait = <3>; }; @@ -204,7 +205,7 @@ regulator-always-on; ti,overcurrent-wait = <3>; }; - fet6 { + fet6: fet6 { regulator-name = "lcd_vdd"; ti,overcurrent-wait = <3>; }; @@ -253,6 +254,15 @@ pinctrl-0 = <&max98095_en>; pinctrl-names = "default"; }; + + ptn3460: lvds-bridge@20 { + compatible = "nxp,ptn3460"; + reg = <0x20>; + powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>; + reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>; + edid-emulation = <5>; + panel = <&panel>; + }; }; i2s0: i2s@03830000 { @@ -300,11 +310,13 @@ vdd_pll-supply = <&ldo8_reg>; }; - backlight { + backlight: backlight { compatible = "pwm-backlight"; pwms = <&pwm 0 1000000 0>; brightness-levels = <0 100 500 1000 1500 2000 2500 2800>; default-brightness-level = <7>; + enable-gpios = <&gpx3 0 GPIO_ACTIVE_HIGH>; + power-supply = <&fet1>; pinctrl-0 = <&pwm0_out>; pinctrl-names = "default"; }; @@ -314,6 +326,12 @@ samsung,invert-vclk; }; + panel: panel { + compatible = "chunghwa,claa101wb01"; + power-supply = <&fet6>; + backlight = <&backlight>; + }; + dp-controller@145B0000 { status = "okay"; pinctrl-names = "default"; @@ -325,22 +343,7 @@ samsung,link-rate = <0x0a>; samsung,lane-count = <2>; samsung,hpd-gpio = <&gpx0 7 0>; - - display-timings { - native-mode = <&timing1>; - - timing1: timing@1 { - clock-frequency = <70589280>; - hactive = <1366>; - vactive = <768>; - hfront-porch = <40>; - hback-porch = <40>; - hsync-len = <32>; - vback-porch = <10>; - vfront-porch = <12>; - vsync-len = <6>; - }; - }; + bridge = <&ptn3460>; }; }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] ARM: dts: Add DT changes for display on peach_pit 2014-08-27 14:48 [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Ajay Kumar @ 2014-08-27 14:48 ` Ajay Kumar 2014-08-27 15:01 ` [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Mark Rutland 1 sibling, 0 replies; 9+ messages in thread From: Ajay Kumar @ 2014-08-27 14:48 UTC (permalink / raw) To: linux-arm-kernel, linux-samsung-soc, devicetree Cc: kgene.kim, kgene.kim, inki.dae, thierry.reding, seanpaul, ajaynumb, jg1.han, joshi, prashanth.g, Ajay Kumar Add DT nodes for ps8622 bridge chip and panel. Add backlight power supply for pwm-backlight. Also add bridge phandle needed by dp to enable display on peach_pit. Note that, peach_pit doesn't support "chunghwa,claa101wb01" panel, but still we choose to reuse the binding since "chunghwa,claa101wb01" has similar LCD timings. Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> --- Changes since V1: -- Remove "simple-panel" compatible string. -- Use GPIO_ACTIVE_HIGH instead of 0. -- Change panel node naming from "panel-simple" to "panel". arch/arm/boot/dts/exynos5420-peach-pit.dts | 36 +++++++++++++++------------- 1 file changed, 19 insertions(+), 17 deletions(-) diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts index 228a6b1..1070a31 100644 --- a/arch/arm/boot/dts/exynos5420-peach-pit.dts +++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts @@ -30,11 +30,12 @@ i2c20 = "/spi@12d40000/cros-ec@0/i2c-tunnel"; }; - backlight { + backlight: backlight { compatible = "pwm-backlight"; pwms = <&pwm 0 1000000 0>; brightness-levels = <0 100 500 1000 1500 2000 2500 2800>; default-brightness-level = <7>; + power-supply = <&tps65090_fet1>; pinctrl-0 = <&pwm0_out>; pinctrl-names = "default"; }; @@ -100,6 +101,12 @@ regulator-boot-on; regulator-always-on; }; + + panel: panel { + compatible = "chunghwa,claa101wb01"; + power-supply = <&tps65090_fet6>; + backlight = <&backlight>; + }; }; &dp { @@ -113,22 +120,7 @@ samsung,link-rate = <0x06>; samsung,lane-count = <2>; samsung,hpd-gpio = <&gpx2 6 0>; - - display-timings { - native-mode = <&timing1>; - - timing1: timing@1 { - clock-frequency = <70589280>; - hactive = <1366>; - vactive = <768>; - hfront-porch = <40>; - hback-porch = <40>; - hsync-len = <32>; - vback-porch = <10>; - vfront-porch = <12>; - vsync-len = <6>; - }; - }; + bridge = <&ps8625>; }; &fimd { @@ -155,6 +147,16 @@ pinctrl-names = "default"; pinctrl-0 = <&max98090_irq>; }; + + ps8625: lvds-bridge@48 { + compatible = "parade,ps8625"; + reg = <0x48>; + sleep-gpios = <&gpx3 5 GPIO_ACTIVE_HIGH>; + reset-gpios = <&gpy7 7 GPIO_ACTIVE_HIGH>; + lane-count = <2>; + panel = <&panel>; + use-external-pwm; + }; }; &hsi2c_9 { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-27 14:48 [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Ajay Kumar 2014-08-27 14:48 ` [PATCH 2/2] ARM: dts: Add DT changes for display on peach_pit Ajay Kumar @ 2014-08-27 15:01 ` Mark Rutland 2014-08-28 5:34 ` Ajay kumar 1 sibling, 1 reply; 9+ messages in thread From: Mark Rutland @ 2014-08-27 15:01 UTC (permalink / raw) To: Ajay Kumar Cc: linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, thierry.reding@gmail.com, seanpaul@google.com, ajaynumb@gmail.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: > Add DT nodes for ptn3460 bridge chip and panel. > Add backlight enable pin and backlight power supply for pwm-backlight. > Also add bridge phandle needed by dp to enable display on snow. > > Note that, snow doesn't support "chunghwa,claa101wb01" panel, > but still we choose to reuse the binding since "chunghwa,claa101wb01" > has similar LCD timings. What does it actually have? It's fine to have "chunghwa,claa101wb01" as a fallback but we should have an identifier for the actual display, too. Thanks, Mark. > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > --- > Changes since V1: > -- Remove "simple-panel" compatible string. > -- Use GPIO_ACTIVE_HIGH instead of 0. > -- Change panel node naming from "panel-simple" to "panel". > > arch/arm/boot/dts/exynos5250-snow.dts | 41 ++++++++++++++++++--------------- > 1 file changed, 22 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts > index f2b8c41..1ac9709 100644 > --- a/arch/arm/boot/dts/exynos5250-snow.dts > +++ b/arch/arm/boot/dts/exynos5250-snow.dts > @@ -9,6 +9,7 @@ > */ > > /dts-v1/; > +#include <dt-bindings/gpio/gpio.h> > #include "exynos5250.dtsi" > #include "exynos5250-cros-common.dtsi" > > @@ -181,7 +182,7 @@ > dcdc3 { > ti,enable-ext-control; > }; > - fet1 { > + fet1: fet1 { > regulator-name = "vcd_led"; > ti,overcurrent-wait = <3>; > }; > @@ -204,7 +205,7 @@ > regulator-always-on; > ti,overcurrent-wait = <3>; > }; > - fet6 { > + fet6: fet6 { > regulator-name = "lcd_vdd"; > ti,overcurrent-wait = <3>; > }; > @@ -253,6 +254,15 @@ > pinctrl-0 = <&max98095_en>; > pinctrl-names = "default"; > }; > + > + ptn3460: lvds-bridge@20 { > + compatible = "nxp,ptn3460"; > + reg = <0x20>; > + powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>; > + edid-emulation = <5>; > + panel = <&panel>; > + }; > }; > > i2s0: i2s@03830000 { > @@ -300,11 +310,13 @@ > vdd_pll-supply = <&ldo8_reg>; > }; > > - backlight { > + backlight: backlight { > compatible = "pwm-backlight"; > pwms = <&pwm 0 1000000 0>; > brightness-levels = <0 100 500 1000 1500 2000 2500 2800>; > default-brightness-level = <7>; > + enable-gpios = <&gpx3 0 GPIO_ACTIVE_HIGH>; > + power-supply = <&fet1>; > pinctrl-0 = <&pwm0_out>; > pinctrl-names = "default"; > }; > @@ -314,6 +326,12 @@ > samsung,invert-vclk; > }; > > + panel: panel { > + compatible = "chunghwa,claa101wb01"; > + power-supply = <&fet6>; > + backlight = <&backlight>; > + }; > + > dp-controller@145B0000 { > status = "okay"; > pinctrl-names = "default"; > @@ -325,22 +343,7 @@ > samsung,link-rate = <0x0a>; > samsung,lane-count = <2>; > samsung,hpd-gpio = <&gpx0 7 0>; > - > - display-timings { > - native-mode = <&timing1>; > - > - timing1: timing@1 { > - clock-frequency = <70589280>; > - hactive = <1366>; > - vactive = <768>; > - hfront-porch = <40>; > - hback-porch = <40>; > - hsync-len = <32>; > - vback-porch = <10>; > - vfront-porch = <12>; > - vsync-len = <6>; > - }; > - }; > + bridge = <&ptn3460>; > }; > }; > > -- > 1.7.9.5 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-27 15:01 ` [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Mark Rutland @ 2014-08-28 5:34 ` Ajay kumar 2014-08-28 9:15 ` Mark Rutland 0 siblings, 1 reply; 9+ messages in thread From: Ajay kumar @ 2014-08-28 5:34 UTC (permalink / raw) To: Mark Rutland Cc: Ajay Kumar, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, thierry.reding@gmail.com, seanpaul@google.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@arm.com> wrote: > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: >> Add DT nodes for ptn3460 bridge chip and panel. >> Add backlight enable pin and backlight power supply for pwm-backlight. >> Also add bridge phandle needed by dp to enable display on snow. >> >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, >> but still we choose to reuse the binding since "chunghwa,claa101wb01" >> has similar LCD timings. > > What does it actually have? > > It's fine to have "chunghwa,claa101wb01" as a fallback but we should > have an identifier for the actual display, too. Actual display used is "AUO,B116XTN0", for which no data currently exists in panel file. Instead of adding a new panel_desc for B116XTN0, why not we reuse existing ones? what is the "identifier" you are talking about? Ajay > Thanks, > Mark. > >> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> >> --- >> Changes since V1: >> -- Remove "simple-panel" compatible string. >> -- Use GPIO_ACTIVE_HIGH instead of 0. >> -- Change panel node naming from "panel-simple" to "panel". >> >> arch/arm/boot/dts/exynos5250-snow.dts | 41 ++++++++++++++++++--------------- >> 1 file changed, 22 insertions(+), 19 deletions(-) >> >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts >> index f2b8c41..1ac9709 100644 >> --- a/arch/arm/boot/dts/exynos5250-snow.dts >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts >> @@ -9,6 +9,7 @@ >> */ >> >> /dts-v1/; >> +#include <dt-bindings/gpio/gpio.h> >> #include "exynos5250.dtsi" >> #include "exynos5250-cros-common.dtsi" >> >> @@ -181,7 +182,7 @@ >> dcdc3 { >> ti,enable-ext-control; >> }; >> - fet1 { >> + fet1: fet1 { >> regulator-name = "vcd_led"; >> ti,overcurrent-wait = <3>; >> }; >> @@ -204,7 +205,7 @@ >> regulator-always-on; >> ti,overcurrent-wait = <3>; >> }; >> - fet6 { >> + fet6: fet6 { >> regulator-name = "lcd_vdd"; >> ti,overcurrent-wait = <3>; >> }; >> @@ -253,6 +254,15 @@ >> pinctrl-0 = <&max98095_en>; >> pinctrl-names = "default"; >> }; >> + >> + ptn3460: lvds-bridge@20 { >> + compatible = "nxp,ptn3460"; >> + reg = <0x20>; >> + powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>; >> + reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>; >> + edid-emulation = <5>; >> + panel = <&panel>; >> + }; >> }; >> >> i2s0: i2s@03830000 { >> @@ -300,11 +310,13 @@ >> vdd_pll-supply = <&ldo8_reg>; >> }; >> >> - backlight { >> + backlight: backlight { >> compatible = "pwm-backlight"; >> pwms = <&pwm 0 1000000 0>; >> brightness-levels = <0 100 500 1000 1500 2000 2500 2800>; >> default-brightness-level = <7>; >> + enable-gpios = <&gpx3 0 GPIO_ACTIVE_HIGH>; >> + power-supply = <&fet1>; >> pinctrl-0 = <&pwm0_out>; >> pinctrl-names = "default"; >> }; >> @@ -314,6 +326,12 @@ >> samsung,invert-vclk; >> }; >> >> + panel: panel { >> + compatible = "chunghwa,claa101wb01"; >> + power-supply = <&fet6>; >> + backlight = <&backlight>; >> + }; >> + >> dp-controller@145B0000 { >> status = "okay"; >> pinctrl-names = "default"; >> @@ -325,22 +343,7 @@ >> samsung,link-rate = <0x0a>; >> samsung,lane-count = <2>; >> samsung,hpd-gpio = <&gpx0 7 0>; >> - >> - display-timings { >> - native-mode = <&timing1>; >> - >> - timing1: timing@1 { >> - clock-frequency = <70589280>; >> - hactive = <1366>; >> - vactive = <768>; >> - hfront-porch = <40>; >> - hback-porch = <40>; >> - hsync-len = <32>; >> - vback-porch = <10>; >> - vfront-porch = <12>; >> - vsync-len = <6>; >> - }; >> - }; >> + bridge = <&ptn3460>; >> }; >> }; >> >> -- >> 1.7.9.5 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe devicetree" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-28 5:34 ` Ajay kumar @ 2014-08-28 9:15 ` Mark Rutland 2014-08-28 9:34 ` Ajay kumar 0 siblings, 1 reply; 9+ messages in thread From: Mark Rutland @ 2014-08-28 9:15 UTC (permalink / raw) To: Ajay kumar Cc: Ajay Kumar, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, thierry.reding@gmail.com, seanpaul@google.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com On Thu, Aug 28, 2014 at 06:34:33AM +0100, Ajay kumar wrote: > On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@arm.com> wrote: > > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: > >> Add DT nodes for ptn3460 bridge chip and panel. > >> Add backlight enable pin and backlight power supply for pwm-backlight. > >> Also add bridge phandle needed by dp to enable display on snow. > >> > >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, > >> but still we choose to reuse the binding since "chunghwa,claa101wb01" > >> has similar LCD timings. > > > > What does it actually have? > > > > It's fine to have "chunghwa,claa101wb01" as a fallback but we should > > have an identifier for the actual display, too. > Actual display used is "AUO,B116XTN0", for which no data currently exists > in panel file. Instead of adding a new panel_desc for B116XTN0, > why not we reuse existing ones? what is the "identifier" you are talking about? The compatible string. So in the dts we'd have compatible = "au0,b116xtn0", "chunghwa,claa101wb01"; That way if we need specific data we can add it later. What vendor is AU0? Can we add a vendor-prefix in parallel? Thanks, Mark. > > Ajay > > Thanks, > > Mark. > > > >> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > >> --- > >> Changes since V1: > >> -- Remove "simple-panel" compatible string. > >> -- Use GPIO_ACTIVE_HIGH instead of 0. > >> -- Change panel node naming from "panel-simple" to "panel". > >> > >> arch/arm/boot/dts/exynos5250-snow.dts | 41 ++++++++++++++++++--------------- > >> 1 file changed, 22 insertions(+), 19 deletions(-) > >> > >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts > >> index f2b8c41..1ac9709 100644 > >> --- a/arch/arm/boot/dts/exynos5250-snow.dts > >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts > >> @@ -9,6 +9,7 @@ > >> */ > >> > >> /dts-v1/; > >> +#include <dt-bindings/gpio/gpio.h> > >> #include "exynos5250.dtsi" > >> #include "exynos5250-cros-common.dtsi" > >> > >> @@ -181,7 +182,7 @@ > >> dcdc3 { > >> ti,enable-ext-control; > >> }; > >> - fet1 { > >> + fet1: fet1 { > >> regulator-name = "vcd_led"; > >> ti,overcurrent-wait = <3>; > >> }; > >> @@ -204,7 +205,7 @@ > >> regulator-always-on; > >> ti,overcurrent-wait = <3>; > >> }; > >> - fet6 { > >> + fet6: fet6 { > >> regulator-name = "lcd_vdd"; > >> ti,overcurrent-wait = <3>; > >> }; > >> @@ -253,6 +254,15 @@ > >> pinctrl-0 = <&max98095_en>; > >> pinctrl-names = "default"; > >> }; > >> + > >> + ptn3460: lvds-bridge@20 { > >> + compatible = "nxp,ptn3460"; > >> + reg = <0x20>; > >> + powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>; > >> + reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>; > >> + edid-emulation = <5>; > >> + panel = <&panel>; > >> + }; > >> }; > >> > >> i2s0: i2s@03830000 { > >> @@ -300,11 +310,13 @@ > >> vdd_pll-supply = <&ldo8_reg>; > >> }; > >> > >> - backlight { > >> + backlight: backlight { > >> compatible = "pwm-backlight"; > >> pwms = <&pwm 0 1000000 0>; > >> brightness-levels = <0 100 500 1000 1500 2000 2500 2800>; > >> default-brightness-level = <7>; > >> + enable-gpios = <&gpx3 0 GPIO_ACTIVE_HIGH>; > >> + power-supply = <&fet1>; > >> pinctrl-0 = <&pwm0_out>; > >> pinctrl-names = "default"; > >> }; > >> @@ -314,6 +326,12 @@ > >> samsung,invert-vclk; > >> }; > >> > >> + panel: panel { > >> + compatible = "chunghwa,claa101wb01"; > >> + power-supply = <&fet6>; > >> + backlight = <&backlight>; > >> + }; > >> + > >> dp-controller@145B0000 { > >> status = "okay"; > >> pinctrl-names = "default"; > >> @@ -325,22 +343,7 @@ > >> samsung,link-rate = <0x0a>; > >> samsung,lane-count = <2>; > >> samsung,hpd-gpio = <&gpx0 7 0>; > >> - > >> - display-timings { > >> - native-mode = <&timing1>; > >> - > >> - timing1: timing@1 { > >> - clock-frequency = <70589280>; > >> - hactive = <1366>; > >> - vactive = <768>; > >> - hfront-porch = <40>; > >> - hback-porch = <40>; > >> - hsync-len = <32>; > >> - vback-porch = <10>; > >> - vfront-porch = <12>; > >> - vsync-len = <6>; > >> - }; > >> - }; > >> + bridge = <&ptn3460>; > >> }; > >> }; > >> > >> -- > >> 1.7.9.5 > >> > >> -- > >> To unsubscribe from this list: send the line "unsubscribe devicetree" in > >> the body of a message to majordomo@vger.kernel.org > >> More majordomo info at http://vger.kernel.org/majordomo-info.html > >> > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-28 9:15 ` Mark Rutland @ 2014-08-28 9:34 ` Ajay kumar 2014-08-28 9:52 ` Mark Rutland 2014-08-28 13:10 ` Thierry Reding 0 siblings, 2 replies; 9+ messages in thread From: Ajay kumar @ 2014-08-28 9:34 UTC (permalink / raw) To: Mark Rutland, thierry.reding@gmail.com Cc: Ajay Kumar, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, seanpaul@google.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com On Thu, Aug 28, 2014 at 2:45 PM, Mark Rutland <mark.rutland@arm.com> wrote: > On Thu, Aug 28, 2014 at 06:34:33AM +0100, Ajay kumar wrote: >> On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@arm.com> wrote: >> > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: >> >> Add DT nodes for ptn3460 bridge chip and panel. >> >> Add backlight enable pin and backlight power supply for pwm-backlight. >> >> Also add bridge phandle needed by dp to enable display on snow. >> >> >> >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, >> >> but still we choose to reuse the binding since "chunghwa,claa101wb01" >> >> has similar LCD timings. >> > >> > What does it actually have? >> > >> > It's fine to have "chunghwa,claa101wb01" as a fallback but we should >> > have an identifier for the actual display, too. >> Actual display used is "AUO,B116XTN0", for which no data currently exists >> in panel file. Instead of adding a new panel_desc for B116XTN0, >> why not we reuse existing ones? what is the "identifier" you are talking about? > > The compatible string. > > So in the dts we'd have compatible = "au0,b116xtn0", "chunghwa,claa101wb01"; Sorry, hardware engineer says actual LCD name is "auo,b116xw03". > That way if we need specific data we can add it later. I think we can add a new panel_desc for "auo,b116xw03". Because its actual LCD dimension is 256x144, but "chungwa, claa101wb01" has LCD dimension 223x125. What is Thierry's opinion on adding this new LCD? > What vendor is AU0? Can we add a vendor-prefix in parallel? its "auo" (already present: AU Optronics Corporation) Ajay > Thanks, > Mark. > >> >> Ajay >> > Thanks, >> > Mark. >> > >> >> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> >> >> --- >> >> Changes since V1: >> >> -- Remove "simple-panel" compatible string. >> >> -- Use GPIO_ACTIVE_HIGH instead of 0. >> >> -- Change panel node naming from "panel-simple" to "panel". >> >> >> >> arch/arm/boot/dts/exynos5250-snow.dts | 41 ++++++++++++++++++--------------- >> >> 1 file changed, 22 insertions(+), 19 deletions(-) >> >> >> >> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts >> >> index f2b8c41..1ac9709 100644 >> >> --- a/arch/arm/boot/dts/exynos5250-snow.dts >> >> +++ b/arch/arm/boot/dts/exynos5250-snow.dts >> >> @@ -9,6 +9,7 @@ >> >> */ >> >> >> >> /dts-v1/; >> >> +#include <dt-bindings/gpio/gpio.h> >> >> #include "exynos5250.dtsi" >> >> #include "exynos5250-cros-common.dtsi" >> >> >> >> @@ -181,7 +182,7 @@ >> >> dcdc3 { >> >> ti,enable-ext-control; >> >> }; >> >> - fet1 { >> >> + fet1: fet1 { >> >> regulator-name = "vcd_led"; >> >> ti,overcurrent-wait = <3>; >> >> }; >> >> @@ -204,7 +205,7 @@ >> >> regulator-always-on; >> >> ti,overcurrent-wait = <3>; >> >> }; >> >> - fet6 { >> >> + fet6: fet6 { >> >> regulator-name = "lcd_vdd"; >> >> ti,overcurrent-wait = <3>; >> >> }; >> >> @@ -253,6 +254,15 @@ >> >> pinctrl-0 = <&max98095_en>; >> >> pinctrl-names = "default"; >> >> }; >> >> + >> >> + ptn3460: lvds-bridge@20 { >> >> + compatible = "nxp,ptn3460"; >> >> + reg = <0x20>; >> >> + powerdown-gpios = <&gpy2 5 GPIO_ACTIVE_HIGH>; >> >> + reset-gpios = <&gpx1 5 GPIO_ACTIVE_HIGH>; >> >> + edid-emulation = <5>; >> >> + panel = <&panel>; >> >> + }; >> >> }; >> >> >> >> i2s0: i2s@03830000 { >> >> @@ -300,11 +310,13 @@ >> >> vdd_pll-supply = <&ldo8_reg>; >> >> }; >> >> >> >> - backlight { >> >> + backlight: backlight { >> >> compatible = "pwm-backlight"; >> >> pwms = <&pwm 0 1000000 0>; >> >> brightness-levels = <0 100 500 1000 1500 2000 2500 2800>; >> >> default-brightness-level = <7>; >> >> + enable-gpios = <&gpx3 0 GPIO_ACTIVE_HIGH>; >> >> + power-supply = <&fet1>; >> >> pinctrl-0 = <&pwm0_out>; >> >> pinctrl-names = "default"; >> >> }; >> >> @@ -314,6 +326,12 @@ >> >> samsung,invert-vclk; >> >> }; >> >> >> >> + panel: panel { >> >> + compatible = "chunghwa,claa101wb01"; >> >> + power-supply = <&fet6>; >> >> + backlight = <&backlight>; >> >> + }; >> >> + >> >> dp-controller@145B0000 { >> >> status = "okay"; >> >> pinctrl-names = "default"; >> >> @@ -325,22 +343,7 @@ >> >> samsung,link-rate = <0x0a>; >> >> samsung,lane-count = <2>; >> >> samsung,hpd-gpio = <&gpx0 7 0>; >> >> - >> >> - display-timings { >> >> - native-mode = <&timing1>; >> >> - >> >> - timing1: timing@1 { >> >> - clock-frequency = <70589280>; >> >> - hactive = <1366>; >> >> - vactive = <768>; >> >> - hfront-porch = <40>; >> >> - hback-porch = <40>; >> >> - hsync-len = <32>; >> >> - vback-porch = <10>; >> >> - vfront-porch = <12>; >> >> - vsync-len = <6>; >> >> - }; >> >> - }; >> >> + bridge = <&ptn3460>; >> >> }; >> >> }; >> >> >> >> -- >> >> 1.7.9.5 >> >> >> >> -- >> >> To unsubscribe from this list: send the line "unsubscribe devicetree" in >> >> the body of a message to majordomo@vger.kernel.org >> >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> >> ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-28 9:34 ` Ajay kumar @ 2014-08-28 9:52 ` Mark Rutland 2014-08-28 13:10 ` Thierry Reding 1 sibling, 0 replies; 9+ messages in thread From: Mark Rutland @ 2014-08-28 9:52 UTC (permalink / raw) To: Ajay kumar Cc: thierry.reding@gmail.com, Ajay Kumar, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, seanpaul@google.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com On Thu, Aug 28, 2014 at 10:34:32AM +0100, Ajay kumar wrote: > On Thu, Aug 28, 2014 at 2:45 PM, Mark Rutland <mark.rutland@arm.com> wrote: > > On Thu, Aug 28, 2014 at 06:34:33AM +0100, Ajay kumar wrote: > >> On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@arm.com> wrote: > >> > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: > >> >> Add DT nodes for ptn3460 bridge chip and panel. > >> >> Add backlight enable pin and backlight power supply for pwm-backlight. > >> >> Also add bridge phandle needed by dp to enable display on snow. > >> >> > >> >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, > >> >> but still we choose to reuse the binding since "chunghwa,claa101wb01" > >> >> has similar LCD timings. > >> > > >> > What does it actually have? > >> > > >> > It's fine to have "chunghwa,claa101wb01" as a fallback but we should > >> > have an identifier for the actual display, too. > >> Actual display used is "AUO,B116XTN0", for which no data currently exists > >> in panel file. Instead of adding a new panel_desc for B116XTN0, > >> why not we reuse existing ones? what is the "identifier" you are talking about? > > > > The compatible string. > > > > So in the dts we'd have compatible = "au0,b116xtn0", "chunghwa,claa101wb01"; > Sorry, hardware engineer says actual LCD name is "auo,b116xw03". Ok. > > That way if we need specific data we can add it later. > I think we can add a new panel_desc for "auo,b116xw03". > Because its actual LCD dimension is 256x144, but "chungwa, claa101wb01" has > LCD dimension 223x125. > What is Thierry's opinion on adding this new LCD? > > > What vendor is AU0? Can we add a vendor-prefix in parallel? > its "auo" (already present: AU Optronics Corporation) Ah, whoops. Mistook 'O' for '0'. Thanks, Mark. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-28 9:34 ` Ajay kumar 2014-08-28 9:52 ` Mark Rutland @ 2014-08-28 13:10 ` Thierry Reding 2014-08-28 14:28 ` Mark Rutland 1 sibling, 1 reply; 9+ messages in thread From: Thierry Reding @ 2014-08-28 13:10 UTC (permalink / raw) To: Ajay kumar Cc: Mark Rutland, Ajay Kumar, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, seanpaul@google.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com [-- Attachment #1: Type: text/plain, Size: 1996 bytes --] On Thu, Aug 28, 2014 at 03:04:32PM +0530, Ajay kumar wrote: > On Thu, Aug 28, 2014 at 2:45 PM, Mark Rutland <mark.rutland@arm.com> wrote: > > On Thu, Aug 28, 2014 at 06:34:33AM +0100, Ajay kumar wrote: > >> On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@arm.com> wrote: > >> > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: > >> >> Add DT nodes for ptn3460 bridge chip and panel. > >> >> Add backlight enable pin and backlight power supply for pwm-backlight. > >> >> Also add bridge phandle needed by dp to enable display on snow. > >> >> > >> >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, > >> >> but still we choose to reuse the binding since "chunghwa,claa101wb01" > >> >> has similar LCD timings. > >> > > >> > What does it actually have? > >> > > >> > It's fine to have "chunghwa,claa101wb01" as a fallback but we should > >> > have an identifier for the actual display, too. > >> Actual display used is "AUO,B116XTN0", for which no data currently exists > >> in panel file. Instead of adding a new panel_desc for B116XTN0, > >> why not we reuse existing ones? what is the "identifier" you are talking about? > > > > The compatible string. > > > > So in the dts we'd have compatible = "au0,b116xtn0", "chunghwa,claa101wb01"; > Sorry, hardware engineer says actual LCD name is "auo,b116xw03". > > > That way if we need specific data we can add it later. > I think we can add a new panel_desc for "auo,b116xw03". > Because its actual LCD dimension is 256x144, but "chungwa, claa101wb01" has > LCD dimension 223x125. > What is Thierry's opinion on adding this new LCD? I think you should simply add a new panel_desc for the panel. It may use similar timings, but it's likely not "compatible" in the way required by device tree. As you say, dimensions are different and those may have an influence on the DPI setting. There's really no reason why this should share the panel_desc with another panel. Thierry [-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow 2014-08-28 13:10 ` Thierry Reding @ 2014-08-28 14:28 ` Mark Rutland 0 siblings, 0 replies; 9+ messages in thread From: Mark Rutland @ 2014-08-28 14:28 UTC (permalink / raw) To: Thierry Reding Cc: Ajay kumar, Ajay Kumar, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kgene.kim@samsung.com, kgene.kim@gmail.com, inki.dae@samsung.com, seanpaul@google.com, jg1.han@samsung.com, joshi@samsung.com, prashanth.g@samsung.com On Thu, Aug 28, 2014 at 02:10:18PM +0100, Thierry Reding wrote: > On Thu, Aug 28, 2014 at 03:04:32PM +0530, Ajay kumar wrote: > > On Thu, Aug 28, 2014 at 2:45 PM, Mark Rutland <mark.rutland@arm.com> wrote: > > > On Thu, Aug 28, 2014 at 06:34:33AM +0100, Ajay kumar wrote: > > >> On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@arm.com> wrote: > > >> > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: > > >> >> Add DT nodes for ptn3460 bridge chip and panel. > > >> >> Add backlight enable pin and backlight power supply for pwm-backlight. > > >> >> Also add bridge phandle needed by dp to enable display on snow. > > >> >> > > >> >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, > > >> >> but still we choose to reuse the binding since "chunghwa,claa101wb01" > > >> >> has similar LCD timings. > > >> > > > >> > What does it actually have? > > >> > > > >> > It's fine to have "chunghwa,claa101wb01" as a fallback but we should > > >> > have an identifier for the actual display, too. > > >> Actual display used is "AUO,B116XTN0", for which no data currently exists > > >> in panel file. Instead of adding a new panel_desc for B116XTN0, > > >> why not we reuse existing ones? what is the "identifier" you are talking about? > > > > > > The compatible string. > > > > > > So in the dts we'd have compatible = "au0,b116xtn0", "chunghwa,claa101wb01"; > > Sorry, hardware engineer says actual LCD name is "auo,b116xw03". > > > > > That way if we need specific data we can add it later. > > I think we can add a new panel_desc for "auo,b116xw03". > > Because its actual LCD dimension is 256x144, but "chungwa, claa101wb01" has > > LCD dimension 223x125. > > What is Thierry's opinion on adding this new LCD? > > I think you should simply add a new panel_desc for the panel. It may use > similar timings, but it's likely not "compatible" in the way required by > device tree. As you say, dimensions are different and those may have an > influence on the DPI setting. There's really no reason why this should > share the panel_desc with another panel. > > Thierry This sounds sane to me. Mark. ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-08-28 14:28 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-08-27 14:48 [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Ajay Kumar 2014-08-27 14:48 ` [PATCH 2/2] ARM: dts: Add DT changes for display on peach_pit Ajay Kumar 2014-08-27 15:01 ` [PATCH V2 1/2] ARM: dts: Add DT changes for display on snow Mark Rutland 2014-08-28 5:34 ` Ajay kumar 2014-08-28 9:15 ` Mark Rutland 2014-08-28 9:34 ` Ajay kumar 2014-08-28 9:52 ` Mark Rutland 2014-08-28 13:10 ` Thierry Reding 2014-08-28 14:28 ` Mark Rutland
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).