Re: [PATCH 3/5] ARM: dts: AM33XX: Fix gpio numbering to match hardware/TRM

All of lore.kernel.org
 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

WARNING: multiple messages have this Message-ID (diff)

From: mporter@ti.com (Matt Porter)
To: linux-arm-kernel@lists.infradead.org
Subject: [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

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

Thread overview: 28+ 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 ` 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 12:59   ` Vaibhav Hiremath
2013-03-27 13:22   ` Peter Korsgaard
2013-03-27 13:22     ` Peter Korsgaard
2013-03-28  5:18     ` Hiremath, Vaibhav
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   ` 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 12:59   ` Vaibhav Hiremath
2013-03-27 13:23   ` Peter Korsgaard
2013-03-27 13:23     ` Peter Korsgaard
2013-03-27 14:11   ` Matt Porter [this message]
2013-03-27 14:11     ` Matt Porter
     [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     ` Vaibhav Hiremath
2013-03-27 12:59 ` [PATCH 5/5] ARM: dts: AM33XX: Add default pinctrl binding for UART0 device Vaibhav Hiremath
2013-03-27 12:59   ` 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
2013-03-27 13:28     ` 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
2013-03-27 13:29     ` 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-27 13:32     ` Matt Porter
2013-03-28  5:17     ` Hiremath, Vaibhav
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.