All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 11/23] ARM: dts: omap5-uevm.dts: add tca6424a
Date: Fri, 25 Apr 2014 14:20:19 +0000	[thread overview]
Message-ID: <535A6F23.1070409@ti.com> (raw)
In-Reply-To: <53594189.1090508@cogentembedded.com>

[-- Attachment #1: Type: text/plain, Size: 938 bytes --]

On 24/04/14 19:53, Sergei Shtylyov wrote:

>> Right. I wonder what the name should be... "gpio" is out, as the name
>> should be more specific.
> 
>    No, it's not out. The name should be "gpio@22", I think it would be
> unique.
> 
>> We already have gpio1-8, which are the gpio
>> banks from the SoC.
> 
>    I don't understand why you are indexing the names while you probably
> have the address part after @ that makes them unique already.

Ah sorry for the confusion, I misunderstood. I was talking about the
label, not the name, of the node.

So the labels for the gpio banks are 'gpio2' etc. The names are
'gpio@123456' etc. And yes, here the name should be 'gpio@22' as you said.

I didn't see any rules about the labels in the ePARP doc, so I guess
'tca6424a' would be a valid one. Maybe not a good one, though,
especially if there are multiple tca6424a chips on the board (I need to
check).

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	Archit Taneja <archit@ti.com>,
	linux-fbdev@vger.kernel.org, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Tony Lindgren <tony@atomide.com>
Subject: Re: [PATCH 11/23] ARM: dts: omap5-uevm.dts: add tca6424a
Date: Fri, 25 Apr 2014 17:20:19 +0300	[thread overview]
Message-ID: <535A6F23.1070409@ti.com> (raw)
In-Reply-To: <53594189.1090508@cogentembedded.com>

[-- Attachment #1: Type: text/plain, Size: 938 bytes --]

On 24/04/14 19:53, Sergei Shtylyov wrote:

>> Right. I wonder what the name should be... "gpio" is out, as the name
>> should be more specific.
> 
>    No, it's not out. The name should be "gpio@22", I think it would be
> unique.
> 
>> We already have gpio1-8, which are the gpio
>> banks from the SoC.
> 
>    I don't understand why you are indexing the names while you probably
> have the address part after @ that makes them unique already.

Ah sorry for the confusion, I misunderstood. I was talking about the
label, not the name, of the node.

So the labels for the gpio banks are 'gpio2' etc. The names are
'gpio@123456' etc. And yes, here the name should be 'gpio@22' as you said.

I didn't see any rules about the labels in the ePARP doc, so I guess
'tca6424a' would be a valid one. Maybe not a good one, though,
especially if there are multiple tca6424a chips on the board (I need to
check).

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: tomi.valkeinen@ti.com (Tomi Valkeinen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 11/23] ARM: dts: omap5-uevm.dts: add tca6424a
Date: Fri, 25 Apr 2014 17:20:19 +0300	[thread overview]
Message-ID: <535A6F23.1070409@ti.com> (raw)
In-Reply-To: <53594189.1090508@cogentembedded.com>

On 24/04/14 19:53, Sergei Shtylyov wrote:

>> Right. I wonder what the name should be... "gpio" is out, as the name
>> should be more specific.
> 
>    No, it's not out. The name should be "gpio at 22", I think it would be
> unique.
> 
>> We already have gpio1-8, which are the gpio
>> banks from the SoC.
> 
>    I don't understand why you are indexing the names while you probably
> have the address part after @ that makes them unique already.

Ah sorry for the confusion, I misunderstood. I was talking about the
label, not the name, of the node.

So the labels for the gpio banks are 'gpio2' etc. The names are
'gpio at 123456' etc. And yes, here the name should be 'gpio at 22' as you said.

I didn't see any rules about the labels in the ePARP doc, so I guess
'tca6424a' would be a valid one. Maybe not a good one, though,
especially if there are multiple tca6424a chips on the board (I need to
check).

 Tomi


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140425/48f2728c/attachment.sig>

  reply	other threads:[~2014-04-25 14:20 UTC|newest]

Thread overview: 135+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-24 10:16 [PATCH 00/23] OMAPDSS: OMAP5 display support Tomi Valkeinen
2014-04-24 10:16 ` Tomi Valkeinen
2014-04-24 10:16 ` Tomi Valkeinen
2014-04-24 10:16 ` [PATCH 01/23] OMAPDSS: HDMI: lane config support Tomi Valkeinen
2014-04-24 10:16   ` Tomi Valkeinen
2014-04-24 10:16   ` Tomi Valkeinen
2014-04-25 10:06   ` Archit Taneja
2014-04-25 10:18     ` Archit Taneja
2014-04-25 10:06     ` Archit Taneja
2014-04-25 10:28     ` Tomi Valkeinen
2014-04-25 10:28       ` Tomi Valkeinen
2014-04-25 10:28       ` Tomi Valkeinen
2014-04-24 10:16 ` [PATCH 02/23] Doc/DT: ti,omap4-dss: hdmi lanes Tomi Valkeinen
2014-04-24 10:16   ` Tomi Valkeinen
2014-04-24 10:16   ` Tomi Valkeinen
2014-04-24 10:16 ` [PATCH 03/23] OMAPDSS: HDMI4: set regulator voltage to 1.8V Tomi Valkeinen
2014-04-24 10:16   ` Tomi Valkeinen
2014-04-24 10:16   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 04/23] OMAPDSS: DSI: " Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 05/23] ARM: OMAP: hwmod: OMAP5 DSS hwmod data Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 06/23] ARM: OMAP: add OMAP5 DSI muxing Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-25 11:11   ` Archit Taneja
2014-04-25 11:23     ` Archit Taneja
2014-04-25 11:11     ` Archit Taneja
2014-04-25 11:18     ` Tomi Valkeinen
2014-04-25 11:18       ` Tomi Valkeinen
2014-04-25 11:18       ` Tomi Valkeinen
2014-04-25 12:58       ` Archit Taneja
2014-04-25 13:10         ` Archit Taneja
2014-04-25 12:58         ` Archit Taneja
2014-04-25 14:08         ` Tomi Valkeinen
2014-04-25 14:08           ` Tomi Valkeinen
2014-04-25 14:08           ` Tomi Valkeinen
2014-04-25 15:31           ` Tony Lindgren
2014-04-25 15:31             ` Tony Lindgren
2014-04-25 15:31             ` Tony Lindgren
2014-04-28  6:52             ` Tomi Valkeinen
2014-04-28  6:52               ` Tomi Valkeinen
2014-04-28  6:52               ` Tomi Valkeinen
2014-04-28 16:45               ` Tony Lindgren
2014-04-28 16:45                 ` Tony Lindgren
2014-04-28 16:45                 ` Tony Lindgren
2014-04-29  5:26                 ` Tomi Valkeinen
2014-04-29  5:26                   ` Tomi Valkeinen
2014-04-29  5:26                   ` Tomi Valkeinen
2014-04-29 15:05                   ` Tony Lindgren
2014-04-29 15:05                     ` Tony Lindgren
2014-04-29 15:05                     ` Tony Lindgren
2014-04-29 16:19                     ` Tomi Valkeinen
2014-04-29 16:19                       ` Tomi Valkeinen
2014-04-29 16:19                       ` Tomi Valkeinen
2014-04-29 16:32                       ` Tomi Valkeinen
2014-04-29 16:32                         ` Tomi Valkeinen
2014-04-29 16:32                         ` Tomi Valkeinen
2014-04-29 17:38                         ` Tony Lindgren
2014-04-29 17:38                           ` Tony Lindgren
2014-04-29 17:38                           ` Tony Lindgren
2014-04-30  6:13                           ` Tomi Valkeinen
2014-04-30  6:13                             ` Tomi Valkeinen
2014-04-30  6:13                             ` Tomi Valkeinen
2014-04-30 17:56                             ` Tony Lindgren
2014-04-30 17:56                               ` Tony Lindgren
2014-04-30 17:56                               ` Tony Lindgren
2014-05-02 13:06                               ` Tomi Valkeinen
2014-05-02 13:06                                 ` Tomi Valkeinen
2014-05-02 13:06                                 ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 07/23] ARM: OMAP: add detection of omap5-dss Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 08/23] ARM: dts: omap5-clocks.dtsi: add dss iclk Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 09/23] ARM: dts: omap5-clocks.dtsi: add ti,set-rate-parent to dss_dss_clk Tomi Valkeinen
2014-04-24 10:17   ` [PATCH 09/23] ARM: dts: omap5-clocks.dtsi: add ti, set-rate-parent " Tomi Valkeinen
2014-04-24 10:17   ` [PATCH 09/23] ARM: dts: omap5-clocks.dtsi: add ti,set-rate-parent " Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 10/23] ARM: dts: omap5.dtsi: add DSS nodes Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 11/23] ARM: dts: omap5-uevm.dts: add tca6424a Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 13:49   ` Sergei Shtylyov
2014-04-24 13:49     ` Sergei Shtylyov
2014-04-24 13:49     ` Sergei Shtylyov
2014-04-24 14:33     ` Tomi Valkeinen
2014-04-24 14:33       ` Tomi Valkeinen
2014-04-24 14:33       ` Tomi Valkeinen
2014-04-24 16:53       ` Sergei Shtylyov
2014-04-24 16:53         ` Sergei Shtylyov
2014-04-24 16:53         ` Sergei Shtylyov
2014-04-25 14:20         ` Tomi Valkeinen [this message]
2014-04-25 14:20           ` Tomi Valkeinen
2014-04-25 14:20           ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 12/23] ARM: dts: omap5-uevm.dts: add display nodes Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 13/23] OMAPDSS: DSS & DISPC DT support for OMAP5 Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 14/23] OMAPDSS: features: fix OMAP5 features Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 15/23] OMAPDSS: DPI: fix LCD3 DSI source Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 16/23] OMAPDSS: DSI: Add OMAP5 DSI module IDs Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 17/23] OMAPDSS: HDMI: improve Makefile Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 18/23] OMAPDSS: HDMI: move irq & phy pwr handling Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 19/23] OMAPDSS: HDMI: support larger register offsets for OMAP5 HDMI core Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 20/23] OMAPDSS: HDMI: PHY changes for OMAP5 Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 21/23] OMAPDSS: HDMI: PLL " Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 22/23] OMAPDSS: HDMI: Add OMAP5 HDMI support Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17 ` [PATCH 23/23] Doc/DT: Add OMAP5 DSS DT bindings Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen
2014-04-24 10:17   ` Tomi Valkeinen

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=535A6F23.1070409@ti.com \
    --to=tomi.valkeinen@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 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.