Re: [PATCH v3 1/2] clk: qcom: Add DT bindings for MSM8939 GCC

From: Rob Herring
Date: Mon May 11 2020 - 16:33:01 EST


On Thu, Apr 23, 2020 at 11:34:05AM +0100, Bryan O'Donoghue wrote:
> Add compatible strings and the include files for the MSM8939 GCC.
>
> Cc: Andy Gross <agross@xxxxxxxxxx>
> Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> Cc: Michael Turquette <mturquette@xxxxxxxxxxxx>
> Cc: Stephen Boyd <sboyd@xxxxxxxxxx>
> Cc: Rob Herring <robh+dt@xxxxxxxxxx>
> Cc: linux-arm-msm@xxxxxxxxxxxxxxx
> Cc: linux-clk@xxxxxxxxxxxxxxx
> Cc: devicetree@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>
> ---
> .../devicetree/bindings/clock/qcom,gcc.yaml | 3 +
> include/dt-bindings/clock/qcom,gcc-msm8939.h | 179 ++++++++++++++++++
> include/dt-bindings/reset/qcom,gcc-msm8939.h | 100 ++++++++++
> 3 files changed, 282 insertions(+)
> create mode 100644 include/dt-bindings/clock/qcom,gcc-msm8939.h
> create mode 100644 include/dt-bindings/reset/qcom,gcc-msm8939.h
>
> diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
> index e533bb0cfd2b..ee0467fb5e31 100644
> --- a/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
> +++ b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml
> @@ -22,6 +22,8 @@ description: |
> - dt-bindings/reset/qcom,gcc-ipq6018.h
> - dt-bindings/clock/qcom,gcc-ipq806x.h (qcom,gcc-ipq8064)
> - dt-bindings/reset/qcom,gcc-ipq806x.h (qcom,gcc-ipq8064)
> + - dt-bindings/clock/qcom,gcc-msm8939.h
> + - dt-bindings/reset/qcom,gcc-msm8939.h
> - dt-bindings/clock/qcom,gcc-msm8660.h
> - dt-bindings/reset/qcom,gcc-msm8660.h
> - dt-bindings/clock/qcom,gcc-msm8974.h
> @@ -41,6 +43,7 @@ properties:
> - qcom,gcc-ipq8064
> - qcom,gcc-msm8660
> - qcom,gcc-msm8916
> + - qcom,gcc-msm8939
> - qcom,gcc-msm8960
> - qcom,gcc-msm8974
> - qcom,gcc-msm8974pro
> diff --git a/include/dt-bindings/clock/qcom,gcc-msm8939.h b/include/dt-bindings/clock/qcom,gcc-msm8939.h
> new file mode 100644
> index 000000000000..e792ef049fc8
> --- /dev/null
> +++ b/include/dt-bindings/clock/qcom,gcc-msm8939.h
> @@ -0,0 +1,179 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */

Okay with GPLv4?

Should be dual licensed ideally. Doesn't matter much to me, but QCom
might care.

Acked-by: Rob Herring <robh@xxxxxxxxxx>