Re: [PATCH 1/1] edac x38: new MC driver module
From: Andrew Morton
Date: Mon Oct 20 2008 - 19:32:46 EST
On Fri, 17 Oct 2008 15:39:45 -0600
> From: Hitoshi Mitake <mitake@xxxxxxxxxxxx>
> I wrote a new module for Intel X38 chipset.
> This chipset is very similar to Intel 3200 chipset,
> but there are some different points,
> so I copyed i3200_edac.c and modified.
> This is a Intel's web page describing this chipset.
> I've tested this new module with broken memory,
> and it seems working well.
> This is a patch, please use.
> Signed-off-by: Hitoshi Mitake <mitake@xxxxxxxxxxxx>
> Signed-off-by: Doug Thompson <dougthompson@xxxxxxxxxxxx>
> Index: linux-2.6.27/drivers/edac/Kconfig
> --- linux-2.6.27.orig/drivers/edac/Kconfig
> +++ linux-2.6.27/drivers/edac/Kconfig
> @@ -102,6 +102,13 @@ config EDAC_I3000
> Support for error detection and correction on the Intel
> 3000 and 3010 server chipsets.
> +config EDAC_X38
> + tristate "Intel X38"
> + depends on EDAC_MM_EDAC && PCI && X86
> + help
> + Support for error detection and correction on the Intel
> + X38 server chipsets.
Is this truly X86, or will this driver only ever be used on x86_64 kernels?
> +static u64 x38_readq(const void __iomem *addr)
> + return readl(addr) | (((u64)readl(addr + 4)) << 32);
Because the x86_64 architecture already implements readq(), and it
would be nice to avoid creating a private version.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/