video: tegra: bitness sanitary cleanup
Peng Du [Fri, 1 Feb 2013 22:41:16 +0000 (14:41 -0800)]
* replace (u32) with (uintptr_t) which is portable;
* use portable print format strings, eg. %lx, %zd;

Change-Id: If9916436b2ee954f0e1487f9b09b9984b97cf4fe
Signed-off-by: Peng Du <pdu@nvidia.com>
Reviewed-on: http://git-master/r/196623
Reviewed-by: Alexander Van Brunt <avanbrunt@nvidia.com>

14 files changed:
drivers/video/tegra/dc/dsi.c
drivers/video/tegra/host/dmabuf.c
drivers/video/tegra/host/gk20a/channel_gk20a.c
drivers/video/tegra/host/gk20a/mm_gk20a.c
drivers/video/tegra/host/host1x/host1x.c
drivers/video/tegra/host/host1x/host1x_cdma.c
drivers/video/tegra/host/nvhost_cdma.c
drivers/video/tegra/host/nvhost_memmgr.c
drivers/video/tegra/host/nvhost_syncpt.h
drivers/video/tegra/host/nvmap.c
drivers/video/tegra/host/t124/t124.c
drivers/video/tegra/nvmap/nvmap_dev.c
drivers/video/tegra/nvmap/nvmap_heap.c
drivers/video/tegra/nvmap/nvmap_heap.h

index d3ed451..907e317 100644 (file)
@@ -470,7 +470,7 @@ static int __maybe_unused tegra_dsi_syncpt(struct tegra_dc_dsi_data *dsi)
        tegra_dsi_writel(dsi, val, DSI_INCR_SYNCPT);
 
        ret = nvhost_syncpt_wait_timeout_ext(dsi->dc->ndev, dsi->syncpt_id,
-               dsi->syncpt_val + 1, MAX_SCHEDULE_TIMEOUT, NULL, NULL);
+               dsi->syncpt_val + 1, (u32)MAX_SCHEDULE_TIMEOUT, NULL, NULL);
        if (ret < 0) {
                dev_err(&dsi->dc->ndev->dev, "DSI sync point failure\n");
                goto fail;
index b02d48f..0bf1558 100644 (file)
@@ -25,7 +25,7 @@
 
 static inline struct dma_buf_attachment *to_dmabuf_att(struct mem_handle *h)
 {
-       return (struct dma_buf_attachment *)(((u32)h) & ~0x3);
+       return (struct dma_buf_attachment *)(((uintptr_t)h) & ~0x3);
 }
 
 static inline struct dma_buf *to_dmabuf(struct mem_handle *h)
@@ -100,7 +100,7 @@ struct mem_handle *nvhost_dmabuf_get(u32 id, struct platform_device *dev)
                }
        }
 
