On Tue, 2017-07-25 at 07:43 +0200, Yegor Yefremov wrote:
> On Mon, Jul 24, 2017 at 8:48 PM, Yegor Yefremov
> <yegorslists@xxxxxxxxxxxxxx> wrote:
> > Hi Andy,
> >
> > On Mon, Jul 24, 2017 at 8:39 PM, Andy Shevchenko
> > <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> > > Hi!
> > >
> > > Since my big ACPI GPIO fix made the vanilla, I think we may return
> > > back
> > > the commit 4ef03d328769 ("tty/serial/8250: use mctrl_gpio
> > > helpers").
> > >
> > > I just have tested it on two x86 boards:
> > > 1) Broxton (without _DSD properties)
> > > 2) ApolloLake (with _DSD properties for RX and CTS pins)
> > >
> > > Opinions, more testing?
Alas, I did more deep testing and the patch breaks the console.
It looks like we need to distinguish what those GPIOs are used for:
a) modem control lines, or
b) wakeup source.
There are few options to distinguish:
1) check if GPIO resource is marked as wakeup source (ACPI only)
2) use "wakeup-source" device property, for now looks like there is no
serial driver is using it (might be collision with the real wake capable
serial drivers)
3) similar to 2), though introduce another property like "oob-wakeup-
source" or any variations of that
4) something else?
> > >
> > > I would like to have more independent testing (Mika? Others?).
> > >
> > > Btw, Yegor, do we still need it?
> >
> > Yes, I do. Thanks for looking at it again. This is almost the only
> > part missing before Baltos systems could be copletely upstreamed.
> >
> > Yegor
> >
> > > P.S. I cherry-picked it on top of recent linux-next and tried. I
> > > can
> > > send a formal patch, but I think it's not needed (it applies
> > > smoothly).
>
> Btw have you also tested this patch [1]?
>
> [1] http://www.spinics.net/lists/linux-serial/msg24053.html
See above. This patch basically is not needed anymore, we need something
else.
--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
