Messages in this thread | | | From | "Kani, Toshi" <> | Subject | RE: [PATCH v4 6/8] EDAC: Add the ghes_edac_preferred check for chipset-specific edac drivers | Date | Fri, 2 Sep 2022 15:48:40 +0000 |
| |
On Wednesday, August 31, 2022 1:40 AM, Jia He wrote: > edac_mc_add_mc* is too late in the init path and that check should happen > as the very first thing in the driver init function. > Add the newly introduced helper ghes_edac_preferred() for the platform > check.
I'd suggest to rephase a bit, something like: == Add ghes_edac_preferred() check to chipset-specific edac drivers to ensure that ghes_edac is used on the platform where ghes_edac is preferred over chipset-specific edac driver.
Unlike the existing edac_get_owner() check, the ghes_edac_preferred() check works independent to the module_init ordering. ==
Otherwise, the patch looks good to me. Reviewed-by: Toshi Kani <toshi.kani@hpe.com>
Toshi
| |