Re: [PATCH] firmware: qcom_scm: Add MDM9607 compatible

[Rob Herring <robh@kernel.org>]

On Sun, Jan 31, 2021 at 02:30:57AM +0100, Konrad Dybcio wrote:
> Add a compatible for MDM9607. It uses the "legacy" calling
> convention.
> 
> Signed-off-by: Konrad Dybcio <konrad.dybcio@somainline.org>
> ---
>  Documentation/devicetree/bindings/firmware/qcom,scm.txt | 1 +
>  drivers/firmware/qcom_scm.c                             | 3 +++
>  2 files changed, 4 insertions(+)

In the future, please split binding changes to separate patch.

Acked-by: Rob Herring <robh@kernel.org>

> 
> diff --git a/Documentation/devicetree/bindings/firmware/qcom,scm.txt b/Documentation/devicetree/bindings/firmware/qcom,scm.txt
> index 78456437df5f..df8379356021 100644
> --- a/Documentation/devicetree/bindings/firmware/qcom,scm.txt
> +++ b/Documentation/devicetree/bindings/firmware/qcom,scm.txt
> @@ -12,6 +12,7 @@ Required properties:
>   * "qcom,scm-ipq4019"
>   * "qcom,scm-ipq806x"
>   * "qcom,scm-ipq8074"
> + * "qcom,scm-mdm9607"
>   * "qcom,scm-msm8660"
>   * "qcom,scm-msm8916"
>   * "qcom,scm-msm8960"
> diff --git a/drivers/firmware/qcom_scm.c b/drivers/firmware/qcom_scm.c
> index 7be48c1bec96..b5b9b13d8d29 100644
> --- a/drivers/firmware/qcom_scm.c
> +++ b/drivers/firmware/qcom_scm.c
> @@ -1265,6 +1265,9 @@ static const struct of_device_id qcom_scm_dt_match[] = {
>  							     SCM_HAS_BUS_CLK)
>  	},
>  	{ .compatible = "qcom,scm-ipq4019" },
> +	{ .compatible = "qcom,scm-mdm9607", .data = (void *)(SCM_HAS_CORE_CLK |
> +							     SCM_HAS_IFACE_CLK |
> +							     SCM_HAS_BUS_CLK) },
>  	{ .compatible = "qcom,scm-msm8660", .data = (void *) SCM_HAS_CORE_CLK },
>  	{ .compatible = "qcom,scm-msm8960", .data = (void *) SCM_HAS_CORE_CLK },
>  	{ .compatible = "qcom,scm-msm8916", .data = (void *)(SCM_HAS_CORE_CLK |
> -- 
> 2.30.0
>