nvmem: core: constify nvmem_cell_read_variable_common() return value
authorDouglas Anderson <dianders@chromium.org>
Fri, 11 Jun 2021 08:33:47 +0000 (09:33 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 11 Jun 2021 08:57:56 +0000 (10:57 +0200)
The caller doesn't modify the memory pointed to by the pointer so it
can be const.

Suggested-by: Stephen Boyd <swboyd@chromium.org>
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Link: https://lore.kernel.org/r/20210611083348.20170-9-srinivas.kandagatla@linaro.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/nvmem/core.c

index bca671f..b3c28a2 100644 (file)
@@ -1606,9 +1606,9 @@ int nvmem_cell_read_u64(struct device *dev, const char *cell_id, u64 *val)
 }
 EXPORT_SYMBOL_GPL(nvmem_cell_read_u64);
 
-static void *nvmem_cell_read_variable_common(struct device *dev,
-                                            const char *cell_id,
-                                            size_t max_len, size_t *len)
+static const void *nvmem_cell_read_variable_common(struct device *dev,
+                                                  const char *cell_id,
+                                                  size_t max_len, size_t *len)
 {
        struct nvmem_cell *cell;
        int nbits;
@@ -1652,7 +1652,7 @@ int nvmem_cell_read_variable_le_u32(struct device *dev, const char *cell_id,
                                    u32 *val)
 {
        size_t len;
-       u8 *buf;
+       const u8 *buf;
        int i;
 
        buf = nvmem_cell_read_variable_common(dev, cell_id, sizeof(*val), &len);
@@ -1683,7 +1683,7 @@ int nvmem_cell_read_variable_le_u64(struct device *dev, const char *cell_id,
                                    u64 *val)
 {
        size_t len;
-       u8 *buf;
+       const u8 *buf;
        int i;
 
        buf = nvmem_cell_read_variable_common(dev, cell_id, sizeof(*val), &len);