From: Eduardo Valentin <eduardo.valentin@ti.com>
To: Benoit Cousson <b-cousson@ti.com>
Cc: Eduardo Valentin <eduardo.valentin@ti.com>,
tony@atomide.com, linux-omap@vger.kernel.org,
Russell King <linux@arm.linux.org.uk>,
devicetree-discuss@lists.ozlabs.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCHv2 3/3] arm: dts: add bandgap entry for OMAP4460 devices
Date: Wed, 29 May 2013 10:11:00 -0400 [thread overview]
Message-ID: <51A60C74.4010307@ti.com> (raw)
In-Reply-To: <5194D0AF.3010407@ti.com>
[-- Attachment #1: Type: text/plain, Size: 904 bytes --]
Salut Monsieur Benoit,
On 16-05-2013 08:27, Eduardo Valentin wrote:
> On 16-05-2013 03:20, Benoit Cousson wrote:
>> Hi Eduardo,
>>
<cut>
>> We need to check.
>>
>
>
> Yeah, I also dont think this will work, because we will reparent the
> interrupt, setting to a different controller. That will break the TALERT
> signal already defined at GIC (check original patch).
>
> I propose keeping the way I sent. Unless there is a way to set two
> different controllers to same device.
>
Any idea on this patch? Shall we keep the way it is?
>> Regards,
>> Benoit
>>
>>
>>>> Where is the gpio controller phandle?
>>>>
>>>> Usually it looks like this:
>>>>
>>>> gpios = <&gpio1 8 0>;
>>>>
>>>>
>>>> Regards,
>>>> Benoit
>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>>
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: eduardo.valentin@ti.com (Eduardo Valentin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2 3/3] arm: dts: add bandgap entry for OMAP4460 devices
Date: Wed, 29 May 2013 10:11:00 -0400 [thread overview]
Message-ID: <51A60C74.4010307@ti.com> (raw)
In-Reply-To: <5194D0AF.3010407@ti.com>
Salut Monsieur Benoit,
On 16-05-2013 08:27, Eduardo Valentin wrote:
> On 16-05-2013 03:20, Benoit Cousson wrote:
>> Hi Eduardo,
>>
<cut>
>> We need to check.
>>
>
>
> Yeah, I also dont think this will work, because we will reparent the
> interrupt, setting to a different controller. That will break the TALERT
> signal already defined at GIC (check original patch).
>
> I propose keeping the way I sent. Unless there is a way to set two
> different controllers to same device.
>
Any idea on this patch? Shall we keep the way it is?
>> Regards,
>> Benoit
>>
>>
>>>> Where is the gpio controller phandle?
>>>>
>>>> Usually it looks like this:
>>>>
>>>> gpios = <&gpio1 8 0>;
>>>>
>>>>
>>>> Regards,
>>>> Benoit
>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>>
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 295 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130529/900f960b/attachment.sig>
WARNING: multiple messages have this Message-ID (diff)
From: Eduardo Valentin <eduardo.valentin@ti.com>
To: Benoit Cousson <b-cousson@ti.com>
Cc: Eduardo Valentin <eduardo.valentin@ti.com>, <tony@atomide.com>,
<linux-omap@vger.kernel.org>,
Russell King <linux@arm.linux.org.uk>,
<devicetree-discuss@lists.ozlabs.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCHv2 3/3] arm: dts: add bandgap entry for OMAP4460 devices
Date: Wed, 29 May 2013 10:11:00 -0400 [thread overview]
Message-ID: <51A60C74.4010307@ti.com> (raw)
In-Reply-To: <5194D0AF.3010407@ti.com>
[-- Attachment #1: Type: text/plain, Size: 904 bytes --]
Salut Monsieur Benoit,
On 16-05-2013 08:27, Eduardo Valentin wrote:
> On 16-05-2013 03:20, Benoit Cousson wrote:
>> Hi Eduardo,
>>
<cut>
>> We need to check.
>>
>
>
> Yeah, I also dont think this will work, because we will reparent the
> interrupt, setting to a different controller. That will break the TALERT
> signal already defined at GIC (check original patch).
>
> I propose keeping the way I sent. Unless there is a way to set two
> different controllers to same device.
>
Any idea on this patch? Shall we keep the way it is?
>> Regards,
>> Benoit
>>
>>
>>>> Where is the gpio controller phandle?
>>>>
>>>> Usually it looks like this:
>>>>
>>>> gpios = <&gpio1 8 0>;
>>>>
>>>>
>>>> Regards,
>>>> Benoit
>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>>
>
>
--
You have got to be excited about what you are doing. (L. Lamport)
Eduardo Valentin
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 295 bytes --]
next prev parent reply other threads:[~2013-05-29 14:11 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-15 14:58 [PATCHv2 0/3] arm: enable TI SoC thermal driver Eduardo Valentin
2013-05-15 14:58 ` [PATCHv2 1/3] arm: introduce config HAS_BANDGAP Eduardo Valentin
2013-05-15 14:58 ` Eduardo Valentin
2013-05-15 14:58 ` Eduardo Valentin
2013-05-31 15:03 ` Eduardo Valentin
2013-05-31 15:03 ` Eduardo Valentin
2013-05-31 15:03 ` Eduardo Valentin
2013-05-31 15:12 ` Fabio Estevam
2013-05-31 15:12 ` Fabio Estevam
2013-06-04 12:32 ` Eduardo Valentin
2013-06-04 12:32 ` Eduardo Valentin
2013-06-04 12:32 ` Eduardo Valentin
2013-06-04 12:42 ` [PATCHv3 " Eduardo Valentin
2013-06-04 12:42 ` Eduardo Valentin
2013-06-04 12:42 ` Eduardo Valentin
2013-05-15 14:58 ` [PATCHv2 2/3] arm: dts: add bandgap entry for OMAP443x devices Eduardo Valentin
2013-05-15 14:58 ` Eduardo Valentin
2013-05-15 14:58 ` Eduardo Valentin
2013-05-15 14:58 ` [PATCHv2 3/3] arm: dts: add bandgap entry for OMAP4460 devices Eduardo Valentin
2013-05-15 14:58 ` Eduardo Valentin
2013-05-15 14:58 ` Eduardo Valentin
2013-05-15 15:23 ` Benoit Cousson
2013-05-15 15:23 ` Benoit Cousson
2013-05-15 15:23 ` Benoit Cousson
2013-05-15 16:36 ` Eduardo Valentin
2013-05-15 16:36 ` Eduardo Valentin
2013-05-15 16:36 ` Eduardo Valentin
2013-05-15 16:57 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-15 16:57 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-16 12:29 ` Eduardo Valentin
2013-05-16 12:29 ` Eduardo Valentin
2013-05-16 12:29 ` Eduardo Valentin
2013-05-31 15:24 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 15:24 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 15:24 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-16 7:20 ` Benoit Cousson
2013-05-16 7:20 ` Benoit Cousson
2013-05-16 7:20 ` Benoit Cousson
2013-05-16 12:27 ` Eduardo Valentin
2013-05-16 12:27 ` Eduardo Valentin
2013-05-16 12:27 ` Eduardo Valentin
2013-05-29 14:11 ` Eduardo Valentin [this message]
2013-05-29 14:11 ` Eduardo Valentin
2013-05-29 14:11 ` Eduardo Valentin
2013-05-29 14:19 ` Cousson, Benoit
2013-05-29 14:19 ` Cousson, Benoit
2013-05-29 14:19 ` Cousson, Benoit
2013-05-29 15:42 ` Eduardo Valentin
2013-05-29 15:42 ` Eduardo Valentin
2013-05-29 15:42 ` Eduardo Valentin
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=51A60C74.4010307@ti.com \
--to=eduardo.valentin@ti.com \
--cc=b-cousson@ti.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=tony@atomide.com \
/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.