ARM: tegra: cardhu: Remove tegra_gpio_enb/disable
Ashwini Ghuge [Wed, 16 May 2012 11:43:28 +0000 (16:43 +0530)]
Removing tegra_gpio_enable and disable calls
from board files as they are supported through
set direction calls in the driver

Bug 984439

Change-Id: I51b17389dbb17b0e94e1635ec6d68acd01c743b4
Signed-off-by: Ashwini Ghuge <aghuge@nvidia.com>
Reviewed-on: http://git-master/r/102529
Reviewed-by: Simone Willett <swillett@nvidia.com>
Tested-by: Simone Willett <swillett@nvidia.com>

arch/arm/mach-tegra/board-cardhu-kbc.c
arch/arm/mach-tegra/board-cardhu-panel.c
arch/arm/mach-tegra/board-cardhu-pinmux.c
arch/arm/mach-tegra/board-cardhu-pm298-power-rails.c
arch/arm/mach-tegra/board-cardhu-pm299-power-rails.c
arch/arm/mach-tegra/board-cardhu-power.c
arch/arm/mach-tegra/board-cardhu-sdhci.c
arch/arm/mach-tegra/board-cardhu-sensors.c
arch/arm/mach-tegra/board-cardhu.c

index da17238..0fb284d 100644 (file)
@@ -253,8 +253,6 @@ int __init cardhu_keys_init(void)
                                if (get_tegra_image_type() == rck_image)
                                        cardhu_keys_e1291_pdata.buttons[i].code
                                                        = KEY_ENTER;
-                       } else {
-                               tegra_gpio_enable(gpio_nr);
                        }
                }
 
@@ -266,8 +264,6 @@ int __init cardhu_keys_init(void)
                        if (gpio_nr < 0) {
                                if (get_tegra_image_type() == rck_image)
                                        cardhu_keys_e1198[i].code = KEY_ENTER;
-                       } else {
-                               tegra_gpio_enable(gpio_nr);
                        }
                }
 
index bebd2ae..1aa0805 100644 (file)
@@ -147,8 +147,6 @@ static int cardhu_backlight_init(struct device *dev)
                pr_err("bl_output array does not have 256 elements\n");
 
 #ifndef CONFIG_TEGRA_CARDHU_DSI
-       tegra_gpio_disable(cardhu_bl_pwm);
-
        ret = gpio_request(cardhu_bl_enb, "backlight_enb");
        if (ret < 0)
                return ret;
@@ -156,8 +154,6 @@ static int cardhu_backlight_init(struct device *dev)
        ret = gpio_direction_output(cardhu_bl_enb, 1);
        if (ret < 0)
                gpio_free(cardhu_bl_enb);
-       else
-               tegra_gpio_enable(cardhu_bl_enb);
 
        return ret;
 #endif
@@ -171,8 +167,6 @@ static int cardhu_backlight_init(struct device *dev)
        ret = gpio_direction_output(cardhu_dsia_bl_enb, 1);
        if (ret < 0)
                gpio_free(cardhu_dsia_bl_enb);
-       else
-               tegra_gpio_enable(cardhu_dsia_bl_enb);
 
        /* Enable back light for DSIb panel */
        ret = gpio_request(cardhu_dsib_bl_enb, "dsib_bl_enable");
@@ -182,8 +176,6 @@ static int cardhu_backlight_init(struct device *dev)
        ret = gpio_direction_output(cardhu_dsib_bl_enb, 1);
        if (ret < 0)
                gpio_free(cardhu_dsib_bl_enb);
-       else
-               tegra_gpio_enable(cardhu_dsib_bl_enb);
 #endif
 
 #if DSI_PANEL_219
@@ -195,8 +187,6 @@ static int cardhu_backlight_init(struct device *dev)
        ret = gpio_direction_output(cardhu_dsia_bl_enb, 1);
        if (ret < 0)
                gpio_free(cardhu_dsia_bl_enb);
-       else
-               tegra_gpio_enable(cardhu_dsia_bl_enb);
 #endif
 
        return ret;
@@ -209,7 +199,6 @@ static void cardhu_backlight_exit(struct device *dev)
        /*ret = gpio_request(cardhu_bl_enb, "backlight_enb");*/
        gpio_set_value(cardhu_bl_enb, 0);
        gpio_free(cardhu_bl_enb);
-       tegra_gpio_disable(cardhu_bl_enb);
        return;
 #endif
 
