Skip to contentSkip to conversation

staging: fsl-mc: restructuring and cleanup

PATCH New Stuart Yoder 1 participant 1 message Aug 23, 2016

lore
Share & tools
lore
#0 lore raw

cover staging: fsl-mc: restructuring and cleanup

In preparation for moving this driver out of staging a review of the .h files intended to be public in include/linux/fsl turned up some needed cleanup. This patch series makes no functional changes to the driver, but does some restructuring to put public/private definitions in the right places and have a clear split between driver sub-components.
-update the public mc.h and mc-bus.h files to contain only public
definitions
-move all private definitions to their corresponding .c files or
to the new fsl-mc-private.h file for definitions private to the bus driver
-rename some source files to improve clarity
-create a new mc-io component to contain all APIs related to the
mc-io abstraction
-remove unneeded includes and forward definitions

Stuart Yoder (14):
staging: fsl-mc: rename mc-private.h to mc-bus.h
staging: fsl-mc: improve naming of source files
staging: fsl-mc: move allocator private definitions out of mc-bus.h
staging: fsl-mc: move resource pool init/cleanup into allocator
staging: fsl-mc: create private include for fsl-mc bus components
staging: fsl-mc: move dprc-driver private definitions out of mc-bus.h
staging: fsl-mc: move bus private definitions into fsl-mc-bus.c
staging: fsl-mc: make fsl_mc_get_root_dprc public
staging: fsl-mc: remove unneeded forward declarations
staging: fsl-mc: move bus definitions from mc.h to mc-bus.h
staging: fsl-mc: remove unneeded includes
staging: fsl-mc: split mc-io definitions into a separate file
staging: fsl-mc: explicitly include files where they are needed
staging: fsl-mc: make order of includes consistent

 drivers/staging/fsl-mc/bus/Makefile                |   7 +-
 drivers/staging/fsl-mc/bus/dpmcp.c                 |   1 +
 drivers/staging/fsl-mc/bus/dpmng.c                 |   1 +
 drivers/staging/fsl-mc/bus/dprc-driver.c           |  67 +----
 drivers/staging/fsl-mc/bus/dprc.c                  |   1 +
 .../bus/{mc-allocator.c => fsl-mc-allocator.c}     | 202 ++++---------
 .../staging/fsl-mc/bus/{mc-bus.c => fsl-mc-bus.c}  |  43 ++-
 .../staging/fsl-mc/bus/{mc-msi.c => fsl-mc-msi.c}  |   4 +-
 drivers/staging/fsl-mc/bus/fsl-mc-private.h        |  52 ++++
 .../staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c |   4 +-
 drivers/staging/fsl-mc/bus/mc-io.c                 | 320 +++++++++++++++++++++
 drivers/staging/fsl-mc/bus/mc-sys.c                | 155 +---------
 .../fsl-mc/include/{mc-private.h => mc-bus.h}      |  91 ++----
 drivers/staging/fsl-mc/include/mc-sys.h            |  15 -
 drivers/staging/fsl-mc/include/mc.h                |  20 --
 15 files changed, 520 insertions(+), 463 deletions(-)
 rename drivers/staging/fsl-mc/bus/{mc-allocator.c => fsl-mc-allocator.c} (80%)
 rename drivers/staging/fsl-mc/bus/{mc-bus.c => fsl-mc-bus.c} (94%)
 rename drivers/staging/fsl-mc/bus/{mc-msi.c => fsl-mc-msi.c} (98%)
 create mode 100644 drivers/staging/fsl-mc/bus/fsl-mc-private.h
 create mode 100644 drivers/staging/fsl-mc/bus/mc-io.c
 rename drivers/staging/fsl-mc/include/{mc-private.h => mc-bus.h} (53%)
Signature · 1 line
1.9.0