From: Sourav Poddar <sourav.poddar@ti.com>
To: Benoit Cousson <bcousson@baylibre.com>
Cc: rob.herring@calxeda.com, mark.rutland@arm.com,
swarren@wwwdotorg.org, devicetree@vger.kernel.org,
tony@atomide.com, balbi@ti.com, linux-omap@vger.kernel.org,
nsekhar@ti.com
Subject: Re: [PATCH 0/3] Add more device nodes for am43x gp evm.
Date: Thu, 19 Dec 2013 08:51:10 +0530 [thread overview]
Message-ID: <52B26626.9020507@ti.com> (raw)
In-Reply-To: <52B1EEAC.2010109@baylibre.com>
On Thursday 19 December 2013 12:21 AM, Benoit Cousson wrote:
> On 18/12/2013 19:15, Sourav Poddar wrote:
>> On Wednesday 18 December 2013 11:19 PM, Benoit Cousson wrote:
>>> Hi Sourav,
>>>
>>> On 18/12/2013 18:43, Sourav Poddar wrote:
>>>> Benoit,
>>>> On Wednesday 27 November 2013 01:01 PM, Sourav Poddar wrote:
>>>>> The patch series adds support for enabling gpio, pwm backlight and
>>>>> matrix gpio keys on am43x-gp-evm.
>>>>>
>>>>> Done on top of 3.13-rc1 + tero clock series(1) + Afzal's basic gp
>>>>> support(2).
>>>>>
>>>>> [1]: https://patchwork.kernel.org/patch/3009541/
>>>>> [2]: https://patchwork.kernel.org/patch/3171761/
>>>>>
>>>>> Tested on am43x-gp-evm.
>>>>>
>>>>> There is a some bug while using regulators through backlight
>>>>> driver on 3.13-rc1. So, tested pwm part with this patch[3].
>>>>>
>>>>> [3]: http://www.spinics.net/lists/arm-kernel/msg288215.html
>>>>>
>>>>> Sourav Poddar (3):
>>>>> arm: dts: am437x-gp-evm: Enable gpio.
>>>>> ARM: dts: am43x-gp-evm: Add matrix gpio keys.
>>>>> ARM: dts: am437x-gp-evm: Add pwm backlight support.
>>>>>
>>>>> arch/arm/boot/dts/am437x-gp-evm.dts | 53
>>>>> +++++++++++++++++++++++++++++++++++
>>>>> 1 files changed, 53 insertions(+), 0 deletions(-)
>>>>>
>>>> Ping on this?
>>>
>>> The series looks good, but you should rebase it on top of for_3.14/dts
>>> that is based on the big cleanup branch Tony has done.
>>> I cannot apply it right now.
>>>
>> Ok. I will rebase it and send you the series, As you can see there is
>> one dependency on afzal basic support patch, as mentioned in
>> the cover letter. If you are ok with that patch, it has to go first. So,
>> if you wish I can work with afzal and send you this
>> series and afzal patch together.
>
> Yes, go ahead and repost the whole series with the depency.
>
> BTW, could you do that as well with your other series?
>
Yes, I will do that.
> Thanks,
> Benoit
prev parent reply other threads:[~2013-12-19 3:21 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-27 7:31 [PATCH 0/3] Add more device nodes for am43x gp evm Sourav Poddar
2013-11-27 7:31 ` [PATCH 1/3] arm: dts: am437x-gp-evm: Add pwm backlight support Sourav Poddar
2013-11-27 7:31 ` [PATCH 2/3] arm: dts: am437x-gp-evm: Enable gpio Sourav Poddar
2013-11-27 7:31 ` [PATCH 3/3] arm: dts: am43x-gp-evm: Add matrix gpio keys Sourav Poddar
2013-12-05 9:48 ` [PATCH 0/3] Add more device nodes for am43x gp evm Sourav Poddar
2013-12-18 17:43 ` Sourav Poddar
2013-12-18 17:49 ` Benoit Cousson
2013-12-18 18:15 ` Sourav Poddar
2013-12-18 18:51 ` Benoit Cousson
2013-12-19 3:21 ` Sourav Poddar [this message]
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=52B26626.9020507@ti.com \
--to=sourav.poddar@ti.com \
--cc=balbi@ti.com \
--cc=bcousson@baylibre.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=nsekhar@ti.com \
--cc=rob.herring@calxeda.com \
--cc=swarren@wwwdotorg.org \
--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 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).