On 06/08/2017 07:12 PM, Rob Herring wrote:
> On Fri, Jun 02, 2017 at 04:37:14PM +0200, Philippe CORNU wrote:
>> This patch adds documentation of device tree bindings for the STM32
>> DSI host driver based on the Synopsys DW MIPI DSI bridge driver.
>> ---
>> .../devicetree/bindings/display/st,stm32-ltdc.txt | 83 +++++++++++++++++++++-
>> 1 file changed, 82 insertions(+), 1 deletion(-)
>>
>> diff --git a/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt b/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt
>> index 90a8459..4cd32bd 100644
>> --- a/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt
>> +++ b/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt
>> @@ -12,8 +12,39 @@
>> Required nodes:
>> - Video port for RGB output.
>>
>> -Example:
>> +* STMicroelectronics STM32 dsi specific extensions to Synopsys DesignWare MIPI
>> + DSI host controller
>>
>> +The STMicroelectronics STM32 dsi driver uses the Synopsys DesignWare MIPI
>> +DSI host controller driver (related documentation in [5]).
>> +
>> +Required properties:
>> +- #address-cells: Should be <1>.
>> +- #size-cells: Should be <0>.
>> +- compatible: "st,stm32-dsi".
>> +- reg: Memory mapped base address and length of the DWC MIPI DSI registers.
>> +- clocks: References to all the clocks specified in the clock-names property
>> + as specified in [1].
>> +- clock-names:
>> + - peripheral clock string name, must be "pclk".
>> + - phy pll reference clock string name, must be "ref".
>
> I still have the same comment as v1.
>
Hi Rob,
I added the "reference to [5]" just few lines above (before the required
properties) and I though it was enough : )
In the v4 version I have just sent, I have tried to be more explicit
regarding this reference to [5] (what is optional and what is mandatory...).
Many thanks for your help,
>> +- resets: References to all the resets specified in the reset-names property
>> + as specified in [2].
>> +- reset-names: string reset name, must be "apb".
>> +
>> +Required nodes:
>> +- dsi input port node: connected to the ltdc rgb output port, see [3] & [4].
>> +- panel or bridge node: see [6].
>
> Need to be explicit here. Port 0 is ... Port 1 is ...
>
Done in v4, many thanks for your comment.
>> +
>> +Note: You can find more documentation
>> +[1] Documentation/devicetree/bindings/clock/clock-bindings.txt
>> +[2] Documentation/devicetree/bindings/reset/reset.txt
>> +[3] Documentation/devicetree/bindings/media/video-interfaces.txt
>> +[4] Documentation/devicetree/bindings/graph.txt
>> +[5] Documentation/devicetree/bindings/display/bridge/dw_mipi_dsi.txt
>> +[6] Documentation/devicetree/bindings/display/mipi-dsi-bus.txt
>> +
>> +Example 1: RGB panel
>> / {
>> ...
>> soc {
>> @@ -33,3 +64,53 @@ Example:
>> };
>> };
>> };
>> +
>> +Example 2: DSI panel
>> +
>> +/ {
>> + ...
>> + soc {
>> + ...
>> + ltdc: display-controller@40016800 {
>> + compatible = "st,stm32-ltdc";
>> + reg = <0x40016800 0x200>;
>> + interrupts = <88>, <89>;
>> + resets = <&rcc STM32F4_APB2_RESET(LTDC)>;
>> + clocks = <&rcc 1 CLK_LCD>;
>> + clock-names = "lcd";
>> +
>> + port {
>> + ltdc_out_dsi: endpoint {
>> + remote-endpoint = <&dsi_in>;
>> + };
>> + };
>> + };
>> +
>> +
>> + dsi: dsi@40016c00 {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + compatible = "st,stm32-dsi";
>> + reg = <0x40016c00 0x800>;
>> + clocks = <&rcc 1 CLK_F469_DSI>, <&clk_hse>;
>> + clock-names = "ref", "pclk";
>> + resets = <&rcc STM32F4_APB2_RESET(DSI)>;
>> + reset-names = "apb";
>> +
>> + port {
>> + dsi_in: endpoint {
>> + remote-endpoint = <<dc_out_dsi>;
>> + };
>> + };
>> +
>> + panel-dsi@0 {
>
> This should use graph binding instead. Or at least we should allow that
> and be clear above that we do.
>
Done in v4, many thanks for your comment.
Philippe
>> + reg = <0>; /* dsi virtual channel (0..3) */
>> + compatible = ...;
>> + enable-gpios = ...;
>> +
>> + };
>> +
>> + };
>> +
>> + };
>> +};
>> --
>> 1.9.1
>>��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