-       return (struct mem_handle *) ((u32)h | mem_mgr_type_dmabuf);
+       return (struct mem_handle *) ((uintptr_t)h | mem_mgr_type_dmabuf);
 }
 
 int nvhost_dmabuf_get_param(struct mem_mgr *memmgr, struct mem_handle *handle,
index c27c2bf..00e3795 100644 (file)
@@ -1112,7 +1112,7 @@ int gk20a_channel_wait(struct channel_gk20a *ch,
        u64 jiffies;
        u32 id;
        u32 offset;
-       u32 timeout;
+       long timeout;
        int remain, ret = 0;
 
        if (args->timeout == NVHOST_NO_TIMEOUT)
@@ -1138,7 +1138,7 @@ int gk20a_channel_wait(struct channel_gk20a *ch,
                        return -ENOMEM;
                }
 
-               notif = (struct notification *)((u32)notif + offset);
+               notif = (struct notification *)((uintptr_t)notif + offset);
 
                /* user should set status pending before
                 * calling this ioctl */
index 8d3c8b5..3715e4e 100644 (file)
@@ -862,7 +862,6 @@ static u64 gk20a_vm_map(struct vm_gk20a *vm,
        nvhost_dbg_info("comptag size=%d start=%d for 0x%llx",
                        bfr.ctag_lines, bfr.ctag_offset, (u64)bfr.addr);
 
-
        /* keep track of the buffer for unmapping */
        /* TBD: check for multiple mapping of same buffer */
        mapped_buffer = kzalloc(sizeof(*mapped_buffer), GFP_KERNEL);
@@ -1223,7 +1222,6 @@ static int gk20a_as_alloc_share(struct nvhost_as_share *as_share)
        }
        nvhost_dbg(dbg_pte, "pdes.kv = 0x%p, pdes.phys = 0x%llx",
                        vm->pdes.kv, (u64)vm->pdes.phys);
-
        /* we could release vm->pdes.kv but it's only one page... */
 
        /* alloc in 4K granularity */
@@ -1544,7 +1542,7 @@ int gk20a_init_pmu_vm(struct mm_gk20a *mm)
        /* IS_ERR throws a warning here (expecting void *) */
        if (IS_ERR_OR_NULL(inst_block->mem.sgt)) {
                inst_pa = 0;
-               err = (int)inst_block->mem.sgt;
+               err = (int)((uintptr_t)inst_block->mem.sgt);
                goto clean_up;
        }
        inst_block->cpu_pa = inst_pa =
index 2d427c7..63c51c4 100644 (file)
@@ -135,7 +135,10 @@ static int nvhost_ioctl_ctrl_syncpt_waitex(struct nvhost_ctrl_userctx *ctx,
        if (args->id >= nvhost_syncpt_nb_pts(&ctx->dev->syncpt))
                return -EINVAL;
        if (args->timeout == NVHOST_NO_TIMEOUT)
-               timeout = MAX_SCHEDULE_TIMEOUT;
+               /* FIXME: MAX_SCHEDULE_TIMEOUT is ulong which can be bigger
+                   than u32 so we should fix nvhost_syncpt_wait_timeout to
+                   take ulong not u32. */
+               timeout = (u32)MAX_SCHEDULE_TIMEOUT;
        else
                timeout = (u32)msecs_to_jiffies(args->timeout);
 
index 67cb2c5..cee45b6 100644 (file)
@@ -153,7 +153,7 @@ static void push_buffer_push_to(struct push_buffer *pb,
                u32 op1, u32 op2)
 {
        u32 cur = pb->cur;
-       u32 *p = (u32 *)((u32)pb->mapped + cur);
+       u32 *p = (u32 *)((uintptr_t)pb->mapped + cur);
        u32 cur_nvmap = (cur/8) & (NVHOST_GATHER_QUEUE_SIZE - 1);
        WARN_ON(cur == pb->fence);
        *(p++) = op1;
@@ -242,7 +242,7 @@ static void cdma_timeout_pb_cleanup(struct nvhost_cdma *cdma, u32 getptr,
        /* NOP all the PB slots */
        getidx = getptr - pb->phys;
        while (nr_slots--) {
-               u32 *p = (u32 *)((u32)pb->mapped + getidx);
+               u32 *p = (u32 *)((uintptr_t)pb->mapped + getidx);
                *(p++) = NVHOST_OPCODE_NOOP;
                *(p++) = NVHOST_OPCODE_NOOP;
                dev_dbg(&dev->dev->dev, "%s: NOP at 0x%llx\n",
index 171ed08..c73fcba 100644 (file)
@@ -444,7 +444,7 @@ static void trace_write_gather(struct nvhost_cdma *cdma,
                for (i = 0; i < words; i += TRACE_MAX_LENGTH) {
                        trace_nvhost_cdma_push_gather(
                                cdma_to_channel(cdma)->dev->name,
-                               (u32)ref,
+                               (u32)((uintptr_t)ref),
                                min(words - i, TRACE_MAX_LENGTH),
                                offset + i * sizeof(u32),
                                mem);
index 1d930ff..2294dab 100644 (file)
@@ -126,7 +126,7 @@ struct mem_handle *nvhost_memmgr_get(struct mem_mgr *mgr,
 
 void nvhost_memmgr_put(struct mem_mgr *mgr, struct mem_handle *handle)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                nvhost_nvmap_put(mgr, handle);
@@ -145,7 +145,7 @@ void nvhost_memmgr_put(struct mem_mgr *mgr, struct mem_handle *handle)
 struct sg_table *nvhost_memmgr_pin(struct mem_mgr *mgr,
                struct mem_handle *handle, struct device *dev)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                return nvhost_nvmap_pin(mgr, handle);
@@ -166,7 +166,7 @@ void nvhost_memmgr_unpin(struct mem_mgr *mgr,
                struct mem_handle *handle, struct device *dev,
                struct sg_table *sgt)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                nvhost_nvmap_unpin(mgr, handle, sgt);
@@ -184,7 +184,7 @@ void nvhost_memmgr_unpin(struct mem_mgr *mgr,
 
 void *nvhost_memmgr_mmap(struct mem_handle *handle)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                return nvhost_nvmap_mmap(handle);
@@ -203,7 +203,7 @@ void *nvhost_memmgr_mmap(struct mem_handle *handle)
 
 void nvhost_memmgr_munmap(struct mem_handle *handle, void *addr)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                nvhost_nvmap_munmap(handle, addr);
@@ -223,7 +223,11 @@ int nvhost_memmgr_get_param(struct mem_mgr *mem_mgr,
                            struct mem_handle *mem_handle,
                            u32 param, u32 *result)
 {
+#ifndef CONFIG_ARM64
        switch (nvhost_memmgr_type((u32)mem_handle)) {
+#else
+       switch (nvhost_memmgr_type((u32)((uintptr_t)mem_handle))) {
+#endif
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                return nvhost_nvmap_get_param(mem_mgr, mem_handle,
@@ -244,7 +248,7 @@ int nvhost_memmgr_get_param(struct mem_mgr *mem_mgr,
 
 void *nvhost_memmgr_kmap(struct mem_handle *handle, unsigned int pagenum)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                return nvhost_nvmap_kmap(handle, pagenum);
@@ -264,7 +268,7 @@ void *nvhost_memmgr_kmap(struct mem_handle *handle, unsigned int pagenum)
 void nvhost_memmgr_kunmap(struct mem_handle *handle, unsigned int pagenum,
                void *addr)
 {
-       switch (nvhost_memmgr_type((u32)handle)) {
+       switch (nvhost_memmgr_type((u32)((uintptr_t)handle))) {
 #ifdef CONFIG_TEGRA_GRHOST_USE_NVMAP
        case mem_mgr_type_nvmap:
                nvhost_nvmap_kunmap(handle, pagenum, addr);
index 397e30d..755adf5 100644 (file)
@@ -141,7 +141,7 @@ int nvhost_syncpt_wait_timeout(struct nvhost_syncpt *sp, u32 id, u32 thresh,
 static inline int nvhost_syncpt_wait(struct nvhost_syncpt *sp, u32 id, u32 thresh)
 {
        return nvhost_syncpt_wait_timeout(sp, id, thresh,
-                                         MAX_SCHEDULE_TIMEOUT, NULL, NULL);
+                                         (u32)MAX_SCHEDULE_TIMEOUT, NULL, NULL);
 }
 
 int nvhost_syncpt_patch_wait(struct nvhost_syncpt *sp, void *patch_addr);
index 7bc6202..3fa7877 100644 (file)
@@ -63,7 +63,7 @@ void nvhost_nvmap_put(struct mem_mgr *mgr, struct mem_handle *handle)
 
 static struct scatterlist *sg_kmalloc(unsigned int nents, gfp_t gfp_mask)
 {
-       return (struct scatterlist *)gfp_mask;
+       return (struct scatterlist *)((uintptr_t)gfp_mask);
 }
 
 struct sg_table *nvhost_nvmap_pin(struct mem_mgr *mgr,
@@ -76,7 +76,7 @@ struct sg_table *nvhost_nvmap_pin(struct mem_mgr *mgr,
        if (!sgt)
                return ERR_PTR(-ENOMEM);
 
-       err = __sg_alloc_table(sgt, 1, 1, (gfp_t)(sgt+1), sg_kmalloc);
+       err = __sg_alloc_table(sgt, 1, 1, (gfp_t)((uintptr_t)(sgt+1)), sg_kmalloc);
        if (err) {
                kfree(sgt);
                return ERR_PTR(err);
index 8ba871f..692f085 100644 (file)
@@ -461,7 +461,7 @@ int nvhost_init_t124_channel_support(struct nvhost_master *host,
        struct nvhost_chip_support *op)
 {
        int i, nb_channels;
-       nvhost_dbg_fn("max channels=%d devices=%d",
+       nvhost_dbg_fn("max channels=%d devices=%zd",
                      NV_HOST1X_CHANNELS,
                      ARRAY_SIZE(t12_devices));
        BUILD_BUG_ON(T124_NVHOST_NUMCHANNELS < ARRAY_SIZE(t12_devices));
index 5d17ffd..dc2d4b5 100644 (file)
@@ -903,7 +903,7 @@ static void allocations_stringify(struct nvmap_client *client,
                        phys_addr_t base = iovmm ? 0 :
                                           (handle->carveout->base);
                        seq_printf(s,
-                               "%-18s %-18s %8llx %10u %8x %6u %6u %6u\n",
+                               "%-18s %-18s %8llx %10zu %8x %6u %6u %6u\n",
                                "", "",
                                (unsigned long long)base, handle->size,
                                handle->userflags,
@@ -931,7 +931,7 @@ static int nvmap_debug_allocations_show(struct seq_file *s, void *unused)
                struct nvmap_client *client =
                        get_client_from_carveout_commit(node, commit);
                client_stringify(client, s);
-               seq_printf(s, " %10u\n", commit->commit);
+               seq_printf(s, " %10zu\n", commit->commit);
                allocations_stringify(client, s, false);
                seq_printf(s, "\n");
                total += commit->commit;
@@ -969,7 +969,7 @@ static int nvmap_debug_clients_show(struct seq_file *s, void *unused)
                struct nvmap_client *client =
                        get_client_from_carveout_commit(node, commit);
                client_stringify(client, s);
-               seq_printf(s, " %10u\n", commit->commit);
+               seq_printf(s, " %10zu\n", commit->commit);
                total += commit->commit;
        }
        seq_printf(s, "%-18s %18s %8u %10u\n", "total", "", 0, total);
@@ -1253,7 +1253,7 @@ static int nvmap_probe(struct platform_device *pdev)
                                            &heap_extra_attr_group))
                        dev_warn(&pdev->dev, "couldn't add extra attributes\n");
 
-               dev_info(&pdev->dev, "created carveout %s (%uKiB)\n",
+               dev_info(&pdev->dev, "created carveout %s (%zuKiB)\n",
                         co->name, co->size / 1024);
 
                if (!IS_ERR_OR_NULL(nvmap_debug_root)) {
index a020a46..1cc19c2 100644 (file)
@@ -286,17 +286,17 @@ static ssize_t heap_stat_show(struct device *dev,
        base = heap_stat(heap, &stat);
 
        if (attr == &heap_stat_total_max)
-               return sprintf(buf, "%u\n", stat.largest);
+               return sprintf(buf, "%zu\n", stat.largest);
        else if (attr == &heap_stat_total_count)
-               return sprintf(buf, "%u\n", stat.count);
+               return sprintf(buf, "%zu\n", stat.count);
        else if (attr == &heap_stat_total_size)
-               return sprintf(buf, "%u\n", stat.total);
+               return sprintf(buf, "%zu\n", stat.total);
        else if (attr == &heap_stat_free_max)
-               return sprintf(buf, "%u\n", stat.free_largest);
+               return sprintf(buf, "%zu\n", stat.free_largest);
        else if (attr == &heap_stat_free_count)
-               return sprintf(buf, "%u\n", stat.free_count);
+               return sprintf(buf, "%zu\n", stat.free_count);
        else if (attr == &heap_stat_free_size)
-               return sprintf(buf, "%u\n", stat.free);
+               return sprintf(buf, "%zu\n", stat.free);
        else if (attr == &heap_stat_base)
                return sprintf(buf, "%08llx\n", (unsigned long long)base);
        else
@@ -697,15 +697,15 @@ struct nvmap_heap *nvmap_heap_create(struct device *parent, const char *name,
        DEFINE_DMA_ATTRS(attrs);
 
        if (WARN_ON(buddy_size && buddy_size < NVMAP_HEAP_MIN_BUDDY_SIZE)) {
-               dev_warn(parent, "%s: buddy_size %u too small\n", __func__,
+               dev_warn(parent, "%s: buddy_size %zu too small\n", __func__,
                        buddy_size);
                buddy_size = 0;
        } else if (WARN_ON(buddy_size >= len)) {
-               dev_warn(parent, "%s: buddy_size %u too large\n", __func__,
+               dev_warn(parent, "%s: buddy_size %zu too large\n", __func__,
                        buddy_size);
                buddy_size = 0;
        } else if (WARN_ON(buddy_size & (buddy_size - 1))) {
-               dev_warn(parent, "%s: buddy_size %u not a power of 2\n",
+               dev_warn(parent, "%s: buddy_size %zu not a power of 2\n",
                         __func__, buddy_size);
                buddy_size = 1 << (ilog2(buddy_size) + 1);
        }
@@ -713,14 +713,14 @@ struct nvmap_heap *nvmap_heap_create(struct device *parent, const char *name,
        if (WARN_ON(buddy_size && (base & (buddy_size - 1)))) {
                phys_addr_t orig = base;
                dev_warn(parent, "%s: base address 0x%llx not aligned to "
-                        "buddy_size %u\n", __func__, (u64)base, buddy_size);
+                        "buddy_size %zu\n", __func__, (u64)base, buddy_size);
                base = ALIGN(base, buddy_size);
                len -= (base - orig);
        }
 
        if (WARN_ON(buddy_size && (len & (buddy_size - 1)))) {
-               dev_warn(parent, "%s: length %u not aligned to "
-                        "buddy_size %u\n", __func__, len, buddy_size);
+               dev_warn(parent, "%s: length %zu not aligned to "
+                        "buddy_size %zu\n", __func__, len, buddy_size);
                len &= ~(buddy_size - 1);
        }
 
index 158a1fa..7b7c522 100644 (file)
@@ -37,7 +37,7 @@ struct nvmap_heap_block {
 
 struct nvmap_heap *nvmap_heap_create(struct device *parent, const char *name,
                                     phys_addr_t base, size_t len,
-                                    unsigned int buddy_size, void *arg);
+                                    size_t buddy_size, void *arg);
 
 void nvmap_heap_destroy(struct nvmap_heap *heap);