Re: [PATCH] pinctrl-baytrail: fix for irq descriptor conflict on ASUS T100TA

[Adam Williamson <awilliam@redhat.com>]

On Fri, 2014-04-18 at 16:44 -0400, Benjamin Tissoires wrote:
> On Wed, Apr 16, 2014 at 8:05 AM, Jin Yao <yao.jin@linux.intel.com> wrote:
> > A crash is triggered on the ASUS T100TA Baytrail-T because of a irq
> > descriptor conflict. There are two gpio triggered acpi events in this
> > device, GPIO 6 and 18. These gpios are translated to irqs by calling
> > gpio_to_irq which in turn will call irq_create_mapping(vg->domain, offset).
> > irq_create_mapping will take care of allocating the irq descriptor, taking
> > the first available number starting from the given value (6 in our case).
> > The 0-15 are already reserved by legacy ISA code, so it gets the first
> > free irq descriptor which is number 16. The i915 driver also uses irq 16,
> > it loads later than gpio and crashes in probe.
> >
> > The bug is reported here:
> > https://bugzilla.kernel.org/show_bug.cgi?id=68291
> 
> Adam, the reporter of this bug told me that his touchscreen was broken
> in its tablet.
> I strongly suspect this patch to be the root cause of this, because
> the touchscreen uses i2c_hid. i2c_hid relies on an IRQ declared in the
> DSDT when it is acpi enumerated, and since the inclusion of this
> patch, no irq are triggered from a driver point of view.
> 
> Adam should still confirm that the revert of the patch makes the
> touchscreen back alive, but if I understood correctly the bug report,
> without the patch, his tablet oopses at boot.

if IRQ allocation is enabled, yeah. What I can do is test with the
#68291 patch dropped and IRQ allocation disabled in pinctrl-baytrail.c -
as is done in Doug Johnson's patch,
http://dougvj.net/baytrail_gpio_quirk.patch , for #67921 , with these
two blocks:

@@ -532,8 +535,8 @@ static int byt_gpio_probe(struct platform_device
*pdev)
 		dev_err(&pdev->dev, "failed adding byt-gpio chip\n");
 		return ret;
 	}
-
 	/* set up interrupts  */
+/*
 	irq_rc = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
 	if (irq_rc && irq_rc->start) {
 		hwirq = irq_rc->start;
@@ -548,7 +551,7 @@ static int byt_gpio_probe(struct platform_device
*pdev)
 
 		irq_set_handler_data(hwirq, vg);
 		irq_set_chained_handler(hwirq, byt_gpio_irq_handler);
-	}
+	}*/
 
 	pm_runtime_enable(dev);
 

so, I'll do a test of that setup, and see how it goes.
-- 
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | XMPP: adamw AT happyassassin . net
http://www.happyassassin.net