All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Stein <alexanders83@web.de>
To: linux-rpi-kernel@lists.infradead.org
Cc: "Stefan Wahren" <info@lategoodbye.de>,
	"Noralf Trønnes" <noralf@tronnes.org>,
	dmaengine@vger.kernel.org, vinod.koul@intel.com,
	dan.j.williams@intel.com, jonathan@raspberrypi.org,
	linux-kernel@vger.kernel.org
Subject: Re: Re: [PATCH] dmaengine: bcm2835: Add slave dma support
Date: Thu, 16 Apr 2015 21:06:49 +0200	[thread overview]
Message-ID: <8408334.G8cbCZ65GO@kongar> (raw)
In-Reply-To: <552EB54A.3060404@lategoodbye.de>

Hi Stefan,

On Wednesday 15 April 2015, 21:00:26 wrote Stefan Wahren:
> Am 15.04.2015 um 11:56 schrieb Noralf Trønnes:
> > Add slave transfer capability to BCM2835 dmaengine driver.
> > This patch is pulled from the bcm2708-dmaengine driver in the
> > Raspberry Pi repo. The work was done by Gellert Weisz.
> >
> > Tested with the bcm2835-mmc driver from the same repo.
> 
> why not with the upstream kernel?

I also looked at slave dma support, especially for use in mmc. It turns our that bcm2835-mmc is written more or less completly new.
Mainline linux uses sdhci "framework" which internally uses the SDMA and/or ADMA (both internal, to SD/MMC controller, DMA units) which can be supported by an SDHCI compatible controller.
AFAIK the SD/MMC controller in bcm2835 lacks both that is why the driver only uses PIO. I dunno if external DMA usage can so easily be integrated into the sdhci, I have my doubts.

Best regards,
Alexander


  reply	other threads:[~2015-04-16 19:07 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-15  9:56 [PATCH] dmaengine: bcm2835: Add slave dma support Noralf Trønnes
2015-04-15 14:37 ` Martin Sperl
2015-04-15 18:53   ` Noralf Trønnes
2015-04-16  6:30     ` Rogier Wolff
2015-04-16 17:28       ` Noralf Trønnes
2015-04-15 19:00 ` Stefan Wahren
2015-04-16 19:06   ` Alexander Stein [this message]
2015-04-16 22:03     ` Noralf Trønnes
2015-04-16 22:09   ` Noralf Trønnes
2015-04-17 17:08     ` Stefan Wahren
2015-04-17 17:19       ` Martin Sperl
2015-04-17 17:20       ` Noralf Trønnes

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=8408334.G8cbCZ65GO@kongar \
    --to=alexanders83@web.de \
    --cc=dan.j.williams@intel.com \
    --cc=dmaengine@vger.kernel.org \
    --cc=info@lategoodbye.de \
    --cc=jonathan@raspberrypi.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rpi-kernel@lists.infradead.org \
    --cc=noralf@tronnes.org \
    --cc=vinod.koul@intel.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.