Commit a65dd5b2 authored by Richard Cochran's avatar Richard Cochran Committed by David S. Miller

cpsw: rename register banks to match the reference manual, part 2

The code mixes up the CPSW_SS and the CPSW_WR register naming. This patch
changes the names to conform to the published Technical Reference Manual
from TI, in order to make working on the code less confusing.
Signed-off-by: default avatarRichard Cochran <richardcochran@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c38e01b8
...@@ -286,7 +286,7 @@ struct cpsw_priv { ...@@ -286,7 +286,7 @@ struct cpsw_priv {
struct platform_device *pdev; struct platform_device *pdev;
struct net_device *ndev; struct net_device *ndev;
struct resource *cpsw_res; struct resource *cpsw_res;
struct resource *cpsw_ss_res; struct resource *cpsw_wr_res;
struct napi_struct napi; struct napi_struct napi;
struct device *dev; struct device *dev;
struct cpsw_platform_data data; struct cpsw_platform_data data;
...@@ -1270,25 +1270,25 @@ static int __devinit cpsw_probe(struct platform_device *pdev) ...@@ -1270,25 +1270,25 @@ static int __devinit cpsw_probe(struct platform_device *pdev)
priv->host_port_regs = regs + data->host_port_reg_ofs; priv->host_port_regs = regs + data->host_port_reg_ofs;
priv->cpts.reg = regs + data->cpts_reg_ofs; priv->cpts.reg = regs + data->cpts_reg_ofs;
priv->cpsw_ss_res = platform_get_resource(pdev, IORESOURCE_MEM, 1); priv->cpsw_wr_res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
if (!priv->cpsw_ss_res) { if (!priv->cpsw_wr_res) {
dev_err(priv->dev, "error getting i/o resource\n"); dev_err(priv->dev, "error getting i/o resource\n");
ret = -ENOENT; ret = -ENOENT;
goto clean_clk_ret; goto clean_clk_ret;
} }
if (!request_mem_region(priv->cpsw_ss_res->start, if (!request_mem_region(priv->cpsw_wr_res->start,
resource_size(priv->cpsw_ss_res), ndev->name)) { resource_size(priv->cpsw_wr_res), ndev->name)) {
dev_err(priv->dev, "failed request i/o region\n"); dev_err(priv->dev, "failed request i/o region\n");
ret = -ENXIO; ret = -ENXIO;
goto clean_clk_ret; goto clean_clk_ret;
} }
regs = ioremap(priv->cpsw_ss_res->start, regs = ioremap(priv->cpsw_wr_res->start,
resource_size(priv->cpsw_ss_res)); resource_size(priv->cpsw_wr_res));
if (!regs) { if (!regs) {
dev_err(priv->dev, "unable to map i/o region\n"); dev_err(priv->dev, "unable to map i/o region\n");
goto clean_cpsw_ss_iores_ret; goto clean_cpsw_wr_iores_ret;
} }
priv->wr_regs = regs; priv->wr_regs = regs;
...@@ -1409,9 +1409,9 @@ static int __devinit cpsw_probe(struct platform_device *pdev) ...@@ -1409,9 +1409,9 @@ static int __devinit cpsw_probe(struct platform_device *pdev)
cpdma_ctlr_destroy(priv->dma); cpdma_ctlr_destroy(priv->dma);
clean_iomap_ret: clean_iomap_ret:
iounmap(priv->regs); iounmap(priv->regs);
clean_cpsw_ss_iores_ret: clean_cpsw_wr_iores_ret:
release_mem_region(priv->cpsw_ss_res->start, release_mem_region(priv->cpsw_wr_res->start,
resource_size(priv->cpsw_ss_res)); resource_size(priv->cpsw_wr_res));
clean_cpsw_iores_ret: clean_cpsw_iores_ret:
release_mem_region(priv->cpsw_res->start, release_mem_region(priv->cpsw_res->start,
resource_size(priv->cpsw_res)); resource_size(priv->cpsw_res));
...@@ -1442,8 +1442,8 @@ static int __devexit cpsw_remove(struct platform_device *pdev) ...@@ -1442,8 +1442,8 @@ static int __devexit cpsw_remove(struct platform_device *pdev)
iounmap(priv->regs); iounmap(priv->regs);
release_mem_region(priv->cpsw_res->start, release_mem_region(priv->cpsw_res->start,
resource_size(priv->cpsw_res)); resource_size(priv->cpsw_res));
release_mem_region(priv->cpsw_ss_res->start, release_mem_region(priv->cpsw_wr_res->start,
resource_size(priv->cpsw_ss_res)); resource_size(priv->cpsw_wr_res));
pm_runtime_disable(&pdev->dev); pm_runtime_disable(&pdev->dev);
clk_put(priv->clk); clk_put(priv->clk);
kfree(priv->slaves); kfree(priv->slaves);
......
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