From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3C5DACA9ED0 for ; Mon, 12 Oct 2026 19:52:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=PQEqSma/x2h0AR3RjM0PIqMO97pynuY49ZTCuyLXjAg=; b=cUwgOrlZAbRYHt onOGEQBIdwnPSybrhXJZNcFJOCJuEwFLSIlF3arDAdgcDtvZrXBe6iBQ7V5daDtZ/VWUE1IA/pTt8 SjHOVDOOgnT/KQOOyXsqYdZVORDUDzaLGZvvf63oiHYTXjJkmHxzBgMFWvNVVTCupXkL+ueXcBjOH 9aW4XraUUq3R1nZ2KChee5bxARojdw5m9tRXO9fyrwmorIJwN66b2C5r1CUBV1vIq+JjzYtnSJEwd 3uG0uXFQ6kE6Yp3DC/j+e681VNFdNYJ8shA6bnWv+68taIeAetHE2cILuKozDecWXCVJuhvOZUr7t ZM1m2PIJZHwGloNzOLXQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xGM4R-0000000BQ0z-3Bpv; Mon, 12 Oct 2026 19:52:27 +0000 Received: from linkmauve.fr ([82.65.109.163]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xGM4M-0000000BPzH-3Lyv; Mon, 12 Oct 2026 19:52:26 +0000 Received: by linkmauve.fr (Postfix, from userid 1000) id 2515A71FB39B; Mon, 12 Oct 2026 21:52:16 +0200 (CEST) From: Link Mauve To: Srinivas Kandagatla Cc: Link Mauve , Sasha Finkelstein , Andy Shevchenko , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Sven Peter , Janne Grunau , Neal Gompa , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Vladimir Zapolskiy , =?UTF-8?q?Andr=C3=A9=20Draszik?= , Orson Zhai , Baolin Wang , Chunyan Zhang , Maxime Coquelin , Alexandre Torgue , Kalyani Akula , Michal Simek , Miguel Ojeda , Boqun Feng , Gary Guo , =?UTF-8?q?Bj=C3=B6rn=20Roy=20Baron?= , Benno Lossin , Andreas Hindborg , Alice Ryhl , Trevor Gross , Danilo Krummrich , Daniel Almeida , Tamir Duberstein , Alexandre Courbot , =?UTF-8?q?Onur=20=C3=96zkan?= , Johan Hovold , Ronald Claveau , Daniel Lezcano , linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev, imx@lists.linux.dev, linux-arm-msm@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, rust-for-linux@vger.kernel.org Subject: [PATCH v7 2/3] nvmem: core: deprecate reg_write callback with reg_write_const Date: Mon, 12 Oct 2026 21:52:09 +0200 Message-ID: <20261012195213.875403-3-linkmauve@linkmauve.fr> X-Mailer: git-send-email 2.56.0 In-Reply-To: <20261012195213.875403-1-linkmauve@linkmauve.fr> References: <20261012195213.875403-1-linkmauve@linkmauve.fr> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20261012_125222_999892_3C7893E6 X-CRM114-Status: GOOD ( 17.58 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org This callback used to take a mutable void * for no reason, which causes the compiler to be unaware that the val buffer should never be modified by the callback. This was found while drafting the nvmem-provider Rust abstraction. Signed-off-by: Link Mauve Reviewed-by: Andy Shevchenko --- drivers/nvmem/core.c | 26 +++++++++++++++++--------- drivers/nvmem/internals.h | 1 + include/linux/nvmem-provider.h | 6 +++++- 3 files changed, 23 insertions(+), 10 deletions(-) diff --git a/drivers/nvmem/core.c b/drivers/nvmem/core.c index 7d4cce8e2742..df36ad5e9695 100644 --- a/drivers/nvmem/core.c +++ b/drivers/nvmem/core.c @@ -71,7 +71,7 @@ static int __nvmem_reg_read(struct nvmem_device *nvmem, unsigned int offset, } static int __nvmem_reg_write(struct nvmem_device *nvmem, unsigned int offset, - void *val, size_t bytes) + const void *val, size_t bytes) { struct nvmem_operations *ops; int ret, wr_ok; @@ -81,14 +81,17 @@ static int __nvmem_reg_write(struct nvmem_device *nvmem, unsigned int offset, if (!ops) return -ENODEV; - if (!ops->reg_write) + if (!ops->reg_write && !ops->reg_write_const) return -EOPNOTSUPP; ret = gpiod_set_value_cansleep(nvmem->wp_gpio, 0); if (ret) return ret; - wr_ok = ops->reg_write(nvmem->priv, offset, val, bytes); + if (ops->reg_write_const) + wr_ok = ops->reg_write_const(nvmem->priv, offset, val, bytes); + else + wr_ok = ops->reg_write(nvmem->priv, offset, (void *)val, bytes); ret = gpiod_set_value_cansleep(nvmem->wp_gpio, 1); if (ret) @@ -309,7 +312,7 @@ static umode_t nvmem_bin_attr_get_umode(struct nvmem_device *nvmem) if (!nvmem->read_only) mode |= 0200; - if (!ops->reg_write) + if (!ops->reg_write && !ops->reg_write_const) mode &= ~0200; if (!ops->reg_read) @@ -346,13 +349,13 @@ static umode_t nvmem_attr_is_visible(struct kobject *kobj, struct nvmem_operations *ops = rcu_dereference_raw(nvmem->ops); /* - * If the device has no .reg_write operation, do not allow - * configuration as read-write. + * If the device has no .reg_write or .reg_write_const operation, do + * not allow configuration as read-write. * If the device is set as read-only by configuration, it * can be forced into read-write mode using the 'force_ro' * attribute. */ - if (attr == &dev_attr_force_ro.attr && !ops->reg_write) + if (attr == &dev_attr_force_ro.attr && !ops->reg_write && !ops->reg_write_const) return 0; /* Attribute not visible */ return attr->mode; @@ -904,7 +907,7 @@ struct nvmem_device *nvmem_register(const struct nvmem_config *config) if (!config->dev) return ERR_PTR(-EINVAL); - if (!config->reg_read && !config->reg_write) + if (!config->reg_read && !config->reg_write && !config->reg_write_const) return ERR_PTR(-EINVAL); nvmem = kzalloc_obj(*nvmem); @@ -960,6 +963,7 @@ struct nvmem_device *nvmem_register(const struct nvmem_config *config) ops->reg_read = config->reg_read; ops->reg_write = config->reg_write; + ops->reg_write_const = config->reg_write_const; nvmem->owner = config->owner; if (!nvmem->owner && config->dev->driver) @@ -994,8 +998,12 @@ struct nvmem_device *nvmem_register(const struct nvmem_config *config) if (rval) goto err_put_device; + if (ops->reg_write) + dev_warn(&nvmem->dev, "Usage of reg_write in nvmem is deprecated, please " + "migrate to reg_write_const.\n"); + nvmem->read_only = device_property_present(config->dev, "read-only") || - config->read_only || !ops->reg_write; + config->read_only || (!ops->reg_write && !ops->reg_write_const); #ifdef CONFIG_NVMEM_SYSFS nvmem->dev.groups = nvmem_dev_groups; diff --git a/drivers/nvmem/internals.h b/drivers/nvmem/internals.h index bc7a99f5aefb..14243d44815d 100644 --- a/drivers/nvmem/internals.h +++ b/drivers/nvmem/internals.h @@ -12,6 +12,7 @@ struct nvmem_operations { nvmem_reg_read_t reg_read; nvmem_reg_write_t reg_write; + nvmem_reg_write_const_t reg_write_const; }; struct nvmem_device { diff --git a/include/linux/nvmem-provider.h b/include/linux/nvmem-provider.h index 6063fe5b7784..e944530999b4 100644 --- a/include/linux/nvmem-provider.h +++ b/include/linux/nvmem-provider.h @@ -20,6 +20,8 @@ typedef int (*nvmem_reg_read_t)(void *priv, unsigned int offset, void *val, size_t bytes); typedef int (*nvmem_reg_write_t)(void *priv, unsigned int offset, void *val, size_t bytes); +typedef int (*nvmem_reg_write_const_t)(void *priv, unsigned int offset, + const void *val, size_t bytes); /* used for vendor specific post processing of cell data */ typedef int (*nvmem_cell_post_process_t)(void *priv, const char *id, int index, unsigned int offset, void *buf, @@ -93,7 +95,8 @@ struct nvmem_cell_info { * @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; return zero if successful. - * @reg_write: Callback to write data; return zero if successful. + * @reg_write: **DEPRECATED** - please use reg_write_const instead. + * @reg_write_const: Callback to write data; return zero if successful. * @size: Device size. * @word_size: Minimum read/write access granularity. * @stride: Minimum read/write access stride. @@ -128,6 +131,7 @@ struct nvmem_config { struct device_node *of_node; nvmem_reg_read_t reg_read; nvmem_reg_write_t reg_write; + nvmem_reg_write_const_t reg_write_const; int size; int word_size; int stride; -- 2.56.0 _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic