[AMD Official Use Only - Internal Distribution Only]
Please fix your coding style in the follow if/else code segment by removing the
unnecessary {}. Other than that, the patch is
Reviewed-by: Hawking Zhang <[email protected]>
+ if (strnstr(atom_ctx->vbios_version,
+ "D342",
+ sizeof(atom_ctx->vbios_version)))
+ {
+ *i2c_addr = EEPROM_I2C_TARGET_ADDR_ARCTURUS_D342;
+ }
+ else
+ {
+ *i2c_addr = EEPROM_I2C_TARGET_ADDR_ARCTURUS;
+ }
Regards,
Hawking
From: Clements, John <[email protected]>
Sent: Tuesday, February 25, 2020 17:58
To: [email protected]; Zhang, Hawking <[email protected]>
Subject: [PATCH] drm/amdgpu: Add Arcturus D342 page retire support
Add support for detecting Arcturus SKU to resolve the correct bad page
retirement EEPROM I2C address.
Thank you,
John Clements
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
