On Wed, Sep 3, 2014 at 2:53 PM, Sebastian Reichel <sre@xxxxxxxxxx> wrote: > On Wed, Sep 03, 2014 at 02:06:47PM -0500, Nishanth Menon wrote: >> Optional properties: >> -- ti,use_poweroff: With this flag, the chip will initiates an ACTIVE-to-OFF or >> - SLEEP-to-OFF transition when the system poweroffs. >> + >> +- ti,system-power-controller OR ti,use_poweroff: This indicates that >> + TWL4030 is the power supply master of the system. With this flag, the >> + chip will initiates an ACTIVE-to-OFF or SLEEP-to-OFF transition when >> + the system poweroffs. > > I suggest the following wording: > > - ti,system-power-controller: This indicates that TWL4030 is the > power supply master of the system. With this flag, the chip will > initiates an ACTIVE-to-OFF or SLEEP-to-OFF transition when the > system poweroffs. > > - ti,use_poweroff: Deprecated name for ti,system-power-controller Yes, this does indeed sound better. if no one has a different opinion or further comments, will update this patch tomorrow. -- --- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html