Re: [PATCH] mfd: devicetree: qcom_rpm: document IPQ8064 resources

From: Bjorn Andersson
Date: Thu Mar 19 2015 - 23:18:45 EST


On Thu 19 Mar 20:02 PDT 2015, Andy Gross wrote:

> From: Josh Cartwright <joshc@xxxxxxxxxxxxxx>
>
> The IPQ8064 SoC has several RPM-controlled resources, an NSS fabric
> clock and four regulator resources. Provide definitions for them.
>
> Signed-off-by: Josh Cartwright <joshc@xxxxxxxxxxxxxx>

I believe you should have a sob here as well.

> ---
> Documentation/devicetree/bindings/mfd/qcom-rpm.txt | 1 +
> include/dt-bindings/mfd/qcom-rpm.h | 6 ++++++
> 2 files changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/mfd/qcom-rpm.txt b/Documentation/devicetree/bindings/mfd/qcom-rpm.txt
> index 85e3198..94d9ae8 100644
> --- a/Documentation/devicetree/bindings/mfd/qcom-rpm.txt
> +++ b/Documentation/devicetree/bindings/mfd/qcom-rpm.txt
> @@ -12,6 +12,7 @@ frequencies.
> "qcom,rpm-apq8064"
> "qcom,rpm-msm8660"
> "qcom,rpm-msm8960"
> + "qcom,rpm-ipq8064"

It would be nice if these where kept sorted...

>
> - reg:
> Usage: required
> diff --git a/include/dt-bindings/mfd/qcom-rpm.h b/include/dt-bindings/mfd/qcom-rpm.h
> index 388a6f3..13a9d4b 100644
> --- a/include/dt-bindings/mfd/qcom-rpm.h
> +++ b/include/dt-bindings/mfd/qcom-rpm.h
> @@ -141,6 +141,12 @@
> #define QCOM_RPM_SYS_FABRIC_MODE 131
> #define QCOM_RPM_USB_OTG_SWITCH 132
> #define QCOM_RPM_VDDMIN_GPIO 133
> +#define QCOM_RPM_NSS_FABRIC_0_CLK 134
> +#define QCOM_RPM_NSS_FABRIC_1_CLK 135
> +#define QCOM_RPM_SMB208_S1a 136
> +#define QCOM_RPM_SMB208_S1b 137
> +#define QCOM_RPM_SMB208_S2a 138
> +#define QCOM_RPM_SMB208_S2b 139

Part of that,

Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxxxxxx>

Regards,
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/