Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752538AbbG2Fji (ORCPT ); Wed, 29 Jul 2015 01:39:38 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:54827 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751307AbbG2Fjg (ORCPT ); Wed, 29 Jul 2015 01:39:36 -0400 Message-ID: <55B86710.4090507@codeaurora.org> Date: Wed, 29 Jul 2015 11:09:28 +0530 From: Archit Taneja User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Srinivas Kandagatla , agross@codeaurora.org, linux-arm-msm@vger.kernel.org CC: Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rob Clark Subject: Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support References: <1438087956-17307-1-git-send-email-srinivas.kandagatla@linaro.org> <1438088076-17606-1-git-send-email-srinivas.kandagatla@linaro.org> In-Reply-To: <1438088076-17606-1-git-send-email-srinivas.kandagatla@linaro.org> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3742 Lines: 143 Hi Srini, On 07/28/2015 06:24 PM, Srinivas Kandagatla wrote: > This patch adds LVDS panel for IFC6410. > > Signed-off-by: Rob Clark > [Rob Clark: WIP patch] > Signed-off-by: Srinivas Kandagatla > --- > arch/arm/boot/dts/qcom-apq8064-ifc6410.dts | 66 ++++++++++++++++++++++++++++++ > 1 file changed, 66 insertions(+) > > diff --git a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts > index 1ab71f1..3bdac02 100644 > --- a/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts > +++ b/arch/arm/boot/dts/qcom-apq8064-ifc6410.dts > @@ -63,6 +63,12 @@ > qcom,switch-mode-frequency = <3200000>; > }; > > + pm8921_l2: l2 { > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + bias-pull-down; > + }; > + > pm8921_l3: l3 { > regulator-min-microvolt = <3050000>; > regulator-max-microvolt = <3300000>; > @@ -96,6 +102,10 @@ > pm8921_lvs1: lvs1 { > bias-pull-down; > }; > + > + pm8921_lvs7: lvs7 { > + bias-pull-down; > + }; > }; > }; > > @@ -119,6 +129,41 @@ > > mdp: qcom,mdp@5100000 { > status = "okay"; > + qcom,lvds-panel = <&panel>; We're trying to switch to the of_graph way of representing connected panels. With that, the above phandle will go away. This needs to be replaced with: port { lvds_out: endpoint { remote_endpoint = <&auo_in>; }; }; > + lvds-vccs-3p3v-supply = <&ext_3p3v>; > + lvds-pll-vdda-supply = <&pm8921_l2>; > + lvds-vdda-supply = <&pm8921_lvs7>; > + }; > + > + panel_3p3v: panel_3p3v { > + compatible = "regulator-fixed"; > + pinctrl-0 = <&disp_en_gpios>; > + pinctrl-names = "default"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "panel_en_3p3v"; > + regulator-type = "voltage"; > + startup-delay-us = <0>; > + gpio = <&pm8921_gpio 36 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + regulator-boot-on; > + }; > + > + backlight: backlight{ > + pinctrl-0 = <&pwm_bl_gpios>; > + pinctrl-names = "default"; > + compatible = "gpio-backlight"; > + gpios = <&pm8921_gpio 26 GPIO_ACTIVE_HIGH>; > + default-on; > + }; > + > + panel: auo,b101xtn01 { > + status = "okay"; > + compatible = "auo,b101xtn01"; > + > + ddc-i2c-bus = <&i2c3>; > + backlight = <&backlight>; > + power-supply = <&panel_3p3v>; and for the panel: port { auo_in: endpoint { remote-endpoint = <&lvds_out>; }; }; Thanks, Archit > }; > > gsbi3: gsbi@16200000 { > @@ -235,6 +280,27 @@ > pm8921_gpio: gpio@150 { > pinctrl-names = "default"; > pinctrl-0 = <&wlan_default_gpios>; > + > + pwm_bl_gpios: pwm-bl-gpios { > + pios { > + pins = "gpio26"; > + bias-disable; > + function = "normal"; > + qcom,drive-strength = ; > + power-source = ; > + }; > + }; > + > + disp_en_gpios: disp-en-gpios { > + pios { > + pins = "gpio36"; > + bias-disable; > + function = "normal"; > + qcom,drive-strength = ; > + power-source = ; > + }; > + }; > + > wlan_default_gpios: wlan-gpios { > pios { > pins = "gpio43"; > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/