Re: [PATCH v3 1/3] drivers: rng: meson: add support for S4

From: Herbert Xu
Date: Wed Sep 20 2023 - 00:56:25 EST


On Mon, Sep 11, 2023 at 01:11:27PM +0300, Alexey Romanov wrote:
>
> +struct meson_rng_priv {
> + int (*read)(struct hwrng *rng, void *buf, size_t max, bool wait);
> +};
>
> struct meson_rng_data {
> void __iomem *base;
> struct hwrng rng;
> + struct device *dev;
> + const struct meson_rng_priv *priv;
> };

Is data->priv really necessary? It doesn't seem to be used anywhere
after the probe.

Thanks,
--
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt