From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] irqchip: omap-intc: fix legacy DMA regression
Date: Tue, 6 Jan 2015 10:30:22 -0800 [thread overview]
Message-ID: <20150106183021.GV4081@atomide.com> (raw)
In-Reply-To: <20150106180532.GO11502@n2100.arm.linux.org.uk>
* Russell King - ARM Linux <linux@arm.linux.org.uk> [150106 10:08]:
> On Tue, Jan 06, 2015 at 10:51:33AM -0600, Felipe Balbi wrote:
> > + /*
> > + * FIXME legacy OMAP DMA driver sitting under arch/arm/plat-omap/dma.c
> > + * depends is still not ready for linear IRQ domains; because of that
> > + * we need to temporarily "blacklist" OMAP2 and OMAP3 devices from using
> > + * linear IRQ Domain until that driver is finally fixed.
>
> "finally fixed" or finally killed off like it really needs to be, once
> all users of it are killed.
>
> We've been trying to do this for, what, three years now... I finally
> pushed a WARN_ON() into that code to make it obvious to anyone who
> uses omap_request_dma() that they really need to update their code.
>
> Here's the list of references to that symbol which *still* need to be
> fixed so that we can kill the legacy DMA driver:
>
> drivers/media/platform/omap/omap_vout_vrfb.c: ret = omap_request_dma(vout->vrfb_dma_tx.dev_id, "VRFB DMA TX",
> drivers/media/platform/omap3isp/isphist.c: ret = omap_request_dma(OMAP24XX_DMA_NO_DEVICE, "DMA_ISP_HIST",
> drivers/media/platform/soc_camera/omap1_camera.c: err = omap_request_dma(OMAP_DMA_CAMERA_IF_RX, DRIVER_NAME,
> drivers/mtd/onenand/omap2.c: r = omap_request_dma(0, pdev->dev.driver->name,
AFAIK we should just remove DMA support from the drivers above.
Nobody seems to be interested in doing anything about them.
> drivers/usb/gadget/udc/omap_udc.c: status = omap_request_dma(dma_channel,
> drivers/usb/gadget/udc/omap_udc.c: status = omap_request_dma(dma_channel,
OK so Aaro picked this one.
> drivers/usb/musb/tusb6010_omap.c: ret = omap_request_dma(chdat->sync_dev, dev_name,
> drivers/usb/musb/tusb6010_omap.c: ret = omap_request_dma(tusb_dma->sync_dev, "TUSB shared",
I'll update this one. FYI, I already have some work-in-progress
MUSB DMA patches that allow building in all the MUSB DMA glue
layers. I just need to finish that series for v3.20:
https://git.kernel.org/cgit/linux/kernel/git/tmlind/linux-omap.git/log/?h=musb-dma-2014-11-25-v2
So converting tusb6010 over to the dmaengine API would be the
next logical step after that series. Probably not going to
happen before v3.21 though..
Regards,
Tony
next prev parent reply other threads:[~2015-01-06 18:30 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20150106123830.GD30544@fuloong-minipc.musicnaut.iki.fi>
2015-01-06 16:51 ` [PATCH] irqchip: omap-intc: fix legacy DMA regression Felipe Balbi
2015-01-06 17:48 ` Aaro Koskinen
2015-01-06 17:52 ` Tony Lindgren
2015-01-06 18:05 ` Russell King - ARM Linux
2015-01-06 18:24 ` Aaro Koskinen
2015-01-06 18:30 ` Tony Lindgren [this message]
2015-01-06 20:38 ` [PATCH v2] " Felipe Balbi
2015-01-07 3:00 ` Jason Cooper
2015-01-19 18:34 ` Tony Lindgren
2015-01-07 11:12 ` [PATCH] " Peter Kümmel
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=20150106183021.GV4081@atomide.com \
--to=tony@atomide.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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).