Re: [PATCH] edac: mpc85xx - Fix building as a module

From: Dave Jiang
Date: Thu May 22 2008 - 12:13:31 EST

Acked-by: Dave Jiang <djiang@xxxxxxxxxx>

Kumar Gala wrote:
including of <asm/mpc85xx.h> causes build problems since it doesn't exist.

Also removed warning:
drivers/edac/mpc85xx_edac.c:45: warning: 'mpc85xx_ctl_name' defined but not used

Signed-off-by: Kumar Gala <galak@xxxxxxxxxxxxxxxxxxx>

This is a fix for 2.6.26.

- k

drivers/edac/mpc85xx_edac.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/drivers/edac/mpc85xx_edac.c b/drivers/edac/mpc85xx_edac.c
index 065732d..d49361b 100644
--- a/drivers/edac/mpc85xx_edac.c
+++ b/drivers/edac/mpc85xx_edac.c
@@ -20,7 +20,6 @@

#include <linux/of_platform.h>
#include <linux/of_device.h>
-#include <asm/mpc85xx.h>
#include "edac_module.h"
#include "edac_core.h"
#include "mpc85xx_edac.h"
@@ -43,8 +42,6 @@ static u32 orig_pci_err_en;
static u32 orig_l2_err_disable;
static u32 orig_hid1;

-static const char *mpc85xx_ctl_name = "MPC85xx";
/************************ MC SYSFS parts ***********************************/

static ssize_t mpc85xx_mc_inject_data_hi_show(struct mem_ctl_info *mci,


Dave Jiang
Software Engineer
MontaVista Software, Inc.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at