arm: tegra: timer: Fix build warning
Sumit Singh [Wed, 5 Nov 2014 07:25:35 +0000 (12:25 +0530)]
Fix warning:
'passing argument 1 of 'IS_ERR_OR_NULL' makes pointer
from integer without a cast' in timerinfo_dev_mmap function in
tegra_timerinfo.c file.

Bug 1566945

Change-Id: I233633839d3d2a38523733446a8d1511319c8b13
Signed-off-by: Sumit Singh <sumsingh@nvidia.com>
Reviewed-on: http://git-master/r/591132
Reviewed-by: Sachin Nikam <snikam@nvidia.com>
Tested-by: Sachin Nikam <snikam@nvidia.com>

drivers/clocksource/tegra-nvtimers.c
drivers/misc/tegra_timerinfo.c

index 9cd4076..cb5d0e9 100644 (file)
@@ -362,8 +362,10 @@ void __init tegra_init_timer(struct device_node *np)
        unsigned long rate;
        struct resource res;
 
-       if (of_address_to_resource(np, 0, &res))
+       if (of_address_to_resource(np, 0, &res)) {
                pr_err("%s:No memory resources found\n", __func__);
+               return;
+       }
 
        timer_reg_base = ioremap(res.start, resource_size(&res));
        if (!timer_reg_base) {
index ad38b75..832f6cb 100644 (file)
@@ -44,11 +44,6 @@ static struct miscdevice timerinfo_dev = {
 static int timerinfo_dev_mmap(struct file *file, struct vm_area_struct *vma)
 {
        /* start at first page containing TIMERUS_CNTR_1US */
-       if (IS_ERR_OR_NULL(timer_reg_base_pa)) {
-               pr_err("%s: Invalid timer_base_address\n", __func__);
-               return -EINVAL;
-       }
-
        if (vma->vm_end  - vma->vm_start != PAGE_SIZE)
                return -EINVAL;
 
@@ -68,6 +63,10 @@ static int timerinfo_dev_mmap(struct file *file, struct vm_area_struct *vma)
 
 static int __init timerinfo_dev_init(void)
 {
+       if (!timer_reg_base_pa) {
+               pr_err("%s: Timer not registered\n", __func__);
+               return 0;
+       }
        return misc_register(&timerinfo_dev);
 }