arm: tegra: resolve compilation time warnings
Sanjay Singh Rawat [Mon, 28 May 2012 07:20:58 +0000 (12:20 +0530)]
Warnings removed are related to unused variables/labels,
structure/argument type mismatch, copyright update, function return
type mismatch and wrong C coding style.

Bug 949219

Change-Id: Ib748d12d5ab3cfc35118be28c29983081cca6cbb
Signed-off-by: Sanjay Singh Rawat <srawat@nvidia.com>
Reviewed-on: http://git-master/r/103770
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Bharat Nihalani <bnihalani@nvidia.com>
Reviewed-by: Sachin Nikam <snikam@nvidia.com>

arch/arm/mach-tegra/board-harmony-pcie.c
arch/arm/mach-tegra/board-trimslice.c
arch/arm/mach-tegra/board-whistler-sdhci.c
arch/arm/mach-tegra/board-whistler.c
arch/arm/mach-tegra/common-t2.c
arch/arm/mach-tegra/common.c
arch/arm/mach-tegra/include/mach/latency_allowance.h
arch/arm/mach-tegra/iovmm-gart.c
arch/arm/mach-tegra/tegra2_clocks.c
arch/arm/mach-tegra/tegra2_mc.c

index f3db0ee..f73cde4 100644 (file)
@@ -4,6 +4,8 @@
  * Copyright (C) 2010 CompuLab, Ltd.
  * Mike Rapoport <mike@compulab.co.il>
  *
+ * Copyright (C) 2011-2012 NVIDIA Corporation.
+ *
  * This software is licensed under the terms of the GNU General Public
  * License version 2, as published by the Free Software Foundation, and
  * may be copied, distributed, and modified under those terms.
@@ -68,7 +70,6 @@ int __init harmony_pcie_init(void)
 
        return 0;
 
-err_pcie:
        tegra_pinmux_set_tristate(TEGRA_PINGROUP_GPV, TEGRA_TRI_TRISTATE);
        tegra_pinmux_set_tristate(TEGRA_PINGROUP_SLXA, TEGRA_TRI_TRISTATE);
        tegra_pinmux_set_tristate(TEGRA_PINGROUP_SLXK, TEGRA_TRI_TRISTATE);
index 823060e..626cd1b 100644 (file)
@@ -6,6 +6,7 @@
  *
  * Based on board-harmony.c
  * Copyright (C) 2010 Google, Inc.
+ * Copyright (C) 2011-2012 NVIDIA Corporation.
  *
  * This software is licensed under the terms of the GNU General Public
  * License version 2, as published by the Free Software Foundation, and
@@ -95,7 +96,7 @@ static struct platform_device *trimslice_devices[] __initdata = {
        &tegra_das_device,
        &tegra_pcm_device,
        &trimslice_audio_device,
-       &trimslice_pci_platform_data,
+       &tegra_pci_device,
 };
 
 static struct i2c_board_info trimslice_i2c3_board_info[] = {
index d98b1d5..2d2a9c8 100644 (file)
@@ -2,7 +2,7 @@
  * arch/arm/mach-tegra/board-whistler-sdhci.c
  *
  * Copyright (C) 2010 Google, Inc.
- * Copyright (C) 2011 NVIDIA Corporation.
+ * Copyright (C) 2011-2012 NVIDIA Corporation.
  *
  * This software is licensed under the terms of the GNU General Public
  * License version 2, as published by the Free Software Foundation, and
@@ -256,8 +256,6 @@ static int __init whistler_wifi_init(void)
 }
 int __init whistler_sdhci_init(void)
 {
-       int ret;
-
        tegra_gpio_enable(WHISTLER_EXT_SDCARD_DETECT);
 
        platform_device_register(&tegra_sdhci_device3);
index 39ed3ea..874ef18 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * arch/arm/mach-tegra/board-whistler.c
  *
- * Copyright (c) 2010 - 2011, NVIDIA Corporation.
+ * Copyright (c) 2010-2012 NVIDIA Corporation.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -482,36 +482,12 @@ static struct tegra_usb_platform_data tegra_ehci1_utmi_pdata = {
        },
 };
 
-static struct tegra_usb_platform_data tegra_ehci3_utmi_pdata = {
-       .port_otg = false,
-       .has_hostpc = false,
-       .phy_intf = TEGRA_USB_PHY_INTF_UTMI,
-       .op_mode        = TEGRA_USB_OPMODE_HOST,
-       .u_data.host = {
-               .vbus_gpio = TEGRA_GPIO_PD3,
-               .vbus_reg = NULL,
-               .hot_plug = true,
-               .remote_wakeup_supported = false,
-               .power_off_on_suspend = true,
-       },
-       .u_cfg.utmi = {
-               .hssync_start_delay = 9,
-               .elastic_limit = 16,
-               .idle_wait_delay = 17,
-               .term_range_adj = 6,
-               .xcvr_setup = 8,
-               .xcvr_lsfslew = 2,
-               .xcvr_lsrslew = 2,
-       },
-};
-
 static struct tegra_usb_otg_data tegra_otg_pdata = {
        .ehci_device = &tegra_ehci1_device,
        .ehci_pdata = &tegra_ehci1_utmi_pdata,
 };
 
 #define SERIAL_NUMBER_LENGTH 20
