Re: [v1,1/1] EDAC: (pnd2) Fix the log level for message "Failed to register device with error %d"

From: Joe Perches
Date: Thu Mar 19 2020 - 21:56:25 EST


On Tue, 2020-03-17 at 06:28 +0000, Amy.Shih@xxxxxxxxxxxxxxxx wrote:
> From: Amy Shih <amy.shih@xxxxxxxxxxxxxxxx>
>
> Fix the log level from "KERN_ERR" to "KERN_INFO" for message "Failed to
> register device with error %d", since it is not the error message but
> the information to notice the user.
[]
> diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c
[]
> @@ -1572,7 +1572,8 @@ static int __init pnd2_init(void)
>
> rc = pnd2_probe();
> if (rc < 0) {
> - pnd2_printk(KERN_ERR, "Failed to register device with error %d.\n", rc);
> + pnd2_printk(KERN_INFO,
> + "Failed to register device with error %d.\n", rc);

So there's no need to use the word "error"
in the output message either.

"Failed to register device - %d\n", rc

would be enough.