RE: [PATCH v3 11/19] crypto: cmh - add DRBG hwrng
"Ousherovitch, Alex" <[email protected]>
| Newsgroups | org.kernel.vger.linux-api,org.infradead.lists.linux-riscv,org.kernel.vger.linux-crypto,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel,org.kernel.vger.linux-kselftest |
|---|---|
| Message-ID | <SA3PR04MB9001AC1DB7241932BA70FFA5D7DD2@SA3PR04MB9001.namprd04.prod.outlook.com> |
On Tue, Aug 11, 2026 at 10:38:13AM +0200, Krzysztof Kozlowski <[email protected]> wrote: > > + dev_info(cmh_dev(), "rng: unregistered rambus-cmh-drbg\n"); > > So every function has dev_info? > > This is one of its kind driver, because I never saw so much of spamming > dmesg. It should raise eyebrows of your internal reviewers, therefore I > insist on performing reviews on the mailing list if you want them to be > tracked. We will remove the dev_info() logging across the driver in v4 (the same cleanup as on patch 02), keeping only real error paths. Understood on on-list reviews. Regards, Alex