<br><br>On Thursday, 14 February 2013, Sylwester Nawrocki <<a href="mailto:sylvester.nawrocki@gmail.com">sylvester.nawrocki@gmail.com</a>> wrote:<br>> On 02/06/2013 12:51 PM, Sachin Kamat wrote:<br>>><br>>> Added documentaion about G2D bindings.<br>
><br>> s/documentaion/documentation<br><br>Ok.<br>><br>>> Signed-off-by: Sachin Kamat<<a href="mailto:sachin.kamat@linaro.org">sachin.kamat@linaro.org</a>><br>>> ---<br>>>   .../devicetree/bindings/gpu/samsung-g2d.txt        |   30 ++++++++++++++++++++<br>
>>   1 files changed, 30 insertions(+), 0 deletions(-)<br>>>   create mode 100644 Documentation/devicetree/bindings/gpu/samsung-g2d.txt<br>>><br>>> diff --git a/Documentation/devicetree/bindings/gpu/samsung-g2d.txt b/Documentation/devicetree/bindings/gpu/samsung-g2d.txt<br>
>> new file mode 100644<br>>> index 0000000..838b26b<br>>> --- /dev/null<br>>> +++ b/Documentation/devicetree/bindings/gpu/samsung-g2d.txt<br>>> @@ -0,0 +1,30 @@<br>>> +* Samsung 2D Graphics Accelerator<br>
>> +<br>><br>>> +Samsung FIMG2D is a 2D graphics accelerator which supports Bit Block Transfer.<br>>> +We set the drawing-context registers for configuring the rendering parameters<br>>> +and then start rendering.<br>
>> +The G2D has V4L2 as well as DRM framework based driver support.<br>><br>> IMHO this whole paragraph is irrelevant and could be dropped.<br><br>Ok. Had formatted this based on mfc document.<br>><br>>> +Required properties:<br>
>> +  - compatible : value should be one among the following<br>><br>> missing colon at the end of line ?<br><br>Hmm. I think so.<br><br>><br>>> +       (a) "samsung,s5pv210-g2d" for G2D IP present in S5PV210&  Exynos4210 SoC<br>
>> +       (b) "samsung,exynos4212-g2d" for G2D IP present in Exynos4x12 SoCs<br>>> +       (b) "samsung,exynos5250-g2d" for G2D IP present in Exynos5250 SoC<br>><br>> s/(b)/(c)<br><br>
Yes, copy paste error :)<br><br>><br>>> +<br>>> +  - reg : Physical base address of the IP registers and length of memory<br>>> +         mapped region.<br>>> +<br>>> +  - interrupts : G2D interrupt number to the CPU.<br>
>> +<br>>> +Optional properties:<br>>> +  - samsung,power-domain : power-domain property defined with a phandle<br>>> +                          to respective power domain.<br>>> +<br>>> +Example:<br>
>> +       g2d@12800000 {<br>>> +               compatible = "samsung,s5pv210-g2d";<br>>> +               reg =<0x12800000 0x1000>;<br>>> +               interrupts =<0 89 0>;<br>
>> +               samsung,power-domain =<&pd_lcd0>;<br>>> +               status = "disabled";<br>>> +       };<br>><br>> Otherwise looks good to me.<br><br>Thanks for reviewing. Does the filename look good or should it be exynos-g2d?<br>
<br>Regards,<br>Sachin.<br><br>-- <br><div>With warm regards,</div>
<div>Sachin</div><br>