Hi Hans, > -----Original Message----- > From: Hans de Goede [mailto:[email protected]] > Sent: Tuesday, June 06, 2017 8:22 AM > To: Andy Shevchenko <[email protected]>; Mani, Rajmohan > <[email protected]> > Cc: [email protected]; [email protected]; linux- > [email protected]; Lee Jones <[email protected]>; Linus Walleij > <[email protected]>; Alexandre Courbot <[email protected]>; Rafael J. > Wysocki <[email protected]>; Len Brown <[email protected]> > Subject: Re: [PATCH v1 3/3] ACPI / PMIC: Add TI PMIC TPS68470 operation > region driver > > Hi, > > On 06/06/2017 04:23 PM, Andy Shevchenko wrote: > > +Cc Hans (that's why didn't delete anything from original mail, just > > adding my comments). > > > > Hans, if you have few minutes it would be appreciated to glance on the > > below for some issues if any since you did pass quite a good quest > > with other PMIC drivers. > > I've gone over this driver, nothing stands out in a bad way to me, IOW this > seems like a normal PMIC OpRegion handler to me. >
Thanks for the reviews and time.
