Re: RFR: 8201568: zForce touchscreen input device fails when closed and immediately reopened [v2]

2020-12-13 Thread Johan Vos
On Sun, 13 Dec 2020 16:21:52 GMT, Johan Vos wrote: >> John Neffenger has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains two additional >> commits s

Re: RFR: 8201568: zForce touchscreen input device fails when closed and immediately reopened [v2]

2020-12-13 Thread Johan Vos
On Fri, 25 Sep 2020 22:21:36 GMT, John Neffenger wrote: >> Fixes [JDK-8201568](https://bugs.openjdk.java.net/browse/JDK-8201568). > > John Neffenger has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought

Re: RFR: 8201568: zForce touchscreen input device fails when closed and immediately reopened [v2]

2020-12-11 Thread John Neffenger
On Tue, 29 Sep 2020 19:47:54 GMT, John Neffenger wrote: >> I don't have access to a Pi right now, so I can't test this (I'll be able to >> test in about 10 days from now though) > > @johanvos, would you mind approving this pull request again? It looks as if > the *ready* label was removed when

Re: RFR: 8201568: zForce touchscreen input device fails when closed and immediately reopened [v2]

2020-09-29 Thread John Neffenger
On Mon, 21 Sep 2020 18:34:29 GMT, John Neffenger wrote: >> Based on my notes below, I believe this change is safe for any Linux input >> device driver because the driver shouldn't >> receive the intermediate *open* and *close* calls at all. >> Nonetheless, it would be reassuring if someone coul

Re: RFR: 8201568: zForce touchscreen input device fails when closed and immediately reopened [v2]

2020-09-25 Thread John Neffenger
> Fixes [JDK-8201568](https://bugs.openjdk.java.net/browse/JDK-8201568). John Neffenger has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional c