Re: [PATCH v3 11/19] crypto: cmh - add DRBG hwrng

Krzysztof Kozlowski <[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 <[email protected]>
On 06/08/2026 21:55, Saravanakrishnan Krishnamoorthy wrote:
> + * cmh_rng_unregister() - Unregister the CMH hardware RNG device
> + *
> + * Unregisters the hwrng device from the kernel hwrng framework if it
> + * was previously registered.
> + */
> +void cmh_rng_unregister(void)
> +{
> +	if (!cmh_rng_registered)
> +		return;
> +	hwrng_unregister(&cmh_hwrng);
> +	cmh_rng_registered = false;
> +	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.

Best regards,
Krzysztof
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.