linux-kernel - Re: [RESEND v6 1/6] dt-bindings: mfd: Document STM32 low power timer bindings

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] [day] [month] [year] [list]

Message-ID: <83a750a8-85ec-dbc8-a429-d61af5be36b4@st.com>
Date:   Mon, 20 Apr 2020 12:15:05 +0000
From:   Benjamin GAIGNARD <benjamin.gaignard@...com>
To:     Fabrice GASNIER <fabrice.gasnier@...com>,
        "lee.jones@...aro.org" <lee.jones@...aro.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mark.rutland@....com" <mark.rutland@....com>,
        "mcoquelin.stm32@...il.com" <mcoquelin.stm32@...il.com>,
        Alexandre TORGUE <alexandre.torgue@...com>,
        "daniel.lezcano@...aro.org" <daniel.lezcano@...aro.org>,
        "tglx@...utronix.de" <tglx@...utronix.de>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-stm32@...md-mailman.stormreply.com" 
        <linux-stm32@...md-mailman.stormreply.com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [RESEND v6 1/6] dt-bindings: mfd: Document STM32 low power timer
 bindings



On 4/20/20 11:34 AM, Fabrice Gasnier wrote:
> On 4/20/20 10:59 AM, Benjamin Gaignard wrote:
>> Add a subnode to STM low power timer bindings to support timer driver
>>
>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@...com>
>> Reviewed-by: Rob Herring <robh@...nel.org>
>> ---
>>   .../devicetree/bindings/mfd/st,stm32-lptimer.yaml     | 19 +++++++++++++++++++
>>   1 file changed, 19 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
>> index ddf190cb800b..64bab1c3bdac 100644
>> --- a/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
>> +++ b/Documentation/devicetree/bindings/mfd/st,stm32-lptimer.yaml
>> @@ -33,6 +33,9 @@ properties:
>>       items:
>>         - const: mux
>>   
>> +  interrupts:
>> +    maxItems: 1
>> +
>>     "#address-cells":
>>       const: 1
>>   
>> @@ -93,6 +96,16 @@ patternProperties:
>>       required:
>>         - compatible
>>   
>> +  timer:
>> +    type: object
>> +
>> +    properties:
>> +      compatible:
>> +        const: st,stm32-lptimer-timer
>> +
>> +    required:
>> +      - compatible
>> +
>>   required:
>>     - "#address-cells"
>>     - "#size-cells"
>> @@ -106,11 +119,13 @@ additionalProperties: false
>>   examples:
>>     - |
>>       #include <dt-bindings/clock/stm32mp1-clks.h>
>> +    #include <dt-bindings/interrupt-controller/arm-gic.h>
>>       timer@...02400 {
>>         compatible = "st,stm32-lptimer";
>>         reg = <0x40002400 0x400>;
>>         clocks = <&timer_clk>;
>>         clock-names = "mux";
>> +      interrupts-extended = <&exti 47 IRQ_TYPE_LEVEL_HIGH>;
>>         #address-cells = <1>;
>>         #size-cells = <0>;
>>   
>> @@ -131,6 +146,10 @@ examples:
>>         timer {
>>           compatible = "st,stm32-lptimer-timer";
>>         };
>> +
>> +      timer {
>> +        compatible = "st,stm32-lptimer-timer";
>> +      };
> Hi Benjamin,
>
> It looks like the compatible is duplicated in this example.
> Also, from "PATCH v6" I don't see the wakeup-source flag in your resend.
> Can you double-check ?
I have rebased the serie on top of v5.7-rc2, I haven't see that the 
bindings got merged
and, unlucky, these hunks can still be applied and becomes duplicated.
I will send a v7.

Benjamin
>
> Thanks,
> Best Regards,
> Fabrice
>
>>       };
>>   
>>   ...
>>