linux-kernel - Re: [PATCH 0/4] Add support for SCMIv3.0 Voltage Domain Protocol and SCMI-Regulator

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]

Message-ID: <8483165a-e413-b2f8-bd33-6da07fe56d62@gmail.com>
Date:   Mon, 5 Oct 2020 17:38:52 -0700
From:   Florian Fainelli <f.fainelli@...il.com>
To:     Cristian Marussi <cristian.marussi@....com>,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Cc:     robh@...nel.org, satyakim@....qualcomm.com, sudeep.holla@....com,
        broonie@...nel.org, james.quinlan@...adcom.com,
        Jonathan.Cameron@...wei.com, etienne.carriere@...aro.org,
        lukasz.luba@....com
Subject: Re: [PATCH 0/4] Add support for SCMIv3.0 Voltage Domain Protocol and
 SCMI-Regulator



On 10/5/2020 3:26 PM, Cristian Marussi wrote:
> Hi,
> 
> this series introduces the support for the new SCMI Voltage Domain Protocol
> defined by the upcoming SCMIv3.0 specification, whose BETA release is
> available at [1].
> 
> Afterwards, a new generic SCMI Regulator driver is developed on top of the
> new SCMI VD Protocol.
> 
> The series is currently based on for-next/scmi [2] on top of:
> 
> commit 66d90f6ecee7 ("firmware: arm_scmi: Enable building as a single
> 		     module")
> 
> Any feedback welcome,

Well, this is just great! We were right about to develop a proprietary 
SCMI protocol in order to control a locked down PMIC accessible behind a 
secured firmware. We would have done essentially just that since the use 
case is to control the various regulators exposed by this PMIC over 
SCMI. Thanks a lot!

> 
> Thanks,
> 
> Cristian
> 
> [1]:https://developer.arm.com/documentation/den0056/c/
> [2]:https://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux.git/log/?h=for-next/scmi
> 
> 
> Cristian Marussi (4):
>    firmware: arm_scmi: Add Voltage Domain Support
>    firmware: arm_scmi: add SCMI Voltage Domain devname
>    regulator: add SCMI driver
>    dt-bindings: arm: add support for SCMI Regulators
> 
>   .../devicetree/bindings/arm/arm,scmi.txt      |  44 ++
>   drivers/firmware/arm_scmi/Makefile            |   2 +-
>   drivers/firmware/arm_scmi/common.h            |   1 +
>   drivers/firmware/arm_scmi/driver.c            |   3 +
>   drivers/firmware/arm_scmi/voltage.c           | 378 ++++++++++++++
>   drivers/regulator/Kconfig                     |   9 +
>   drivers/regulator/Makefile                    |   1 +
>   drivers/regulator/scmi-regulator.c            | 488 ++++++++++++++++++
>   include/linux/scmi_protocol.h                 |  64 +++
>   9 files changed, 989 insertions(+), 1 deletion(-)
>   create mode 100644 drivers/firmware/arm_scmi/voltage.c
>   create mode 100644 drivers/regulator/scmi-regulator.c
> 

-- 
Florian