[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.c...@amd.com<mailto:guchun.c...@amd.com>

Regards,
Guchun

From: Clements, John <john.cleme...@amd.com>
Sent: Monday, August 3, 2020 2:48 PM
To: amd-gfx list <amd-gfx@lists.freedesktop.org>; Chen, Guchun 
<guchun.c...@amd.com>; Zhang, Hawking <hawking.zh...@amd.com>
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@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to