Linux输入驱动程序工作不正常

Linux输入驱动程序工作不正常,linux,input,kernel,driver,device,Linux,Input,Kernel,Driver,Device,我编写了一个小型linux输入驱动程序,它读取gpio的状态。 驱动程序注册正常,中断也被触发,但事件并不总是显示。 驱动程序运行在Android和内核版本3.8.13的Beagleboneblack上 要测试它,我可以执行以下操作之一: cat/dev/input/event2 或者运行我编写的用户空间应用程序。 该应用程序能够从其他输入设备正确检索事件。 要提到/dev/input中的“event2”是正确的事件,已经选中了 驾驶员代码: #include <linux/module.

我编写了一个小型linux输入驱动程序,它读取gpio的状态。 驱动程序注册正常,中断也被触发,但事件并不总是显示。 驱动程序运行在Android和内核版本3.8.13的Beagleboneblack上 要测试它,我可以执行以下操作之一:

cat/dev/input/event2

或者运行我编写的用户空间应用程序。 该应用程序能够从其他输入设备正确检索事件。 要提到/dev/input中的“event2”是正确的事件,已经选中了

驾驶员代码:

#include <linux/module.h>                                                                                                                                                                                
#include <linux/interrupt.h>
#include <linux/platform_device.h>
...

struct button_drv {

·   int··   ·   ·   ·   btn_irq;
·   int ·   ·   ·   ·   btn_gpio;
·   struct device·  ·   *dev;
·   struct input_dev·   *btn_input;
};

static irqreturn_t btn_irq(int irq_nb, void *data) {

·   struct button_drv *btn_drv = (struct button_drv*)data;
·   struct input_dev *btn_input = btn_drv->btn_input;
·   int btn_gpio = btn_drv->btn_gpio;

·   dev_info(btn_drv->dev,"Inside button IRQ!\n");
·   input_report_key(btn_input,BTN_0,gpio_get_value(btn_gpio));
·   input_sync(btn_input);

·   return IRQ_HANDLED;
}

static int btn_probe(struct platform_device *pdev) {

·   int ret,btn_irq,btn_gpio;
·   struct button_drv *btn_drv;
·   struct input_dev *btn_input;
·   struct device_node *node = pdev->dev.of_node;
·   
·   dev_info(&pdev->dev,"Inside btn probe driver!\n");
·   btn_drv = devm_kzalloc(&pdev->dev,sizeof(struct button_drv),GFP_KERNEL);
·   if(!btn_drv)
·   {
·   ·   dev_err(&pdev->dev,"Failed to allocate memory for btn device!\n");
·   ·   return -ENOMEM;
·   }

·   platform_set_drvdata(pdev,btn_drv);
·   btn_gpio = of_get_named_gpio(node,"btn-gpios",0);
·   
·   ret = devm_gpio_request_one(&pdev->dev,btn_gpio,GPIOF_DIR_IN,"btn");
·   if(IS_ERR_VALUE(ret))
    {
  ·   dev_err(&pdev->dev,"Failed to allocate btn gpio!\n");
        ret = -ENODEV;
·   ·   goto err_mem;
   }
·   
·   btn_irq = gpio_to_irq(btn_gpio);
·   if(IS_ERR_VALUE(btn_irq))
·   {
   ·   dev_err(&pdev->dev,"Failed to get corresponding btn IRQ!\n");
   ·   ret = -ENODEV;
   ·   goto err_out;
·   }

·   ret = devm_request_irq(&pdev->dev,btn_irq,btn_irq,
·   ·   ·   IRQF_TRIGGER_FALLING | IRQF_ONESHOT,pdev->name, btn_drv);
·   if(IS_ERR_VALUE(ret))
·   {
·   ·   dev_err(&pdev->dev,"Failed to map btn IRQ!\n");
·   ·   ret = -EINVAL;
·   ·   goto err_out;
·   }

·   btn_input = input_allocate_device();
·   if(IS_ERR(btn_input))
·   {
·   ·   dev_err(&pdev->dev,"No memory for btn input device!");
·   ·   ret = -ENODEV;                                                                                                                                                                                   
·   ·   goto err_out;
·   }
·   btn_input->evbit[0] = BIT_MASK(EV_KEY);
·   btn_input->keybit[BIT_WORD(BTN_0)] = BIT_MASK(BTN_0);
·   btn_input->name = "button1";
    btn_input->phys = "button1/input0";
·   ret = input_register_device(btn_input);
·   if(IS_ERR_VALUE(ret))
·   {
·   ·   dev_err(&pdev->dev,"Failed to register btn input device!\n");
·   ·   ret = -ENODEV;
·   ·   goto err_out;
·   }

·   btn_drv->btn_gpio · = btn_gpio;
·   btn_drv->btn_irq ·  = btn_irq;
·   btn_drv->btn_input ·= btn_input;
·   btn_drv->dev·   ·   = &pdev->dev;

·   dev_info(&pdev->dev,"Registered btn input device!");
·   
·   return 0;

err_out:
·   devm_gpio_free(&pdev->dev,btn_gpio);

err_mem:
·   platform_set_drvdata(pdev,NULL);
·   devm_kfree(&pdev->dev, btn_drv);
·   return ret;
}

