diff mbox series

EDAC/armada-xp: Fix output of uncorrectable error counter

Message ID 20211006121332.58788-1-hans.potsch@nokia.com (mailing list archive)
State New, archived
Headers show
Series EDAC/armada-xp: Fix output of uncorrectable error counter | expand

Commit Message

Hans Potsch Oct. 6, 2021, 12:13 p.m. UTC
Incorrect parameter is passed to the edac_mc_handle_error() function.
Therefore number of correctable errors is displayed as uncorrectable
errors. Changed to correct parameter.

Signed-off-by: Hans Potsch <hans.potsch@nokia.com>
---
 drivers/edac/armada_xp_edac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Borislav Petkov Oct. 14, 2021, 9:56 a.m. UTC | #1
On Wed, Oct 06, 2021 at 02:13:32PM +0200, Hans Potsch wrote:
> Incorrect parameter is passed to the edac_mc_handle_error() function.
> Therefore number of correctable errors is displayed as uncorrectable
> errors. Changed to correct parameter.
> 
> Signed-off-by: Hans Potsch <hans.potsch@nokia.com>
> ---
>  drivers/edac/armada_xp_edac.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/edac/armada_xp_edac.c b/drivers/edac/armada_xp_edac.c
> index e3e757513d1b..b1f46a974b9e 100644
> --- a/drivers/edac/armada_xp_edac.c
> +++ b/drivers/edac/armada_xp_edac.c
> @@ -178,7 +178,7 @@ static void axp_mc_check(struct mem_ctl_info *mci)
>  				     "details unavailable (multiple errors)");
>  	if (cnt_dbe)
>  		edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci,
> -				     cnt_sbe, /* error count */
> +				     cnt_dbe, /* error count */
>  				     0, 0, 0, /* pfn, offset, syndrome */
>  				     -1, -1, -1, /* top, mid, low layer */
>  				     mci->ctl_name,
> -- 

Applied, thanks.
diff mbox series

Patch

diff --git a/drivers/edac/armada_xp_edac.c b/drivers/edac/armada_xp_edac.c
index e3e757513d1b..b1f46a974b9e 100644
--- a/drivers/edac/armada_xp_edac.c
+++ b/drivers/edac/armada_xp_edac.c
@@ -178,7 +178,7 @@  static void axp_mc_check(struct mem_ctl_info *mci)
 				     "details unavailable (multiple errors)");
 	if (cnt_dbe)
 		edac_mc_handle_error(HW_EVENT_ERR_UNCORRECTED, mci,
-				     cnt_sbe, /* error count */
+				     cnt_dbe, /* error count */
 				     0, 0, 0, /* pfn, offset, syndrome */
 				     -1, -1, -1, /* top, mid, low layer */
 				     mci->ctl_name,