-static char usb_serial_num[SERIAL_NUMBER_LENGTH];
 static void whistler_usb_init(void)
 {
        tegra_otg_device.dev.platform_data = &tegra_otg_pdata;
index 6f9b177..f90dfce 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Tegra 2 SoC-specific initialization (memory controller, etc.)
  *
- * Copyright (c) 2009-2011, NVIDIA Corporation.
+ * Copyright (c) 2009-2012 NVIDIA Corporation.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -177,16 +177,18 @@ out:
        return IRQ_HANDLED;
 }
 
-void __init tegra_mc_init(void)
+static int __init tegra_mc_init(void)
 {
        if (request_irq(INT_MC_GENERAL, tegra_mc_error_isr, 0,
                        "mc_status", NULL)) {
                pr_err("%s: unable to register MC error interrupt\n", __func__);
+               return -EINVAL;
        } else {
                void __iomem *mc = IO_ADDRESS(TEGRA_MC_BASE);
                u32 reg = MC_INT_SECURITY_VIOLATION | MC_INT_INVALID_GART_PAGE |
                        MC_INT_DECERR_EMEM_OTHERS;
                writel(reg, mc + MC_INT_MASK);
        }
+       return 0;
 }
 arch_initcall(tegra_mc_init);
index 78bd77d..2986f1c 100644 (file)
@@ -974,12 +974,14 @@ void cpufreq_store_default_gov(void)
 
 int cpufreq_change_gov(char *target_gov)
 {
-       unsigned int cpu = 0;
+       unsigned int cpu = 0, ret = -EINVAL;
 
 #ifndef CONFIG_TEGRA_AUTO_HOTPLUG
        for_each_online_cpu(cpu)
 #endif
-       return cpufreq_set_gov(target_gov, cpu);
+       ret = cpufreq_set_gov(target_gov, cpu);
+
+       return ret;
 }
 
 int cpufreq_restore_default_gov(void)
index f0d27f0..8644075 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * arch/arm/mach-tegra/include/mach/latency_allowance.h
  *
- * Copyright (C) 2011, NVIDIA Corporation.
+ * Copyright (C) 2011-2012 NVIDIA Corporation.
  *
  * This software is licensed under the terms of the GNU General Public
  * License version 2, as published by the Free Software Foundation, and
@@ -104,7 +104,7 @@ static inline int tegra_enable_latency_scaling(enum tegra_la_id id,
 
 static inline void tegra_disable_latency_scaling(enum tegra_la_id id)
 {
-       return 0;
+       return;
 }
 #else
 int tegra_set_latency_allowance(enum tegra_la_id id,
index ad1ce3a..130901c 100644 (file)
@@ -4,7 +4,7 @@
  * Tegra I/O VMM implementation for GART devices in Tegra and Tegra 2 series
  * systems-on-a-chip.
  *
- * Copyright (c) 2010-2012, NVIDIA Corporation.
+ * Copyright (c) 2010-2012 NVIDIA Corporation.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -73,7 +73,7 @@ static int gart_map(struct tegra_iovmm_domain *, struct tegra_iovmm_area *);
 static void gart_unmap(struct tegra_iovmm_domain *,
        struct tegra_iovmm_area *, bool);
 static void gart_map_pfn(struct tegra_iovmm_domain *,
-       struct tegra_iovmm_area *, tegra_iovmm_addr_t, unsigned long);
+       struct tegra_iovmm_area *, unsigned long, unsigned long);
 static struct tegra_iovmm_domain *gart_alloc_domain(
        struct tegra_iovmm_device *, struct tegra_iovmm_client *);
 
@@ -330,7 +330,7 @@ static void gart_unmap(struct tegra_iovmm_domain *domain,
 }
 
 static void gart_map_pfn(struct tegra_iovmm_domain *domain,
-       struct tegra_iovmm_area *iovma, tegra_iovmm_addr_t offs,
+       struct tegra_iovmm_area *iovma, unsigned long offs,
        unsigned long pfn)
 {
        struct gart_device *gart =
index 135ae6a..fa88b11 100644 (file)
@@ -779,8 +779,9 @@ static int tegra2_pll_clk_set_rate(struct clk *c, unsigned long rate)
 
        if (sel->input_rate == 0) {
                unsigned long cfreq;
-               BUG_ON(c->flags & PLLU);
                struct clk_pll_freq_table cfg;
+
+               BUG_ON(c->flags & PLLU);
                sel = &cfg;
 
                switch (input_rate) {
index 6df9c23..1b72af0 100644 (file)
@@ -896,8 +896,10 @@ static enum hrtimer_restart sample_timer_function(struct hrtimer *handle)
 #define REGISTER_SYSFS(_name, _val)                                    \
        tegra_mc_dram_##_name##_kobj =                                  \
                kobject_create_and_add(#_name, tegra_mc_dram_kobj);     \
-       sysfs_create_group(tegra_mc_dram_##_name##_kobj,                \
-                          &tegra_mc_dram_##_name##_attr_group);
+       if (sysfs_create_group(tegra_mc_dram_##_name##_kobj,            \
+                          &tegra_mc_dram_##_name##_attr_group))        \
+               printk(KERN_ERR "\n sysfs_create_group failed at %s"    \
+                               " line %d\n", __FILE__, __LINE__);
 
 static int tegra_mc_init(void)
 {