[AMD Official Use Only - Internal Distribution Only]

Hi John,

How about adding the shell function at the beginning of 
amdgpu_fru_get_product_info. It make sense to me to check the FRU eeprom 
availability before access. But having the check in amdgpu_fru_read_eeprom 
would results to unnecessary drm error message for all the following access in 
amdgpu_fru_get_product_info.

Regards,
Hawking
From: Clements, John <john.cleme...@amd.com>
Sent: Monday, March 23, 2020 16:51
To: amd-gfx@lists.freedesktop.org; Zhang, Hawking <hawking.zh...@amd.com>
Subject: [PATCH] drm/amdgpu: disabled fru eeprom access


[AMD Official Use Only - Internal Distribution Only]

Submitting patch to:

  *   Disable FRU EEPROM access by default
  *   Add shell function to be used for supported device detection

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