Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1194810 > unrolled thread
| Started by | Archit Taneja <architt@codeaurora.org> |
|---|---|
| First post | 2015-07-29 07:40 +0200 |
| Last post | 2015-07-29 16:10 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support Archit Taneja <architt@codeaurora.org> - 2015-07-29 07:40 +0200
Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support Srinivas Kandagatla <srinivas.kandagatla@linaro.org> - 2015-07-29 16:10 +0200
| From | Archit Taneja <architt@codeaurora.org> |
|---|---|
| Date | 2015-07-29 07:40 +0200 |
| Subject | Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support |
| Message-ID | <pRsfT-7gC-13@gated-at.bofh.it> |
Hi Srini,
On 07/28/2015 06:24 PM, Srinivas Kandagatla wrote:
> This patch adds LVDS panel for IFC6410.
>
> Signed-off-by: Rob Clark <robdclark@gmail.com>
> [Rob Clark: WIP patch]
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> ---
> 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 = <PMIC_GPIO_STRENGTH_HIGH>;
> + power-source = <PM8921_GPIO_S4>;
> + };
> + };
> +
> + disp_en_gpios: disp-en-gpios {
> + pios {
> + pins = "gpio36";
> + bias-disable;
> + function = "normal";
> + qcom,drive-strength = <PMIC_GPIO_STRENGTH_HIGH>;
> + power-source = <PM8921_GPIO_S4>;
> + };
> + };
> +
> 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/
[toc] | [next] | [standalone]
| From | Srinivas Kandagatla <srinivas.kandagatla@linaro.org> |
|---|---|
| Date | 2015-07-29 16:10 +0200 |
| Message-ID | <pRAds-1Te-21@gated-at.bofh.it> |
| In reply to | #1194810 |
On 29/07/15 06:39, Archit Taneja wrote:
> Hi Srini,
>
> On 07/28/2015 06:24 PM, Srinivas Kandagatla wrote:
>> This patch adds LVDS panel for IFC6410.
>>
>> Signed-off-by: Rob Clark <robdclark@gmail.com>
>> [Rob Clark: WIP patch]
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>> 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:
>
Thanks for the input, I modified the panel support to use graphs which
works perfectly fine.
thanks,
srini
> 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 =
>> <PMIC_GPIO_STRENGTH_HIGH>;
>> + power-source = <PM8921_GPIO_S4>;
>> + };
>> + };
>> +
>> + disp_en_gpios: disp-en-gpios {
>> + pios {
>> + pins = "gpio36";
>> + bias-disable;
>> + function = "normal";
>> + qcom,drive-strength =
>> <PMIC_GPIO_STRENGTH_HIGH>;
>> + power-source = <PM8921_GPIO_S4>;
>> + };
>> + };
>> +
>> wlan_default_gpios: wlan-gpios {
>> pios {
>> pins = "gpio43";
>>
>
--
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/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web