[PATCH] gpu: host1x: Store physical address to all bufs

[PATCH] gpu: host1x: Store physical address to all bufs

[David Ung]

From: Arto Merilainen <amerilainen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Signed-off-by: Arto Merilainen <amerilainen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
 drivers/gpu/host1x/job.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c
index 63bd63f..7f3cfff 100644
--- a/drivers/gpu/host1x/job.c
+++ b/drivers/gpu/host1x/job.c
@@ -538,9 +538,12 @@ int host1x_job_pin(struct host1x_job *job, struct device *dev)
 
 		g->base = job->gather_addr_phys[i];
 
-		for (j = i + 1; j < job->num_gathers; j++)
-			if (job->gathers[j].bo == g->bo)
+		for (j = i + 1; j < job->num_gathers; j++) {
+			if (job->gathers[j].bo == g->bo) {
 				job->gathers[j].handled = true;
+				job->gathers[j].base = g->base;
+			}
+		}
 
 		err = do_relocs(job, g->bo);
 		if (err)
-- 
1.8.1.5

Re: [PATCH] gpu: host1x: Store physical address to all bufs

[Dmitry Osipenko]

17.01.2015 04:07, David Ung пишет:
> From: Arto Merilainen <amerilainen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>
> Signed-off-by: Arto Merilainen <amerilainen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>   drivers/gpu/host1x/job.c | 7 +++++--
>   1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c
> index 63bd63f..7f3cfff 100644
> --- a/drivers/gpu/host1x/job.c
> +++ b/drivers/gpu/host1x/job.c
> @@ -538,9 +538,12 @@ int host1x_job_pin(struct host1x_job *job, struct device *dev)
>
>   		g->base = job->gather_addr_phys[i];
>
> -		for (j = i + 1; j < job->num_gathers; j++)
> -			if (job->gathers[j].bo == g->bo)
> +		for (j = i + 1; j < job->num_gathers; j++) {
> +			if (job->gathers[j].bo == g->bo) {
>   				job->gathers[j].handled = true;
> +				job->gathers[j].base = g->base;
> +			}
> +		}
>
>   		err = do_relocs(job, g->bo);
>   		if (err)
>
No commit description? Is it bugfix or optimization...

-- 
Dmitry

RE: [PATCH] gpu: host1x: Store physical address to all bufs

[David Ung]

> -----Original Message-----
> From: Dmitry Osipenko [mailto:digetx@gmail.com]
> Sent: Friday, January 16, 2015 5:43 PM
> To: David Ung; airlied@linux.ie
> Cc: linux-tegra@vger.kernel.org; thierry.reding@gmail.com; Terje
> Bergstrom; swarren@wwwdotorg.org; Arto Merilainen
> Subject: Re: [PATCH] gpu: host1x: Store physical address to all bufs
> 
> 17.01.2015 04:07, David Ung пишет:
> > From: Arto Merilainen <amerilainen@nvidia.com>
> >
> > Signed-off-by: Arto Merilainen <amerilainen@nvidia.com>
> > ---
> >   drivers/gpu/host1x/job.c | 7 +++++--
> >   1 file changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c index
> > 63bd63f..7f3cfff 100644
> > --- a/drivers/gpu/host1x/job.c
> > +++ b/drivers/gpu/host1x/job.c
> > @@ -538,9 +538,12 @@ int host1x_job_pin(struct host1x_job *job, struct
> > device *dev)
> >
> >   		g->base = job->gather_addr_phys[i];
> >
> > -		for (j = i + 1; j < job->num_gathers; j++)
> > -			if (job->gathers[j].bo == g->bo)
> > +		for (j = i + 1; j < job->num_gathers; j++) {
> > +			if (job->gathers[j].bo == g->bo) {
> >   				job->gathers[j].handled = true;
> > +				job->gathers[j].base = g->base;
> > +			}
> > +		}
> >
> >   		err = do_relocs(job, g->bo);
> >   		if (err)
> >
> No commit description? Is it bugfix or optimization...

Sorry this is actually a part of the series of patches that adds other host1x clients.  So its more like a features patch.
I shall post this as part of a series, so please ignore this for now.

David