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

From: Jin, Yao
Date: Mon Apr 21 2014 - 09:28:57 EST




On 2014/4/21 14:27, Jin, Yao wrote:
>
>
> On 2014/4/20 23:28, Adam Williamson wrote:
>> On Sun, 2014-04-20 at 20:08 +0800, Jin, Yao wrote:
>>> For the issue that touch screen doesn't work, could you check power
>>> state of LPSS devices? For example:
>>>
>>> cd /sys/bus/acpi/devices
>>> grep -H . */power_state
>>>
>>> If they are D3cold, it should be the reason why touch screen doesn't
>>> work. That's another issue, unrelated to this gpio patch.
>>
>> The touch screen worked fine with kernel 3.14, and Benjamin looked at
>> debug output from the attempt to load the touchscreen driver when
>> diagnosing the problem, he's not just guessing. I am building a kernel
>> without your patch to confirm that fixes it.
>>
>
> I tried the clean kernel 3.14 with the boot option "nomodeset text" on
> Dell Venue 11 Pro (If without "nomodeset",my ubuntu is being "black
> screen", but that should be another i915 issue).
>
> After system starup, I executed the "startx" to launch the xwindow. In
> xwindow, the touchscreen work.
>
> I tried the kernel 3.14 again with my gpio patch applied, the result was
> the same, the touchscreen work.
>
> I also check the DSDT table of Dell Venue 11 Pro by searching the
> keyword "Interrupt", I can't find any clue for the irq conflict.
>
> I'm sorry I don't have a Dell Venue 8 for testing, but I guess it's
> similar to Dell Venue 11 Pro.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>

Could you also add following change to disable the runtime PM of HSUART?
After that then check if your touchscreen work? It's to solve a known
issue.The change is based on 3.14.

--- a/drivers/tty/serial/8250/8250_dw.c
+++ b/drivers/tty/serial/8250/8250_dw.c
@@ -368,8 +368,6 @@ static int dw8250_probe(struct platform_device *pdev)

platform_set_drvdata(pdev, data);

- pm_runtime_set_active(&pdev->dev);
- pm_runtime_enable(&pdev->dev);

return 0;
}
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/