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