diff mbox series

[v11,2/4] dt-bindings: mtd: Add a property to declare secure regions in NAND chips

Message ID 20210402150128.29128-3-manivannan.sadhasivam@linaro.org
State Accepted
Commit ee590106c331ad54df9da3052b5c52014b2edb0b
Headers show
Series Add support for secure regions in NAND | expand

Commit Message

Manivannan Sadhasivam April 2, 2021, 3:01 p.m. UTC 
On a typical end product, a vendor may choose to secure some regions in
the NAND memory which are supposed to stay intact between FW upgrades.
The access to those regions will be blocked by a secure element like
Trustzone. So the normal world software like Linux kernel should not
touch these regions (including reading).

So let's add a property for declaring such secure regions so that the
drivers can skip touching them.

Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
 Documentation/devicetree/bindings/mtd/nand-controller.yaml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Miquel Raynal April 7, 2021, 9:06 a.m. UTC | #1 
On Fri, 2021-04-02 at 15:01:26 UTC, Manivannan Sadhasivam wrote:
> On a typical end product, a vendor may choose to secure some regions in

> the NAND memory which are supposed to stay intact between FW upgrades.

> The access to those regions will be blocked by a secure element like

> Trustzone. So the normal world software like Linux kernel should not

> touch these regions (including reading).

> 

> So let's add a property for declaring such secure regions so that the

> drivers can skip touching them.

> 

> Reviewed-by: Rob Herring <robh@kernel.org>

> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>


Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Miquel
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/mtd/nand-controller.yaml b/Documentation/devicetree/bindings/mtd/nand-controller.yaml
index d0e422f4b3e0..678b39952502 100644
--- a/Documentation/devicetree/bindings/mtd/nand-controller.yaml
+++ b/Documentation/devicetree/bindings/mtd/nand-controller.yaml
@@ -143,6 +143,13 @@  patternProperties:
           Ready/Busy pins. Active state refers to the NAND ready state and
           should be set to GPIOD_ACTIVE_HIGH unless the signal is inverted.
 
+      secure-regions:
+        $ref: /schemas/types.yaml#/definitions/uint64-matrix
+        description:
+          Regions in the NAND chip which are protected using a secure element
+          like Trustzone. This property contains the start address and size of
+          the secure regions present.
+
     required:
       - reg