[2/3] dt-bindings: watchdog: Add compatible for QCS404, SC7180, SDM845, SM8150
diff mbox series

Message ID 3f871ae3818b46423430074689e33bc34b28aa1c.1576211720.git.saiprakash.ranjan@codeaurora.org
State Changes Requested
Headers show
Series
  • Convert QCOM watchdog timer bindings to YAML
Related show

Checks

Context Check Description
robh/checkpatch success

Commit Message

Sai Prakash Ranjan Dec. 13, 2019, 4:53 a.m. UTC
Add missing compatible for watchdog timer on QCS404,
SC7180, SDM845 and SM8150 SoCs.

Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
---
 .../devicetree/bindings/watchdog/qcom-wdt.yaml       | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Rob Herring Dec. 19, 2019, 11:28 p.m. UTC | #1
On Fri, Dec 13, 2019 at 10:23:19AM +0530, Sai Prakash Ranjan wrote:
> Add missing compatible for watchdog timer on QCS404,
> SC7180, SDM845 and SM8150 SoCs.
> 
> Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
> ---
>  .../devicetree/bindings/watchdog/qcom-wdt.yaml       | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> index 4a42f4261322..ec25ce1c9e2e 100644
> --- a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
> @@ -12,6 +12,18 @@ maintainers:
>  properties:
>    compatible:
>      oneOf:
> +      - items:
> +          - const: qcom,apss-wdt-sc7180
> +          - const: qcom,kpss-wdt
> +      - items:
> +          - const: qcom,apss-wdt-sdm845
> +          - const: qcom,kpss-wdt
> +      - items:
> +          - const: qcom,apss-wdt-sm8150
> +          - const: qcom,kpss-wdt
> +      - items:
> +          - const: qcom,apss-wdt-qcs404
> +          - const: qcom,kpss-wdt

This can be one entry:

- items:
    - enum:
        - ...
    - const: qcom,kpss-wdt

>        - const: qcom,kpss-timer
>        - const: qcom,kpss-wdt
>        - const: qcom,kpss-wdt-apq8064
> -- 
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
> of Code Aurora Forum, hosted by The Linux Foundation
>
Sai Prakash Ranjan Jan. 13, 2020, 5:47 a.m. UTC | #2
Hi Rob,

On 2019-12-20 04:58, Rob Herring wrote:
> On Fri, Dec 13, 2019 at 10:23:19AM +0530, Sai Prakash Ranjan wrote:
>> Add missing compatible for watchdog timer on QCS404,
>> SC7180, SDM845 and SM8150 SoCs.
>> 
>> Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
>> ---
>>  .../devicetree/bindings/watchdog/qcom-wdt.yaml       | 12 
>> ++++++++++++
>>  1 file changed, 12 insertions(+)
>> 
>> diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml 
>> b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
>> index 4a42f4261322..ec25ce1c9e2e 100644
>> --- a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
>> +++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
>> @@ -12,6 +12,18 @@ maintainers:
>>  properties:
>>    compatible:
>>      oneOf:
>> +      - items:
>> +          - const: qcom,apss-wdt-sc7180
>> +          - const: qcom,kpss-wdt
>> +      - items:
>> +          - const: qcom,apss-wdt-sdm845
>> +          - const: qcom,kpss-wdt
>> +      - items:
>> +          - const: qcom,apss-wdt-sm8150
>> +          - const: qcom,kpss-wdt
>> +      - items:
>> +          - const: qcom,apss-wdt-qcs404
>> +          - const: qcom,kpss-wdt
> 
> This can be one entry:
> 
> - items:
>     - enum:
>         - ...
>     - const: qcom,kpss-wdt
> 

Will change in next version.

Thanks,
Sai

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
index 4a42f4261322..ec25ce1c9e2e 100644
--- a/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.yaml
@@ -12,6 +12,18 @@  maintainers:
 properties:
   compatible:
     oneOf:
+      - items:
+          - const: qcom,apss-wdt-sc7180
+          - const: qcom,kpss-wdt
+      - items:
+          - const: qcom,apss-wdt-sdm845
+          - const: qcom,kpss-wdt
+      - items:
+          - const: qcom,apss-wdt-sm8150
+          - const: qcom,kpss-wdt
+      - items:
+          - const: qcom,apss-wdt-qcs404
+          - const: qcom,kpss-wdt
       - const: qcom,kpss-timer
       - const: qcom,kpss-wdt
       - const: qcom,kpss-wdt-apq8064