On Wed, Jan 31, 2018 at 6:20 PM, Fabio Estevam <feste...@gmail.com> wrote: > Hi Daniel,
Fabio, +1 to your review. It seems it repeats some of the points I made. >> + ak5558->pdn_gpio = of_get_named_gpio(np, "ak5558,pdn-gpio", 0); > > It does not match the property in the binding doc: asahi-kasei,pdn-gpios Btw, it must be simple "reset" as a widely used for the same purpose. -- With Best Regards, Andy Shevchenko