Re: [PATCH v1 1/5] dt-bindings: display: stm32: add supply property to DSI controller

[Philippe CORNU <philippe.cornu@xxxxxx>]

On 4/3/19 10:16 AM, Yannick Fertré wrote:
> This patch adds documentation of a new property phy-dsi-supply to the
> STM32 DSI controller.
> 
> Signed-off-by: Yannick Fertré <yannick.fertre@xxxxxx>
> ---
>   Documentation/devicetree/bindings/display/st,stm32-ltdc.txt | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt b/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt
> index 3eb1b48..60c54da 100644
> --- a/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt
> +++ b/Documentation/devicetree/bindings/display/st,stm32-ltdc.txt
> @@ -40,6 +40,8 @@ Mandatory nodes specific to STM32 DSI:
>   - panel or bridge node: A node containing the panel or bridge description as
>     documented in [6].
>     - port: panel or bridge port node, connected to the DSI output port (port@1).
> +Optional properties:
> +- phy-dsi-supply: phandle of the regulator that provides the supply voltage.
>   
>   Note: You can find more documentation in the following references
>   [1] Documentation/devicetree/bindings/clock/clock-bindings.txt
> @@ -101,6 +103,7 @@ Example 2: DSI panel
>   			clock-names = "pclk", "ref";
>   			resets = <&rcc STM32F4_APB2_RESET(DSI)>;
>   			reset-names = "apb";
> +			phy-dsi-supply = <&reg18>;

Dear Yannick,
Thank you for your patch,

Reviewed-by: Philippe Cornu <philippe.cornu@xxxxxx>

Philippe :)


>   
>   			ports {
>   				#address-cells = <1>;
>