[AMD Public Use] - if (adev->asic_type != CHIP_VEGA20 && adev->asic_type != CHIP_ARCTURUS) + if (adev->asic_type != CHIP_VEGA20 && + adev->asic_type != CHIP_ARCTURUS && + adev->asic_type != CHIP_SIENNA_CICHLID) return 0;
Does it make sense to check UMC RAS availability through amdgpu_ras_is_supported, instead of check specific ASIC type one by one? Also, it would be good to merge the upcoming logic from Guchun where we have a gloal flag to mark the availability of bad page retirement. Thoughts? Regards, Hawking From: Clements, John <john.cleme...@amd.com> Sent: Tuesday, July 21, 2020 18:02 To: amd-gfx list <amd-gfx@lists.freedesktop.org>; Zhang, Hawking <hawking.zh...@amd.com> Subject: [PATCH] drm/amdgpu: add RAS EEPROM support for sienna chichlid [AMD Public Use] Submitting patch to enable RAS EEPROM support for sienna chichlid
_______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx