Re: [PATCH 0/3] mmc: sdhci-iproc: add bcm2835 support

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

 



Looks clean Stefan. I'll try and revisit the caps setting in the future and may be able to provide additional settings in a later patch.

This patch series:
Acked-by: Scott Branden <sbranden@xxxxxxxxxxxx>


On 16-01-27 02:25 PM, Stefan Wahren wrote:
This patch series based on the suggestion of Scott Branden to implement
bcm2835 support into sdhci-iproc [1].

These changes provide the following advantages:
   * no need to maintain a separate driver for bcm2835
   * improve support and test coverage of bcm2835 mmc support
   * get the rid of SDHCI_CAPABILITIES hack in bcm2835_sdhci_readl
   * increase write performance by avoiding delays in write ops

Patch 1 prepare sdhci-iproc for another platform. Patch 2 and 3
are the actually changes for bcm2835 support.

These series has been tested on Raspberry Pi B with dd (direct I/O):

sdhci-bcm2835

378 kB/s WRITE
10,6 MB/s READ

sdhci-iproc (after applying patch series)

881 kB/s WRITE
10,8 MB/s READ

Changes since RFC:
   * drop patch which tries to add quirks for bcm2835 in DT and define all
     quirks in driver instead to keep old DTs working with sdhci-iproc
     (suggestion by Stephen Warren)
   * define MMC caps in shdci-iproc platform data to make adding new
     platforms more easily (suggestion by Stephen Warren)

[1] - https://lkml.org/lkml/2015/12/22/525

Stefan Wahren (3):
   mmc: sdhci-iproc: define MMC caps in platform data
   mmc: sdhci-iproc: add bcm2835 support
   DT: sdhci-iproc: add bcm2835 compatible

  .../devicetree/bindings/mmc/brcm,sdhci-iproc.txt   |    5 ++++-
  drivers/mmc/host/Kconfig                           |    6 +++---
  drivers/mmc/host/sdhci-iproc.c                     |   20 ++++++++++++++++++--
  3 files changed, 25 insertions(+), 6 deletions(-)


--
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