On Thu, Sep 01, 2016 at 07:06:52PM +0530, Archit Taneja wrote: > Make the following changes in the HDMI gpio bindings: > > - Use "-gpios" as the suffix for all the gpio names > - Move all the gpios to optional, since there are platforms that use none > of them. > - The HPD gpio is a standard one, remove the "qcom,hdmi-tx-" prefix from > it. > - Add a missing lpm gpio used on some platforms. > > Make the necessary changes in the driver to incorporate these changes. > > There hasn't been any upstream DT that uses the HDMI bindings, so it's > okay to change and move around these properties. > > Cc: Rob Herring <robh@xxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > Signed-off-by: Archit Taneja <architt@xxxxxxxxxxxxxx> > --- > v2: > - Keep "qcom,hdmi-tx-" suffix for all gpios except for hpd. > - Use "-gpios" suffix instead of "-gpio". > - Move all the gpios to optional properties. > > .../devicetree/bindings/display/msm/hdmi.txt | 11 ++++++----- > drivers/gpu/drm/msm/hdmi/hdmi.c | 21 +++++++++++++++++++-- > 2 files changed, 25 insertions(+), 7 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/msm/hdmi.txt b/Documentation/devicetree/bindings/display/msm/hdmi.txt > index ce84459..f1a83ab 100644 > --- a/Documentation/devicetree/bindings/display/msm/hdmi.txt > +++ b/Documentation/devicetree/bindings/display/msm/hdmi.txt > @@ -13,17 +13,18 @@ Required properties: > - interrupts: The interrupt signal from the hdmi block. > - clocks: device clocks > See ../clocks/clock-bindings.txt for details. > -- qcom,hdmi-tx-ddc-clk-gpio: ddc clk pin > -- qcom,hdmi-tx-ddc-data-gpio: ddc data pin > -- qcom,hdmi-tx-hpd-gpio: hpd pin > - core-vdda-supply: phandle to supply regulator > - hdmi-mux-supply: phandle to mux regulator > - phys: the phandle for the HDMI PHY device > - phy-names: the name of the corresponding PHY device > > Optional properties: > -- qcom,hdmi-tx-mux-en-gpio: hdmi mux enable pin > -- qcom,hdmi-tx-mux-sel-gpio: hdmi mux select pin > +- qcom,hdmi-tx-ddc-clk-gpios: ddc clk pin > +- qcom,hdmi-tx-ddc-data-gpios: ddc data pin Sorry, for raising another point, but couldn't you use the i2c-gpio binding for these instead? > +- hpd-gpios: hpd pin > +- qcom,hdmi-tx-mux-en-gpios: hdmi mux enable pin > +- qcom,hdmi-tx-mux-sel-gpios: hdmi mux select pin > +- qcom,hdmi-tx-mux-lpm-gpios: hdmi mux lpm pin > - power-domains: reference to the power domain(s), if available. > - pinctrl-names: the pin control state names; should contain "default" > - pinctrl-0: the default pinctrl state (active) -- 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
