RE: [PATCH] drm/amdgpu: added RAS EEPROM device support check

["Chen, Guchun" <Guchun.Chen@xxxxxxx>]

[AMD Public Use]

 

Please put the check __is_ras_eeprom_supported behind the line ‘*exceed_err_limit = false;’ in function amdgpu_ras_eeprom_init and amdgpu_ras_eeprom_check_err_threshold respectively. That promises, even when eeprom is not available on several ASICs, exceed_err_limit must set to be false.

 

With above fixed, the patch is:

Reviewed-by: Guchun Chen guchun.chen@xxxxxxx

 

Regards,

Guchun

 

From: Clements, John <John.Clements@xxxxxxx>
Sent: Monday, August 3, 2020 2:48 PM
To: amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>; Chen, Guchun <Guchun.Chen@xxxxxxx>; Zhang, Hawking <Hawking.Zhang@xxxxxxx>
Subject: [PATCH] drm/amdgpu: added RAS EEPROM device support check

 

[AMD Public Use]

 

Submitting patch to with added device support check before trying to access RAS EEPROM.

 

Thank you,

John Clements

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx