[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 [email protected]<mailto:[email protected]>

Regards,
Guchun

From: Clements, John <[email protected]>
Sent: Monday, August 3, 2020 2:48 PM
To: amd-gfx list <[email protected]>; Chen, Guchun 
<[email protected]>; Zhang, Hawking <[email protected]>
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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to