mbox series

[0/4] Add support for SCMIv3.0 Voltage Domain Protocol and SCMI-Regulator

Message ID 20201005222623.1123-1-cristian.marussi@arm.com (mailing list archive)
Headers show
Series Add support for SCMIv3.0 Voltage Domain Protocol and SCMI-Regulator | expand

Message

Cristian Marussi Oct. 5, 2020, 10:26 p.m. UTC
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,

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

Comments

Florian Fainelli Oct. 6, 2020, 12:38 a.m. UTC | #1
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
>
Cristian Marussi Oct. 9, 2020, 12:42 p.m. UTC | #2
On Mon, Oct 05, 2020 at 05:38:52PM -0700, Florian Fainelli wrote:
> 
> 
> 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!
> 

You're welcome :D ... any feedback from your independent testing will be
appreciated.

Thanks

Cristian
> >
> >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