static int btn_remove(struct platform_device *pdev) {

·.......

·   return 0;
}                                                      


static struct platform_driver btn_drv = {
·   .probe· ·   = btn_probe,
·   .remove··   = btn_remove,
...
};


static int __init btn_init(void) {

·   return platform_driver_register(&btn_drv);
}
module_init(btn_init);

static void __exit btn_exit(void) {

·   platform_driver_unregister(&btn_drv);
}
module_exit(btn_exit);


MODULE_DESCRIPTION("Button driver");
MODULE_AUTHOR("Test");
MODULE_LICENSE("GPL");        
我在irq例程中放置了一个
dev\u info
,以便在调用
input\u report\u键时查看。
有时我得到更多的事件,有时更多的IRQ,有时我只得到IRQ消息

因此,我没有遗漏任何IRQ,似乎与关键报告事件有关。我在想,如果按下按钮时IRQ彼此靠近,可能会重叠,这会影响输入流

将尝试更改代码,强制每10毫秒报告一次密钥,以查看是否有任何区别。 我没有别的想法。 非常感谢您的帮助

更新:所以,我修改了代码,每10毫秒报告一次,每秒报告一次,没有区别,一定是其他原因

有什么想法吗

谢谢,, 丹尼尔看来我找到了问题所在。 设置中断时,它应该在两个边缘上触发,因为linux输入系统只报告事件值的变化,因此当同一边缘触发IRQ时,大多数时间事件都是相同的,在我的例子中是“低”。这意味着开关被释放,但在实际代码中,开关从未被视为被按下(除非gpio上的电平还不稳定,可以读取高电平而不是低电平)。看来这是我收到事件报告的唯一案例

通过以下更改,驱动程序将按预期工作:

在btn_探头更换中:

ret = devm_request_irq(&pdev->dev,btn_irq,btn_irq,
        IRQF_TRIGGER_FALLING | IRQF_ONESHOT,pdev->name, btn_drv);

重新编写IRQ以避免在短时间内出现多个IRQ:

static irqreturn_t btn_irq(int irq_nb, void *data) {

·   struct button_drv *btn_drv = (struct button_drv*)data;
    struct input_dev *btn_input = btn_drv->btn_input;
·   int btn_gpio = btn_drv.btn_gpio;
·   int val;

·   if(time_before(jiffies,btn_drv.new_time))
·   {
·   ·   btn_drv.scan_disc++;
·   ·   return IRQ_HANDLED;
·   }

·   if(!gpio_get_value(btn_gpio))
·   {
·   ·   pr_info("btn-drv: switch pressed!\n");
·   ·   val = 1;
·   }
·   else
·   {
·   ·   pr_info("btn-drv: switch released!\n");
·   ·   val = 0;
·   }

·   input_report_key(btn_input,BTN_0,val);
·   input_sync(btn_input);

·   btn_drv.new_time = jiffies + SCAN_DELTA;

·   return IRQ_HANDLED;
}
与:

#定义扫描增量∙ 微秒到微秒(100)∙  //毫秒

new\u time
scan\u disc
btn\u drv结构中的新条目

ret = devm_request_irq(&pdev->dev,btn_irq,btn_irq,
        IRQF_TRIGGER_FALLING | IRQF_ONESHOT,pdev->name, btn_drv);
ret = devm_request_irq(&pdev->dev,btn_irq,btn_irq,
        IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING | IRQF_ONESHOT,pdev->name, btn_drv);
static irqreturn_t btn_irq(int irq_nb, void *data) {

·   struct button_drv *btn_drv = (struct button_drv*)data;
    struct input_dev *btn_input = btn_drv->btn_input;
·   int btn_gpio = btn_drv.btn_gpio;
·   int val;

·   if(time_before(jiffies,btn_drv.new_time))
·   {
·   ·   btn_drv.scan_disc++;
·   ·   return IRQ_HANDLED;
·   }

·   if(!gpio_get_value(btn_gpio))
·   {
·   ·   pr_info("btn-drv: switch pressed!\n");
·   ·   val = 1;
·   }
·   else
·   {
·   ·   pr_info("btn-drv: switch released!\n");
·   ·   val = 0;
·   }

·   input_report_key(btn_input,BTN_0,val);
·   input_sync(btn_input);

·   btn_drv.new_time = jiffies + SCAN_DELTA;

·   return IRQ_HANDLED;
}