devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Matt Porter <mporter@ti.com>
To: Vaibhav Hiremath <hvaibhav@ti.com>
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	b-cousson@ti.com, devicetree-discuss@lists.ozlabs.org
Subject: Re: [PATCH 3/5] ARM: dts: AM33XX: Fix gpio numbering to match hardware/TRM
Date: Wed, 27 Mar 2013 10:11:55 -0400	[thread overview]
Message-ID: <20130327141155.GK18335@beef> (raw)
In-Reply-To: <1364389156-8273-4-git-send-email-hvaibhav@ti.com>

On Wed, Mar 27, 2013 at 06:29:14PM +0530, Vaibhav Hiremath wrote:
> With DT support, where naming convention is based on base-addr
> and not id, so we should follow TRM/Spec numbering label.
> 
> This patch changes GPIO numbering as per TRM, as gpio0-3.
> 
> Matt Porter had submitted base patch sometime back but it
> didn't make it to the mainline -
> https://patchwork.kernel.org/patch/1433001/
>
> Later DT nodes for gpio based keypad and led driver got
> merged, so this fix needs to propagate to board dts files
> as well.
> 
> Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> Signed-off-by: Matt Porter <mporter@ti.com>
> Cc: Benoit Cousson <b-cousson@ti.com>

Thanks for picking this up, Vaibhav. It'll eliminate all
the pain people have with the DT not matching the TRM.

-Matt

  parent reply	other threads:[~2013-03-27 14:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-27 12:59 [PATCH 0/5] ARM: dts: AM33XX: Cleanup and pinctrl binding support Vaibhav Hiremath
2013-03-27 12:59 ` [PATCH 1/5] ARM: dts: AM33XX: Fix the i2c numbering to match hardware/TRM Vaibhav Hiremath
2013-03-27 13:22   ` Peter Korsgaard
2013-03-28  5:18     ` Hiremath, Vaibhav
2013-03-27 12:59 ` [PATCH 2/5] ARM: dts: AM33XX: Add default pinctrl binding for I2C device Vaibhav Hiremath
2013-03-27 12:59 ` [PATCH 3/5] ARM: dts: AM33XX: Fix gpio numbering to match hardware/TRM Vaibhav Hiremath
2013-03-27 13:23   ` Peter Korsgaard
2013-03-27 14:11   ` Matt Porter [this message]
     [not found] ` <1364389156-8273-1-git-send-email-hvaibhav-l0cyMroinI0@public.gmane.org>
2013-03-27 12:59   ` [PATCH 4/5] ARM: dts: AM33XX: Add pinctrl binding to gpio-leds node Vaibhav Hiremath
2013-03-27 12:59 ` [PATCH 5/5] ARM: dts: AM33XX: Add default pinctrl binding for UART0 device Vaibhav Hiremath
     [not found] ` <686d89bc7b6d4bf1bc1c0524e4b586dd@DLEE51.ent.ti.com>
2013-03-27 13:28   ` [PATCH 1/5] ARM: dts: AM33XX: Fix the i2c numbering to match hardware/TRM Matt Porter
     [not found] ` <a20a6c7b5b5b45328684e5afab385ae5@DFLE51.ent.ti.com>
2013-03-27 13:29   ` [PATCH 2/5] ARM: dts: AM33XX: Add default pinctrl binding for I2C device Matt Porter
     [not found] ` <896a1261daec485db6be66630bc4019a@DFLE51.ent.ti.com>
2013-03-27 13:32   ` [PATCH 5/5] ARM: dts: AM33XX: Add default pinctrl binding for UART0 device Matt Porter
2013-03-28  5:17     ` Hiremath, Vaibhav

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=20130327141155.GK18335@beef \
    --to=mporter@ti.com \
    --cc=b-cousson@ti.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=hvaibhav@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.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).