On 10/20/2015 02:30 PM, Krzysztof Kozlowski wrote: > W dniu 20.10.2015 o 18:22, Andrzej Hajda pisze: >> DECON-TV(Display and Enhancement Controller for TV) is a variation >> of DECON IP. Its main purpose is to produce video stream for HDMI IP. >> >> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx> >> --- >> .../devicetree/bindings/video/exynos5433-decon.txt | 21 ++++++++++++--------- >> 1 file changed, 12 insertions(+), 9 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >> index 3dff78b..2a88c8d 100644 >> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >> @@ -5,24 +5,27 @@ Exynos series of SoCs which transfers the image data from a video memory >> buffer to an external LCD interface. >> >> Required properties: >> -- compatible: value should be "samsung,exynos5433-decon"; >> +- compatible: value should be one of: >> + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; > Until this point it looked good. > >> - reg: physical base address and length of the DECON registers set. >> -- interrupts: should contain a list of all DECON IP block interrupts in the >> - order: VSYNC, LCD_SYSTEM. The interrupt specifier format >> - depends on the interrupt controller used. >> -- interrupt-names: should contain the interrupt names: "vsync", "lcd_sys" >> - in the same order as they were listed in the interrupts >> - property. >> +- interrupts: should contain interrupt specifier of VSYNC and optionally >> + LCD_SYSTEM. The interrupt specifier format depends on >> + the interrupt controller used. >> +- interrupt-names: should contain the interrupt name "vsync" and optionally >> + "lcd_sys" in the same order as they were listed in >> + the interrupts property. > The driver already did not require both interrupts, right? Only one of them? Right. More precisely it did not require since beginning. > >> - clocks: must include clock specifiers corresponding to entries in the >> clock-names property. >> - clock-names: list of clock names sorted in the same order as the clocks >> property. Must contain "pclk", "aclk_decon", "aclk_smmu_decon0x", >> "aclk_xiu_decon0x", "pclk_smmu_decon0x", clk_decon_vclk", >> "sclk_decon_eclk" >> + >> +Optional properties: >> - ports: contains a port which is connected to mic node. address-cells and >> - size-cells must 1 and 0, respectively. >> + size-cells must be 1 and 0, respectively. >> - port: contains an endpoint node which is connected to the endpoint in the mic >> - node. The reg value muset be 0. >> + node. The reg value must be 0. >> - i80-if-timings: specify whether the panel which is connected to decon uses >> i80 lcd interface or mipi video interface. This node contains >> no timing information as that of fimd does. Because there is >> > This is cleanup, please split it. Ok. I will split it into cleanup/fix patch, and 2nd one adding compatible. Regards Andrzej > > Best regards, > Krzysztof > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
