spi: reorganize drivers
Subsystems (3)
PATCH 1/1 spi: reorganize drivers
Sort the SPI makefile and enforce the naming convention spi_*.c for spi drivers.
This change also rolls the contents of atmel_spi.h into the .c file since there is only one user of that particular include file.
---
drivers/spi/Makefile | 80 ++++++++-----------
drivers/spi/atmel_spi.h | 167 ---------------------------------------
drivers/spi/spi_ath79.c | 0
drivers/spi/spi_atmel.c | 155 ++++++++++++++++++++++++++++++++++++
drivers/spi/spi_au1550.c | 0
drivers/spi/spi_coldfire_qspi.c | 0
drivers/spi/spi_davinci.c | 0
drivers/spi/spi_dw.c | 2
drivers/spi/spi_dw.h | 0
drivers/spi/spi_dw_mid.c | 2
drivers/spi/spi_dw_mmio.c | 2
drivers/spi/spi_dw_pci.c | 2
drivers/spi/spi_ep93xx.c | 0
drivers/spi/spi_mpc512x_psc.c | 0
drivers/spi/spi_mpc52xx.c | 0
drivers/spi/spi_mpc52xx_psc.c | 0
drivers/spi/spi_omap2_mcspi.c | 0
drivers/spi/spi_omap_100k.c | 0
drivers/spi/spi_omap_uwire.c | 0
drivers/spi/spi_orion.c | 0
drivers/spi/spi_pl022.c | 0
drivers/spi/spi_pxa2xx.c | 0
drivers/spi/spi_pxa2xx_pci.c | 0
drivers/spi/spi_ti_ssp.c | 0
drivers/spi/spi_tle62x0.c | 0
drivers/spi/spi_xilinx.c | 0
26 files changed, 192 insertions(+), 218 deletions(-)
delete mode 100644 drivers/spi/atmel_spi.h
rename drivers/spi/{ath79_spi.c => spi_ath79.c} (100%)
rename drivers/spi/{atmel_spi.c => spi_atmel.c} (85%)
rename drivers/spi/{au1550_spi.c => spi_au1550.c} (100%)
rename drivers/spi/{coldfire_qspi.c => spi_coldfire_qspi.c} (100%)
rename drivers/spi/{davinci_spi.c => spi_davinci.c} (100%)
rename drivers/spi/{dw_spi.c => spi_dw.c} (99%)
rename drivers/spi/{dw_spi.h => spi_dw.h} (100%)
rename drivers/spi/{dw_spi_mid.c => spi_dw_mid.c} (99%)
rename drivers/spi/{dw_spi_mmio.c => spi_dw_mmio.c} (99%)
rename drivers/spi/{dw_spi_pci.c => spi_dw_pci.c} (99%)
rename drivers/spi/{ep93xx_spi.c => spi_ep93xx.c} (100%)
rename drivers/spi/{mpc512x_psc_spi.c => spi_mpc512x_psc.c} (100%)
rename drivers/spi/{mpc52xx_spi.c => spi_mpc52xx.c} (100%)
rename drivers/spi/{mpc52xx_psc_spi.c => spi_mpc52xx_psc.c} (100%)
rename drivers/spi/{omap2_mcspi.c => spi_omap2_mcspi.c} (100%)
rename drivers/spi/{omap_spi_100k.c => spi_omap_100k.c} (100%)
rename drivers/spi/{omap_uwire.c => spi_omap_uwire.c} (100%)
rename drivers/spi/{orion_spi.c => spi_orion.c} (100%)
rename drivers/spi/{amba-pl022.c => spi_pl022.c} (100%)
rename drivers/spi/{pxa2xx_spi.c => spi_pxa2xx.c} (100%)
rename drivers/spi/{pxa2xx_spi_pci.c => spi_pxa2xx_pci.c} (100%)
rename drivers/spi/{ti-ssp-spi.c => spi_ti_ssp.c} (100%)
rename drivers/spi/{tle62x0.c => spi_tle62x0.c} (100%)
rename drivers/spi/{xilinx_spi.c => spi_xilinx.c} (100%)drivers/spi/Makefile +34 -46
@@ -7,68 +7,56 @@ ccflags-$(CONFIG_SPI_DEBUG) := -DDEBUG # small core, mostly translating board-specific # config declarations into driver model code obj-$(CONFIG_SPI_MASTER) += spi.o +obj-$(CONFIG_SPI_SPIDEV) += spidev.o # SPI master controller drivers (bus) obj-$(CONFIG_SPI_ALTERA) += spi_altera.o -obj-$(CONFIG_SPI_ATMEL) += atmel_spi.o -obj-$(CONFIG_SPI_ATH79) += ath79_spi.o +obj-$(CONFIG_SPI_ATMEL) += spi_atmel.o +obj-$(CONFIG_SPI_ATH79) += spi_ath79.o +obj-$(CONFIG_SPI_AU1550) += spi_au1550.o obj-$(CONFIG_SPI_BFIN) += spi_bfin5xx.o obj-$(CONFIG_SPI_BFIN_SPORT) += spi_bfin_sport.o obj-$(CONFIG_SPI_BITBANG) += spi_bitbang.o -obj-$(CONFIG_SPI_AU1550) += au1550_spi.o obj-$(CONFIG_SPI_BUTTERFLY) += spi_butterfly.o -obj-$(CONFIG_SPI_COLDFIRE_QSPI) += coldfire_qspi.o -obj-$(CONFIG_SPI_DAVINCI) += davinci_spi.o -obj-$(CONFIG_SPI_DESIGNWARE) += dw_spi.o -obj-$(CONFIG_SPI_DW_PCI) += dw_spi_midpci.o -dw_spi_midpci-objs := dw_spi_pci.o dw_spi_mid.o -obj-$(CONFIG_SPI_DW_MMIO) += dw_spi_mmio.o -obj-$(CONFIG_SPI_EP93XX) += ep93xx_spi.o +obj-$(CONFIG_SPI_COLDFIRE_QSPI) += spi_coldfire_qspi.o +obj-$(CONFIG_SPI_DAVINCI) += spi_davinci.o +obj-$(CONFIG_SPI_DESIGNWARE) += spi_dw.o +obj-$(CONFIG_SPI_DW_MMIO) += spi_dw_mmio.o +obj-$(CONFIG_SPI_DW_PCI) += spi_dw_midpci.o +spi_dw_midpci-objs := spi_dw_pci.o spi_dw_mid.o +obj-$(CONFIG_SPI_EP93XX) += spi_ep93xx.o +obj-$(CONFIG_SPI_FSL_LIB) += spi_fsl_lib.o +obj-$(CONFIG_SPI_FSL_ESPI) += spi_fsl_espi.o +obj-$(CONFIG_SPI_FSL_SPI) += spi_fsl_spi.o obj-$(CONFIG_SPI_GPIO) += spi_gpio.o obj-$(CONFIG_SPI_IMX) += spi_imx.o obj-$(CONFIG_SPI_LM70_LLP) += spi_lm70llp.o -obj-$(CONFIG_SPI_PXA2XX) += pxa2xx_spi.o -obj-$(CONFIG_SPI_PXA2XX_PCI) += pxa2xx_spi_pci.o +obj-$(CONFIG_SPI_MPC512x_PSC) += spi_mpc512x_psc.o +obj-$(CONFIG_SPI_MPC52xx_PSC) += spi_mpc52xx_psc.o +obj-$(CONFIG_SPI_MPC52xx) += spi_mpc52xx.o +obj-$(CONFIG_SPI_NUC900) += spi_nuc900.o obj-$(CONFIG_SPI_OC_TINY) += spi_oc_tiny.o -obj-$(CONFIG_SPI_OMAP_UWIRE) += omap_uwire.o -obj-$(CONFIG_SPI_OMAP24XX) += omap2_mcspi.o -obj-$(CONFIG_SPI_OMAP_100K) += omap_spi_100k.o -obj-$(CONFIG_SPI_ORION) += orion_spi.o -obj-$(CONFIG_SPI_PL022) += amba-pl022.o -obj-$(CONFIG_SPI_MPC512x_PSC) += mpc512x_psc_spi.o -obj-$(CONFIG_SPI_MPC52xx_PSC) += mpc52xx_psc_spi.o -obj-$(CONFIG_SPI_MPC52xx) += mpc52xx_spi.o -obj-$(CONFIG_SPI_FSL_LIB) += spi_fsl_lib.o -obj-$(CONFIG_SPI_FSL_ESPI) += spi_fsl_espi.o -obj-$(CONFIG_SPI_FSL_SPI) += spi_fsl_spi.o +obj-$(CONFIG_SPI_OMAP_UWIRE) += spi_omap_uwire.o +obj-$(CONFIG_SPI_OMAP_100K) += spi_omap_100k.o +obj-$(CONFIG_SPI_OMAP24XX) += spi_omap2_mcspi.o +obj-$(CONFIG_SPI_ORION) += spi_orion.o +obj-$(CONFIG_SPI_PL022) += spi_pl022.o obj-$(CONFIG_SPI_PPC4xx) += spi_ppc4xx.o +obj-$(CONFIG_SPI_PXA2XX) += spi_pxa2xx.o +obj-$(CONFIG_SPI_PXA2XX_PCI) += spi_pxa2xx_pci.o obj-$(CONFIG_SPI_S3C24XX_GPIO) += spi_s3c24xx_gpio.o obj-$(CONFIG_SPI_S3C24XX) += spi_s3c24xx_hw.o +spi_s3c24xx_hw-y := spi_s3c24xx.o +spi_s3c24xx_hw-$(CONFIG_SPI_S3C24XX_FIQ) += spi_s3c24xx_fiq.o obj-$(CONFIG_SPI_S3C64XX) += spi_s3c64xx.o -obj-$(CONFIG_SPI_TEGRA) += spi_tegra.o -obj-$(CONFIG_SPI_TI_SSP) += ti-ssp-spi.o -obj-$(CONFIG_SPI_TOPCLIFF_PCH) += spi_topcliff_pch.o -obj-$(CONFIG_SPI_TXX9) += spi_txx9.o -obj-$(CONFIG_SPI_XILINX) += xilinx_spi.o obj-$(CONFIG_SPI_SH) += spi_sh.o -obj-$(CONFIG_SPI_SH_SCI) += spi_sh_sci.o obj-$(CONFIG_SPI_SH_MSIOF) += spi_sh_msiof.o +obj-$(CONFIG_SPI_SH_SCI) += spi_sh_sci.o obj-$(CONFIG_SPI_STMP3XXX) += spi_stmp.o -obj-$(CONFIG_SPI_NUC900) += spi_nuc900.o - -# special build for s3c24xx spi driver with fiq support -spi_s3c24xx_hw-y := spi_s3c24xx.o -spi_s3c24xx_hw-$(CONFIG_SPI_S3C24XX_FIQ) += spi_s3c24xx_fiq.o - -# ... add above this line ... - -# SPI protocol drivers (device/link on bus) -obj-$(CONFIG_SPI_SPIDEV) += spidev.o -obj-$(CONFIG_SPI_TLE62X0) += tle62x0.o -# ... add above this line ... - -# SPI slave controller drivers (upstream link) -# ... add above this line ... +obj-$(CONFIG_SPI_TEGRA) += spi_tegra.o +obj-$(CONFIG_SPI_TI_SSP) += spi_ti_ssp.o +obj-$(CONFIG_SPI_TLE62X0) += spi_tle62x0.o +obj-$(CONFIG_SPI_TOPCLIFF_PCH) += spi_topcliff_pch.o +obj-$(CONFIG_SPI_TXX9) += spi_txx9.o +obj-$(CONFIG_SPI_XILINX) += spi_xilinx.o -# SPI slave drivers (protocol for that link) -# ... add above this line ...
drivers/spi/atmel_spi.h deleted +0 -167
167 lines not on this page · raw message
drivers/spi/spi_ath79.c renamed from drivers/spi/ath79_spi.c +0 -0
drivers/spi/spi_atmel.c renamed from drivers/spi/atmel_spi.c +154 -1
161 lines not on this page · raw message
drivers/spi/spi_au1550.c renamed from drivers/spi/au1550_spi.c +0 -0
drivers/spi/spi_coldfire_qspi.c renamed from drivers/spi/coldfire_qspi.c +0 -0
drivers/spi/spi_davinci.c renamed from drivers/spi/davinci_spi.c +0 -0
drivers/spi/spi_dw.c renamed from drivers/spi/dw_spi.c +1 -1
@@ -24,7 +24,7 @@#include <linux/slab.h> #include <linux/spi/spi.h> -#include "dw_spi.h" +#include "spi_dw.h" #ifdef CONFIG_DEBUG_FS #include <linux/debugfs.h>
drivers/spi/spi_dw.h renamed from drivers/spi/dw_spi.h +0 -0
drivers/spi/spi_dw_mid.c renamed from drivers/spi/dw_spi_mid.c +1 -1
@@ -23,7 +23,7 @@#include <linux/slab.h> #include <linux/spi/spi.h> -#include "dw_spi.h" +#include "spi_dw.h" #ifdef CONFIG_SPI_DW_MID_DMA #include <linux/intel_mid_dma.h>
drivers/spi/spi_dw_mmio.c renamed from drivers/spi/dw_spi_mmio.c +1 -1
@@ -16,7 +16,7 @@#include <linux/spi/spi.h> #include <linux/scatterlist.h> -#include "dw_spi.h" +#include "spi_dw.h" #define DRIVER_NAME "dw_spi_mmio"
drivers/spi/spi_dw_pci.c renamed from drivers/spi/dw_spi_pci.c +1 -1
@@ -22,7 +22,7 @@#include <linux/slab.h> #include <linux/spi/spi.h> -#include "dw_spi.h" +#include "spi_dw.h" #define DRIVER_NAME "dw_spi_pci"
drivers/spi/spi_ep93xx.c renamed from drivers/spi/ep93xx_spi.c +0 -0
drivers/spi/spi_mpc512x_psc.c renamed from drivers/spi/mpc512x_psc_spi.c +0 -0
drivers/spi/spi_mpc52xx.c renamed from drivers/spi/mpc52xx_spi.c +0 -0
drivers/spi/spi_mpc52xx_psc.c renamed from drivers/spi/mpc52xx_psc_spi.c +0 -0
drivers/spi/spi_omap2_mcspi.c renamed from drivers/spi/omap2_mcspi.c +0 -0
drivers/spi/spi_omap_100k.c renamed from drivers/spi/omap_spi_100k.c +0 -0
drivers/spi/spi_omap_uwire.c renamed from drivers/spi/omap_uwire.c +0 -0
drivers/spi/spi_orion.c renamed from drivers/spi/orion_spi.c +0 -0
drivers/spi/spi_pl022.c renamed from drivers/spi/amba-pl022.c +0 -0
drivers/spi/spi_pxa2xx.c renamed from drivers/spi/pxa2xx_spi.c +0 -0
drivers/spi/spi_pxa2xx_pci.c renamed from drivers/spi/pxa2xx_spi_pci.c +0 -0
drivers/spi/spi_ti_ssp.c renamed from drivers/spi/ti-ssp-spi.c +0 -0
drivers/spi/spi_tle62x0.c renamed from drivers/spi/tle62x0.c +0 -0
drivers/spi/spi_xilinx.c renamed from drivers/spi/xilinx_spi.c +0 -0
Quoted 2 lines · Grant Likely
Sort the SPI makefile and enforce the naming convention spi_*.c for spi drivers.
(...)
Quoted 1 line
rename
drivers/spi/{amba-pl022.c=>spi_pl022.c} (100%)
Whatever the subsystem maintainer likes, that file was named like that since many other AMBA-based drivers are named like that.
I might have a few patches to that file next cycle, I guess I'll just base them on your tree FTM.
Yours,
Linus Walleij
Quoted 7 lines · Linus Walleij
Quoted 2 lines · Grant Likely
Sort the SPI makefile and enforce the naming convention spi_*.c for spi drivers.
(...)
Quoted 1 line
rename
drivers/spi/{amba-pl022.c=>spi_pl022.c} (100%)
Quoted 7 lines
Whatever the subsystem maintainer likes, that file was named like that since many other AMBA-based drivers are named like that.
Acked-by Linus Walleij <[email protected]>I might have a few patches to that file next cycle, I guess I'll just base them on your tree FTM.
Sure, although file renames are not a big deal when picking up patches. I'm also okay with patches based on Linus' tree.
g.