@@ -217,12 +206,11 @@ static void cardhu_backlight_exit(struct device *dev)
        /* Disable back light for DSIa panel */
        gpio_set_value(cardhu_dsia_bl_enb, 0);
        gpio_free(cardhu_dsia_bl_enb);
-       tegra_gpio_disable(cardhu_dsia_bl_enb);
+
 
        /* Disable back light for DSIb panel */
        gpio_set_value(cardhu_dsib_bl_enb, 0);
        gpio_free(cardhu_dsib_bl_enb);
-       tegra_gpio_disable(cardhu_dsib_bl_enb);
 
        gpio_set_value(cardhu_lvds_shutdown, 1);
        mdelay(20);
@@ -232,7 +220,7 @@ static void cardhu_backlight_exit(struct device *dev)
        /* Disable back light for DSIa panel */
        gpio_set_value(cardhu_dsia_bl_enb, 0);
        gpio_free(cardhu_dsia_bl_enb);
-       tegra_gpio_disable(cardhu_dsia_bl_enb);
+
 
        gpio_set_value(cardhu_lvds_shutdown, 1);
        mdelay(20);
@@ -765,8 +753,6 @@ static int cardhu_dsi_panel_enable(void)
        ret = gpio_direction_output(AVDD_LCD, 1);
        if (ret < 0)
                gpio_free(AVDD_LCD);
-       else
-               tegra_gpio_enable(AVDD_LCD);
 
 #if DSI_PANEL_219
 
@@ -777,8 +763,7 @@ static int cardhu_dsi_panel_enable(void)
        if (ret < 0) {
                gpio_free(cardhu_bl_pwm);
                return ret;
-       } else
-               tegra_gpio_enable(cardhu_bl_pwm);
+       }
 
        ret = gpio_request(cardhu_bl_enb, "bl_enb");
        if (ret < 0)
@@ -787,8 +772,7 @@ static int cardhu_dsi_panel_enable(void)
        if (ret < 0) {
                gpio_free(cardhu_bl_enb);
                return ret;
-       } else
-               tegra_gpio_enable(cardhu_bl_enb);
+       }
 
        gpio_set_value(cardhu_lvds_shutdown, 1);
        mdelay(20);
@@ -808,8 +792,7 @@ static int cardhu_dsi_panel_enable(void)
        if (ret < 0) {
                gpio_free(cardhu_dsi_218_panel_reset);
                return ret;
-       } else
-               tegra_gpio_enable(cardhu_dsi_218_panel_reset);
+       }
 
        gpio_set_value(cardhu_dsi_218_panel_reset, 1);
        gpio_set_value(cardhu_dsi_218_panel_reset, 0);
@@ -826,8 +809,7 @@ static int cardhu_dsi_panel_enable(void)
        if (ret < 0) {
                gpio_free(cardhu_dsi_219_panel_reset);
                return ret;
-       } else
-               tegra_gpio_enable(cardhu_dsi_219_panel_reset);
+       }
 
        gpio_set_value(cardhu_dsi_219_panel_reset, 0);
        gpio_set_value(cardhu_dsi_219_panel_reset, 1);
@@ -850,18 +832,13 @@ static int cardhu_dsi_panel_disable(void)
        printk(KERN_INFO "DSI panel disable\n");
 
 #if DSI_PANEL_219
-       tegra_gpio_disable(cardhu_dsi_219_panel_reset);
        gpio_free(cardhu_dsi_219_panel_reset);
-       tegra_gpio_disable(cardhu_bl_enb);
        gpio_free(cardhu_bl_enb);
-       tegra_gpio_disable(cardhu_bl_pwm);
        gpio_free(cardhu_bl_pwm);
-       tegra_gpio_disable(cardhu_lvds_shutdown);
        gpio_free(cardhu_lvds_shutdown);
 #endif
 
 #if DSI_PANEL_218
-       tegra_gpio_disable(cardhu_dsi_218_panel_reset);
        gpio_free(cardhu_dsi_218_panel_reset);
 #endif
 
@@ -885,7 +862,6 @@ static int cardhu_dsi_panel_postsuspend(void)
        }
 
 #if DSI_PANEL_218
-       tegra_gpio_disable(AVDD_LCD);
        gpio_free(AVDD_LCD);
 #endif
 
@@ -1225,28 +1201,22 @@ int __init cardhu_panel_init(void)
                /* lvds configuration */
                err = gpio_request(pm313_R_FDE, "R_FDE");
                err |= gpio_direction_output(pm313_R_FDE, 1);
-               tegra_gpio_enable(pm313_R_FDE);
 
                err |= gpio_request(pm313_R_FB, "R_FB");
                err |= gpio_direction_output(pm313_R_FB, 1);
-               tegra_gpio_enable(pm313_R_FB);
 
                err |= gpio_request(pm313_MODE0, "MODE0");
                err |= gpio_direction_output(pm313_MODE0, 1);
-               tegra_gpio_enable(pm313_MODE0);
 
                err |= gpio_request(pm313_MODE1, "MODE1");
                err |= gpio_direction_output(pm313_MODE1, 0);
-               tegra_gpio_enable(pm313_MODE1);
 
                err |= gpio_request(pm313_BPP, "BPP");
                err |= gpio_direction_output(pm313_BPP, PM313_LVDS_PANEL_BPP);
-               tegra_gpio_enable(pm313_BPP);
 
                err = gpio_request(pm313_lvds_shutdown, "lvds_shutdown");
                /* free ride provided by bootloader */
                err |= gpio_direction_output(pm313_lvds_shutdown, 1);
-               tegra_gpio_enable(pm313_lvds_shutdown);
 
                if (err)
                        printk(KERN_ERR "ERROR(s) in LVDS configuration\n");
@@ -1257,15 +1227,12 @@ int __init cardhu_panel_init(void)
                                (board_info.board_id == BOARD_PM311)) {
                gpio_request(e1247_pm269_lvds_shutdown, "lvds_shutdown");
                gpio_direction_output(e1247_pm269_lvds_shutdown, 1);
-               tegra_gpio_enable(e1247_pm269_lvds_shutdown);
        } else {
                gpio_request(cardhu_lvds_shutdown, "lvds_shutdown");
                gpio_direction_output(cardhu_lvds_shutdown, 1);
-               tegra_gpio_enable(cardhu_lvds_shutdown);
        }
 #endif
 
-       tegra_gpio_enable(cardhu_hdmi_hpd);
        gpio_request(cardhu_hdmi_hpd, "hdmi_hpd");
        gpio_direction_input(cardhu_hdmi_hpd);
 
index 996fbe7..3767a04 100644 (file)
@@ -548,11 +548,8 @@ static __initdata struct tegra_pingroup_config gmi_pins_269[] = {
 
 static void __init cardhu_pinmux_audio_init(void)
 {
-       tegra_gpio_enable(TEGRA_GPIO_CDC_IRQ);
        gpio_request(TEGRA_GPIO_CDC_IRQ, "wm8903");
        gpio_direction_input(TEGRA_GPIO_CDC_IRQ);
-
-       tegra_gpio_enable(TEGRA_GPIO_HP_DET);
 }
 
 #define GPIO_INIT_PIN_MODE(_gpio, _is_input, _value)   \
@@ -761,7 +758,6 @@ static void set_unused_pin_gpio(struct gpio_init_pin_info *lpm_pin_info,
                        gpio_free(pin_info->gpio_nr);
                        continue;
                }
-               tegra_gpio_enable(pin_info->gpio_nr);
        }
 }
 
index 39c0d50..3375c49 100644 (file)
@@ -700,12 +700,5 @@ int __init cardhu_pm298_gpio_switch_regulator_init(void)
                break;
        }
 
-       for (i = 0; i < nfixreg_devs; ++i) {
-               struct fixed_voltage_config *fixed_reg_pdata =
-                               fixed_reg_devs[i]->dev.platform_data;
-               int gpio_nr = fixed_reg_pdata->gpio;
-               if (gpio_nr < TEGRA_NR_GPIOS)
-                       tegra_gpio_enable(gpio_nr);
-       }
        return platform_add_devices(fixed_reg_devs, nfixreg_devs);
 }
index 3be66db..d402485 100644 (file)
@@ -690,12 +690,5 @@ int __init cardhu_pm299_gpio_switch_regulator_init(void)
                break;
        }
 
-       for (i = 0; i < nfixreg_devs; ++i) {
-               struct fixed_voltage_config *fixed_reg_pdata =
-                               fixed_reg_devs[i]->dev.platform_data;
-               int gpio_nr = fixed_reg_pdata->gpio;
-               if (gpio_nr < TEGRA_NR_GPIOS)
-                       tegra_gpio_enable(gpio_nr);
-       }
        return platform_add_devices(fixed_reg_devs, nfixreg_devs);
 }
