lkml.org 
[lkml]   [2020]   [Nov]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH v3 0/5] Add support of SECVIO from SNVS on iMX8q/x
Date
From: Franck LENORMAND <franck.lenormand@oss.nxp.com>

This patchset aims to add support for the SECurity VIOlation (SECVIO) of the
SNVS. A secvio is a signal emitted by the SNVS when a hardware attack
is detected. On imx8x and imx8q SoC, the SNVS is controlled by the
SECO and it is possible to interact with it using the SCU using the SC APIs.

For the driver to communicate with the SNVS via the SCU and the SECO, I had to:
- Add support for exchange of big message with the SCU (needed for
imx_scu_irq_get_status)
- Add API to check linux can control the SECVIO (imx_sc_rm_is_resource_owned)
- Add APIs for the driver to read the state of the SECVIO registers of the
SNVS and DGO (imx_sc_seco_secvio_enable and imx_sc_seco_secvio_enable).

To check the state of the SECVIO IRQ in the SCU, I added the
imx_scu_irq_get_status API.

The secvio driver is designed to receive the IRQ produced by the
SNVS in case of hardware attack and notify the status to the
audit framework which can be used by the user.

The goal of the driver is to be self suficient but can be extended by the
user to perform custom operations on values read (imx_sc_seco_secvio_enable)

v2:
- Removed (firmware: imx: scu-rm: Add Resource Management APIs)
-> Code required is already present
- Removed (firmware: imx: scu: Support reception of messages of any size)
-> The imx-scu is already working in fast-ipc mode
- (soc: imx8: Add the SC SECVIO driver):
- Fixed the warnings reported by kernel test robot

v3: Apply review from Aisheng
- Fixed the "SECurity VIOlation" to "security violation"
- (firmware: imx: scu-seco: Add Secure Controller APIS):
- Rewrite doc of return error code
- Rework request and response structure into one
- (firmware: imx: scu-irq: Add API to retrieve status of IRQ):
- Rewrite doc of return error code
- Rework imx_scu_irq_work_handler to use added imx_scu_irq_get_status
- (dt-bindings: arm: imx: Documentation of the SC secvio driver):
- Use reference of nvmem

Franck LENORMAND (5):
firmware: imx: scu-seco: Add Secure Controller APIS
firmware: imx: scu-irq: Add API to retrieve status of IRQ
dt-bindings: firmware: imx-scu: Add SECVIO resource
dt-bindings: arm: imx: Documentation of the SC secvio driver
soc: imx8: Add the SC SECVIO driver

.../bindings/arm/freescale/fsl,imx-sc-secvio.yaml | 34 +
drivers/firmware/imx/Makefile | 2 +-
drivers/firmware/imx/imx-scu-irq.c | 48 +-
drivers/firmware/imx/imx-scu.c | 8 +-
drivers/firmware/imx/seco.c | 253 ++++++
drivers/soc/imx/Kconfig | 10 +
drivers/soc/imx/Makefile | 1 +
drivers/soc/imx/secvio/Kconfig | 10 +
drivers/soc/imx/secvio/Makefile | 3 +
drivers/soc/imx/secvio/imx-secvio-audit.c | 39 +
drivers/soc/imx/secvio/imx-secvio-debugfs.c | 379 +++++++++
drivers/soc/imx/secvio/imx-secvio-sc-int.h | 84 ++
drivers/soc/imx/secvio/imx-secvio-sc.c | 858 +++++++++++++++++++++
include/dt-bindings/firmware/imx/rsrc.h | 3 +-
include/linux/firmware/imx/ipc.h | 1 +
include/linux/firmware/imx/sci.h | 5 +
include/linux/firmware/imx/svc/seco.h | 73 ++
include/soc/imx/imx-secvio-sc.h | 177 +++++
18 files changed, 1970 insertions(+), 18 deletions(-)
create mode 100644 Documentation/devicetree/bindings/arm/freescale/fsl,imx-sc-secvio.yaml
create mode 100644 drivers/firmware/imx/seco.c
create mode 100644 drivers/soc/imx/secvio/Kconfig
create mode 100644 drivers/soc/imx/secvio/Makefile
create mode 100644 drivers/soc/imx/secvio/imx-secvio-audit.c
create mode 100644 drivers/soc/imx/secvio/imx-secvio-debugfs.c
create mode 100644 drivers/soc/imx/secvio/imx-secvio-sc-int.h
create mode 100644 drivers/soc/imx/secvio/imx-secvio-sc.c
create mode 100644 include/linux/firmware/imx/svc/seco.h
create mode 100644 include/soc/imx/imx-secvio-sc.h

--
2.7.4

\
 
 \ /
  Last update: 2020-11-10 15:13    [W:0.125 / U:1.844 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site