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

[Rob Herring]

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>