From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
Lee Jones <lee.jones@linaro.org>
Cc: linux-pwm@vger.kernel.org, kernel@stlinux.com,
devicetree@vger.kernel.org, srinivas.kandagatla@gmail.com,
linux-kernel@vger.kernel.org, thierry.reding@gmail.com,
linux-arm-kernel@lists.infradead.org
Subject: Re: [STLinux Kernel] [PATCH 4/7] ARM: stih416-b2020e: Enable PWM on the B2020 Rev-E
Date: Thu, 19 Jun 2014 12:59:27 +0200 [thread overview]
Message-ID: <53A2C28F.1080007@st.com> (raw)
In-Reply-To: <20140619101236.GA7390@griffinp-ThinkPad-X1-Carbon-2nd>
[-- Attachment #1: Type: text/plain, Size: 1175 bytes --]
Hi Peter,
On 06/19/2014 12:12 PM, Peter Griffin wrote:
> Hi Lee,
>
>>> Why are you only enabling this for rev-E board?
>>
>> Purely for testing purposes, as it's the only board I have. If
>> anyone has the other revisions and would be happy to test the driver
>> for me then great.
>
> Ok that's what I suspected.
>
> If you tell me how, I can test on a Rev C board. But as there are several other
> revisions I think checking the board schematics would be the quickest way forward.
Only Rev C & E are in the wild now, so checking these two boards will
be enough.
Thanks,
Maxime
>
>>> If you make the stih415 DT PWM nodes part of this series as well, then
>>> assuming there is no good reason to the contary, the PWM could
>>> be enabled for both stih415/6 in the common stih41x-b2020 DTS file which
>>> is also included by rev-E (assuming that PWM wiring is not some rev-E specific
>>> thing).
>
> Can you add the stih415 bindings in V2 as well?
>
> regards,
>
> Peter
>
> _______________________________________________
> Kernel mailing list
> Kernel@stlinux.com
> http://www.stlinux.com/mailman/listinfo/kernel
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: maxime.coquelin@st.com (Maxime Coquelin)
To: linux-arm-kernel@lists.infradead.org
Subject: [STLinux Kernel] [PATCH 4/7] ARM: stih416-b2020e: Enable PWM on the B2020 Rev-E
Date: Thu, 19 Jun 2014 12:59:27 +0200 [thread overview]
Message-ID: <53A2C28F.1080007@st.com> (raw)
In-Reply-To: <20140619101236.GA7390@griffinp-ThinkPad-X1-Carbon-2nd>
Hi Peter,
On 06/19/2014 12:12 PM, Peter Griffin wrote:
> Hi Lee,
>
>>> Why are you only enabling this for rev-E board?
>>
>> Purely for testing purposes, as it's the only board I have. If
>> anyone has the other revisions and would be happy to test the driver
>> for me then great.
>
> Ok that's what I suspected.
>
> If you tell me how, I can test on a Rev C board. But as there are several other
> revisions I think checking the board schematics would be the quickest way forward.
Only Rev C & E are in the wild now, so checking these two boards will
be enough.
Thanks,
Maxime
>
>>> If you make the stih415 DT PWM nodes part of this series as well, then
>>> assuming there is no good reason to the contary, the PWM could
>>> be enabled for both stih415/6 in the common stih41x-b2020 DTS file which
>>> is also included by rev-E (assuming that PWM wiring is not some rev-E specific
>>> thing).
>
> Can you add the stih415 bindings in V2 as well?
>
> regards,
>
> Peter
>
> _______________________________________________
> Kernel mailing list
> Kernel at stlinux.com
> http://www.stlinux.com/mailman/listinfo/kernel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140619/4cb3411c/attachment.sig>
WARNING: multiple messages have this Message-ID (diff)
From: Maxime Coquelin <maxime.coquelin@st.com>
To: Peter Griffin <peter.griffin@linaro.org>,
Lee Jones <lee.jones@linaro.org>
Cc: <linux-pwm@vger.kernel.org>, <kernel@stlinux.com>,
<devicetree@vger.kernel.org>, <srinivas.kandagatla@gmail.com>,
<linux-kernel@vger.kernel.org>, <thierry.reding@gmail.com>,
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [STLinux Kernel] [PATCH 4/7] ARM: stih416-b2020e: Enable PWM on the B2020 Rev-E
Date: Thu, 19 Jun 2014 12:59:27 +0200 [thread overview]
Message-ID: <53A2C28F.1080007@st.com> (raw)
In-Reply-To: <20140619101236.GA7390@griffinp-ThinkPad-X1-Carbon-2nd>
[-- Attachment #1: Type: text/plain, Size: 1175 bytes --]
Hi Peter,
On 06/19/2014 12:12 PM, Peter Griffin wrote:
> Hi Lee,
>
>>> Why are you only enabling this for rev-E board?
>>
>> Purely for testing purposes, as it's the only board I have. If
>> anyone has the other revisions and would be happy to test the driver
>> for me then great.
>
> Ok that's what I suspected.
>
> If you tell me how, I can test on a Rev C board. But as there are several other
> revisions I think checking the board schematics would be the quickest way forward.
Only Rev C & E are in the wild now, so checking these two boards will
be enough.
Thanks,
Maxime
>
>>> If you make the stih415 DT PWM nodes part of this series as well, then
>>> assuming there is no good reason to the contary, the PWM could
>>> be enabled for both stih415/6 in the common stih41x-b2020 DTS file which
>>> is also included by rev-E (assuming that PWM wiring is not some rev-E specific
>>> thing).
>
> Can you add the stih415 bindings in V2 as well?
>
> regards,
>
> Peter
>
> _______________________________________________
> Kernel mailing list
> Kernel@stlinux.com
> http://www.stlinux.com/mailman/listinfo/kernel
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2014-06-19 10:59 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-18 14:52 [PATCH 1/7] ARM: stih407: Add DT nodes for for PWM Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 14:52 ` [PATCH 2/7] ARM: stih416: Add Pinctrl settings " Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 14:52 ` [PATCH 3/7] ARM: stih416: Add DT nodes " Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 15:08 ` Gabriel Fernandez
2014-06-18 15:08 ` Gabriel Fernandez
2014-06-18 15:17 ` Lee Jones
2014-06-18 15:17 ` Lee Jones
2014-06-18 15:28 ` [PATCH v2 " Lee Jones
2014-06-18 15:28 ` Lee Jones
2014-06-18 14:52 ` [PATCH 4/7] ARM: stih416-b2020e: Enable PWM on the B2020 Rev-E Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-19 9:05 ` [STLinux Kernel] " Peter Griffin
2014-06-19 9:05 ` Peter Griffin
2014-06-19 9:20 ` Lee Jones
2014-06-19 9:20 ` Lee Jones
2014-06-19 10:12 ` Peter Griffin
2014-06-19 10:12 ` Peter Griffin
2014-06-19 10:59 ` Maxime Coquelin [this message]
2014-06-19 10:59 ` Maxime Coquelin
2014-06-19 10:59 ` Maxime Coquelin
2014-06-19 11:22 ` Lee Jones
2014-06-19 11:22 ` Lee Jones
2014-06-19 13:29 ` Peter Griffin
2014-06-19 13:29 ` Peter Griffin
2014-06-18 14:52 ` [PATCH 5/7] ARM: multi_v7_defconfig: Enable ST's PWM driver Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 14:52 ` [PATCH 6/7] pwm: st: Add new driver for ST's PWM IP Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 23:11 ` Thierry Reding
2014-06-18 23:11 ` Thierry Reding
2014-06-19 8:44 ` Lee Jones
2014-06-19 8:44 ` Lee Jones
2014-06-19 8:44 ` Lee Jones
2014-06-19 11:32 ` Russell King - ARM Linux
2014-06-19 11:32 ` Russell King - ARM Linux
2014-06-19 14:27 ` Ajit Pal
2014-06-19 14:27 ` Ajit Pal
[not found] ` <53A2F342.7030606-qxv4g6HH51o@public.gmane.org>
2014-06-20 22:27 ` Thierry Reding
2014-06-20 22:27 ` Thierry Reding
2014-06-20 22:27 ` Thierry Reding
2014-06-20 22:16 ` Thierry Reding
2014-06-20 22:16 ` Thierry Reding
2014-06-18 14:52 ` [PATCH 7/7] pwm: st: Supply Device Tree binding documentation " Lee Jones
2014-06-18 14:52 ` Lee Jones
2014-06-18 15:08 ` Gabriel Fernandez
2014-06-18 15:08 ` Gabriel Fernandez
2014-06-18 15:16 ` Lee Jones
2014-06-18 15:16 ` Lee Jones
2014-06-18 15:29 ` [PATCH v2 " Lee Jones
2014-06-18 15:29 ` Lee Jones
2014-06-18 15:07 ` [PATCH 1/7] ARM: stih407: Add DT nodes for for PWM Gabriel Fernandez
2014-06-18 15:07 ` Gabriel Fernandez
2014-06-18 15:16 ` Lee Jones
2014-06-18 15:16 ` Lee Jones
2014-06-18 15:26 ` [PATCH v2 " Lee Jones
2014-06-18 15:26 ` Lee Jones
2014-06-19 8:38 ` [STLinux Kernel] [PATCH " Peter Griffin
2014-06-19 8:38 ` Peter Griffin
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=53A2C28F.1080007@st.com \
--to=maxime.coquelin@st.com \
--cc=devicetree@vger.kernel.org \
--cc=kernel@stlinux.com \
--cc=lee.jones@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=peter.griffin@linaro.org \
--cc=srinivas.kandagatla@gmail.com \
--cc=thierry.reding@gmail.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.