lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110616131536.GE31534@ponder.secretlab.ca>
Date:	Thu, 16 Jun 2011 07:15:36 -0600
From:	Grant Likely <grant.likely@...retlab.ca>
To:	dirk.brandewie@...il.com
Cc:	linux-kernel@...r.kernel.org,
	spi-devel-general@...ts.sourceforge.net
Subject: Re: [PATCH 4/5] spi/spi-dw: update function naming convention

On Wed, Jun 15, 2011 at 10:23:07AM -0700, dirk.brandewie@...il.com wrote:
> From: Dirk Brandewie <dirk.brandewie@...il.com>
> 
> Update internal maning to match new file names now that the naming
> convention for driver files is being enforced.
> 
> Signed-off-by: Dirk Brandewie <dirk.brandewie@...il.com>

You should probably move this patch to before patch 2 & 3 when you
repost so that I can pick it up without needing all the
comments/concerns on patch 3 to be address first.

g.

> ---
>  drivers/spi/spi-dw-mid.c   |    8 ++--
>  drivers/spi/spi-dw-mmio.c  |   34 ++++++------
>  drivers/spi/spi-dw-pci.c   |   10 ++--
>  drivers/spi/spi-dw.c       |  116 ++++++++++++++++++++++---------------------
>  drivers/spi/spi-dw.h       |   46 +++++++++---------
>  include/linux/spi/spi-dw.h |    2 +-
>  6 files changed, 109 insertions(+), 107 deletions(-)
> 
> diff --git a/drivers/spi/spi-dw-mid.c b/drivers/spi/spi-dw-mid.c
> index e44e37f..fb41ea4 100644
> --- a/drivers/spi/spi-dw-mid.c
> +++ b/drivers/spi/spi-dw-mid.c
> @@ -118,7 +118,7 @@ static int mid_spi_dma_transfer(struct dw_spi *dws)
>  
>  	/* 1. setup DMA related registers */
>  
> -	dw_spi_disable(dws);
> +	spi_dw_disable(dws);
>  	dw_writew(dws, dmardlr, 0xf);
>  	dw_writew(dws, dmatdlr, 0x10);
>  	if (dws->xfer.tx_dma)
> @@ -126,7 +126,7 @@ static int mid_spi_dma_transfer(struct dw_spi *dws)
>  	if (dws->xfer.rx_dma)
>  		dma_ctrl |= 0x1;
>  	dw_writew(dws, dmacr, dma_ctrl);
> -	dw_spi_enable(dws);
> +	spi_dw_enable(dws);
>  
>  
>  	dws->dma_chan_done = 0;
> @@ -183,7 +183,7 @@ static int mid_spi_dma_transfer(struct dw_spi *dws)
>  	return 0;
>  }
>  
> -static struct dw_spi_dma_ops mid_dma_ops = {
> +static struct spi_dw_dma_ops mid_dma_ops = {
>  	.dma_init	= mid_spi_dma_init,
>  	.dma_exit	= mid_spi_dma_exit,
>  	.dma_transfer	= mid_spi_dma_transfer,
> @@ -200,7 +200,7 @@ static struct dw_spi_dma_ops mid_dma_ops = {
>  #define CLK_SPI_CDIV_MASK	0x00000e00
>  #define CLK_SPI_DISABLE_OFFSET	8
>  
> -int dw_spi_mid_init(struct dw_spi *dws)
> +int spi_dw_mid_init(struct dw_spi *dws)
>  {
>  	u32 *clk_reg, clk_cdiv;
>  
> diff --git a/drivers/spi/spi-dw-mmio.c b/drivers/spi/spi-dw-mmio.c
> index 34eb665..c978d35 100644
> --- a/drivers/spi/spi-dw-mmio.c
> +++ b/drivers/spi/spi-dw-mmio.c
> @@ -18,21 +18,21 @@
>  
>  #include "spi-dw.h"
>  
> -#define DRIVER_NAME "dw_spi_mmio"
> +#define DRIVER_NAME "spi_dw_mmio"
>  
> -struct dw_spi_mmio {
> +struct spi_dw_mmio {
>  	struct dw_spi  dws;
>  	struct clk     *clk;
>  };
>  
> -static int __devinit dw_spi_mmio_probe(struct platform_device *pdev)
> +static int __devinit spi_dw_mmio_probe(struct platform_device *pdev)
>  {
> -	struct dw_spi_mmio *dwsmmio;
> +	struct spi_dw_mmio *dwsmmio;
>  	struct dw_spi *dws;
>  	struct resource *mem, *ioarea;
>  	int ret;
>  
> -	dwsmmio = kzalloc(sizeof(struct dw_spi_mmio), GFP_KERNEL);
> +	dwsmmio = kzalloc(sizeof(struct spi_dw_mmio), GFP_KERNEL);
>  	if (!dwsmmio) {
>  		ret = -ENOMEM;
>  		goto err_end;
> @@ -82,7 +82,7 @@ static int __devinit dw_spi_mmio_probe(struct platform_device *pdev)
>  	dws->num_cs = 4;
>  	dws->max_freq = clk_get_rate(dwsmmio->clk);
>  
> -	ret = dw_spi_add_host(dws);
> +	ret = spi_dw_add_host(dws);
>  	if (ret)
>  		goto err_clk;
>  
> @@ -105,9 +105,9 @@ err_end:
>  	return ret;
>  }
>  
> -static int __devexit dw_spi_mmio_remove(struct platform_device *pdev)
> +static int __devexit spi_dw_mmio_remove(struct platform_device *pdev)
>  {
> -	struct dw_spi_mmio *dwsmmio = platform_get_drvdata(pdev);
> +	struct spi_dw_mmio *dwsmmio = platform_get_drvdata(pdev);
>  	struct resource *mem;
>  
>  	platform_set_drvdata(pdev, NULL);
> @@ -117,7 +117,7 @@ static int __devexit dw_spi_mmio_remove(struct platform_device *pdev)
>  	dwsmmio->clk = NULL;
>  
>  	free_irq(dwsmmio->dws.irq, &dwsmmio->dws);
> -	dw_spi_remove_host(&dwsmmio->dws);
> +	spi_dw_remove_host(&dwsmmio->dws);
>  	iounmap(dwsmmio->dws.regs);
>  	kfree(dwsmmio);
>  
> @@ -126,25 +126,25 @@ static int __devexit dw_spi_mmio_remove(struct platform_device *pdev)
>  	return 0;
>  }
>  
> -static struct platform_driver dw_spi_mmio_driver = {
> -	.remove		= __devexit_p(dw_spi_mmio_remove),
> +static struct platform_driver spi_dw_mmio_driver = {
> +	.remove		= __devexit_p(spi_dw_mmio_remove),
>  	.driver		= {
>  		.name	= DRIVER_NAME,
>  		.owner	= THIS_MODULE,
>  	},
>  };
>  
> -static int __init dw_spi_mmio_init(void)
> +static int __init spi_dw_mmio_init(void)
>  {
> -	return platform_driver_probe(&dw_spi_mmio_driver, dw_spi_mmio_probe);
> +	return platform_driver_probe(&spi_dw_mmio_driver, spi_dw_mmio_probe);
>  }
> -module_init(dw_spi_mmio_init);
> +module_init(spi_dw_mmio_init);
>  
> -static void __exit dw_spi_mmio_exit(void)
> +static void __exit spi_dw_mmio_exit(void)
>  {
> -	platform_driver_unregister(&dw_spi_mmio_driver);
> +	platform_driver_unregister(&spi_dw_mmio_driver);
>  }
> -module_exit(dw_spi_mmio_exit);
> +module_exit(spi_dw_mmio_exit);
>  
>  MODULE_AUTHOR("Jean-Hugues Deschenes <jean-hugues.deschenes@...asic.com>");
>  MODULE_DESCRIPTION("Memory-mapped I/O interface driver for DW SPI Core");
> diff --git a/drivers/spi/spi-dw-pci.c b/drivers/spi/spi-dw-pci.c
> index c5f37f0..eb35fa5 100644
> --- a/drivers/spi/spi-dw-pci.c
> +++ b/drivers/spi/spi-dw-pci.c
> @@ -80,12 +80,12 @@ static int __devinit spi_pci_probe(struct pci_dev *pdev,
>  	 * clock rate, FIFO depth.
>  	 */
>  	if (pdev->device == 0x0800) {
> -		ret = dw_spi_mid_init(dws);
> +		ret = spi_dw_mid_init(dws);
>  		if (ret)
>  			goto err_unmap;
>  	}
>  
> -	ret = dw_spi_add_host(dws);
> +	ret = spi_dw_add_host(dws);
>  	if (ret)
>  		goto err_unmap;
>  
> @@ -109,7 +109,7 @@ static void __devexit spi_pci_remove(struct pci_dev *pdev)
>  	struct dw_spi_pci *dwpci = pci_get_drvdata(pdev);
>  
>  	pci_set_drvdata(pdev, NULL);
> -	dw_spi_remove_host(&dwpci->dws);
> +	spi_dw_remove_host(&dwpci->dws);
>  	iounmap(dwpci->dws.regs);
>  	pci_release_region(pdev, 0);
>  	kfree(dwpci);
> @@ -122,7 +122,7 @@ static int spi_suspend(struct pci_dev *pdev, pm_message_t state)
>  	struct dw_spi_pci *dwpci = pci_get_drvdata(pdev);
>  	int ret;
>  
> -	ret = dw_spi_suspend_host(&dwpci->dws);
> +	ret = spi_dw_suspend_host(&dwpci->dws);
>  	if (ret)
>  		return ret;
>  	pci_save_state(pdev);
> @@ -141,7 +141,7 @@ static int spi_resume(struct pci_dev *pdev)
>  	ret = pci_enable_device(pdev);
>  	if (ret)
>  		return ret;
> -	return dw_spi_resume_host(&dwpci->dws);
> +	return spi_dw_resume_host(&dwpci->dws);
>  }
>  #else
>  #define spi_suspend	NULL
> diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c
> index 2dacb8f..20f94fa 100644
> --- a/drivers/spi/spi-dw.c
> +++ b/drivers/spi/spi-dw.c
> @@ -1,5 +1,5 @@
>  /*
> - * dw_spi.c - Designware SPI core controller driver
> + * spi-dw.c - Designware SPI core controller driver
>   *
>   * Copyright (c) 2009, Intel Corporation.
>   *
> @@ -36,7 +36,7 @@
>  
>  
>  /* Slave spi_dev related */
> -struct chip_data {
> +struct slave_cfg {
>  	struct spi_device *spi_dev;
>  	u32 cr0;
>  	u32 cs;			/* chip select pin */
> @@ -73,7 +73,7 @@ static ssize_t  spi_show_regs(struct file *file, char __user *user_buf,
>  		return 0;
>  
>  	len += snprintf(buf + len, SPI_REGS_BUFSIZE - len,
> -			"MRST SPI0 registers:\n");
> +			"DW SPI0 registers:\n");
>  	len += snprintf(buf + len, SPI_REGS_BUFSIZE - len,
>  			"=================================\n");
>  	len += snprintf(buf + len, SPI_REGS_BUFSIZE - len,
> @@ -114,42 +114,42 @@ static ssize_t  spi_show_regs(struct file *file, char __user *user_buf,
>  	return ret;
>  }
>  
> -static const struct file_operations mrst_spi_regs_ops = {
> +static const struct file_operations spi_dw_regs_ops = {
>  	.owner		= THIS_MODULE,
>  	.open		= spi_show_regs_open,
>  	.read		= spi_show_regs,
>  	.llseek		= default_llseek,
>  };
>  
> -static int mrst_spi_debugfs_init(struct dw_spi *dws)
> +static int spi_dw_debugfs_init(struct dw_spi *dws)
>  {
> -	dws->debugfs = debugfs_create_dir("mrst_spi", NULL);
> +	dws->debugfs = debugfs_create_dir("spi_dw", NULL);
>  	if (!dws->debugfs)
>  		return -ENOMEM;
>  
>  	debugfs_create_file("registers", S_IFREG | S_IRUGO,
> -		dws->debugfs, (void *)dws, &mrst_spi_regs_ops);
> +		dws->debugfs, (void *)dws, &spi_dw_regs_ops);
>  	return 0;
>  }
>  
> -static void mrst_spi_debugfs_remove(struct dw_spi *dws)
> +static void spi_dw_debugfs_remove(struct dw_spi *dws)
>  {
>  	if (dws->debugfs)
>  		debugfs_remove_recursive(dws->debugfs);
>  }
>  
>  #else
> -static inline int mrst_spi_debugfs_init(struct dw_spi *dws)
> +static inline int spi_dw_debugfs_init(struct dw_spi *dws)
>  {
>  	return 0;
>  }
>  
> -static inline void mrst_spi_debugfs_remove(struct dw_spi *dws)
> +static inline void spi_dw_debugfs_remove(struct dw_spi *dws)
>  {
>  }
>  #endif /* CONFIG_DEBUG_FS */
>  
> -static irqreturn_t dw_spi_irq(int irq, void *dev_id)
> +static irqreturn_t spi_dw_irq(int irq, void *dev_id)
>  {
>  	struct dw_spi *dws = dev_id;
>  	u16  irq_mask = 0x3f;
> @@ -164,15 +164,16 @@ static irqreturn_t dw_spi_irq(int irq, void *dev_id)
>  		dw_readw(dws, rxoicr);
>  		dw_readw(dws, rxuicr);
>  		dws->xfer.err = -EIO;
> -		dw_spi_disable(dws);
> +		spi_dw_disable(dws);
>  		complete(&dws->xfer.complete);
>  		return IRQ_HANDLED;
>  	}
>  
>  	/* disable interrupts */
> -	dw_spi_mask_intr(dws, irq_mask);
> +	spi_dw_mask_intr(dws, irq_mask);
>  	return IRQ_WAKE_THREAD;
>  }
> +
>  struct spi_message *get_message(struct dw_spi *dws)
>  {
>  	struct spi_message *message = NULL;
> @@ -187,6 +188,7 @@ struct spi_message *get_message(struct dw_spi *dws)
>  	spin_unlock_irqrestore(&dws->lock, flags);
>  	return message;
>  }
> +
>  static inline u32 tx_max(struct dw_spi *dws)
>  {
>  	u32 tx_left, tx_room;
> @@ -204,7 +206,7 @@ static inline u32 rx_max(struct dw_spi *dws)
>  	return min(rx_left, (u32)dw_readw(dws, rxflr));
>  }
>  
> -static int transfer_setup(struct dw_spi *dws, struct chip_data *controller,
> +static int transfer_setup(struct dw_spi *dws, struct slave_cfg *controller,
>  			struct spi_transfer *transfer, struct spi_message *msg)
>  {
>  	int err = 0;
> @@ -253,7 +255,7 @@ static int transfer_setup(struct dw_spi *dws, struct chip_data *controller,
>  	if (unlikely(!controller->clk_div)) {
>  		controller->clk_div = dws->max_freq / controller->speed_hz;
>  		controller->clk_div = (controller->clk_div + 1) & 0xfffe;
> -		dw_spi_set_clk(dws, controller->clk_div);
> +		spi_dw_set_clk(dws, controller->clk_div);
>  		dev_err(&dws->master->dev, "setting default clk_div");
>  		err = 1;
>  	}
> @@ -292,11 +294,11 @@ static int transfer_setup(struct dw_spi *dws, struct chip_data *controller,
>  	controller->cr0 = cr0;
>  
>  	if (err || dw_readw(dws, ctrl0) != cr0) {
> -		dw_spi_disable(dws);
> -		dw_spi_chip_sel(dws, controller->spi_dev->chip_select);
> +		spi_dw_disable(dws);
> +		spi_dw_chip_sel(dws, controller->spi_dev->chip_select);
>  		dw_writew(dws, ctrl0, cr0);
> -		dw_spi_set_clk(dws, controller->clk_div);
> -		dw_spi_enable(dws);
> +		spi_dw_set_clk(dws, controller->clk_div);
> +		spi_dw_enable(dws);
>  		err = 0;
>  	}
>  out:
> @@ -357,7 +359,7 @@ static inline void do_pio(struct dw_spi *dws)
>  	complete(&dws->xfer.complete);
>  }
>  
> -static irqreturn_t dw_spi_irq_thread_handler(int irq, void *dev_id)
> +static irqreturn_t spi_dw_irq_thread_handler(int irq, void *dev_id)
>  {
>  	struct dw_spi *dws = dev_id;
>  
> @@ -377,17 +379,17 @@ static irqreturn_t dw_spi_irq_thread_handler(int irq, void *dev_id)
>  		goto out;
>  	}
>  
> -	dw_spi_umask_intr(dws, SPI_INT_ALL);
> +	spi_dw_umask_intr(dws, SPI_INT_ALL);
>  out:
>  	return IRQ_HANDLED;
>  }
>  
>  static inline void do_int_xfer(struct dw_spi *dws)
>  {
> -	dw_spi_disable(dws);
> +	spi_dw_disable(dws);
>  	dw_writew(dws, txfltr, dws->xfer.tx_threshold);
>  	dw_writew(dws, rxfltr, dws->xfer.rx_threshold);
> -	dw_spi_enable(dws);
> +	spi_dw_enable(dws);
>  	dw_readw(dws, icr);
>  	tx_fifo_fill(dws);
>  	dw_writew(dws, imr, SPI_INT_ALL);
> @@ -433,7 +435,7 @@ static void pump_messages(struct work_struct *work)
>  		container_of(work, struct dw_spi, pump_messages);
>  	struct spi_transfer *transfer;
>  	struct spi_message *message;
> -	struct chip_data *controller;
> +	struct slave_cfg *controller;
>  	int err = 0;
>  
>  
> @@ -474,7 +476,7 @@ static void pump_messages(struct work_struct *work)
>  }
>  
>  /* spi_device use this to queue in their spi_msg */
> -static int dw_spi_transfer(struct spi_device *spi, struct spi_message *msg)
> +static int spi_dw_transfer(struct spi_device *spi, struct spi_message *msg)
>  {
>  	struct dw_spi *dws = spi_master_get_devdata(spi->master);
>  	unsigned long flags;
> @@ -496,10 +498,10 @@ static int dw_spi_transfer(struct spi_device *spi, struct spi_message *msg)
>  }
>  
>  /* This may be called twice for each spi dev */
> -static int dw_spi_setup(struct spi_device *spi)
> +static int spi_dw_setup(struct spi_device *spi)
>  {
> -	struct dw_spi_chip *chip_info = NULL;
> -	struct chip_data *chip;
> +	struct spi_dw_chip *chip_info = NULL;
> +	struct slave_cfg *chip;
>  
>  	if (spi->bits_per_word != 8 && spi->bits_per_word != 16)
>  		return -EINVAL;
> @@ -512,7 +514,7 @@ static int dw_spi_setup(struct spi_device *spi)
>  	/* Only alloc on first setup */
>  	chip = spi_get_ctldata(spi);
>  	if (!chip) {
> -		chip = kzalloc(sizeof(struct chip_data), GFP_KERNEL);
> +		chip = kzalloc(sizeof(struct slave_cfg), GFP_KERNEL);
>  		if (!chip)
>  			return -ENOMEM;
>  	}
> @@ -545,9 +547,9 @@ static int dw_spi_setup(struct spi_device *spi)
>  	return 0;
>  }
>  
> -static void dw_spi_cleanup(struct spi_device *spi)
> +static void spi_dw_cleanup(struct spi_device *spi)
>  {
> -	struct chip_data *chip = spi_get_ctldata(spi);
> +	struct slave_cfg *chip = spi_get_ctldata(spi);
>  	kfree(chip);
>  }
>  
> @@ -568,7 +570,7 @@ static int __devinit init_queue(struct dw_spi *dws)
>  }
>  
>  
> -int dw_spi_stop_queue(struct dw_spi *dws)
> +int spi_dw_stop_queue(struct dw_spi *dws)
>  {
>  	unsigned long flags;
>  	int status = 0;
> @@ -582,13 +584,13 @@ int dw_spi_stop_queue(struct dw_spi *dws)
>  
>  	return status;
>  }
> -EXPORT_SYMBOL_GPL(dw_spi_stop_queue);
> +EXPORT_SYMBOL_GPL(spi_dw_stop_queue);
>  
>  static int destroy_queue(struct dw_spi *dws)
>  {
>  	int status;
>  
> -	status = dw_spi_stop_queue(dws);
> +	status = spi_dw_stop_queue(dws);
>  	if (status != 0)
>  		return status;
>  	destroy_workqueue(dws->workqueue);
> @@ -598,15 +600,15 @@ static int destroy_queue(struct dw_spi *dws)
>  /* Restart the controller, disable all interrupts, clean rx fifo */
>  static void spi_hw_init(struct dw_spi *dws)
>  {
> -	dw_spi_disable(dws);
> -	dw_spi_mask_intr(dws, 0xff);
> +	spi_dw_disable(dws);
> +	spi_dw_mask_intr(dws, 0xff);
>  	dw_readw(dws, icr);
> -	dw_spi_enable(dws);
> +	spi_dw_enable(dws);
>  
>  	BUG_ON(!dws->fifo_len);
>  }
>  
> -int __devinit dw_spi_add_host(struct dw_spi *dws)
> +int __devinit spi_dw_add_host(struct dw_spi *dws)
>  {
>  	struct spi_master *master;
>  	int ret;
> @@ -627,8 +629,8 @@ int __devinit dw_spi_add_host(struct dw_spi *dws)
>  	/* Change to address of FIFO */
>  	dws->dma_addr = (dma_addr_t)(dws->paddr + 0x60);
>  
> -	ret = request_threaded_irq(dws->irq, dw_spi_irq,
> -				dw_spi_irq_thread_handler,
> +	ret = request_threaded_irq(dws->irq, spi_dw_irq,
> +				spi_dw_irq_thread_handler,
>  				IRQF_SHARED, "dw_spi", dws);
>  	if (ret < 0) {
>  		dev_err(&master->dev, "can not get IRQ\n");
> @@ -638,9 +640,9 @@ int __devinit dw_spi_add_host(struct dw_spi *dws)
>  	master->mode_bits = SPI_CPOL | SPI_CPHA;
>  	master->bus_num = dws->bus_num;
>  	master->num_chipselect = dws->num_cs;
> -	master->cleanup = dw_spi_cleanup;
> -	master->setup = dw_spi_setup;
> -	master->transfer = dw_spi_transfer;
> +	master->cleanup = spi_dw_cleanup;
> +	master->setup = spi_dw_setup;
> +	master->transfer = spi_dw_transfer;
>  
>  	/* Basic HW init */
>  	spi_hw_init(dws);
> @@ -667,7 +669,7 @@ int __devinit dw_spi_add_host(struct dw_spi *dws)
>  		goto err_queue_alloc;
>  	}
>  
> -	mrst_spi_debugfs_init(dws);
> +	spi_dw_debugfs_init(dws);
>  	return 0;
>  
>  err_queue_alloc:
> @@ -675,59 +677,59 @@ err_queue_alloc:
>  	if (dws->dma_ops && dws->dma_ops->dma_exit)
>  		dws->dma_ops->dma_exit(dws);
>  err_diable_hw:
> -	dw_spi_disable(dws);
> +	spi_dw_disable(dws);
>  	free_irq(dws->irq, dws);
>  err_free_master:
>  	spi_master_put(master);
>  exit:
>  	return ret;
>  }
> -EXPORT_SYMBOL_GPL(dw_spi_add_host);
> +EXPORT_SYMBOL_GPL(spi_dw_add_host);
>  
> -void __devexit dw_spi_remove_host(struct dw_spi *dws)
> +void __devexit spi_dw_remove_host(struct dw_spi *dws)
>  {
>  	int status = 0;
>  
>  	if (!dws)
>  		return;
> -	mrst_spi_debugfs_remove(dws);
> +	spi_dw_debugfs_remove(dws);
>  
>  	/* Remove the queue */
>  	status = destroy_queue(dws);
>  	if (status != 0)
> -		dev_err(&dws->master->dev, "dw_spi_remove: workqueue will not "
> +		dev_err(&dws->master->dev, "spi_dw_remove: workqueue will not "
>  			"complete, message memory not freed\n");
>  
>  	if (dws->dma_ops && dws->dma_ops->dma_exit)
>  		dws->dma_ops->dma_exit(dws);
> -	dw_spi_disable(dws);
> +	spi_dw_disable(dws);
>  	dw_readw(dws, icr);
>  	free_irq(dws->irq, dws);
>  
>  	/* Disconnect from the SPI framework */
>  	spi_unregister_master(dws->master);
>  }
> -EXPORT_SYMBOL_GPL(dw_spi_remove_host);
> +EXPORT_SYMBOL_GPL(spi_dw_remove_host);
>  
> -int dw_spi_suspend_host(struct dw_spi *dws)
> +int spi_dw_suspend_host(struct dw_spi *dws)
>  {
>  	int ret = 0;
>  
> -	ret = dw_spi_stop_queue(dws);
> +	ret = spi_dw_stop_queue(dws);
>  	if (ret)
>  		return ret;
> -	dw_spi_disable(dws);
> +	spi_dw_disable(dws);
>  	return ret;
>  }
> -EXPORT_SYMBOL_GPL(dw_spi_suspend_host);
> +EXPORT_SYMBOL_GPL(spi_dw_suspend_host);
>  
> -int dw_spi_resume_host(struct dw_spi *dws)
> +int spi_dw_resume_host(struct dw_spi *dws)
>  {
>  	spi_hw_init(dws);
>  	dws->run = QUEUE_RUNNING;
>  	return 0;
>  }
> -EXPORT_SYMBOL_GPL(dw_spi_resume_host);
> +EXPORT_SYMBOL_GPL(spi_dw_resume_host);
>  
>  MODULE_AUTHOR("Feng Tang <feng.tang@...el.com>");
>  MODULE_DESCRIPTION("Driver for DesignWare SPI controller core");
> diff --git a/drivers/spi/spi-dw.h b/drivers/spi/spi-dw.h
> index b016b85..6acb67c 100644
> --- a/drivers/spi/spi-dw.h
> +++ b/drivers/spi/spi-dw.h
> @@ -3,7 +3,7 @@
>  
>  #include <linux/io.h>
>  #include <linux/scatterlist.h>
> -#include <linux/spi/dw_spi.h>
> +#include <linux/spi/spi-dw.h>
>  
>  /* Bit fields in CTRLR0 */
>  #define SPI_DFS_OFFSET			0
> @@ -53,7 +53,7 @@
>  /* TX RX interrupt level threshold, max can be 256 */
>  #define SPI_INT_THRESHOLD		32
>  
> -struct dw_spi_reg {
> +struct spi_dw_reg {
>  	u32	ctrl0;
>  	u32	ctrl1;
>  	u32	ssienr;
> @@ -83,7 +83,7 @@ struct dw_spi_reg {
>  } __packed;
>  
>  struct dw_spi;
> -struct dw_spi_dma_ops {
> +struct spi_dw_dma_ops {
>  	int (*dma_init)(struct dw_spi *dws);
>  	void (*dma_exit)(struct dw_spi *dws);
>  	int (*dma_transfer)(struct dw_spi *dws);
> @@ -149,7 +149,7 @@ struct dw_spi {
>  	u32			dma_chan_done;
>  	struct device		*dma_dev;
>  	dma_addr_t		dma_addr; /* phy address of the Data register */
> -	struct dw_spi_dma_ops	*dma_ops;
> +	struct spi_dw_dma_ops	*dma_ops;
>  	void			*dma_priv; /* platform relate info */
>  	struct pci_dev		*dmac;
>  
> @@ -161,34 +161,34 @@ struct dw_spi {
>  };
>  
>  #define dw_readl(dw, name) \
> -	__raw_readl(&(((struct dw_spi_reg *)dw->regs)->name))
> +	__raw_readl(&(((struct spi_dw_reg *)dw->regs)->name))
>  #define dw_writel(dw, name, val) \
> -	__raw_writel((val), &(((struct dw_spi_reg *)dw->regs)->name))
> +	__raw_writel((val), &(((struct spi_dw_reg *)dw->regs)->name))
>  #define dw_readw(dw, name) \
> -	__raw_readw(&(((struct dw_spi_reg *)dw->regs)->name))
> +	__raw_readw(&(((struct spi_dw_reg *)dw->regs)->name))
>  #define dw_writew(dw, name, val) \
> -	__raw_writew((val), &(((struct dw_spi_reg *)dw->regs)->name))
> +	__raw_writew((val), &(((struct spi_dw_reg *)dw->regs)->name))
>  #define dw_readb(dw, name) \
> -	__raw_readb(&(((struct dw_spi_reg *)dw->regs)->name))
> +	__raw_readb(&(((struct spi_dw_reg *)dw->regs)->name))
>  #define dw_writeb(dw, name, val) \
> -	__raw_writeb((val), &(((struct dw_spi_reg *)dw->regs)->name))
> +	__raw_writeb((val), &(((struct spi_dw_reg *)dw->regs)->name))
>  
> -static inline void dw_spi_disable(struct dw_spi *dws)
> +static inline void spi_dw_disable(struct dw_spi *dws)
>  {
>  	dw_writel(dws, ssienr, 0);
>  }
>  
> -static inline void dw_spi_enable(struct dw_spi *dws)
> +static inline void spi_dw_enable(struct dw_spi *dws)
>  {
>  	dw_writel(dws, ssienr, 1);
>  }
>  
> -static inline void dw_spi_set_clk(struct dw_spi *dws, u16 div)
> +static inline void spi_dw_set_clk(struct dw_spi *dws, u16 div)
>  {
>  	dw_writel(dws, baudr, div);
>  }
>  
> -static inline void dw_spi_chip_sel(struct dw_spi *dws, u16 cs)
> +static inline void spi_dw_chip_sel(struct dw_spi *dws, u16 cs)
>  {
>  	if (cs > dws->num_cs)
>  		return;
> @@ -197,7 +197,7 @@ static inline void dw_spi_chip_sel(struct dw_spi *dws, u16 cs)
>  }
>  
>  /* Disable IRQ bits */
> -static inline void dw_spi_mask_intr(struct dw_spi *dws, u32 mask)
> +static inline void spi_dw_mask_intr(struct dw_spi *dws, u32 mask)
>  {
>  	u32 new_mask;
>  
> @@ -206,7 +206,7 @@ static inline void dw_spi_mask_intr(struct dw_spi *dws, u32 mask)
>  }
>  
>  /* Enable IRQ bits */
> -static inline void dw_spi_umask_intr(struct dw_spi *dws, u32 mask)
> +static inline void spi_dw_umask_intr(struct dw_spi *dws, u32 mask)
>  {
>  	u32 new_mask;
>  
> @@ -214,13 +214,13 @@ static inline void dw_spi_umask_intr(struct dw_spi *dws, u32 mask)
>  	dw_writel(dws, imr, new_mask);
>  }
>  
> -extern int dw_spi_add_host(struct dw_spi *dws);
> -extern void dw_spi_remove_host(struct dw_spi *dws);
> -extern int dw_spi_suspend_host(struct dw_spi *dws);
> -extern int dw_spi_resume_host(struct dw_spi *dws);
> -extern void dw_spi_xfer_done(struct dw_spi *dws);
> -extern int dw_spi_stop_queue(struct dw_spi *dws);
> +extern int spi_dw_add_host(struct dw_spi *dws);
> +extern void spi_dw_remove_host(struct dw_spi *dws);
> +extern int spi_dw_suspend_host(struct dw_spi *dws);
> +extern int spi_dw_resume_host(struct dw_spi *dws);
> +extern void spi_dw_xfer_done(struct dw_spi *dws);
> +extern int spi_dw_stop_queue(struct dw_spi *dws);
>  
>  /* platform related setup */
> -extern int dw_spi_mid_init(struct dw_spi *dws); /* Intel MID platforms */
> +extern int spi_dw_mid_init(struct dw_spi *dws); /* Intel MID platforms */
>  #endif /* DW_SPI_HEADER_H */
> diff --git a/include/linux/spi/spi-dw.h b/include/linux/spi/spi-dw.h
> index 75dba03..c4a96be 100644
> --- a/include/linux/spi/spi-dw.h
> +++ b/include/linux/spi/spi-dw.h
> @@ -13,7 +13,7 @@ enum dw_ssi_type {
>   * which can be save in the "controller_data" member of the
>   * struct spi_device
>   */
> -struct dw_spi_chip {
> +struct spi_dw_chip {
>  	u8 type;	/* SPI/SSP/Micrwire */
>  	u8 enable_dma;
>  };
> -- 
> 1.7.3.4
> 
> 
> ------------------------------------------------------------------------------
> EditLive Enterprise is the world's most technically advanced content
> authoring tool. Experience the power of Track Changes, Inline Image
> Editing and ensure content is compliant with Accessibility Checking.
> http://p.sf.net/sfu/ephox-dev2dev
> _______________________________________________
> spi-devel-general mailing list
> spi-devel-general@...ts.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/spi-devel-general
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