[PATCH 1/3] nvmem: specify ->reg_read/reg_write() expected return values
From: Théo Lebrun
Date: Wed Jul 24 2024 - 09:57:30 EST
Both ->reg_read() and ->reg_write() return values are not easy to
deduce. Such type of callbacks could return the number of bytes in the
success case, which would be translated to errors in the nvmem core.
Such mistake has been done in drivers/nvmem/rmem.c:
commit 28b008751aa2 ("nvmem: rmem: Fix return value of rmem_read()").
Signed-off-by: Théo Lebrun <theo.lebrun@xxxxxxxxxxx>
---
include/linux/nvmem-provider.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/linux/nvmem-provider.h b/include/linux/nvmem-provider.h
index 3ebeaa0ded00..515676ebe598 100644
--- a/include/linux/nvmem-provider.h
+++ b/include/linux/nvmem-provider.h
@@ -92,8 +92,8 @@ struct nvmem_cell_info {
* @read_only: Device is read-only.
* @root_only: Device is accessibly to root only.
* @of_node: If given, this will be used instead of the parent's of_node.
- * @reg_read: Callback to read data.
- * @reg_write: Callback to write data.
+ * @reg_read: Callback to read data; return zero if successful.
+ * @reg_write: Callback to write data; return zero if successful.
* @size: Device size.
* @word_size: Minimum read/write access granularity.
* @stride: Minimum read/write access stride.
--
2.45.2