Re: [PATCH v2 1/2] drivers: caam: add RNG software self-test

[Sascha Hauer <s.hauer@pengutronix.de>]

On Mon, Dec 03, 2018 at 11:52:26AM +0100, Roland Hieber wrote:
> Hi Sascha,
> 
> > +	pr_notice("RNG software self-test passed\n");
> 
> I noticed that you downgraded this to a pr_info() when applying the
> patch, is there a reason?  pr_warn()s have the "WARNING:" prefix,
> pr_notice()s have a "NOTICE:" prefix, but pr_info()s don't have any
> prefix. I think this one should be a pr_notice because it overrides the
> pr_warn("RNG self-test failure detected...") from HAB and tells the user
> that this warning is no longer relevant. With pr_info(), I think it will
> get lost between all the other infos.

I can follow this argumentation, but then I argue we should lower the
priority of the failure detected message to to pr_info or even pr_debug,
because the failure is pretty much expected, it is correctly being
worked around and there's nothing we can do about the failure.

People tend to bother about warnings and ask questions what is wrong.
When everything you can do is to tell that everything is fine then probably
the warning shouldn't be there in the first place.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox