From 07a6899fe7d43d4bcba7f762a26c81204c6c004b Mon Sep 17 00:00:00 2001 From: Beomho Seo Date: Fri, 18 Apr 2014 18:31:24 +0900 Subject: [PATCH] modem_if: Change hardcoded link enable gpio value Currently, link enable gpio is hardecoded to zero. Zero equal gpa0-0 value. It is value for uart rxd line. If link enable gpio is unused, it is don't have to call gpio request. This patch change to set line enable gpio value optionally, and remove unused variable 'ret'. Change-Id: I4095d57d76362722abd5c5fd92ef529772680d97 Signed-off-by: Beomho Seo --- drivers/misc/modem_if/sipc4_modem.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/misc/modem_if/sipc4_modem.c b/drivers/misc/modem_if/sipc4_modem.c index c4cd872..89892fa 100644 --- a/drivers/misc/modem_if/sipc4_modem.c +++ b/drivers/misc/modem_if/sipc4_modem.c @@ -416,12 +416,10 @@ static struct modem_data *modem_parse_dt(struct platform_device *pdev, struct de mc->link_pm_data->gpio_link_enable = of_get_named_gpio(np, "link-enable-gpio", 0); /* link enable is optional */ -/* if (!gpio_is_valid(mc->link_pm_data->gpio_link_enable)) { - mif_err("failed to get link-enable gpio\n"); - return ERR_PTR(-EINVAL); - }*/ + if (gpio_is_valid(mc->link_pm_data->gpio_link_enable)) + devm_gpio_request(&pdev->dev, mc->link_pm_data->gpio_link_enable, + "gpio_link_enable"); - mc->link_pm_data->gpio_link_enable = 0; devm_gpio_request(&pdev->dev, mc->gpio_reset_req_n, "gpio_reset_req_n"); devm_gpio_request(&pdev->dev, mc->gpio_cp_on, "gpio_cp_on"); @@ -432,7 +430,6 @@ static struct modem_data *modem_parse_dt(struct platform_device *pdev, struct de devm_gpio_request(&pdev->dev, mc->link_pm_data->gpio_link_slavewake, "gpio_link_slavewake"); devm_gpio_request(&pdev->dev, mc->link_pm_data->gpio_link_hostwake, "gpio_link_hostwake"); devm_gpio_request(&pdev->dev, mc->link_pm_data->gpio_link_active, "gpio_link_active"); - devm_gpio_request(&pdev->dev, mc->link_pm_data->gpio_link_enable, "gpio_link_enable"); platform_set_drvdata(pdev, mc); @@ -449,7 +446,7 @@ dt_parse_err: static int modem_probe(struct platform_device *pdev) { - int i, ret; + int i; struct modem_data *pdata = pdev->dev.platform_data; struct modem_shared *msd = NULL; struct modem_ctl *modemctl = NULL; -- 2.7.4