diff options
| author |   Roland Stigge <stigge@antcom.de> | 2012-06-18 11:28:26 +0200 |
|---|---|---|
| committer |   Linus Walleij <linus.walleij@linaro.org> | 2012-07-05 15:02:18 +0200 |
| commit | 42b14cb037fefa33a2ff51c4d3915a49c71de3d5 (patch) | |
| tree | 5accb630636ab2ce17de936e694bc3404b6fed55 /arch/mips/pci | |
| parent | gpio: export devm_gpio_request_one (diff) | |
| download | linux-dev-42b14cb037fefa33a2ff51c4d3915a49c71de3d5.tar.xz linux-dev-42b14cb037fefa33a2ff51c4d3915a49c71de3d5.zip | |
mips: pci-lantiq: Fix check for valid gpio
This patch fixes two checks for valid gpio number, formerly (wrongly)
considering zero as invalid, now using gpio_is_valid().
Signed-off-by: Roland Stigge <stigge@antcom.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'arch/mips/pci')
| -rw-r--r-- | arch/mips/pci/pci-lantiq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/mips/pci/pci-lantiq.c b/arch/mips/pci/pci-lantiq.c index ea453532a33c..075d87acd12a 100644 --- a/arch/mips/pci/pci-lantiq.c +++ b/arch/mips/pci/pci-lantiq.c @@ -129,7 +129,7 @@ static int __devinit ltq_pci_startup(struct platform_device *pdev) /* setup reset gpio used by pci */ reset_gpio = of_get_named_gpio(node, "gpio-reset", 0); - if (reset_gpio > 0) + if (gpio_is_valid(reset_gpio)) devm_gpio_request(&pdev->dev, reset_gpio, "pci-reset"); /* enable auto-switching between PCI and EBU */ @@ -192,7 +192,7 @@ static int __devinit ltq_pci_startup(struct platform_device *pdev) ltq_ebu_w32(ltq_ebu_r32(LTQ_EBU_PCC_IEN) | 0x10, LTQ_EBU_PCC_IEN); /* toggle reset pin */ - if (reset_gpio > 0) { + if (gpio_is_valid(reset_gpio)) { __gpio_set_value(reset_gpio, 0); wmb(); mdelay(1); |