Messages in this thread | | | Date | Fri, 8 Nov 2019 17:21:05 -0800 | From | Bjorn Andersson <> | Subject | Re: [PATCH] remoteproc: fix argument 2 of rproc_mem_entry_init |
| |
On Thu 17 Oct 04:59 PDT 2019, Ben Dooks (Codethink) wrote:
> The rproc_mem_entry_init() call takes a pointer to a vm > as the second argument. The code is currently using a > plain 0 as "NULL". Change to using NULL to fix the > following sparse warnings: > > drivers/remoteproc/remoteproc_core.c:339:49: warning: Using plain integer as NULL pointer > drivers/remoteproc/remoteproc_core.c:916:46: warning: Using plain integer as NULL pointer >
Applied, thanks
> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > --- > Cc: Ohad Ben-Cohen <ohad@wizery.com> > Cc: Bjorn Andersson <bjorn.andersson@linaro.org> > Cc: linux-remoteproc@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > drivers/remoteproc/remoteproc_core.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c > index 3c5fbbbfb0f1..943af836fa0f 100644 > --- a/drivers/remoteproc/remoteproc_core.c > +++ b/drivers/remoteproc/remoteproc_core.c > @@ -336,7 +336,8 @@ int rproc_alloc_vring(struct rproc_vdev *rvdev, int i) > return -ENOMEM; > } else { > /* Register carveout in in list */ > - mem = rproc_mem_entry_init(dev, 0, 0, size, rsc->vring[i].da, > + mem = rproc_mem_entry_init(dev, NULL, 0, > + size, rsc->vring[i].da, > rproc_alloc_carveout, > rproc_release_carveout, > "vdev%dvring%d", > @@ -913,7 +914,7 @@ static int rproc_handle_carveout(struct rproc *rproc, > } > > /* Register carveout in in list */ > - carveout = rproc_mem_entry_init(dev, 0, 0, rsc->len, rsc->da, > + carveout = rproc_mem_entry_init(dev, NULL, 0, rsc->len, rsc->da, > rproc_alloc_carveout, > rproc_release_carveout, rsc->name); > if (!carveout) { > -- > 2.23.0 >
| |