Commit eed008e6 authored by Ladislav Michl's avatar Ladislav Michl Committed by Mauro Carvalho Chehab

[media] media: rc: gpio-ir-recv: use gpiolib API

Gpiolib API is preferred way to access gpios.
Signed-off-by: default avatarLadislav Michl <ladis@linux-mips.org>
Signed-off-by: default avatarSean Young <sean@mess.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 5c95878f
...@@ -26,29 +26,19 @@ ...@@ -26,29 +26,19 @@
struct gpio_rc_dev { struct gpio_rc_dev {
struct rc_dev *rcdev; struct rc_dev *rcdev;
int gpio_nr; struct gpio_desc *gpiod;
bool active_low; int irq;
}; };
static irqreturn_t gpio_ir_recv_irq(int irq, void *dev_id) static irqreturn_t gpio_ir_recv_irq(int irq, void *dev_id)
{ {
int val;
struct gpio_rc_dev *gpio_dev = dev_id; struct gpio_rc_dev *gpio_dev = dev_id;
int gval;
int rc = 0;
gval = gpio_get_value(gpio_dev->gpio_nr); val = gpiod_get_value(gpio_dev->gpiod);
if (val >= 0)
ir_raw_event_store_edge(gpio_dev->rcdev, val == 1);
if (gval < 0)
goto err_get_value;
if (gpio_dev->active_low)
gval = !gval;
rc = ir_raw_event_store_edge(gpio_dev->rcdev, gval == 1);
if (rc < 0)
goto err_get_value;
err_get_value:
return IRQ_HANDLED; return IRQ_HANDLED;
} }
...@@ -57,7 +47,6 @@ static int gpio_ir_recv_probe(struct platform_device *pdev) ...@@ -57,7 +47,6 @@ static int gpio_ir_recv_probe(struct platform_device *pdev)
struct device *dev = &pdev->dev; struct device *dev = &pdev->dev;
struct device_node *np = dev->of_node; struct device_node *np = dev->of_node;
struct gpio_rc_dev *gpio_dev; struct gpio_rc_dev *gpio_dev;
enum of_gpio_flags flags;
struct rc_dev *rcdev; struct rc_dev *rcdev;
int rc; int rc;
...@@ -68,15 +57,17 @@ static int gpio_ir_recv_probe(struct platform_device *pdev) ...@@ -68,15 +57,17 @@ static int gpio_ir_recv_probe(struct platform_device *pdev)
if (!gpio_dev) if (!gpio_dev)
return -ENOMEM; return -ENOMEM;
rc = of_get_gpio_flags(np, 0, &flags); gpio_dev->gpiod = devm_gpiod_get(dev, NULL, GPIOD_IN);
if (rc < 0) { if (IS_ERR(gpio_dev->gpiod)) {
rc = PTR_ERR(gpio_dev->gpiod);
/* Just try again if this happens */
if (rc != -EPROBE_DEFER) if (rc != -EPROBE_DEFER)
dev_err(dev, "Failed to get gpio flags (%d)\n", rc); dev_err(dev, "error getting gpio (%d)\n", rc);
return rc; return rc;
} }
gpio_dev->irq = gpiod_to_irq(gpio_dev->gpiod);
gpio_dev->gpio_nr = rc; if (gpio_dev->irq < 0)
gpio_dev->active_low = (flags & OF_GPIO_ACTIVE_LOW); return gpio_dev->irq;
rcdev = devm_rc_allocate_device(dev, RC_DRIVER_IR_RAW); rcdev = devm_rc_allocate_device(dev, RC_DRIVER_IR_RAW);
if (!rcdev) if (!rcdev)
...@@ -101,11 +92,6 @@ static int gpio_ir_recv_probe(struct platform_device *pdev) ...@@ -101,11 +92,6 @@ static int gpio_ir_recv_probe(struct platform_device *pdev)
gpio_dev->rcdev = rcdev; gpio_dev->rcdev = rcdev;
rc = devm_gpio_request_one(dev, gpio_dev->gpio_nr, GPIOF_DIR_IN,
"gpio-ir-recv");
if (rc < 0)
return rc;
rc = devm_rc_register_device(dev, rcdev); rc = devm_rc_register_device(dev, rcdev);
if (rc < 0) { if (rc < 0) {
dev_err(dev, "failed to register rc device (%d)\n", rc); dev_err(dev, "failed to register rc device (%d)\n", rc);
...@@ -114,8 +100,7 @@ static int gpio_ir_recv_probe(struct platform_device *pdev) ...@@ -114,8 +100,7 @@ static int gpio_ir_recv_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, gpio_dev); platform_set_drvdata(pdev, gpio_dev);
return devm_request_irq(dev, gpio_to_irq(gpio_dev->gpio_nr), return devm_request_irq(dev, gpio_dev->irq, gpio_ir_recv_irq,
gpio_ir_recv_irq,
IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING, IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING,
"gpio-ir-recv-irq", gpio_dev); "gpio-ir-recv-irq", gpio_dev);
} }
...@@ -123,26 +108,24 @@ static int gpio_ir_recv_probe(struct platform_device *pdev) ...@@ -123,26 +108,24 @@ static int gpio_ir_recv_probe(struct platform_device *pdev)
#ifdef CONFIG_PM #ifdef CONFIG_PM
static int gpio_ir_recv_suspend(struct device *dev) static int gpio_ir_recv_suspend(struct device *dev)
{ {
struct platform_device *pdev = to_platform_device(dev); struct gpio_rc_dev *gpio_dev = dev_get_drvdata(dev);
struct gpio_rc_dev *gpio_dev = platform_get_drvdata(pdev);
if (device_may_wakeup(dev)) if (device_may_wakeup(dev))
enable_irq_wake(gpio_to_irq(gpio_dev->gpio_nr)); enable_irq_wake(gpio_dev->irq);
else else
disable_irq(gpio_to_irq(gpio_dev->gpio_nr)); disable_irq(gpio_dev->irq);
return 0; return 0;
} }
static int gpio_ir_recv_resume(struct device *dev) static int gpio_ir_recv_resume(struct device *dev)
{ {
struct platform_device *pdev = to_platform_device(dev); struct gpio_rc_dev *gpio_dev = dev_get_drvdata(dev);
struct gpio_rc_dev *gpio_dev = platform_get_drvdata(pdev);
if (device_may_wakeup(dev)) if (device_may_wakeup(dev))
disable_irq_wake(gpio_to_irq(gpio_dev->gpio_nr)); disable_irq_wake(gpio_dev->irq);
else else
enable_irq(gpio_to_irq(gpio_dev->gpio_nr)); enable_irq(gpio_dev->irq);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment