Hi Archit,

Thank you for the patch.

On Friday 02 August 2013 19:33:43 Archit Taneja wrote:
> Add a DT node for VPE in dra7.dtsi. This is experimental because we might
> need to split the VPE address space a bit more, and also because the IRQ
> line described is accessible the IRQ crossbar driver is added for DRA7XX.
> 
> Cc: Rajendra Nayak <rna...@ti.com>
> Cc: Sricharan R <r.sricha...@ti.com>
> Signed-off-by: Archit Taneja <arc...@ti.com>
> ---
>  arch/arm/boot/dts/dra7.dtsi | 11 +++++++++++

Documentation is missing :-) As this is an experimental patch you can probably 
document the bindings later.

>  1 file changed, 11 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
> index ce9a0f0..3237972 100644
> --- a/arch/arm/boot/dts/dra7.dtsi
> +++ b/arch/arm/boot/dts/dra7.dtsi
> @@ -484,6 +484,17 @@
>                       dmas = <&sdma 70>, <&sdma 71>;
>                       dma-names = "tx0", "rx0";
>               };
> +
> +             vpe {
> +                     compatible = "ti,vpe";
> +                     ti,hwmods = "vpe";
> +                     reg = <0x489d0000 0xd000>, <0x489dd000 0x400>;
> +                     reg-names = "vpe", "vpdma";
> +                     interrupts = <0 159 0x4>;
> +                     #address-cells = <1>;
> +                     #size-cells = <0>;

Are #address-cells and #size-cells really needed ?

> +             };
> +
>       };
> 
>       clocks {
-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to