Messages in this thread | | | Date | Mon, 09 Nov 2020 22:47:27 +0000 | From | Paul Cercueil <> | Subject | Re: [PATCH 1/2] remoteproc: ingenic: Constify ingenic_rproc_ops |
| |
Hi,
Le dim. 8 nov. 2020 à 0:36, Rikard Falkeborn <rikard.falkeborn@gmail.com> a écrit : > The only usage of ingenic_rproc_ops is to pass its address to > devm_rproc_alloc(), which accepts a const pointer. Make it const to > allow the compiler to put it in read-only memory. > > Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Acked-by: Paul Cercueil <paul@crapouillou.net>
Cheers, -Paul
> --- > drivers/remoteproc/ingenic_rproc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/remoteproc/ingenic_rproc.c > b/drivers/remoteproc/ingenic_rproc.c > index 1c2b21a5d178..26e19e6143b7 100644 > --- a/drivers/remoteproc/ingenic_rproc.c > +++ b/drivers/remoteproc/ingenic_rproc.c > @@ -135,7 +135,7 @@ static void *ingenic_rproc_da_to_va(struct rproc > *rproc, u64 da, size_t len) > return (__force void *)va; > } > > -static struct rproc_ops ingenic_rproc_ops = { > +static const struct rproc_ops ingenic_rproc_ops = { > .prepare = ingenic_rproc_prepare, > .unprepare = ingenic_rproc_unprepare, > .start = ingenic_rproc_start, > -- > 2.29.2 >
| |