linux-kernel - Re: [PATCH 2/3] dt-bindings: watchdog: Add compatible for QCS404, SC7180, SDM845, SM8150

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 13 Jan 2020 11:17:05 +0530
From:   Sai Prakash Ranjan <saiprakash.ranjan@...eaurora.org>
To:     Rob Herring <robh@...nel.org>
Cc:     Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Douglas Anderson <dianders@...omium.org>,
        Guenter Roeck <linux@...ck-us.net>, devicetree@...r.kernel.org,
        Stephen Boyd <swboyd@...omium.org>,
        Matthias Kaehlcke <mka@...omium.org>,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        linux-arm-msm@...r.kernel.org
Subject: Re: [PATCH 2/3] dt-bindings: watchdog: Add compatible for QCS404,
 SC7180, SDM845, SM8150

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@...eaurora.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

-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a 
member
of Code Aurora Forum, hosted by The Linux Foundation