index 8ce417f..ab7fac8 100644 (file)
@@ -1074,13 +1074,6 @@ int __init cardhu_fixed_regulator_init(void)
                break;
        }
 
-       for (i = 0; i < nfixreg_devs; ++i) {
-               struct fixed_voltage_config *fixed_reg_pdata =
-                               fixed_reg_devs[i]->dev.platform_data;
-               int gpio_nr = fixed_reg_pdata->gpio;
-               if (gpio_nr < TEGRA_NR_GPIOS)
-                       tegra_gpio_enable(gpio_nr);
-       }
        return platform_add_devices(fixed_reg_devs, nfixreg_devs);
 }
 subsys_initcall_sync(cardhu_fixed_regulator_init);
index 15b2174..f87874a 100644 (file)
@@ -271,10 +271,6 @@ static int __init cardhu_wifi_init(void)
        if (rc)
                pr_err("WLAN_WOW gpio request failed:%d\n", rc);
 
-       tegra_gpio_enable(CARDHU_WLAN_PWR);
-       tegra_gpio_enable(CARDHU_WLAN_RST);
-       tegra_gpio_enable(CARDHU_WLAN_WOW);
-
        rc = gpio_direction_output(CARDHU_WLAN_PWR, 0);
        if (rc)
                pr_err("WLAN_PWR gpio direction configuration failed:%d\n", rc);
