[PATCH 5/5] of: Modify c_can binding documentation
Stephen Warren
swarren at wwwdotorg.org
Sun Sep 2 12:02:38 EST 2012
On 09/01/2012 12:05 AM, AnilKumar, Chimata wrote:
> On Fri, Aug 31, 2012 at 14:59:21, AnilKumar, Chimata wrote:
>> Modify c_can binding documentation according to recent review comments
>> on device tree data addition patches.
>> diff --git a/Documentation/devicetree/bindings/net/can/c_can.txt b/Documentation/devicetree/bindings/net/can/c_can.txt
>> index a43f083..90a70be 100644
>> --- a/Documentation/devicetree/bindings/net/can/c_can.txt
>> +++ b/Documentation/devicetree/bindings/net/can/c_can.txt
>> @@ -8,6 +8,8 @@ Required properties:
>> registers map
>> - interrupts : property with a value describing the interrupt
>> number
>> +- status : describes the node status either "disabled" or
>> + "okay"
That's a standrd property that applies to any node, and doesn't describe
data required by the device itself (as do regs/interrupts) by simply
whether the node is activated; I'm not sure it's worth mentioning it in
a device-specific binding.
A similar comment exists for the pre-existing description of
interrupt-parent below.
>> Optional properties:
>> - interrupt-parent : The parent interrupt controller
More information about the devicetree-discuss
mailing list