diff mbox

gpu: host1x: Store physical address to all bufs

Message ID 1421456866-31670-1-git-send-email-davidu@nvidia.com
State Deferred
Headers show

Commit Message

David Ung Jan. 17, 2015, 1:07 a.m. UTC
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(-)

Comments

Dmitry Osipenko Jan. 17, 2015, 1:43 a.m. UTC | #1
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...
David Ung Jan. 17, 2015, 6:27 a.m. UTC | #2
> -----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
diff mbox

Patch

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)