On 11/05/16 17:07, Stephen Warren wrote: > On 05/11/2016 03:19 AM, Linus Walleij wrote: >> On Mon, May 2, 2016 at 2:17 PM, Laxman Dewangan <[email protected]> >> wrote: >> >>> NVIDIA Tegra210 supports the IO pads which can operate at 1.8V >>> or 3.3V I/O voltage levels. Also the IO pads can be configured >>> for power down state if it is not used. SW needs to configure the >>> voltage level of IO pads based on IO rail voltage and its power >>> state based on platform usage. >>> >>> The voltage and power state configurations of pads are provided >>> through pin control frameworks. Add pin control driver for Tegra's >>> IO pads' voltage and power state configurations. >>> >>> Signed-off-by: Laxman Dewangan <[email protected]> >>> >>> --- >>> Changes from V1: >>> - Use the generic pincontrol property for low power enable/disable. >>> - sync with latest and remove all MODULE* macros >>> - Use renamed APIs form PMC >> >> Looks OK to me, need Stephen's ACK though! > > This driver is for HW that's separate from the main Tegra pinmux module. > Thierry Reding and Jon Hunter are more appropriate reviewers for this one.
There will be another spin of this, so let's wait for the V2. Cheers Jon

