[PATCH v1 4/7] regmap: Add reg_seal operation for hardware protection
Oleksij Rempel
o.rempel at pengutronix.de
Fri May 30 04:41:03 PDT 2025
Add a new 'reg_seal' operation to the regmap bus interface, along
with a public API `regmap_seal()`. This is needed for drivers that
must perform hardware-level "sealing" or permanent write-protection
on registers/words, a capability not covered by standard regmap
read/write ops.
The initial use case is for the STM32 BSEC driver (coming in a
follow-up patch) to lock One-Time Programmable (OTP) memory words.
This gives explicit control to make OTP entries permanently read-only
after programming.
Signed-off-by: Oleksij Rempel <o.rempel at pengutronix.de>
---
drivers/base/regmap/internal.h | 2 ++
drivers/base/regmap/regmap.c | 31 +++++++++++++++++++++++++++++++
include/linux/regmap.h | 33 +++++++++++++++++++++++++++++++++
3 files changed, 66 insertions(+)
diff --git a/drivers/base/regmap/internal.h b/drivers/base/regmap/internal.h
index 6f6a34edc7a8..f2b95cc6361e 100644
--- a/drivers/base/regmap/internal.h
+++ b/drivers/base/regmap/internal.h
@@ -39,6 +39,8 @@ struct regmap {
unsigned int *val);
int (*reg_write)(void *context, unsigned int reg,
unsigned int val);
+ int (*reg_seal)(void *context, unsigned int reg,
+ unsigned int flags);
};
struct regmap_field {
diff --git a/drivers/base/regmap/regmap.c b/drivers/base/regmap/regmap.c
index 777636c0b319..dc65b9e965ce 100644
--- a/drivers/base/regmap/regmap.c
+++ b/drivers/base/regmap/regmap.c
@@ -81,6 +81,17 @@ static int _regmap_bus_reg_write(void *context, unsigned int reg,
return map->bus->reg_write(map->bus_context, reg, val);
}
+static int _regmap_bus_reg_seal(void *context, unsigned int reg,
+ unsigned int flags)
+{
+ struct regmap *map = context;
+
+ if (!map->bus->reg_seal)
+ return -EOPNOTSUPP;
+
+ return map->bus->reg_seal(map->bus_context, reg, flags);
+}
+
/*
* regmap_init - initialize and register a regmap
*
@@ -124,6 +135,9 @@ struct regmap *regmap_init(struct device *dev,
}
}
+ if (!map->reg_seal)
+ map->reg_seal = _regmap_bus_reg_seal;
+
list_add_tail(&map->list, ®maps);
return map;
@@ -186,6 +200,23 @@ int regmap_read(struct regmap *map, unsigned int reg, unsigned int *val)
return map->reg_read(map, reg, val);
}
+/**
+ * regmap_seal() - Seal a register in a map
+ *
+ * @map: Register map to seal
+ * @reg: Register to seal
+ * @flag: Flag to set in the register
+ *
+ * This function is used to seal a register, preventing further writes to it.
+ * The flag is typically used to indicate that the register is sealed.
+ *
+ * Returns zero for success, a negative number on error.
+ */
+int regmap_seal(struct regmap *map, unsigned int reg, unsigned int flags)
+{
+ return map->reg_seal(map, reg, flags);
+}
+
/**
* regmap_update_bits() - Perform a read/modify/write cycle on a register
*
diff --git a/include/linux/regmap.h b/include/linux/regmap.h
index c24b877712cd..0a460f9f541b 100644
--- a/include/linux/regmap.h
+++ b/include/linux/regmap.h
@@ -65,6 +65,35 @@ typedef int (*regmap_hw_reg_read)(void *context, unsigned int reg,
unsigned int *val);
typedef int (*regmap_hw_reg_write)(void *context, unsigned int reg,
unsigned int val);
+typedef int (*regmap_hw_reg_seal)(void *context, unsigned int reg,
+ unsigned int flags);
+
+/**
+ * @REGMAP_SEAL_WRITE_PROTECT: Request to make the register(s) write-protected.
+ * If this flag is set, the bus-specific seal operation should attempt to
+ * prevent further writes to the specified register or range.
+ */
+#define REGMAP_SEAL_WRITE_PROTECT BIT(0)
+
+/**
+ * @REGMAP_SEAL_PERMANENT: Signifies the sealing operation is intended to be
+ * permanent and irreversible. If this flag is not set (and REGMAP_SEAL_CLEAR
+ * is also not set), the protection might be temporary or its permanence
+ * is undefined by this generic flag (bus-specific behavior would dictate).
+ * For operations like OTP locking, this flag should be used with
+ * REGMAP_SEAL_WRITE_PROTECT.
+ */
+#define REGMAP_SEAL_PERMANENT BIT(1)
+
+/**
+ * @REGMAP_SEAL_CLEAR: Request to clear a previously set protection.
+ * This flag should be used in conjunction with other flags (e.g.,
+ * REGMAP_SEAL_WRITE_PROTECT) to specify which type of protection to attempt
+ * to clear. The underlying hardware must support clearing the protection.
+ * If the protection is permanent, an attempt to clear it should fail with
+ * an appropriate error code (e.g., -EOPNOTSUPP or -EPERM).
+ */
+#define REGMAP_SEAL_CLEAR BIT(2)
/**
* struct regmap_bus - Description of a hardware bus for the register map
@@ -73,6 +102,8 @@ typedef int (*regmap_hw_reg_write)(void *context, unsigned int reg,
* @reg_write: Write a single register value to the given register address. This
* write operation has to complete when returning from the function.
* @reg_read: Read a single register value from a given register address.
+ * @reg_seal: Optional. Perform a hardware operation to seal or permanently
+ * protect a register or region (e.g., OTP write-locking).
* @read: Read operation. Data is returned in the buffer used to transmit
* data.
* @write: Write operation.
@@ -88,6 +119,7 @@ typedef int (*regmap_hw_reg_write)(void *context, unsigned int reg,
struct regmap_bus {
regmap_hw_reg_write reg_write;
regmap_hw_reg_read reg_read;
+ regmap_hw_reg_seal reg_seal;
int (*read)(void *context,
const void *reg_buf, size_t reg_size,
@@ -202,6 +234,7 @@ int regmap_multi_register_cdev(struct regmap *map8,
int regmap_write(struct regmap *map, unsigned int reg, unsigned int val);
int regmap_read(struct regmap *map, unsigned int reg, unsigned int *val);
+int regmap_seal(struct regmap *map, unsigned int reg, unsigned int flags);
#ifndef regmap_bulk_read
#define regmap_bulk_read regmap_bulk_read
--
2.39.5
More information about the barebox
mailing list