Re: [PATCH 2/2] ARM: dts: doc: Document missing binding for ti-sdma controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Nov 9, 2013 at 5:53 AM, Sricharan R <r.sricharan@xxxxxx> wrote:
> Hi Rob,
>
> On Saturday 09 November 2013 03:24 AM, Rob Herring wrote:
>> On Fri, Nov 8, 2013 at 4:38 AM, Sricharan R <r.sricharan@xxxxxx> wrote:
>>> The binding and support for system DMA module present in OMAP2+ Socs
>>> and DRA7 with dma-engine is already present. So adding the missing
>>> documentation for the same.
>>>
>>> Cc: Benoit Cousson <bcousson@xxxxxxxxxxxx>
>>> Signed-off-by: Sricharan R <r.sricharan@xxxxxx>
>>> ---
>>>  Documentation/devicetree/bindings/dma/ti-sdma.txt |   28 +++++++++++++++++++++
>>>  1 file changed, 28 insertions(+)
>>>  create mode 100644 Documentation/devicetree/bindings/dma/ti-sdma.txt
>>>
>>> diff --git a/Documentation/devicetree/bindings/dma/ti-sdma.txt b/Documentation/devicetree/bindings/dma/ti-sdma.txt
>>> new file mode 100644
>>> index 0000000..7fb2342
>>> --- /dev/null
>>> +++ b/Documentation/devicetree/bindings/dma/ti-sdma.txt
>>> @@ -0,0 +1,28 @@
>>> +TI SDMA
>>> +
>>> +Required Properties:
>>> +- compatible : Should be "ti,omap2420-sdma", "ti,omap2430-sdma" for OMAP2
>>> +               Should be "ti,omap3430-sdma", "ti,omap3630-sdma" for OMAP3
>>> +               Should be "ti,omap4430-sdma" for OMAP4, OMAP5 and DRA7
>>> +- reg: Memory map for accessing module
>>> +- interrupts: Interrupt request number from the dma controller to
>> s/dma/DMA/
>>
>>> +             interrupt controller
>>> +- dma-cells: Should be set to <1>
>>> +- dma-channels: Total number of logical DMA channels supported by the
>>> +               controller
>>> +- dma-requests: Total number of dma request signals supported by the
>> s/dma/DMA/
>  Ok, will correct case.
>>> +               controller
>> These should have the # included.
>  Sorry, i do not follow this ?

You are missing the # in the property names as the example has:

+       #dma-cells = <1>;
+       #dma-channels = <32>;
+       #dma-requests = <64>;

Rob
--
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




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux