From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Vinod Koul <vinod.koul@intel.com>
Cc: nsekhar@ti.com, linux@arm.linux.org.uk, olof@lixom.net,
arnd@arndb.de, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org,
dmaengine@vger.kernel.org
Subject: Re: [PATCH v5 00/24] dmaengine/ARM: Merge the edma drivers into one
Date: Thu, 15 Oct 2015 10:16:12 +0300 [thread overview]
Message-ID: <561F52BC.7040903@ti.com> (raw)
In-Reply-To: <20151014143124.GU27370@localhost>
On 10/14/2015 05:31 PM, Vinod Koul wrote:
> On Wed, Oct 14, 2015 at 02:42:42PM +0300, Peter Ujfalusi wrote:
>> Hi,
>>
>> Cover letter:
>>
>> with this series the edma two driver setup will be changed to have only one
>> driver to support eDMA3. The legacy edma interface will be removed and eDMA can
>> only be used via dmaengine API from this point on.
>> In order to do the merge the following improvements has been done:
>> - One driver instance per eDMA:
>> - Any number of eDMA instances are supported (both legacy and DT boot)
>> - Not relying on global variables, arrays, etc
>> - Code simplification and optimizations in several places
>>
>> This change will also help us to do bigger changes in the eDMA driver since,
>> since now we have only one driver to work with.
>
> I have applied this now. I got a conlfict on 3rd one while applying and also
> I got conflict while merging to next
>
> Pls verify all is well!
Thanks Vinod,
It looks fine.
It is strange since I'm working on top of linux-next as well. The following
commit caused the conflict:
ddfe4d0cce78 dmaengine: edma: remove redundant conditions
I have it in my linux-next, but for some reason it is missing from your
topic/edma, so the series applied with conflicts there and when merging on top
of linux-next there were again conflict.
Thank you,
Péter
WARNING: multiple messages have this Message-ID (diff)
From: peter.ujfalusi@ti.com (Peter Ujfalusi)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 00/24] dmaengine/ARM: Merge the edma drivers into one
Date: Thu, 15 Oct 2015 10:16:12 +0300 [thread overview]
Message-ID: <561F52BC.7040903@ti.com> (raw)
In-Reply-To: <20151014143124.GU27370@localhost>
On 10/14/2015 05:31 PM, Vinod Koul wrote:
> On Wed, Oct 14, 2015 at 02:42:42PM +0300, Peter Ujfalusi wrote:
>> Hi,
>>
>> Cover letter:
>>
>> with this series the edma two driver setup will be changed to have only one
>> driver to support eDMA3. The legacy edma interface will be removed and eDMA can
>> only be used via dmaengine API from this point on.
>> In order to do the merge the following improvements has been done:
>> - One driver instance per eDMA:
>> - Any number of eDMA instances are supported (both legacy and DT boot)
>> - Not relying on global variables, arrays, etc
>> - Code simplification and optimizations in several places
>>
>> This change will also help us to do bigger changes in the eDMA driver since,
>> since now we have only one driver to work with.
>
> I have applied this now. I got a conlfict on 3rd one while applying and also
> I got conflict while merging to next
>
> Pls verify all is well!
Thanks Vinod,
It looks fine.
It is strange since I'm working on top of linux-next as well. The following
commit caused the conflict:
ddfe4d0cce78 dmaengine: edma: remove redundant conditions
I have it in my linux-next, but for some reason it is missing from your
topic/edma, so the series applied with conflicts there and when merging on top
of linux-next there were again conflict.
Thank you,
P?ter
WARNING: multiple messages have this Message-ID (diff)
From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: Vinod Koul <vinod.koul@intel.com>
Cc: <nsekhar@ti.com>, <linux@arm.linux.org.uk>, <olof@lixom.net>,
<arnd@arndb.de>, <linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>, <linux-omap@vger.kernel.org>,
<dmaengine@vger.kernel.org>
Subject: Re: [PATCH v5 00/24] dmaengine/ARM: Merge the edma drivers into one
Date: Thu, 15 Oct 2015 10:16:12 +0300 [thread overview]
Message-ID: <561F52BC.7040903@ti.com> (raw)
In-Reply-To: <20151014143124.GU27370@localhost>
On 10/14/2015 05:31 PM, Vinod Koul wrote:
> On Wed, Oct 14, 2015 at 02:42:42PM +0300, Peter Ujfalusi wrote:
>> Hi,
>>
>> Cover letter:
>>
>> with this series the edma two driver setup will be changed to have only one
>> driver to support eDMA3. The legacy edma interface will be removed and eDMA can
>> only be used via dmaengine API from this point on.
>> In order to do the merge the following improvements has been done:
>> - One driver instance per eDMA:
>> - Any number of eDMA instances are supported (both legacy and DT boot)
>> - Not relying on global variables, arrays, etc
>> - Code simplification and optimizations in several places
>>
>> This change will also help us to do bigger changes in the eDMA driver since,
>> since now we have only one driver to work with.
>
> I have applied this now. I got a conlfict on 3rd one while applying and also
> I got conflict while merging to next
>
> Pls verify all is well!
Thanks Vinod,
It looks fine.
It is strange since I'm working on top of linux-next as well. The following
commit caused the conflict:
ddfe4d0cce78 dmaengine: edma: remove redundant conditions
I have it in my linux-next, but for some reason it is missing from your
topic/edma, so the series applied with conflicts there and when merging on top
of linux-next there were again conflict.
Thank you,
Péter
next prev parent reply other threads:[~2015-10-15 7:16 UTC|newest]
Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-14 11:42 [PATCH v5 00/24] dmaengine/ARM: Merge the edma drivers into one Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 01/24] ARM: common: edma: Fix channel parameter for irq callbacks Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 02/24] ARM: common: edma: Remove unused functions Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 03/24] dmaengine: edma: Simplify and optimize the edma_execute path Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 04/24] ARM: davinci/common: Convert edma driver to handle one eDMA instance per driver Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 05/24] ARM/dmaengine: edma: Move of_dma_controller_register to the dmaengine driver Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 06/24] ARM: common: edma: Internal API to use pointer to 'struct edma' Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 07/24] ARM/dmaengine: edma: Public API to use private struct pointer Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 08/24] ARM/dmaengine: edma: Remove limitation on the number of eDMA controllers Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 09/24] ARM: davinci: Use platform_device_register_full() to create pdev for eDMA Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 10/24] ARM: davinci: Add dma_mask to eDMA devices Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 11/24] ARM/dmaengine: edma: Merge the two drivers under drivers/dma/ Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 12/24] dmaengine: edma: Allocate memory dynamically for bitmaps and structures Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 13/24] dmaengine: edma: Use devm_kcalloc when possible Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 14/24] dmaengine: edma: Cleanup regarding the use of dev around the code Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 15/24] dmaengine: edma: Use dev_dbg instead pr_debug Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 16/24] dmaengine: edma: Use the edma_write_slot instead open coded memcpy_toio Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` [PATCH v5 17/24] dmaengine: edma: Print warning when linking slots from different eDMA Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:42 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 18/24] dmaengine: edma: Consolidate the comments for functions Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 19/24] dmaengine: edma: Simplify the interrupt handling Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 20/24] dmaengine: edma: Move the pending error check into helper function Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 21/24] dmaengine: edma: Simplify and optimize ccerr interrupt handler Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 22/24] dmaengine: edma: Read channel mapping support only once from HW Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 23/24] dmaengine: edma: Rename bitfields for slot and channel usage tracking Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` [PATCH v5 24/24] dmaengine: edma: Dynamic paRAM slot handling if HW supports it Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 11:43 ` Peter Ujfalusi
2015-10-14 14:31 ` [PATCH v5 00/24] dmaengine/ARM: Merge the edma drivers into one Vinod Koul
2015-10-14 14:31 ` Vinod Koul
2015-10-15 7:16 ` Peter Ujfalusi [this message]
2015-10-15 7:16 ` Peter Ujfalusi
2015-10-15 7:16 ` Peter Ujfalusi
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=561F52BC.7040903@ti.com \
--to=peter.ujfalusi@ti.com \
--cc=arnd@arndb.de \
--cc=dmaengine@vger.kernel.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=nsekhar@ti.com \
--cc=olof@lixom.net \
--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.