On 11/12/2014 04:54 PM, Naitik Amin wrote:
HI there, I am trying to use the gpio driver for pca953x with IRQ enabled config. When the device gets probbed, I gives me an error saying : "failed to set trigger mode 8" The probe called, ret = devm_request_threaded_irq(&client->dev, client->irq, NULL, pca953x_irq_handler, IRQF_TRIGGER_LOW | IRQF_ONESHOT, dev_name(&client->dev), chip); with IRQF_TRIGGER_LOW as a flag
try to remove IRQF_TRIGGER_LOW
I did some tracing and found out, irq set type, went to gic_set_type in irq- gic.c ( http://lxr.free-electrons.com/source/drivers/irqchip/irq-gic.c#L187 ) But gic_set_type, only wants to see either IRQF_TRIGGER_HIGH OR IRQF_TRIGGER_LOW, thats why it returned.
if (type != IRQ_TYPE_LEVEL_HIGH && type != IRQ_TYPE_EDGE_RISING)
Questions 1. Is it correct that irq set type went to gic_set_type, bcoz pca953x does provide one of itself.
Valid IRQ types are IRQ controller specific values. You don't need all this magic in code any more if you are using DT, which should provide valid configuration of IRQ type. if not-DT - res.flags should be used. regards, -grygorii -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
