Hi Kani
As per your review for previous patches, I will update the commit log in the next version.
Please see my comments below:
> -----Original Message-----
> > @@ -454,7 +437,7 @@ int ghes_edac_register(struct ghes *ghes, struct
> > device *dev)
> > pr_info("This system has a very crappy BIOS: It doesn't even list
> > the DIMMS.\n");
> > pr_info("Its SMBIOS info is wrong. It is doubtful that the error
> > report would\n");
> > pr_info("work on such system. Use this driver with caution\n");
> > - } else if (idx < 0) {
> > + } else if (ghes_edac_force_load) {
>
> This change causes the following messages to start showing up on Arm.
> Is that what you intend to do?
>
No
> The messages can be avoided by not setting the force flag on Arm
> unconditionally.
> This will need some change to the flag check in ghes_edac_unregister() though.
Yes, looks like I can remove the ghes_edac_force_load check in the ghes_edac_unregister()
since it had been checked in the ghes_edac_init
