* Re: + mmc-add-dma-support-to-tmio_mmc-driver-when-used-on-superh.patch [not found] ` <AANLkTik4gzuZvrAwbK-dyCXVYf5gLD0RGrpvM55YLzOK@mail.gmail.com> @ 2010-05-13 13:20 ` Guennadi Liakhovetski 2010-05-13 14:18 ` Guennadi Liakhovetski 0 siblings, 1 reply; 3+ messages in thread From: Guennadi Liakhovetski @ 2010-05-13 13:20 UTC (permalink / raw) To: Ian Molton Cc: Andrew Morton, mm-commits, dan.j.williams, Paul Mundt, Magnus Damm, Russell King, sameo, linux-sh@vger.kernel.org, linux-mmc (re-adding CC) On Thu, 13 May 2010, Ian Molton wrote: > Might be worth exposing for now, whilst the feature is new. but yes, > automatic select yes on all the platforms you are comfortable with. > > Im not sure if it should be per MFD or per platform. I tend to think > the latter, given that its slave DMA and thus I could have the same > MFD in platforms with different CPUs (eg. e740 and e800) where a CPU > bug might upset one but not the other. So far I went for the least effort - not exposed, selected by MFDs. If we ever have a need to change this, e.g., make it user-selectable or per platform, we can always do so later. For now all SuperH / SH-Mobile platforms I tested can use this driver with DMA. Expect updated patches shortly. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: + mmc-add-dma-support-to-tmio_mmc-driver-when-used-on-superh.patch 2010-05-13 13:20 ` + mmc-add-dma-support-to-tmio_mmc-driver-when-used-on-superh.patch Guennadi Liakhovetski @ 2010-05-13 14:18 ` Guennadi Liakhovetski 2010-05-13 21:46 ` Ian Molton 0 siblings, 1 reply; 3+ messages in thread From: Guennadi Liakhovetski @ 2010-05-13 14:18 UTC (permalink / raw) To: Ian Molton Cc: Andrew Morton, mm-commits, dan.j.williams, Paul Mundt, Magnus Damm, Russell King, sameo, linux-sh@vger.kernel.org, linux-mmc On Thu, 13 May 2010, Guennadi Liakhovetski wrote: > On Thu, 13 May 2010, Ian Molton wrote: > > > Might be worth exposing for now, whilst the feature is new. but yes, > > automatic select yes on all the platforms you are comfortable with. > > > > Im not sure if it should be per MFD or per platform. I tend to think > > the latter, given that its slave DMA and thus I could have the same > > MFD in platforms with different CPUs (eg. e740 and e800) where a CPU > > bug might upset one but not the other. > > So far I went for the least effort - not exposed, selected by MFDs. If we > ever have a need to change this, e.g., make it user-selectable or per > platform, we can always do so later. For now all SuperH / SH-Mobile > platforms I tested can use this driver with DMA. Expect updated patches > shortly. One more thing I forgot to mention - even if your MFD enables DMA, it still can further decide when (on which platforms) to enable and when to disable DMA by providing or not providing DMA slave configuration to tmio_mmc. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: + mmc-add-dma-support-to-tmio_mmc-driver-when-used-on-superh.patch 2010-05-13 14:18 ` Guennadi Liakhovetski @ 2010-05-13 21:46 ` Ian Molton 0 siblings, 0 replies; 3+ messages in thread From: Ian Molton @ 2010-05-13 21:46 UTC (permalink / raw) To: Guennadi Liakhovetski Cc: Andrew Morton, mm-commits, dan.j.williams, Paul Mundt, Magnus Damm, Russell King, sameo, linux-sh@vger.kernel.org, linux-mmc Sounds good to me, and you're right - the MFD level is good for this config option. -- Ian Molton Linux, Automotive, and other hacking: http://www.mnementh.co.uk/ On 13 May 2010 15:18, Guennadi Liakhovetski <g.liakhovetski@gmx.de> wrote: > On Thu, 13 May 2010, Guennadi Liakhovetski wrote: > >> On Thu, 13 May 2010, Ian Molton wrote: >> >> > Might be worth exposing for now, whilst the feature is new. but yes, >> > automatic select yes on all the platforms you are comfortable with. >> > >> > Im not sure if it should be per MFD or per platform. I tend to think >> > the latter, given that its slave DMA and thus I could have the same >> > MFD in platforms with different CPUs (eg. e740 and e800) where a CPU >> > bug might upset one but not the other. >> >> So far I went for the least effort - not exposed, selected by MFDs. If we >> ever have a need to change this, e.g., make it user-selectable or per >> platform, we can always do so later. For now all SuperH / SH-Mobile >> platforms I tested can use this driver with DMA. Expect updated patches >> shortly. > > One more thing I forgot to mention - even if your MFD enables DMA, it > still can further decide when (on which platforms) to enable and when to > disable DMA by providing or not providing DMA slave configuration to > tmio_mmc. > > Thanks > Guennadi > --- > Guennadi Liakhovetski, Ph.D. > Freelance Open-Source Software Developer > http://www.open-technology.de/ > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-05-13 21:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <201005122255.o4CMtmvN025653@imap1.linux-foundation.org>
[not found] ` <AANLkTim5U-JbrVCBKHnERgHol2GcwTfweQyI4onR1Skh@mail.gmail.com>
[not found] ` <Pine.LNX.4.64.1005131357500.4671@axis700.grange>
[not found] ` <AANLkTinPW2W6aV1smLsQObLOAS3bECDZ7mhSGCm2aqgO@mail.gmail.com>
[not found] ` <Pine.LNX.4.64.1005131415460.4671@axis700.grange>
[not found] ` <AANLkTik4gzuZvrAwbK-dyCXVYf5gLD0RGrpvM55YLzOK@mail.gmail.com>
2010-05-13 13:20 ` + mmc-add-dma-support-to-tmio_mmc-driver-when-used-on-superh.patch Guennadi Liakhovetski
2010-05-13 14:18 ` Guennadi Liakhovetski
2010-05-13 21:46 ` Ian Molton
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).