From: Joel Fernandes <joelf@ti.com>
To: Sekhar Nori <nsekhar@ti.com>
Cc: Balaji TK <balajitk@ti.com>, Arnd Bergmann <arnd@arndb.de>,
Mark Jackson <mpfj-list@newflow.co.uk>,
Tony Lindgren <tony@atomide.com>,
Pantel Antoniou <panto@antoniou-consulting.com>,
Linux MMC List <linux-mmc@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Vinod Koul <vinod.koul@intel.com>,
Jason Kridner <jkridner@beagleboard.org>,
Benoit Cousson <benoit.cousson@linaro.org>,
Linux OMAP List <linux-omap@vger.kernel.org>,
Linux ARM Kernel List <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2] ARM: EDMA: Fix clearing of unused list for DT DMA resources
Date: Tue, 30 Jul 2013 00:05:25 -0500 [thread overview]
Message-ID: <51F74995.2050202@ti.com> (raw)
In-Reply-To: <51F74688.3090702@ti.com>
On 07/29/2013 11:52 PM, Sekhar Nori wrote:> On Tuesday 30 July 2013
09:23 AM, Joel Fernandes wrote:
>> On 07/29/2013 02:04 AM, Sekhar Nori wrote:
>>> On Sunday 28 July 2013 05:02 AM, Joel Fernandes wrote:
>>>> Hi Tony or Sekhar,
>>>>
>>>> If this patch looks ok, could you pick it up for -rc cycle?
>>>>
>>>> It fixes DMA breakages after the merge window for devices for which DMA
>>>> resources are being populated in device tree instead pdev.
>>>
>>> But which DT-enabled platform in kernel is using EDMA? A grep for edma
>>> over arch/arm/boot/dts/* brings up nothing.
>>> Does this really have to go into the -rc cycle or can it wait till
>>> v3.12?
>>
>>
>> But unused channel list is also populated for private EDMA callers like
>> davinci-pcm no? IIRC, HWMOD data is also removed for McASP so EDMA for
>> those may break.
>
> Yes, but which DT-enabled platform has davinci-pcm working today?
> AFAICS, this bug affects DT-enabled platforms only.
Yes, you're right. I didn't realize nothing else is working yet. I guess
its fine then for next kernel release.
Thanks,
-Joel
WARNING: multiple messages have this Message-ID (diff)
From: joelf@ti.com (Joel Fernandes)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] ARM: EDMA: Fix clearing of unused list for DT DMA resources
Date: Tue, 30 Jul 2013 00:05:25 -0500 [thread overview]
Message-ID: <51F74995.2050202@ti.com> (raw)
In-Reply-To: <51F74688.3090702@ti.com>
On 07/29/2013 11:52 PM, Sekhar Nori wrote:> On Tuesday 30 July 2013
09:23 AM, Joel Fernandes wrote:
>> On 07/29/2013 02:04 AM, Sekhar Nori wrote:
>>> On Sunday 28 July 2013 05:02 AM, Joel Fernandes wrote:
>>>> Hi Tony or Sekhar,
>>>>
>>>> If this patch looks ok, could you pick it up for -rc cycle?
>>>>
>>>> It fixes DMA breakages after the merge window for devices for which DMA
>>>> resources are being populated in device tree instead pdev.
>>>
>>> But which DT-enabled platform in kernel is using EDMA? A grep for edma
>>> over arch/arm/boot/dts/* brings up nothing.
>>> Does this really have to go into the -rc cycle or can it wait till
>>> v3.12?
>>
>>
>> But unused channel list is also populated for private EDMA callers like
>> davinci-pcm no? IIRC, HWMOD data is also removed for McASP so EDMA for
>> those may break.
>
> Yes, but which DT-enabled platform has davinci-pcm working today?
> AFAICS, this bug affects DT-enabled platforms only.
Yes, you're right. I didn't realize nothing else is working yet. I guess
its fine then for next kernel release.
Thanks,
-Joel
WARNING: multiple messages have this Message-ID (diff)
From: Joel Fernandes <joelf@ti.com>
To: Sekhar Nori <nsekhar@ti.com>
Cc: Tony Lindgren <tony@atomide.com>,
Vinod Koul <vinod.koul@intel.com>,
Benoit Cousson <benoit.cousson@linaro.org>,
Balaji TK <balajitk@ti.com>, Arnd Bergmann <arnd@arndb.de>,
Jason Kridner <jkridner@beagleboard.org>,
Mark Jackson <mpfj-list@newflow.co.uk>,
Linux OMAP List <linux-omap@vger.kernel.org>,
Linux ARM Kernel List <linux-arm-kernel@lists.infradead.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Linux MMC List <linux-mmc@vger.kernel.org>,
Pantel Antoniou <panto@antoniou-consulting.com>
Subject: Re: [PATCH v2] ARM: EDMA: Fix clearing of unused list for DT DMA resources
Date: Tue, 30 Jul 2013 00:05:25 -0500 [thread overview]
Message-ID: <51F74995.2050202@ti.com> (raw)
In-Reply-To: <51F74688.3090702@ti.com>
On 07/29/2013 11:52 PM, Sekhar Nori wrote:> On Tuesday 30 July 2013
09:23 AM, Joel Fernandes wrote:
>> On 07/29/2013 02:04 AM, Sekhar Nori wrote:
>>> On Sunday 28 July 2013 05:02 AM, Joel Fernandes wrote:
>>>> Hi Tony or Sekhar,
>>>>
>>>> If this patch looks ok, could you pick it up for -rc cycle?
>>>>
>>>> It fixes DMA breakages after the merge window for devices for which DMA
>>>> resources are being populated in device tree instead pdev.
>>>
>>> But which DT-enabled platform in kernel is using EDMA? A grep for edma
>>> over arch/arm/boot/dts/* brings up nothing.
>>> Does this really have to go into the -rc cycle or can it wait till
>>> v3.12?
>>
>>
>> But unused channel list is also populated for private EDMA callers like
>> davinci-pcm no? IIRC, HWMOD data is also removed for McASP so EDMA for
>> those may break.
>
> Yes, but which DT-enabled platform has davinci-pcm working today?
> AFAICS, this bug affects DT-enabled platforms only.
Yes, you're right. I didn't realize nothing else is working yet. I guess
its fine then for next kernel release.
Thanks,
-Joel
next prev parent reply other threads:[~2013-07-30 5:05 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-22 17:59 [PATCH v2] ARM: EDMA: Fix clearing of unused list for DT DMA resources Joel Fernandes
2013-07-22 17:59 ` Joel Fernandes
2013-07-22 17:59 ` Joel Fernandes
2013-07-27 23:32 ` Joel Fernandes
2013-07-27 23:32 ` Joel Fernandes
2013-07-29 7:04 ` Sekhar Nori
2013-07-29 7:04 ` Sekhar Nori
2013-07-29 7:04 ` Sekhar Nori
2013-07-30 3:53 ` Joel Fernandes
2013-07-30 3:53 ` Joel Fernandes
2013-07-30 4:52 ` Sekhar Nori
2013-07-30 4:52 ` Sekhar Nori
2013-07-30 4:52 ` Sekhar Nori
2013-07-30 5:05 ` Joel Fernandes [this message]
2013-07-30 5:05 ` Joel Fernandes
2013-07-30 5:05 ` Joel Fernandes
2013-07-29 7:01 ` Sekhar Nori
2013-07-29 7:01 ` Sekhar Nori
2013-07-30 3:47 ` Joel Fernandes
2013-07-30 3:47 ` Joel Fernandes
2013-07-30 16:29 ` Sekhar Nori
2013-07-30 16:29 ` Sekhar Nori
2013-07-30 16:29 ` Sekhar Nori
2013-07-31 5:06 ` Joel Fernandes
2013-07-31 5:06 ` Joel Fernandes
-- strict thread matches above, loose matches on Subject: below --
2013-08-23 19:53 Joel Fernandes
2013-08-23 19:53 ` Joel Fernandes
2013-08-23 19:53 ` Joel Fernandes
[not found] ` <1377287613-16491-1-git-send-email-joelf-l0cyMroinI0@public.gmane.org>
2013-08-26 10:46 ` Sekhar Nori
2013-08-26 10:46 ` Sekhar Nori
2013-08-26 10:46 ` Sekhar Nori
[not found] ` <521B3212.3050405-l0cyMroinI0@public.gmane.org>
2013-08-26 16:52 ` Joel Fernandes
2013-08-26 16:52 ` Joel Fernandes
2013-08-26 16:52 ` Joel Fernandes
[not found] ` <521B87C0.1090507-l0cyMroinI0@public.gmane.org>
2013-08-26 19:34 ` Sekhar Nori
2013-08-26 19:34 ` Sekhar Nori
2013-08-26 19:34 ` Sekhar Nori
2013-09-06 19:13 ` Mark Jackson
2013-09-06 19:13 ` Mark Jackson
2013-09-06 19:13 ` Mark Jackson
2013-09-06 19:15 ` Mark Jackson
2013-09-06 19:15 ` Mark Jackson
[not found] ` <522A29CF.2090001-2FZW7xY0fHgqdlJmJB21zg@public.gmane.org>
2013-09-06 19:51 ` Joel Fernandes
2013-09-06 19:51 ` Joel Fernandes
2013-09-06 19:51 ` Joel Fernandes
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=51F74995.2050202@ti.com \
--to=joelf@ti.com \
--cc=arnd@arndb.de \
--cc=balajitk@ti.com \
--cc=benoit.cousson@linaro.org \
--cc=jkridner@beagleboard.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=mpfj-list@newflow.co.uk \
--cc=nsekhar@ti.com \
--cc=panto@antoniou-consulting.com \
--cc=tony@atomide.com \
--cc=vinod.koul@intel.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.