linux-kernel - Re: [RFC v2 3/3] dt-bindings: firmware: arm,scmi: Add support for pinctrl protocol

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:   Thu, 27 Apr 2023 07:19:32 +0000
From:   Oleksii Moisieiev <Oleksii_Moisieiev@...m.com>
To:     Michal Simek <michal.simek@....com>,
        "sudeep.holla@....com" <sudeep.holla@....com>
CC:     Linus Walleij <linus.walleij@...aro.org>,
        Cristian Marussi <cristian.marussi@....com>,
        Peng Fan <peng.fan@....nxp.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-gpio@...r.kernel.org" <linux-gpio@...r.kernel.org>
Subject: Re: [RFC v2 3/3] dt-bindings: firmware: arm,scmi: Add support for
 pinctrl protocol



On 27.04.23 10:07, Michal Simek wrote:
> 
> 
> On 4/26/23 15:26, Oleksii Moisieiev wrote:
>> Add new SCMI v3.2 pinctrl protocol bindings definitions and example.
>>
>> Signed-off-by: Oleksii Moisieiev <oleksii_moisieiev@...m.com>
>> ---
>>   .../bindings/firmware/arm,scmi.yaml           | 77 +++++++++++++++++++
>>   1 file changed, 77 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml 
>> b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
>> index 2f7c51c75e85..41ba5b8d8151 100644
>> --- a/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
>> +++ b/Documentation/devicetree/bindings/firmware/arm,scmi.yaml
>> @@ -212,6 +212,63 @@ properties:
>>         reg:
>>           const: 0x18
>> +  protocol@19:
>> +    $ref: '#/$defs/protocol-node'
>> +
>> +    properties:
>> +      reg:
>> +        const: 0x19
>> +
>> +      '#pinctrl-cells':
>> +        const: 0
>> +
> 
> As I said. pinctrl has also gpio part and based on discussion with SCMI 
> guys this protocol can be also used for simple gpio. That's why here 
> should be also listed #gpio-cells to be able to use this functionality.
> 
> And for complete implementation it should be also added to driver it but 
> of course this can be added later by separate patch.
> 
> Thanks,
> Michal
> 

Hi Michal,

The plan is to add gpio support as the separate patch on top of this 
patch series.

Oleksii.