From mboxrd@z Thu Jan 1 00:00:00 1970 From: Archit Taneja Subject: Re: [PATCH v2 22/25] dt-bindings: msm/dsi: Remove unused properties Date: Fri, 26 Aug 2016 10:25:25 +0530 Message-ID: <57BFCBBD.4010504@codeaurora.org> References: <1466077007-26792-1-git-send-email-architt@codeaurora.org> <1466691210-22779-1-git-send-email-architt@codeaurora.org> <1466691210-22779-23-git-send-email-architt@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:57213 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752020AbcHZEzb (ORCPT ); Fri, 26 Aug 2016 00:55:31 -0400 In-Reply-To: <1466691210-22779-23-git-send-email-architt@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: robh@kernel.org Cc: robdclark@gmail.com, dri-devel@lists.freedesktop.org, linux-arm-msm@vger.kernel.org Hi Rob, On 06/23/2016 07:43 PM, Archit Taneja wrote: > Remove the DSI index properties from the DSI host and PHY binding > documentation. The indices aren't a valid property and shouldn't > be a part of the DT binding. > Can I get an Ack on this? Thanks, Archit > Signed-off-by: Archit Taneja > --- > Documentation/devicetree/bindings/display/msm/dsi.txt | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/msm/dsi.txt b/Documentation/devicetree/bindings/display/msm/dsi.txt > index 6b1cab1..e6933a8 100644 > --- a/Documentation/devicetree/bindings/display/msm/dsi.txt > +++ b/Documentation/devicetree/bindings/display/msm/dsi.txt > @@ -7,8 +7,6 @@ Required properties: > - reg: Physical base address and length of the registers of controller > - reg-names: The names of register regions. The following regions are required: > * "dsi_ctrl" > -- qcom,dsi-host-index: The ID of DSI controller hardware instance. This should > - be 0 or 1, since we have 2 DSI controllers at most for now. > - interrupts: The interrupt signal from the DSI block. > - power-domains: Should be <&mmcc MDSS_GDSC>. > - clocks: Phandles to device clocks. > @@ -96,8 +94,6 @@ Required properties: > * "dsi_phy_regulator" > - clock-cells: Must be 1. The DSI PHY block acts as a clock provider, creating > 2 clocks: A byte clock (index 0), and a pixel clock (index 1). > -- qcom,dsi-phy-index: The ID of DSI PHY hardware instance. This should > - be 0 or 1, since we have 2 DSI PHYs at most for now. > - power-domains: Should be <&mmcc MDSS_GDSC>. > - clocks: Phandles to device clocks. See [1] for details on clock bindings. > - clock-names: the following clocks are required: > @@ -116,7 +112,6 @@ Optional properties: > Example: > dsi0: dsi@fd922800 { > compatible = "qcom,mdss-dsi-ctrl"; > - qcom,dsi-host-index = <0>; > interrupt-parent = <&mdp>; > interrupts = <4 0>; > reg-names = "dsi_ctrl"; > @@ -199,7 +194,6 @@ Example: > > dsi_phy0: dsi-phy@fd922a00 { > compatible = "qcom,dsi-phy-28nm-hpm"; > - qcom,dsi-phy-index = <0>; > reg-names = > "dsi_pll", > "dsi_phy", > -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project