Re: Cherryview wake up events

From: Mika Westerberg
Date: Fri Sep 23 2016 - 04:28:02 EST


On Wed, Sep 21, 2016 at 11:16:35AM +0200, Johannes Stezenbach wrote:
> On Wed, Sep 21, 2016 at 12:06:14PM +0300, Mika Westerberg wrote:
> > On Tue, Sep 20, 2016 at 11:11:53PM +0200, Johannes Stezenbach wrote:
> > > Or it is because the PNP0C40 device depends on GpioInt from PMIC
> > > which isn't available...
> > >
> > > Method (_CRS, 0, NotSerialized) // _CRS: Current Resource Settings
> > > {
> > > Name (CBUF, ResourceTemplate ()
> > > {
> > > GpioInt (Edge, ActiveBoth, ExclusiveAndWake, PullUp, 0x0BB8,
> > > "\\_SB.PCI0.I2C7.PMI2", 0x00, ResourceConsumer, ,
> > > )
> > > { // Pin list
> > > 0x0016
> > > }
> > > })
> > > Return (CBUF) /* \_SB_.TBAD._CRS.CBUF */
> > > }
> >
> > Most likely this is the reason. I'll try to find if we have an existing
> > driver for this PMIC somewhere. I guess this is the Dollar Cove which is
> > successor of Crystal Cove IIRC which is already supported by the
> > mainline kernel.
>
> There is an ADBG ("TI_DCOVE") in PMI2._STA, so Dollar Cove
> sounds like a good guess.

David (CC'd) is working on getting the Dollar Cove PMIC driver
upstreamed to the mainline kernel.