Re: [PATCH/RFC 5/5] mmc: renesas-sdhi: remove gen3 support from sysc dmac driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 09, 2017 at 05:40:15PM +0900, Magnus Damm wrote:
> Hi Geert and Simon,
> 
> On Fri, Jun 9, 2017 at 4:40 PM, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
> > Hi Simon,
> >
> > On Thu, Jun 8, 2017 at 3:09 PM, Simon Horman <horms+renesas@xxxxxxxxxxxx> wrote:
> >> Gen3 SoCs are now supported by the internal dmac variant of the SDHI driver.
> >> Remove them from the sys dmac variant where only PIO mode is supported.
> >>
> >> Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
> >
> > Is it possible to fall back to PIO with the new DMAC driver?
> > If not, is this desirable?
> 
> I believe R-Car Gen3 SDHI may depending on SoC use either on-chip bus
> mastering DMA or SYS-DMAC via DMA Engine. In both cases fall back to
> PIO is most likely valid. For any given SoC which is better of on-chip
> DMA or SYS-DMAC needs to be investigated together with IPMMU.

My answer is a bit different:

With this patchset there are two variants of the SDHI driver, one that
supports SYS-DMAC via DMA Engine and one that supports on-chip bus
mastering DMA.

My understanding is that prior to this patch set the SYS-DMAC and only
variant of the SDHI driver supported fallback to PIO. And as the internal
DMAC variant uses the same core code my assumption is that it should also
have that property.  I would need to investigate things in more detail to
confirm that assumption.

In terms of supporting Gen3 with SYS-DMAC or on-chip bus mastering DMA,
I believe those cases should use compat strings and soc-match in each of
the two variants of the SDHI driver as appropriate.
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux