lkml.org 
[lkml]   [2010]   [Apr]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 06/11] drivers/edac/cpc925_edac.c: Rename dev_info to cdi
Date
There is a macro called dev_info that prints struct device specific
information. Having variables with the same name can be confusing and
prevents conversion of the macro to a function.

Rename the existing dev_info variables to something else in preparation
to converting the dev_info macro to a function.

Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/edac/cpc925_edac.c | 122 ++++++++++++++++++++++----------------------
1 files changed, 61 insertions(+), 61 deletions(-)

diff --git a/drivers/edac/cpc925_edac.c b/drivers/edac/cpc925_edac.c
index 3d50274..484e77c 100644
--- a/drivers/edac/cpc925_edac.c
+++ b/drivers/edac/cpc925_edac.c
@@ -286,8 +286,8 @@ struct cpc925_dev_info {
char *ctl_name;
int edac_idx;
struct edac_device_ctl_info *edac_dev;
- void (*init)(struct cpc925_dev_info *dev_info);
- void (*exit)(struct cpc925_dev_info *dev_info);
+ void (*init)(struct cpc925_dev_info *cdi);
+ void (*exit)(struct cpc925_dev_info *cdi);
void (*check)(struct edac_device_ctl_info *edac_dev);
};

@@ -581,19 +581,19 @@ static void cpc925_mc_check(struct mem_ctl_info *mci)

/******************** CPU err device********************************/
/* Enable CPU Errors detection */
-static void cpc925_cpu_init(struct cpc925_dev_info *dev_info)
+static void cpc925_cpu_init(struct cpc925_dev_info *cdi)
{
u32 apimask;

- apimask = __raw_readl(dev_info->vbase + REG_APIMASK_OFFSET);
+ apimask = __raw_readl(cdi->vbase + REG_APIMASK_OFFSET);
if ((apimask & CPU_MASK_ENABLE) == 0) {
apimask |= CPU_MASK_ENABLE;
- __raw_writel(apimask, dev_info->vbase + REG_APIMASK_OFFSET);
+ __raw_writel(apimask, cdi->vbase + REG_APIMASK_OFFSET);
}
}

/* Disable CPU Errors detection */
-static void cpc925_cpu_exit(struct cpc925_dev_info *dev_info)
+static void cpc925_cpu_exit(struct cpc925_dev_info *cdi)
{
/*
* WARNING:
@@ -612,16 +612,16 @@ static void cpc925_cpu_exit(struct cpc925_dev_info *dev_info)
/* Check for CPU Errors */
static void cpc925_cpu_check(struct edac_device_ctl_info *edac_dev)
{
- struct cpc925_dev_info *dev_info = edac_dev->pvt_info;
+ struct cpc925_dev_info *cdi = edac_dev->pvt_info;
u32 apiexcp;
u32 apimask;

/* APIEXCP is cleared when read */
- apiexcp = __raw_readl(dev_info->vbase + REG_APIEXCP_OFFSET);
+ apiexcp = __raw_readl(cdi->vbase + REG_APIEXCP_OFFSET);
if ((apiexcp & CPU_EXCP_DETECTED) == 0)
return;

- apimask = __raw_readl(dev_info->vbase + REG_APIMASK_OFFSET);
+ apimask = __raw_readl(cdi->vbase + REG_APIMASK_OFFSET);
cpc925_printk(KERN_INFO, "Processor Interface Fault\n"
"Processor Interface register dump:\n");
cpc925_printk(KERN_INFO, "APIMASK 0x%08x\n", apimask);
@@ -632,35 +632,35 @@ static void cpc925_cpu_check(struct edac_device_ctl_info *edac_dev)

/******************** HT Link err device****************************/
/* Enable HyperTransport Link Error detection */
-static void cpc925_htlink_init(struct cpc925_dev_info *dev_info)
+static void cpc925_htlink_init(struct cpc925_dev_info *cdi)
{
u32 ht_errctrl;

- ht_errctrl = __raw_readl(dev_info->vbase + REG_ERRCTRL_OFFSET);
+ ht_errctrl = __raw_readl(cdi->vbase + REG_ERRCTRL_OFFSET);
if ((ht_errctrl & HT_ERRCTRL_ENABLE) == 0) {
ht_errctrl |= HT_ERRCTRL_ENABLE;
- __raw_writel(ht_errctrl, dev_info->vbase + REG_ERRCTRL_OFFSET);
+ __raw_writel(ht_errctrl, cdi->vbase + REG_ERRCTRL_OFFSET);
}
}

/* Disable HyperTransport Link Error detection */
-static void cpc925_htlink_exit(struct cpc925_dev_info *dev_info)
+static void cpc925_htlink_exit(struct cpc925_dev_info *cdi)
{
u32 ht_errctrl;

- ht_errctrl = __raw_readl(dev_info->vbase + REG_ERRCTRL_OFFSET);
+ ht_errctrl = __raw_readl(cdi->vbase + REG_ERRCTRL_OFFSET);
ht_errctrl &= ~HT_ERRCTRL_ENABLE;
- __raw_writel(ht_errctrl, dev_info->vbase + REG_ERRCTRL_OFFSET);
+ __raw_writel(ht_errctrl, cdi->vbase + REG_ERRCTRL_OFFSET);
}

/* Check for HyperTransport Link errors */
static void cpc925_htlink_check(struct edac_device_ctl_info *edac_dev)
{
- struct cpc925_dev_info *dev_info = edac_dev->pvt_info;
- u32 brgctrl = __raw_readl(dev_info->vbase + REG_BRGCTRL_OFFSET);
- u32 linkctrl = __raw_readl(dev_info->vbase + REG_LINKCTRL_OFFSET);
- u32 errctrl = __raw_readl(dev_info->vbase + REG_ERRCTRL_OFFSET);
- u32 linkerr = __raw_readl(dev_info->vbase + REG_LINKERR_OFFSET);
+ struct cpc925_dev_info *cdi = edac_dev->pvt_info;
+ u32 brgctrl = __raw_readl(cdi->vbase + REG_BRGCTRL_OFFSET);
+ u32 linkctrl = __raw_readl(cdi->vbase + REG_LINKCTRL_OFFSET);
+ u32 errctrl = __raw_readl(cdi->vbase + REG_ERRCTRL_OFFSET);
+ u32 linkerr = __raw_readl(cdi->vbase + REG_LINKERR_OFFSET);

if (!((brgctrl & BRGCTRL_DETSERR) ||
(linkctrl & HT_LINKCTRL_DETECTED) ||
@@ -682,24 +682,24 @@ static void cpc925_htlink_check(struct edac_device_ctl_info *edac_dev)
/* Clear by write 1 */
if (brgctrl & BRGCTRL_DETSERR)
__raw_writel(BRGCTRL_DETSERR,
- dev_info->vbase + REG_BRGCTRL_OFFSET);
+ cdi->vbase + REG_BRGCTRL_OFFSET);

if (linkctrl & HT_LINKCTRL_DETECTED)
__raw_writel(HT_LINKCTRL_DETECTED,
- dev_info->vbase + REG_LINKCTRL_OFFSET);
+ cdi->vbase + REG_LINKCTRL_OFFSET);

/* Initiate Secondary Bus Reset to clear the chain failure */
if (errctrl & ERRCTRL_CHN_FAL)
__raw_writel(BRGCTRL_SECBUSRESET,
- dev_info->vbase + REG_BRGCTRL_OFFSET);
+ cdi->vbase + REG_BRGCTRL_OFFSET);

if (errctrl & ERRCTRL_RSP_ERR)
__raw_writel(ERRCTRL_RSP_ERR,
- dev_info->vbase + REG_ERRCTRL_OFFSET);
+ cdi->vbase + REG_ERRCTRL_OFFSET);

if (linkerr & HT_LINKERR_DETECTED)
__raw_writel(HT_LINKERR_DETECTED,
- dev_info->vbase + REG_LINKERR_OFFSET);
+ cdi->vbase + REG_LINKERR_OFFSET);

edac_device_handle_ce(edac_dev, 0, 0, edac_dev->ctl_name);
}
@@ -729,21 +729,21 @@ static struct cpc925_dev_info cpc925_devs[] = {
*/
static void cpc925_add_edac_devices(void __iomem *vbase)
{
- struct cpc925_dev_info *dev_info;
+ struct cpc925_dev_info *cdi;

if (!vbase) {
cpc925_printk(KERN_ERR, "MMIO not established yet\n");
return;
}

- for (dev_info = &cpc925_devs[0]; dev_info->init; dev_info++) {
- dev_info->vbase = vbase;
- dev_info->pdev = platform_device_register_simple(
- dev_info->ctl_name, 0, NULL, 0);
- if (IS_ERR(dev_info->pdev)) {
+ for (cdi = &cpc925_devs[0]; cdi->init; cdi++) {
+ cdi->vbase = vbase;
+ cdi->pdev = platform_device_register_simple(
+ cdi->ctl_name, 0, NULL, 0);
+ if (IS_ERR(cdi->pdev)) {
cpc925_printk(KERN_ERR,
"Can't register platform device for %s\n",
- dev_info->ctl_name);
+ cdi->ctl_name);
continue;
}

@@ -751,45 +751,45 @@ static void cpc925_add_edac_devices(void __iomem *vbase)
* Don't have to allocate private structure but
* make use of cpc925_devs[] instead.
*/
- dev_info->edac_idx = edac_device_alloc_index();
- dev_info->edac_dev =
- edac_device_alloc_ctl_info(0, dev_info->ctl_name,
- 1, NULL, 0, 0, NULL, 0, dev_info->edac_idx);
- if (!dev_info->edac_dev) {
+ cdi->edac_idx = edac_device_alloc_index();
+ cdi->edac_dev =
+ edac_device_alloc_ctl_info(0, cdi->ctl_name,
+ 1, NULL, 0, 0, NULL, 0, cdi->edac_idx);
+ if (!cdi->edac_dev) {
cpc925_printk(KERN_ERR, "No memory for edac device\n");
goto err1;
}

- dev_info->edac_dev->pvt_info = dev_info;
- dev_info->edac_dev->dev = &dev_info->pdev->dev;
- dev_info->edac_dev->ctl_name = dev_info->ctl_name;
- dev_info->edac_dev->mod_name = CPC925_EDAC_MOD_STR;
- dev_info->edac_dev->dev_name = dev_name(&dev_info->pdev->dev);
+ cdi->edac_dev->pvt_info = cdi;
+ cdi->edac_dev->dev = &cdi->pdev->dev;
+ cdi->edac_dev->ctl_name = cdi->ctl_name;
+ cdi->edac_dev->mod_name = CPC925_EDAC_MOD_STR;
+ cdi->edac_dev->dev_name = dev_name(&cdi->pdev->dev);

if (edac_op_state == EDAC_OPSTATE_POLL)
- dev_info->edac_dev->edac_check = dev_info->check;
+ cdi->edac_dev->edac_check = cdi->check;

- if (dev_info->init)
- dev_info->init(dev_info);
+ if (cdi->init)
+ cdi->init(cdi);

- if (edac_device_add_device(dev_info->edac_dev) > 0) {
+ if (edac_device_add_device(cdi->edac_dev) > 0) {
cpc925_printk(KERN_ERR,
"Unable to add edac device for %s\n",
- dev_info->ctl_name);
+ cdi->ctl_name);
goto err2;
}

debugf0("%s: Successfully added edac device for %s\n",
- __func__, dev_info->ctl_name);
+ __func__, cdi->ctl_name);

continue;

err2:
- if (dev_info->exit)
- dev_info->exit(dev_info);
- edac_device_free_ctl_info(dev_info->edac_dev);
+ if (cdi->exit)
+ cdi->exit(cdi);
+ edac_device_free_ctl_info(cdi->edac_dev);
err1:
- platform_device_unregister(dev_info->pdev);
+ platform_device_unregister(cdi->pdev);
}
}

@@ -799,20 +799,20 @@ err1:
*/
static void cpc925_del_edac_devices(void)
{
- struct cpc925_dev_info *dev_info;
+ struct cpc925_dev_info *cdi;

- for (dev_info = &cpc925_devs[0]; dev_info->init; dev_info++) {
- if (dev_info->edac_dev) {
- edac_device_del_device(dev_info->edac_dev->dev);
- edac_device_free_ctl_info(dev_info->edac_dev);
- platform_device_unregister(dev_info->pdev);
+ for (cdi = &cpc925_devs[0]; cdi->init; cdi++) {
+ if (cdi->edac_dev) {
+ edac_device_del_device(cdi->edac_dev->dev);
+ edac_device_free_ctl_info(cdi->edac_dev);
+ platform_device_unregister(cdi->pdev);
}

- if (dev_info->exit)
- dev_info->exit(dev_info);
+ if (cdi->exit)
+ cdi->exit(cdi);

debugf0("%s: Successfully deleted edac device for %s\n",
- __func__, dev_info->ctl_name);
+ __func__, cdi->ctl_name);
}
}

--
1.7.0.3.311.g6a6955


\
 
 \ /
  Last update: 2010-04-05 21:11    [W:0.149 / U:2.404 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site