> Date: Wed, 19 Jul 2017 04:32:45 +0300 > From: Artturi Alm <artturi....@gmail.com> > > On Sun, Jul 16, 2017 at 11:13:35PM +0200, Mark Kettenis wrote: > > > Date: Sun, 9 Jul 2017 20:34:29 +0300 > > > From: Artturi Alm <artturi....@gmail.com> > > > > > > Hi, > > > > > > revived the diff below, i2c tested via pmic's shutdown(), for working > > > "shutdown -p now" operation. > > > there was only two i2c's w/"status: 'okay'" in the FDT, so not all of > > > them do attach. > > > > > > related part of dmesg: > > > > > > com0: console > > > sxitwi0 at simplebus0 > > > iic0 at sxitwi0 > > > axppmic0 at iic0 addr 0x34: AXP209, ACIN > > > sxitwi1 at simplebus0 > > > iic1 at sxitwi1 > > > dwge0 at simplebus0 > > > > > > Comments? > > > -Artturi > > > > It's a pity that the PSCI "firmware" doesn't do an actual shutdown. > > But having i2c support is worth having in its own right. > > > > A bit of a step backwards to add code under the old-style 4-clause BSD > > license, but I believe that is still acceptable. > > > > I don't think we'll ever support the Marvell Discovery hardware, so > > I'd just fold the gttwsi_core.c code into sxitwi.c and get rid of the > > GTTWSI_ALLWINNER hack. > > > > done in diff below, hope i understood correctly what you meant.
Pretty much. I've made a few more cleanups and committed it. Thanks! Mark
