[PATCH v2 00/12] mmc: core: A start to slim down public mmc headers

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

 



Changes in v2:
	- As patch 1->11 was applied from version 1, those have been dropped
	from the series.
	- Folded in a new change in patch 1/12, to solve a build issue for imx
	which is using the mxcmmc platform data header.
	- Added reviews/acks.


Ideally we should not expose functions nor structures as being public available
as the interface for mmc - unless they really are intended to be used like that.

Unfortunate this is not the case today and which has lead to the mmc interface
in some cases being abused. That of course has its own problems, but we can
still move things in the right direction, which is what this series intend to
do. More precisely, it moves things that can be easily moved from the public mmc
headers to the private mmc headers.

This should be considered as a first step of improving the situation, further
additional changes will have to continue this path.


Ulf Hansson (12):
  mmc: mxcmmc: Include interrupt.h in the platform data header
  mmc: core: First step in cleaning up public mmc header files
  mmc: core: First step in cleaning up private mmc header files
  mmc: core: Move public functions from core.h to private headers
  mmc: core: Move some host specific public functions to host.h
  mmc: core: Move erase/trim/discard defines from public core.h to mmc.h
  mmc: core: Remove unused struct _mmc_csd from public mmc.h header
  mmc: omap: Don't use mmc_card_present() when validating for inserted
    card
  mmc: vub300: Don't use mmc_card_present() when validating for inserted
    card
  mmc: core: Move public functions from card.h to private headers
  mmc: core: Move public functions from host.h to private headers
  mmc: core: Don't use extern declarations of public mmc functions

 drivers/mmc/core/block.c                 |   6 +
 drivers/mmc/core/block.h                 |   8 +
 drivers/mmc/core/bus.c                   |   2 +
 drivers/mmc/core/bus.h                   |  16 +-
 drivers/mmc/core/card.h                  | 223 ++++++++++++++++++++++++++++
 drivers/mmc/core/core.c                  |   1 +
 drivers/mmc/core/core.h                  |  41 +++++-
 drivers/mmc/core/debugfs.c               |   2 +
 drivers/mmc/core/host.h                  |  48 ++++++
 drivers/mmc/core/mmc.c                   |  13 ++
 drivers/mmc/core/mmc_ops.h               |  14 ++
 drivers/mmc/core/mmc_test.c              |   5 +
 drivers/mmc/core/pwrseq.h                |   6 +-
 drivers/mmc/core/queue.c                 |   2 +
 drivers/mmc/core/queue.h                 |   6 +-
 drivers/mmc/core/quirks.c                |   2 +
 drivers/mmc/core/sd.c                    |   2 +
 drivers/mmc/core/sd.h                    |   5 +-
 drivers/mmc/core/sd_ops.h                |   9 ++
 drivers/mmc/core/sdio.c                  |   2 +
 drivers/mmc/core/sdio_bus.c              |   1 +
 drivers/mmc/core/sdio_bus.h              |   3 +
 drivers/mmc/core/sdio_cis.h              |   3 +
 drivers/mmc/core/sdio_io.c               |   2 +
 drivers/mmc/core/sdio_irq.c              |   2 +
 drivers/mmc/core/sdio_ops.h              |   5 +
 drivers/mmc/core/slot-gpio.h             |   2 +
 drivers/mmc/host/omap.c                  |   2 +-
 drivers/mmc/host/vub300.c                |   6 +-
 include/linux/mmc/card.h                 | 242 -------------------------------
 include/linux/mmc/core.h                 |  86 ++---------
 include/linux/mmc/host.h                 |  63 ++------
 include/linux/mmc/mmc.h                  |  59 ++------
 include/linux/mmc/slot-gpio.h            |   3 +
 include/linux/platform_data/mmc-mxcmmc.h |   1 +
 35 files changed, 467 insertions(+), 426 deletions(-)
 create mode 100644 drivers/mmc/core/card.h

-- 
1.9.1

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