Skip to content

mmc: sdhci-s3c: use the sdhci-pltfm.c and move the header file

Jaehoon Chung linux-mmc linux-samsung-soc 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest4 patches 10 repliesIn review
  2. v24 patches 7 repliesSuperseded
  3. v14 patches 14 repliesSuperseded

Changes in v3 from the cover letter

- Modify mmc/host/Makefile for sdhci-s3c.c
- remove unnecessary code

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4mmc: sdhci-s3c: use the sdhci-pltfm for Samsung-SoCdrivers/mmc/host/Kconfig drivers/mmc/host/Makefile drivers/mmc/host/sdhci-s3c.c+93 -1810 replies
2/4ARM: SAMSUNG: move the header file to driver directoryarch/arm/mach-exynos/setup-sdhci-gpio.c arch/arm/mach-s5pc100/setup-sdhci-gpio.c arch/arm/mach-s5pv210/setup-sdhci-gpio.c +2 more+89 -900 replies
3/4mmc: sdhci-s3c: use the sdhci-s3c.h instead of regs-sdhci.hdrivers/mmc/host/sdhci-s3c.c+5 -70 replies
4/4ARM: SAMSUNG: move the mmc platdata header file to linux/mmc/arch/arm/plat-samsung/include/plat/sdhci.h include/linux/mmc/sdhci-s3c.h+85 -520 replies

Reviews trailers per patch

None of the 4 patches has a Reviewed-by or Acked-by.

PatchMark Brown
SeriesReviwed-by
1/4
2/4
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCH v3 0/4] mmc: sdhci-s3c: use the sdhci-pltfm.c and move the header file

This patchset is the purpose for using sdhci-pltfm.c and
removing the arch dependency.
The sdhci-s3c has too heavy arch dependency.
For example, if work for sdhci-s3c.c at mmc-tree, sometime need to change sdhci.h for Samsung-Soc. But now we need to work at both mmc-next tree and Samsung-SoC tree.
I think this is very inefficient and can conflict to merge.
So remained at least dependency and moved the header file to mmc folder.

And sdhci-s3c changed to use the sdhci-pltfm.c.
There is no reason that didn't use the sdhci-pltfm.c.
If we use the sdhci-pltfm.c,we can easily maintain the common part.

In future, i will work more removing the arch dependency.

This patchset based on mmc-next tree.

Changelog v3:
- Modify mmc/host/Makefile for sdhci-s3c.c
- remove unnecessary code

Changelog V2:
- modified driver/mmc/host/Kconfig
- Add the comments for s3c64xx
- based-on latest mmc-tree.

Jaehoon Chung (4):
mmc: sdhci-s3c: use the sdhci-pltfm for Samsung-SoC
ARM: SAMSUNG: move the sdhci header file to driver directory
mmc: sdhci-s3c: use the sdhci-s3c.h instead of regs-sdhci.h
ARM: SAMSUNG: move the mmc platdata header file into linux/mmc/

 arch/arm/mach-exynos/setup-sdhci-gpio.c         |    1 -
 arch/arm/mach-s5pc100/setup-sdhci-gpio.c        |    1 -
 arch/arm/mach-s5pv210/setup-sdhci-gpio.c        |    1 -
 arch/arm/plat-samsung/include/plat/regs-sdhci.h |   87 --------
 arch/arm/plat-samsung/include/plat/sdhci.h      |   53 +-----
 drivers/mmc/host/Kconfig                        |   20 +-
 drivers/mmc/host/Makefile                       |    2 +-
 drivers/mmc/host/sdhci-s3c.c                    |  265 ++++++++---------------
 drivers/mmc/host/sdhci-s3c.h                    |   89 ++++++++
 include/linux/mmc/sdhci-s3c.h                   |   84 +++++++
 10 files changed, 273 insertions(+), 330 deletions(-)
 delete mode 100644 arch/arm/plat-samsung/include/plat/regs-sdhci.h
 create mode 100644 drivers/mmc/host/sdhci-s3c.h
 create mode 100644 include/linux/mmc/sdhci-s3c.h