On Tue, Apr 24, 2018 at 07:02:50PM +0530, Amit Nischal wrote: > Add device tree bindings for video clock controller for Qualcomm > Technology Inc's SoCs. > > Signed-off-by: Amit Nischal <anischal@xxxxxxxxxxxxxx> > --- > .../devicetree/bindings/clock/qcom,videocc.txt | 18 ++++++++++++++++ > include/dt-bindings/clock/qcom,videocc-sdm845.h | 25 ++++++++++++++++++++++ > 2 files changed, 43 insertions(+) > create mode 100644 Documentation/devicetree/bindings/clock/qcom,videocc.txt > create mode 100644 include/dt-bindings/clock/qcom,videocc-sdm845.h > > diff --git a/Documentation/devicetree/bindings/clock/qcom,videocc.txt b/Documentation/devicetree/bindings/clock/qcom,videocc.txt > new file mode 100644 > index 0000000..1c23b41 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/qcom,videocc.txt > @@ -0,0 +1,18 @@ > +Qualcomm Video Clock & Reset Controller Binding > +----------------------------------------------- > + > +Required properties : > +- compatible : shall contain "qcom,videocc-sdm845" '<vendor>,<soc>-<block>' is the preferred order. Every single QCom binding... Can someone spread the word in QCom. > +- reg : shall contain base register location and length > +- #clock-cells : shall contain 1 > +- #reset-cells : shall contain 1 > +- #power-domain-cells : shall contain 1 No header definitions for resets and power-domain? There's no requirement to have headers, but the binding should be complete even if you don't have a driver yet. > + > +Example: > + videocc: qcom,videocc@ab00000 { clock-controller@... > + compatible = "qcom,videocc-sdm845"; > + reg = <0xab00000 0x10000>; > + #clock-cells = <1>; > + #reset-cells = <1>; > + #power-domain-cells = <1>; > + }; > diff --git a/include/dt-bindings/clock/qcom,videocc-sdm845.h b/include/dt-bindings/clock/qcom,videocc-sdm845.h > new file mode 100644 > index 0000000..f5f7599 > --- /dev/null > +++ b/include/dt-bindings/clock/qcom,videocc-sdm845.h > @@ -0,0 +1,25 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* Copyright (c) 2016-2018, The Linux Foundation. All rights reserved. */ > + > +#ifndef _DT_BINDINGS_CLK_MSM_VIDEO_CC_SDM845_H > +#define _DT_BINDINGS_CLK_MSM_VIDEO_CC_SDM845_H > + > +#define VIDEO_CC_APB_CLK 0 > +#define VIDEO_CC_AT_CLK 1 > +#define VIDEO_CC_QDSS_TRIG_CLK 2 > +#define VIDEO_CC_QDSS_TSCTR_DIV8_CLK 3 > +#define VIDEO_CC_VCODEC0_AXI_CLK 4 > +#define VIDEO_CC_VCODEC0_CORE_CLK 5 > +#define VIDEO_CC_VCODEC1_AXI_CLK 6 > +#define VIDEO_CC_VCODEC1_CORE_CLK 7 > +#define VIDEO_CC_VENUS_AHB_CLK 8 > +#define VIDEO_CC_VENUS_CLK_SRC 9 > +#define VIDEO_CC_VENUS_CTL_AXI_CLK 10 > +#define VIDEO_CC_VENUS_CTL_CORE_CLK 11 > +#define VIDEO_PLL0 12 > + > +#define VENUS_GDSC 0 > +#define VCODEC0_GDSC 1 > +#define VCODEC1_GDSC 2 > + > +#endif > -- > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member > of Code Aurora Forum, hosted by The Linux Foundation > > -- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html