On 17.05.2018 18:34, Thierry Reding wrote:
> From: Thierry Reding <treding@xxxxxxxxxx>
>
> The number of words and the offset in a gather don't need to be
> explicitly sized, so make them unsigned int instead.
>
> Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> ---
> drivers/gpu/host1x/job.c | 13 ++++++++-----
> drivers/gpu/host1x/job.h | 4 ++--
> include/linux/host1x.h | 4 ++--
> 3 files changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c
> index 9d6d3e151291..1a8344dbdb2b 100644
> --- a/drivers/gpu/host1x/job.c
> +++ b/drivers/gpu/host1x/job.c
> @@ -96,13 +96,16 @@ void host1x_job_put(struct host1x_job *job)
> EXPORT_SYMBOL(host1x_job_put);
>
> void host1x_job_add_gather(struct host1x_job *job, struct host1x_bo *bo,
> - u32 words, u32 offset)
> + unsigned int words, unsigned int offset)
> {
> - struct host1x_job_gather *cur_gather = &job->gathers[job->num_gathers];
> + struct host1x_job_gather *gather = &job->gathers[job->num_gathers];
> +
> + WARN_ON(job->num_gathers >= job->num_cmdbufs);
> +
> + gather->words = words;
> + gather->bo = bo;
> + gather->offset = offset;
>
> - cur_gather->words = words;
> - cur_gather->bo = bo;
> - cur_gather->offset = offset;
> job->num_gathers++;
> }
> EXPORT_SYMBOL(host1x_job_add_gather);
> diff --git a/drivers/gpu/host1x/job.h b/drivers/gpu/host1x/job.h
> index 4bda51d503ec..188400e00192 100644
> --- a/drivers/gpu/host1x/job.h
> +++ b/drivers/gpu/host1x/job.h
> @@ -20,10 +20,10 @@
> #define __HOST1X_JOB_H
>
> struct host1x_job_gather {
> - u32 words;
> + unsigned int words;
> dma_addr_t base;
> struct host1x_bo *bo;
> - u32 offset;
> + unsigned int offset;
> bool handled;
> };
>
> diff --git a/include/linux/host1x.h b/include/linux/host1x.h
> index dcb6140d39d7..89110d896d72 100644
> --- a/include/linux/host1x.h
> +++ b/include/linux/host1x.h
> @@ -251,8 +251,8 @@ struct host1x_job {
>
> struct host1x_job *host1x_job_alloc(struct host1x_channel *ch,
> u32 num_cmdbufs, u32 num_relocs);
> -void host1x_job_add_gather(struct host1x_job *job, struct host1x_bo *mem_id,
> - u32 words, u32 offset);
> +void host1x_job_add_gather(struct host1x_job *job, struct host1x_bo *bo,
> + unsigned int words, unsigned int offset);
> struct host1x_job *host1x_job_get(struct host1x_job *job);
> void host1x_job_put(struct host1x_job *job);
> int host1x_job_pin(struct host1x_job *job, struct device *dev);
>
Reviewed-by: Dmitry Osipenko <digetx@xxxxxxxxx>
Tested-by: Dmitry Osipenko <digetx@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
