EDAC, skx_edac: Fix logical channel intermediate decoding bug

[Borislav Petkov <bp@alien8.de>]

On Tue, Oct 09, 2018 at 10:20:25AM -0700, Tony Luck wrote:
> From: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
> 
> The code "lchan = (lchan << 1) | ~lchan" for logical channel
> intermediate decoding is wrong. The wrong intermediate decoding
> result is {0xffffffff, 0xfffffffe}.
> 
> Fix it by replacing '~' with '!'. The correct intermediate
> decoding result is {0x1, 0x2}.
> 
> Cc: stable@vger.kernel.org
> Signed-off-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
> Signed-off-by: Tony Luck <tony.luck@intel.com>
> ---
>  drivers/edac/skx_edac.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/edac/skx_edac.c b/drivers/edac/skx_edac.c
> index fae095162c01..45c5a8931a58 100644
> --- a/drivers/edac/skx_edac.c
> +++ b/drivers/edac/skx_edac.c
> @@ -668,7 +668,7 @@ static bool skx_sad_decode(struct decoded_addr *res)
>  			break;
>  		case 2:
>  			lchan = (addr >> shift) % 2;
> -			lchan = (lchan << 1) | ~lchan;
> +			lchan = (lchan << 1) | !lchan;
>  			break;
>  		case 3:
>  			lchan = ((addr >> shift) % 2) << 1;
> -- 

Applied, thanks.