Re: [PATCH 6/7] edac: mpc85xx fsb ecc

[Peter Tyser]

Hi Doug and Konstantin,

On Wed, 2010-02-03 at 23:46 -0700, dougthompson@xxxxxxxxxxxx wrote:
> From: Konstantin Olifer <kolifer@xxxxxxxxx>
> 
> Patch is for Kernel 2.6.33. e752x_edac suppose that fsb ECC exist, for example, on
> Celeron + 3100. That is wrong. 
> ECC FSB supported only on Xeon for our vision (www.setdsp.ru) and datasheet from developer intel www. 
> Fixed.
> Signed-off-by:  Konstantin Olifer <kolifer@xxxxxxxxx>
> Signed-off-by:  Doug Thompson <dougthompson@xxxxxxxxxxxx>
> ---
>  e752x_edac.c |   10 ++--------
>  1 file changed, 2 insertions(+), 8 deletions(-)

It looks like 6/7 and 7/7 in this series have the wrong subject/title.
Patches 6 and 7 are for the e752x instead of the mpc85xx.

When changing the subjects, it might be nice to change this commit
message to something more straightforward such as:
"FSB parity is only supported on the Xeon processor.  Previously it was
incorrectly enabled for the Celeron as well."

Best,
Peter

--
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/