Re: [PATCH 1/2] Documentation: devicetree: mfd: twl4030-power: Use the standard 'ti,system-power-controller' to mark power control

From: Nishanth Menon
Date: Wed Sep 03 2014 - 16:31:51 EST


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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/