Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1445383
| From | Peter Ujfalusi <peter.ujfalusi@ti.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 7/7] dmaengine: omap-dma: Support for LinkedList transfer of slave_sg |
| Date | 2016-07-18 13:20 +0200 |
| Message-ID | <rWeKB-5sp-13@gated-at.bofh.it> (permalink) |
| References | <rUOfv-1kh-5@gated-at.bofh.it> <rUOfw-1kh-13@gated-at.bofh.it> <rWehA-53k-17@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 07/18/16 13:42, Russell King - ARM Linux wrote:
> On Thu, Jul 14, 2016 at 03:42:42PM +0300, Peter Ujfalusi wrote:
>> struct omap_desc {
>> + struct omap_chan *c;
>> struct virt_dma_desc vd;
>
> No need for this. to_omap_dma_chan(foo->vd.tx.chan) will give you the
> omap_chan for the descriptor. In any case, I question whether you
> actually need this (see below.)
I don't know how I missed that. Works and looks better!
>> + bool using_ll;
>> enum dma_transfer_direction dir;
>> dma_addr_t dev_addr;
>>
>> @@ -81,6 +109,9 @@ struct omap_desc {
>> };
>>
>> enum {
>> + CAPS_0_SUPPORT_LL123 = BIT(20), /* Linked List type1/2/3 */
>> + CAPS_0_SUPPORT_LL4 = BIT(21), /* Linked List type4 */
>> +
>> CCR_FS = BIT(5),
>> CCR_READ_PRIORITY = BIT(6),
>> CCR_ENABLE = BIT(7),
>> @@ -151,6 +182,19 @@ enum {
>> CICR_SUPER_BLOCK_IE = BIT(14), /* OMAP2+ only */
>>
>> CLNK_CTRL_ENABLE_LNK = BIT(15),
>> +
>> + CDP_DST_VALID_INC = 0 << 0,
>> + CDP_DST_VALID_RELOAD = 1 << 0,
>> + CDP_DST_VALID_REUSE = 2 << 0,
>> + CDP_SRC_VALID_INC = 0 << 2,
>> + CDP_SRC_VALID_RELOAD = 1 << 2,
>> + CDP_SRC_VALID_REUSE = 2 << 2,
>> + CDP_NTYPE_TYPE1 = 1 << 4,
>> + CDP_NTYPE_TYPE2 = 2 << 4,
>> + CDP_NTYPE_TYPE3 = 3 << 4,
>> + CDP_TMODE_NORMAL = 0 << 8,
>> + CDP_TMODE_LLIST = 1 << 8,
>> + CDP_FAST = BIT(10),
>> };
>>
>> static const unsigned es_bytes[] = {
>> @@ -180,7 +224,64 @@ static inline struct omap_desc *to_omap_dma_desc(struct dma_async_tx_descriptor
>>
>> static void omap_dma_desc_free(struct virt_dma_desc *vd)
>> {
>> - kfree(container_of(vd, struct omap_desc, vd));
>> + struct omap_desc *d = container_of(vd, struct omap_desc, vd);
>
> struct omap_desc *d = to_omap_dma_desc(&vd->tx);
>
> works just as well, and looks much nicer, and follows the existing code
> pattern.
Yes, I missed this as well.
>> +
>> + if (d->using_ll) {
>> + struct omap_chan *c = d->c;
>> + int i;
>> +
>> + for (i = 0; i < d->sglen; i++) {
>> + if (d->sg[i].t2_desc)
>> + dma_pool_free(c->desc_pool, d->sg[i].t2_desc,
>> + d->sg[i].t2_desc_paddr);
>
> Why do you need a per-channel pool of descriptors? Won't a per-device
> descriptor pool be much better, and simplify the code here?
I was planning to try per-device pool after this series. I think I went with
per-channel pool as for example bcm2835-dma was doing the same.
In code wise I don't think it is going to simplify much as we still need to
free here what we have allocated. I can test this out.
--
Péter
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/7] dmaengine:omap-dma: Linked List transfer for slave_sg Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-14 14:50 +0200
[PATCH 1/7] dmaengine: omap-dma: Simplify omap_dma_start_sg parameter list Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-14 14:50 +0200
[PATCH 7/7] dmaengine: omap-dma: Support for LinkedList transfer of slave_sg Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-14 14:50 +0200
Re: [PATCH 7/7] dmaengine: omap-dma: Support for LinkedList transfer of slave_sg Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-07-18 12:50 +0200
Re: [PATCH 7/7] dmaengine: omap-dma: Support for LinkedList transfer of slave_sg Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-18 13:20 +0200
[PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-14 14:50 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-07-18 12:40 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-19 14:40 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-07-19 18:30 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-19 21:30 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Vinod Koul <vinod.koul@intel.com> - 2016-07-24 09:40 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Robert Jarzmik <robert.jarzmik@free.fr> - 2016-07-20 08:30 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-21 11:40 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-21 11:40 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-07-21 11:50 +0200
Re: [PATCH 2/7] dmaengine: omap-dma: Complete the cookie first on transfer completion Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-22 13:10 +0200
[PATCH 4/7] dmaengine: omap-dma: Dynamically allocate memory for lch_map Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-14 14:50 +0200
[PATCH 6/7] dmaengine: omap-dma: Use pointer to omap_sg in slave_sg setup's loop Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-14 14:50 +0200
Re: [PATCH 0/7] dmaengine:omap-dma: Linked List transfer for slave_sg Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-07-18 12:40 +0200
Re: [PATCH 0/7] dmaengine:omap-dma: Linked List transfer for slave_sg Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-18 14:10 +0200
Re: [PATCH 0/7] dmaengine:omap-dma: Linked List transfer for slave_sg Russell King - ARM Linux <linux@armlinux.org.uk> - 2016-07-18 14:30 +0200
Re: [PATCH 0/7] dmaengine:omap-dma: Linked List transfer for slave_sg Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-07-18 14:40 +0200
csiph-web