Commit c3dce24c authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Mark Brown

spi: pxa2xx: Utilize struct device from struct ssp_device

We have a duplication of struct device in the struct driver_data,
get rid of it and reuse member from struct ssp_device instead.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210423182441.50272-4-andriy.shevchenko@linux.intel.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent 9e43c9a8
...@@ -111,7 +111,7 @@ pxa2xx_spi_dma_prepare_one(struct driver_data *drv_data, ...@@ -111,7 +111,7 @@ pxa2xx_spi_dma_prepare_one(struct driver_data *drv_data,
ret = dmaengine_slave_config(chan, &cfg); ret = dmaengine_slave_config(chan, &cfg);
if (ret) { if (ret) {
dev_warn(&drv_data->pdev->dev, "DMA slave config failed\n"); dev_warn(drv_data->ssp->dev, "DMA slave config failed\n");
return NULL; return NULL;
} }
...@@ -125,7 +125,7 @@ irqreturn_t pxa2xx_spi_dma_transfer(struct driver_data *drv_data) ...@@ -125,7 +125,7 @@ irqreturn_t pxa2xx_spi_dma_transfer(struct driver_data *drv_data)
status = pxa2xx_spi_read(drv_data, SSSR) & drv_data->mask_sr; status = pxa2xx_spi_read(drv_data, SSSR) & drv_data->mask_sr;
if (status & SSSR_ROR) { if (status & SSSR_ROR) {
dev_err(&drv_data->pdev->dev, "FIFO overrun\n"); dev_err(drv_data->ssp->dev, "FIFO overrun\n");
dmaengine_terminate_async(drv_data->controller->dma_rx); dmaengine_terminate_async(drv_data->controller->dma_rx);
dmaengine_terminate_async(drv_data->controller->dma_tx); dmaengine_terminate_async(drv_data->controller->dma_tx);
...@@ -145,16 +145,14 @@ int pxa2xx_spi_dma_prepare(struct driver_data *drv_data, ...@@ -145,16 +145,14 @@ int pxa2xx_spi_dma_prepare(struct driver_data *drv_data,
tx_desc = pxa2xx_spi_dma_prepare_one(drv_data, DMA_MEM_TO_DEV, xfer); tx_desc = pxa2xx_spi_dma_prepare_one(drv_data, DMA_MEM_TO_DEV, xfer);
if (!tx_desc) { if (!tx_desc) {
dev_err(&drv_data->pdev->dev, dev_err(drv_data->ssp->dev, "failed to get DMA TX descriptor\n");
"failed to get DMA TX descriptor\n");
err = -EBUSY; err = -EBUSY;
goto err_tx; goto err_tx;
} }
rx_desc = pxa2xx_spi_dma_prepare_one(drv_data, DMA_DEV_TO_MEM, xfer); rx_desc = pxa2xx_spi_dma_prepare_one(drv_data, DMA_DEV_TO_MEM, xfer);
if (!rx_desc) { if (!rx_desc) {
dev_err(&drv_data->pdev->dev, dev_err(drv_data->ssp->dev, "failed to get DMA RX descriptor\n");
"failed to get DMA RX descriptor\n");
err = -EBUSY; err = -EBUSY;
goto err_rx; goto err_rx;
} }
...@@ -191,8 +189,8 @@ void pxa2xx_spi_dma_stop(struct driver_data *drv_data) ...@@ -191,8 +189,8 @@ void pxa2xx_spi_dma_stop(struct driver_data *drv_data)
int pxa2xx_spi_dma_setup(struct driver_data *drv_data) int pxa2xx_spi_dma_setup(struct driver_data *drv_data)
{ {
struct pxa2xx_spi_controller *pdata = drv_data->controller_info; struct pxa2xx_spi_controller *pdata = drv_data->controller_info;
struct device *dev = &drv_data->pdev->dev;
struct spi_controller *controller = drv_data->controller; struct spi_controller *controller = drv_data->controller;
struct device *dev = drv_data->ssp->dev;
dma_cap_mask_t mask; dma_cap_mask_t mask;
dma_cap_zero(mask); dma_cap_zero(mask);
......
...@@ -239,6 +239,7 @@ static int pxa2xx_spi_pci_probe(struct pci_dev *dev, ...@@ -239,6 +239,7 @@ static int pxa2xx_spi_pci_probe(struct pci_dev *dev,
spi_pdata.dma_burst_size = c->dma_burst_size ? c->dma_burst_size : 1; spi_pdata.dma_burst_size = c->dma_burst_size ? c->dma_burst_size : 1;
ssp = &spi_pdata.ssp; ssp = &spi_pdata.ssp;
ssp->dev = &dev->dev;
ssp->phys_base = pci_resource_start(dev, 0); ssp->phys_base = pci_resource_start(dev, 0);
ssp->mmio_base = pcim_iomap_table(dev)[0]; ssp->mmio_base = pcim_iomap_table(dev)[0];
ssp->port_id = (c->port_id >= 0) ? c->port_id : dev->devfn; ssp->port_id = (c->port_id >= 0) ? c->port_id : dev->devfn;
......
...@@ -628,7 +628,7 @@ static void int_error_stop(struct driver_data *drv_data, const char *msg) ...@@ -628,7 +628,7 @@ static void int_error_stop(struct driver_data *drv_data, const char *msg)
pxa2xx_spi_flush(drv_data); pxa2xx_spi_flush(drv_data);
pxa2xx_spi_off(drv_data); pxa2xx_spi_off(drv_data);
dev_err(&drv_data->pdev->dev, "%s\n", msg); dev_err(drv_data->ssp->dev, "%s\n", msg);
drv_data->controller->cur_msg->status = -EIO; drv_data->controller->cur_msg->status = -EIO;
spi_finalize_current_transfer(drv_data->controller); spi_finalize_current_transfer(drv_data->controller);
...@@ -731,8 +731,7 @@ static void handle_bad_msg(struct driver_data *drv_data) ...@@ -731,8 +731,7 @@ static void handle_bad_msg(struct driver_data *drv_data)
pxa2xx_spi_write(drv_data, SSTO, 0); pxa2xx_spi_write(drv_data, SSTO, 0);
write_SSSR_CS(drv_data, drv_data->clear_sr); write_SSSR_CS(drv_data, drv_data->clear_sr);
dev_err(&drv_data->pdev->dev, dev_err(drv_data->ssp->dev, "bad message state in interrupt handler\n");
"bad message state in interrupt handler\n");
} }
static irqreturn_t ssp_int(int irq, void *dev_id) static irqreturn_t ssp_int(int irq, void *dev_id)
...@@ -748,7 +747,7 @@ static irqreturn_t ssp_int(int irq, void *dev_id) ...@@ -748,7 +747,7 @@ static irqreturn_t ssp_int(int irq, void *dev_id)
* the IRQ was not for us (we shouldn't be RPM suspended when the * the IRQ was not for us (we shouldn't be RPM suspended when the
* interrupt is enabled). * interrupt is enabled).
*/ */
if (pm_runtime_suspended(&drv_data->pdev->dev)) if (pm_runtime_suspended(drv_data->ssp->dev))
return IRQ_NONE; return IRQ_NONE;
/* /*
...@@ -1158,7 +1157,7 @@ static int pxa2xx_spi_slave_abort(struct spi_controller *controller) ...@@ -1158,7 +1157,7 @@ static int pxa2xx_spi_slave_abort(struct spi_controller *controller)
pxa2xx_spi_flush(drv_data); pxa2xx_spi_flush(drv_data);
pxa2xx_spi_off(drv_data); pxa2xx_spi_off(drv_data);
dev_dbg(&drv_data->pdev->dev, "transfer aborted\n"); dev_dbg(drv_data->ssp->dev, "transfer aborted\n");
drv_data->controller->cur_msg->status = -EINTR; drv_data->controller->cur_msg->status = -EINTR;
spi_finalize_current_transfer(drv_data->controller); spi_finalize_current_transfer(drv_data->controller);
...@@ -1645,7 +1644,7 @@ static int pxa2xx_spi_fw_translate_cs(struct spi_controller *controller, ...@@ -1645,7 +1644,7 @@ static int pxa2xx_spi_fw_translate_cs(struct spi_controller *controller,
{ {
struct driver_data *drv_data = spi_controller_get_devdata(controller); struct driver_data *drv_data = spi_controller_get_devdata(controller);
if (has_acpi_companion(&drv_data->pdev->dev)) { if (has_acpi_companion(drv_data->ssp->dev)) {
switch (drv_data->ssp_type) { switch (drv_data->ssp_type) {
/* /*
* For Atoms the ACPI DeviceSelection used by the Windows * For Atoms the ACPI DeviceSelection used by the Windows
...@@ -1711,7 +1710,6 @@ static int pxa2xx_spi_probe(struct platform_device *pdev) ...@@ -1711,7 +1710,6 @@ static int pxa2xx_spi_probe(struct platform_device *pdev)
drv_data = spi_controller_get_devdata(controller); drv_data = spi_controller_get_devdata(controller);
drv_data->controller = controller; drv_data->controller = controller;
drv_data->controller_info = platform_info; drv_data->controller_info = platform_info;
drv_data->pdev = pdev;
drv_data->ssp = ssp; drv_data->ssp = ssp;
controller->dev.of_node = pdev->dev.of_node; controller->dev.of_node = pdev->dev.of_node;
......
...@@ -12,7 +12,6 @@ ...@@ -12,7 +12,6 @@
#include <linux/errno.h> #include <linux/errno.h>
#include <linux/io.h> #include <linux/io.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/platform_device.h>
#include <linux/pxa2xx_ssp.h> #include <linux/pxa2xx_ssp.h>
#include <linux/scatterlist.h> #include <linux/scatterlist.h>
#include <linux/sizes.h> #include <linux/sizes.h>
...@@ -20,9 +19,6 @@ ...@@ -20,9 +19,6 @@
#include <linux/spi/pxa2xx_spi.h> #include <linux/spi/pxa2xx_spi.h>
struct driver_data { struct driver_data {
/* Driver model hookup */
struct platform_device *pdev;
/* SSP Info */ /* SSP Info */
struct ssp_device *ssp; struct ssp_device *ssp;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment