• Peter Tyser's avatar
    edac: mpc85xx mask ecc syndrome correctly · 21768639
    Peter Tyser authored
    With a 64-bit wide data bus only the lowest 8-bits of the ECC syndrome are
    relevant.  With a 32-bit wide data bus only the lowest 16-bits are
    relevant on most architectures.
    
    Without this change, the ECC syndrome displayed can be mildly confusing,
    eg:
    
      EDAC MPC85xx MC1: syndrome: 0x25252525
    
    When in reality the ECC syndrome is 0x25.
    
    A variety of Freescale manuals say a variety of different things about how
    to decode the CAPTURE_ECC (syndrome) register.  I don't have a system with
    a 32-bit bus to test on, but I believe the change is correct.  It'd be
    good to get an ACK from someone at Freescale about this change though.
    Signed-off-by: default avatarPeter Tyser <ptyser@xes-inc.com>
    Signed-off-by: default avatarDoug Thompson <dougthompson@xmission.com>
    Cc: Kumar Gala <galak@gate.crashing.org>
    Cc: Dave Jiang <djiang@mvista.com>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
    21768639
mpc85xx_edac.h 4.59 KB