linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: ricardo.neri@ti.com (Ricardo Neri)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/6] ARM/dts: omap4-panda: Add pinmux configuration for HDMI
Date: Mon, 5 Nov 2012 23:24:41 -0600	[thread overview]
Message-ID: <50989F19.70401@ti.com> (raw)
In-Reply-To: <5097BC67.5050508@ti.com>

Hi Tomi,

On 11/05/2012 07:17 AM, Tomi Valkeinen wrote:
> Hi Ricardo,
>
> On 2012-11-05 15:14, Tomi Valkeinen wrote:
>> From: Ricardo Neri <ricardo.neri@ti.com>
>>
>> Add the pinmux configuration for HDMI and TPD12S015A. Configure the
>> gpios for the TPD12S015A and SDA, SCL and CEC for HDMI.
>>
>> Signed-off-by: Ricardo Neri <ricardo.neri@ti.com>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> ---
>>   arch/arm/boot/dts/omap4-panda-a4.dts |   17 +++++++++++++++++
>>   arch/arm/boot/dts/omap4-panda.dts    |   18 ++++++++++++++++++
>>   arch/arm/boot/dts/omap4-pandaES.dts  |    9 +++++++++
>>   3 files changed, 44 insertions(+)
>>   create mode 100644 arch/arm/boot/dts/omap4-panda-a4.dts
>
> Ricardo, I modified this and the other dts patch to add new .dts files
> for the board revisions with external pull on the SCL/SDA lines. Can you
> verify that these are correct?

I verified the pinmux values to make sure that the OMAP internal pull is 
not enabled in Pandaboard A4/4430 ES2.3+. It looks correct to me.

I also tested it on 4430 ES2.1 and 4430 ES2.3 SDPs and works fine.

BR,

Ricardo

Tested-by: Ricardo Neri <ricardo.neri@ti.com>
Reviewed-by: Ricardo Neri <ricardo.neri@ti.com>

>
>   Tomi
>
>> diff --git a/arch/arm/boot/dts/omap4-panda-a4.dts b/arch/arm/boot/dts/omap4-panda-a4.dts
>> new file mode 100644
>> index 0000000..75466d2
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/omap4-panda-a4.dts
>> @@ -0,0 +1,17 @@
>> +/*
>> + * Copyright (C) 2012 Texas Instruments Incorporated - http://www.ti.com/
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License version 2 as
>> + * published by the Free Software Foundation.
>> + */
>> +/include/ "omap4-panda.dts"
>> +
>> +/* Pandaboard Rev A4+ have external pullups on SCL & SDA */
>> +&dss_hdmi_pins {
>> +	pinctrl-single,pins = <
>> +		0x5a 0x118	/* hdmi_cec.hdmi_cec INPUT PULLUP | MODE 0 */
>> +		0x5c 0x100	/* hdmi_scl.hdmi_scl INPUT | MODE 0 */
>> +		0x5e 0x100	/* hdmi_sda.hdmi_sda INPUT | MODE 0 */
>> +		>;
>> +};
>> diff --git a/arch/arm/boot/dts/omap4-panda.dts b/arch/arm/boot/dts/omap4-panda.dts
>> index e8f927c..8060f5b 100644
>> --- a/arch/arm/boot/dts/omap4-panda.dts
>> +++ b/arch/arm/boot/dts/omap4-panda.dts
>> @@ -65,6 +65,8 @@
>>   			&twl6040_pins
>>   			&mcpdm_pins
>>   			&mcbsp1_pins
>> +			&dss_hdmi_pins
>> +			&tpd12s015_pins
>>   	>;
>>
>>   	twl6040_pins: pinmux_twl6040_pins {
>> @@ -92,6 +94,22 @@
>>   			0xc4 0x100	/* abe_mcbsp1_fsx.abe_mcbsp1_fsx INPUT | MODE0 */
>>   		>;
>>   	};
>> +
>> +	dss_hdmi_pins: pinmux_dss_hdmi_pins {
>> +		pinctrl-single,pins = <
>> +			0x5a 0x118	/* hdmi_cec.hdmi_cec INPUT PULLUP | MODE 0 */
>> +			0x5c 0x118	/* hdmi_scl.hdmi_scl INPUT PULLUP | MODE 0 */
>> +			0x5e 0x118	/* hdmi_sda.hdmi_sda INPUT PULLUP | MODE 0 */
>> +		>;
>> +	};
>> +
>> +	tpd12s015_pins: pinmux_tpd12s015_pins {
>> +		pinctrl-single,pins = <
>> +			0x22 0x3	/* gpmc_a17.gpio_41 OUTPUT | MODE3 */
>> +			0x48 0x3	/* gpmc_nbe1.gpio_60 OUTPUT | MODE3 */
>> +			0x58 0x10b	/* hdmi_hpd.gpio_63 INPUT PULLDOWN | MODE3 */
>> +		>;
>> +	};
>>   };
>>
>>   &i2c1 {
>> diff --git a/arch/arm/boot/dts/omap4-pandaES.dts b/arch/arm/boot/dts/omap4-pandaES.dts
>> index d4ba43a..73bc1a6 100644
>> --- a/arch/arm/boot/dts/omap4-pandaES.dts
>> +++ b/arch/arm/boot/dts/omap4-pandaES.dts
>> @@ -22,3 +22,12 @@
>>   		"AFML", "Line In",
>>   		"AFMR", "Line In";
>>   };
>> +
>> +/* PandaboardES has external pullups on SCL & SDA */
>> +&dss_hdmi_pins {
>> +	pinctrl-single,pins = <
>> +		0x5a 0x118	/* hdmi_cec.hdmi_cec INPUT PULLUP | MODE 0 */
>> +		0x5c 0x100	/* hdmi_scl.hdmi_scl INPUT | MODE 0 */
>> +		0x5e 0x100	/* hdmi_sda.hdmi_sda INPUT | MODE 0 */
>> +		>;
>> +};
>>
>
>

  reply	other threads:[~2012-11-06  5:24 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-05 13:14 [PATCH 0/6] OMAPDSS: enable DSS for Panda & SDP with devtree Tomi Valkeinen
2012-11-05 13:14 ` [PATCH 1/6] ARM/dts: omap4-panda: Add pinmux configuration for HDMI Tomi Valkeinen
2012-11-05 13:17   ` Tomi Valkeinen
2012-11-06  5:24     ` Ricardo Neri [this message]
2012-11-05 17:43   ` Benoit Cousson
2012-11-07 10:56   ` Benoit Cousson
2012-11-13 19:10   ` Ricardo Neri
2012-11-14  7:39     ` Tomi Valkeinen
2012-11-05 13:14 ` [PATCH 2/6] ARM/dts: omap4-sdp: " Tomi Valkeinen
2012-11-07 10:57   ` Benoit Cousson
2012-11-05 13:14 ` [PATCH 3/6] OMAPDSS: HACK: look for regulators with omap4 names Tomi Valkeinen
2012-11-05 13:14 ` [PATCH 4/6] OMAP: panda: move display init from board file to dss-common.c Tomi Valkeinen
2012-11-05 13:14 ` [PATCH 5/6] OMAP: omap4sdp: " Tomi Valkeinen
2012-11-05 13:14 ` [PATCH 6/6] OMAP: board-generic: enable DSS for panda & sdp boards Tomi Valkeinen
2012-11-07  1:04 ` [PATCH 0/6] OMAPDSS: enable DSS for Panda & SDP with devtree Tony Lindgren
2012-11-07 10:58   ` Benoit Cousson
2012-11-07 21:34     ` Tony Lindgren

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50989F19.70401@ti.com \
    --to=ricardo.neri@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).