<div dir="ltr"><span style="font-size:12.8000001907349px">Tested by Brian Loften, "Brain Wreck" </span><a href="mailto:bloften80@gmail.com" target="_blank" style="font-size:12.8000001907349px">bloften80@gmail.com</a><span style="font-size:12.8000001907349px"> confirmed working on ASUS T100TA, kernel 20150629 -next running 15.04 i386 Ubuntu Gnome -- suspend resume is functioning normally, backlight controls work before and after resume using slide controls and meta keys on keyboard</span><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 28, 2015 at 4:18 PM, Paul Gortmaker <span dir="ltr"><<a href="mailto:paul.gortmaker@windriver.com" target="_blank">paul.gortmaker@windriver.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">[Re: [Intel-gfx] [v3 0/7] Crystalcove (CRC) PMIC based panel and pwm control] On 26/06/2015 (Fri 20:47) Ville Syrjälä wrote:<br>
<br>
> On Fri, Jun 26, 2015 at 06:31:37PM +0200, Daniel Vetter wrote:<br>
> > On Fri, Jun 26, 2015 at 02:32:03PM +0530, Shobhit Kumar wrote:<br>
> > > Hi,<br>
> > > Next update of the series reviewed at<br>
> > > <a href="https://lkml.org/lkml/2015/6/22/155" rel="noreferrer" target="_blank">https://lkml.org/lkml/2015/6/22/155</a><br>
> > ><br>
> > > Major changes are few review comments from Varka and Ville being addressed. Also except<br>
> > > for intel-gfx patches, all patches reviesion history is moved out of commit message.<br>
> > ><br>
> > > Hope this series finally finds its mark.<br>
> > ><br>
> > > Regards<br>
> > > Shobhit<br>
> > ><br>
> > > Shobhit Kumar (7):<br>
> > >   gpiolib: Add support for removing registered consumer lookup table<br>
> > >   mfd: intel_soc_pmic_core: Add lookup table for Panel Control as GPIO<br>
> > >     signal<br>
> > >   mfd: intel_soc_pmic_crc: Add PWM cell device for Crystalcove PMIC<br>
> > >   mfd: intel_soc_pmic_core: ADD PWM lookup table for CRC PMIC based PWM<br>
> > >   pwm: crc: Add Crystalcove (CRC) PWM driver<br>
> > >   drm/i915: Use the CRC gpio for panel enable/disable<br>
> > >   drm/i915: Backlight control using CRC PMIC based PWM driver<br>
> ><br>
> > I think we have r-b/acks on all the patches now. Ok if I pull this in<br>
> > through drm-intel.git for 4.3? Or should I make a topic branch with tag<br>
> > and then send out pull requests to everyone? Or will each maintainer merge<br>
> > on their own since it's all only coupled at runtime anyway? Any of these<br>
> > would suit me.<br>
><br>
> I forgot to mention that I had a build failure due to<br>
> builtin_platform_driver() when I tried this (just changed it to<br>
> module_platform_driver() to get past it). So I'm not sure if this<br>
> now depends on some tree which isn't included in -nightly...<br>
<br>
</div></div>builtin_platform_register does not yet exist in mainline; as Paul (the<br>
other one) said earlier.  So you can either open-code what it does for<br>
now, or use  module_platform_register.  If you do the latter, then<br>
ensure you (temorarily) also include module.h or you risk additional<br>
breakage in the future.<br>
<br>
Paul.<br>
--<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> --<br>
> Ville Syrjälä<br>
> Intel OTC<br>
</div></div></blockquote></div><br></div>