index de5b69a..239bace 100644 (file)
@@ -87,24 +87,20 @@ static int cardhu_camera_init(void)
         * and donot have TCA6416 exp for camera */
        if ((board_info.board_id == BOARD_E1198) ||
                (board_info.board_id == BOARD_E1291)) {
-               tegra_gpio_enable(CAM1_POWER_DWN_GPIO);
                ret = gpio_request(CAM1_POWER_DWN_GPIO, "camera_power_en");
                if (ret < 0)
                        pr_err("%s: gpio_request failed for gpio %s\n",
                                __func__, "CAM1_POWER_DWN_GPIO");
-                               tegra_gpio_enable(CAM3_POWER_DWN_GPIO);
                ret = gpio_request(CAM3_POWER_DWN_GPIO, "cam3_power_en");
                if (ret < 0)
                        pr_err("%s: gpio_request failed for gpio %s\n",
                                __func__, "CAM3_POWER_DWN_GPIO");
 
-               tegra_gpio_enable(CAM2_POWER_DWN_GPIO);
                ret = gpio_request(CAM2_POWER_DWN_GPIO, "camera2_power_en");
                if (ret < 0)
                        pr_err("%s: gpio_request failed for gpio %s\n",
                                __func__, "CAM2_POWER_DWN_GPIO");
 
-               tegra_gpio_enable(OV5650_RESETN_GPIO);
                ret = gpio_request(OV5650_RESETN_GPIO, "camera_reset");
                if (ret < 0)
                        pr_err("%s: gpio_request failed for gpio %s\n",
@@ -124,7 +120,6 @@ static int cardhu_camera_init(void)
        }
 
        /* To select the CSIB MUX either for cam2 or cam3 */
-       tegra_gpio_enable(CAMERA_CSI_MUX_SEL_GPIO);
        ret = gpio_request(CAMERA_CSI_MUX_SEL_GPIO, "camera_csi_sel");
        if (ret < 0)
                pr_err("%s: gpio_request failed for gpio %s\n",
@@ -785,8 +780,6 @@ static int cardhu_nct1008_init(void)
                ret = gpio_direction_input(nct1008_port);
                if (ret < 0)
                        gpio_free(nct1008_port);
-               else
-                       tegra_gpio_enable(nct1008_port);
        }
 
        return ret;
@@ -916,7 +909,6 @@ static void mpuirq_init(void)
 #if (MPU_GYRO_TYPE == MPU_TYPE_MPU3050)
 #if    MPU_ACCEL_IRQ_GPIO
        /* ACCEL-IRQ assignment */
-       tegra_gpio_enable(MPU_ACCEL_IRQ_GPIO);
        ret = gpio_request(MPU_ACCEL_IRQ_GPIO, MPU_ACCEL_NAME);
        if (ret < 0) {
                pr_err("%s: gpio_request failed %d\n", __func__, ret);
@@ -933,7 +925,6 @@ static void mpuirq_init(void)
 #endif
 
        /* MPU-IRQ assignment */
-       tegra_gpio_enable(MPU_GYRO_IRQ_GPIO);
        ret = gpio_request(MPU_GYRO_IRQ_GPIO, MPU_GYRO_NAME);
        if (ret < 0) {
                pr_err("%s: gpio_request failed %d\n", __func__, ret);
index 07121ad..abf2152 100644 (file)
@@ -173,8 +173,6 @@ static struct platform_device cardhu_bluesleep_device = {
 static noinline void __init cardhu_setup_bluesleep(void)
 {
        platform_device_register(&cardhu_bluesleep_device);
-       tegra_gpio_enable(TEGRA_GPIO_PU6);
-       tegra_gpio_enable(TEGRA_GPIO_PU1);
        return;
 }
 
@@ -848,9 +846,6 @@ static int __init cardhu_touch_init(void)
 
                touch_init_raydium(TEGRA_GPIO_PH4, TEGRA_GPIO_PH6, 2);
        } else {
-               tegra_gpio_enable(TEGRA_GPIO_PH4);
-               tegra_gpio_enable(TEGRA_GPIO_PH6);
-
                gpio_request(TEGRA_GPIO_PH4, "atmel-irq");
                gpio_direction_input(TEGRA_GPIO_PH4);
 
@@ -1008,19 +1003,6 @@ static void cardhu_usb_init(void)
 static void cardhu_usb_init(void) { }
 #endif
 
-static void cardhu_gps_init(void)
-{
-       tegra_gpio_enable(TEGRA_GPIO_PU2);
-       tegra_gpio_enable(TEGRA_GPIO_PU3);
-}
-
-static void cardhu_nfc_init(void)
-{
-       tegra_gpio_enable(TEGRA_GPIO_PX0);
-       tegra_gpio_enable(TEGRA_GPIO_PP3);
-       tegra_gpio_enable(TEGRA_GPIO_PO7);
-}
-
 static struct baseband_power_platform_data tegra_baseband_power_data = {
        .baseband_type = BASEBAND_XMM,
        .modem = {
@@ -1098,7 +1080,7 @@ static void cardhu_modem_init(void)
                } else {
                        w_disable_gpio = TEGRA_GPIO_PDD5;
                }
-               tegra_gpio_enable(w_disable_gpio);
+
                ret = gpio_request(w_disable_gpio, "w_disable_gpio");
                if (ret < 0)
                        pr_err("%s: gpio_request failed for gpio %d\n",
@@ -1118,22 +1100,9 @@ static void cardhu_modem_init(void)
                                break;
                        }
                        gpio_direction_output(TEGRA_GPIO_PH7, 1);
-                       tegra_gpio_enable(TEGRA_GPIO_PH7);
                }
                break;
        case BOARD_E1186:
-               tegra_gpio_enable(
-                       tegra_baseband_power_data.modem.xmm.bb_rst);
-               tegra_gpio_enable(
-                       tegra_baseband_power_data.modem.xmm.bb_on);
-               tegra_gpio_enable(
-                       tegra_baseband_power_data.modem.xmm.ipc_bb_wake);
-               tegra_gpio_enable(
-                       tegra_baseband_power_data.modem.xmm.ipc_ap_wake);
-               tegra_gpio_enable(
-                       tegra_baseband_power_data.modem.xmm.ipc_hsic_active);
-               tegra_gpio_enable(
-                       tegra_baseband_power_data.modem.xmm.ipc_hsic_sus_req);
                platform_device_register(&tegra_baseband_power_device);
                platform_device_register(&tegra_baseband_power2_device);
                break;
@@ -1172,7 +1141,6 @@ static void __init tegra_cardhu_init(void)
        cardhu_dtv_init();
        cardhu_suspend_init();
        cardhu_touch_init();
-       cardhu_gps_init();
        cardhu_modem_init();
        cardhu_kbc_init();
        cardhu_scroll_init();
@@ -1186,7 +1154,6 @@ static void __init tegra_cardhu_init(void)
        cardhu_pins_state_init();
        cardhu_emc_init();
        tegra_release_bootloader_fb();
-       cardhu_nfc_init();
        cardhu_pci_init();
 #ifdef CONFIG_TEGRA_WDT_RECOVERY
        tegra_wdt_recovery_init();