gpu: host1x: Cleanup loop variable usage

Use unsigned int where possible and don't unnecessarily initialize the
loop variable.

Reviewed-by: Dmitry Osipenko <digetx@gmail.com>
Tested-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Thierry Reding <treding@nvidia.com>

diff --git a/drivers/gpu/host1x/debug.c b/drivers/gpu/host1x/debug.c
index dc77ec452ffc6f796b1984b61f5eaf1c05283193..329e4a3d8ae7b3b9444e0accd135a94179ed9fd8 100644 (file)
--- a/drivers/gpu/host1x/debug.c
+++ b/drivers/gpu/host1x/debug.c
@@ -103,7 +103,7 @@ static void show_syncpts(struct host1x *m, struct output *o)
 
 static void show_all(struct host1x *m, struct output *o, bool show_fifo)
 {
-       int i;
+       unsigned int i;
 
        host1x_hw_show_mlocks(m, o);
        show_syncpts(m, o);

diff --git a/drivers/gpu/host1x/intr.c b/drivers/gpu/host1x/intr.c
index 8b4fad0ab35d24de11380ec00f7e8e6802db5c3a..6028cf7b681f19eb3061cb360204fb971a492848 100644 (file)
--- a/drivers/gpu/host1x/intr.c
+++ b/drivers/gpu/host1x/intr.c
@@ -144,7 +144,7 @@ static const action_handler action_handlers[HOST1X_INTR_ACTION_COUNT] = {
 static void run_handlers(struct list_head completed[HOST1X_INTR_ACTION_COUNT])
 {
        struct list_head *head = completed;
-       int i;
+       unsigned int i;
 
        for (i = 0; i < HOST1X_INTR_ACTION_COUNT; ++i, ++head) {
                action_handler handler = action_handlers[i];

diff --git a/drivers/gpu/host1x/job.c b/drivers/gpu/host1x/job.c
index 3cbfc6e37668372c76f86ce31be0b9836a566006..2be0bcaf828834b6d4967d6c584cb8efce09f40c 100644 (file)
--- a/drivers/gpu/host1x/job.c
+++ b/drivers/gpu/host1x/job.c
@@ -196,10 +196,10 @@ unpin:
 
 static int do_relocs(struct host1x_job *job, struct host1x_job_gather *g)
 {
-       int i = 0;
        u32 last_page = ~0;
        void *cmdbuf_page_addr = NULL;
        struct host1x_bo *cmdbuf = g->bo;
+       unsigned int i;
 
        /* pin & patch the relocs for one gather */
        for (i = 0; i < job->num_relocs; i++) {
@@ -451,7 +451,7 @@ static inline int copy_gathers(struct host1x_job *job, struct device *dev)
        struct host1x_firewall fw;
        size_t size = 0;
        size_t offset = 0;
-       int i;
+       unsigned int i;
 
        fw.job = job;
        fw.dev = dev;

diff --git a/drivers/gpu/host1x/syncpt.c b/drivers/gpu/host1x/syncpt.c
index a108669188e88ab29ee642d8308c2fd6816a452e..088c05dd884c334b439b61a71af3576f69b244a0 100644 (file)
--- a/drivers/gpu/host1x/syncpt.c
+++ b/drivers/gpu/host1x/syncpt.c
@@ -57,8 +57,8 @@ static struct host1x_syncpt *host1x_syncpt_alloc(struct host1x *host,
                                                 struct host1x_client *client,
                                                 unsigned long flags)
 {
-       int i;
        struct host1x_syncpt *sp = host->syncpt;
+       unsigned int i;
        char *name;
 
        mutex_lock(&host->syncpt_mutex);