Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1295378

Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various paths

Show all headers | View raw

Hi,

On Fri, Dec 18, 2015 at 10:39:48AM -0800, Tony Lindgren wrote:
> * Brian Norris <computersforpeace@gmail.com> [151218 10:11]:
> > On Mon, Dec 14, 2015 at 11:49:32AM +0200, Peter Ujfalusi wrote:
> > > We have 4 functions containing almost identical DMA setup code. Create one
> > > function which can set up the DMA for both read and write and use this in
> > > place for the setup code in the driver.
> > > The new function will use wait_for_completion_timeout() and it will figure
> > > out the best data_type to be used for the transfer instead of hardwiring
> > > 32 or 16 bit data.
> > > 
> > > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> > 
> > Does anyone use this driver? I've seen practically zero activity on the
> > entire OneNAND codebase in the last few years, and I presumed it was
> > essentially dead.
> > 
> > If it's not dead, I'd like to know some contingency of people who are
> > willing to actually maintain (or at least review) this stuff.
> > 
> > Kyungmin, are you still out there? Or Tony, do you know of any users for
> > this?
> > 
> > Peter, are you actually using this, or are you just refactoring for the
> > fun of it?
> 
> It's used for n8x0 and n900, but mostly in read-only mode. I suggest we

It's also used on n9/n950 phones. Also write functionality is needed
for mtdoops and kernel flashing.

> remove the DMA support for it completely because of the following:
> 
> 1. The DMA support for this driver is not done correctly. The pin used
>    as GPIO should be used as external DMA request line.
> 
> 2. AFAIK the DMA for this driver is mostly disabld, probably largely
>    due to #1 above
> 
> 3. If we remove DMA support, we can then easily switch to use the
>    generic onenand driver.
> 
> I'd like to hear Aaro's comments too before doing this tough.

Probably DMA support is not that critical. Looks like with DT boot it's
not even possible to enable it at the moment.

A.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] mtd: onenand: omap2: Simplify the DMA setup for various paths Peter Ujfalusi <peter.ujfalusi@ti.com> - 2015-12-14 11:00 +0100
  Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various  paths Brian Norris <computersforpeace@gmail.com> - 2015-12-18 19:20 +0100
    Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various  paths Tony Lindgren <tony@atomide.com> - 2015-12-18 19:40 +0100
      Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various  paths Aaro Koskinen <aaro.koskinen@iki.fi> - 2015-12-19 14:40 +0100
        Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various  paths Peter Ujfalusi <peter.ujfalusi@ti.com> - 2015-12-21 09:20 +0100
    Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various  paths Peter Ujfalusi <peter.ujfalusi@ti.com> - 2015-12-18 20:50 +0100
      Re: [PATCH] mtd: onenand: omap2: Simplify the DMA setup for various  paths Brian Norris <computersforpeace@gmail.com> - 2015-12-18 23:40 +0100

csiph-web