From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from luna.linkmauve.fr (82-65-109-163.subs.proxad.net [82.65.109.163]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A3AFF3B0AD3; Wed, 15 Jul 2026 19:09:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=82.65.109.163 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784142564; cv=none; b=HTXuEwoeIkcAj1eYneg+1LUstY4B6uSjlKCyOjsi5qf+BTFXAAVnO0YI7IjTpfKrZvSEdV1T2QLjhGKhbVFgsc0PZ03Ak0LCp0pbKeU4608VWnysimd849cgdpTE5RuyKn5c4nFF1sfoX4Z9ww5js1dv13Yhk9Juty2FMGAm304= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784142564; c=relaxed/simple; bh=66x5PoM5bVTx1CXpsNEXknXuxFVHsPvPsPCs90jO+30=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=t29mwJ/enc23qVyp5cCkrk0wc/fRp29+yjg5K9sB9wxk6QZ/j5IiFJHuk6vvWFzJOSRkyT8+FyciWknHaCGbZFaz2K1OvxrxayckAAq+qSnnmohFwXpHBDsZTVWcEIoibUJ/Cr1kmvtIJ6iZmFChIL4K61SMpMXbf6g1epjBJpk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linkmauve.fr; spf=pass smtp.mailfrom=linkmauve.fr; arc=none smtp.client-ip=82.65.109.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linkmauve.fr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linkmauve.fr Received: by luna.linkmauve.fr (Postfix, from userid 1000) id 7F6AFF40D2D; Wed, 15 Jul 2026 21:09:06 +0200 (CEST) Date: Wed, 15 Jul 2026 21:09:06 +0200 From: Link Mauve To: Andy Shevchenko Cc: Link Mauve , Srinivas Kandagatla , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Jonathan Cameron , David Lechner , Nuno =?iso-8859-1?Q?S=E1?= , Andy Shevchenko , Sakari Ailus , Tianshu Qiu , Bingbu Cao , Mauro Carvalho Chehab , Aaro Koskinen , Andreas Kemnade , Roger Quadros , Tony Lindgren , Lee Jones , Arnd Bergmann , Greg Kroah-Hartman , Bartosz Golaszewski , "Vaibhaav Ram T.L" , Kumaravel Thiagarajan , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Vladimir Zapolskiy , =?iso-8859-1?Q?Andr=E9?= Draszik , Orson Zhai , Baolin Wang , Chunyan Zhang , Maxime Coquelin , Alexandre Torgue , Praveen Teja Kundanala , Kalyani Akula , Michal Simek , Alexandre Belloni , Joshua Kinard , Antoniu Miclaus , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Miguel Ojeda , Boqun Feng , Gary Guo , =?iso-8859-1?Q?Bj=F6rn?= Roy Baron , Benno Lossin , Andreas Hindborg , Alice Ryhl , Trevor Gross , Danilo Krummrich , Daniel Almeida , Tamir Duberstein , Alexandre Courbot , Onur =?iso-8859-1?Q?=D6zkan?= , Daniel Lezcano , Johan Hovold , Ronald Claveau , Salah Triki , Yury Norov , David Carlier , Achim Gratz , Alexander Sverdlin , Patrick Wicki , Markus Heidelberg , Markus Perkins , Uwe =?iso-8859-1?Q?Kleine-K=F6nig_=28The_Capable_Hub=29?= , Bjorn Helgaas , Abd-Alrhman Masalkhi , Chen Ni , Kees Cook , linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, linux-media@vger.kernel.org, linux-omap@vger.kernel.org, mfd@lists.linux.dev, linux-i2c@vger.kernel.org, linux-gpio@vger.kernel.org, imx@lists.linux.dev, linux-arm-msm@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-rtc@vger.kernel.org, linux-sunxi@lists.linux.dev, rust-for-linux@vger.kernel.org Subject: Re: [PATCH 0/8] nvmem: make reg_write() take a const void * Message-ID: References: <20260715175229.24672-1-linkmauve@linkmauve.fr> Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Jabber-ID: linkmauve@linkmauve.fr On Wed, Jul 15, 2026 at 10:02:13PM +0300, Andy Shevchenko wrote: > On Wed, Jul 15, 2026 at 07:52:16PM +0200, Link Mauve wrote: > > 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. > > > > I’ve split this series based on the subsystem each patch affects, but I > > think it should go through the nvmem tree in one go after gathering the > > acks from the other subsystem maintainers. > > It's an option. The other option (and often we go this way) is to introduce > a new name, convert users, convert the name back. Thanks for the hint, I’ll go for this route for v2 then. > > See, for example, how GPIO subsystem changed a proto of .set() callback. I will, thanks. > > > I’ve been told previously to not include too many maintainers in Cc, but > > I don’t know which ones to remove from the get_maintainers.pl output, so > > I’ve still included everyone, sorry about that. > > With this, rather standard, way you will fulfill the above requirement > automatically. Also you won't need treewide change to bother everybody > with this. And also increase the chances to go in (maybe in a few releases > span). I originally thought given how few users of this API there are in the kernel, it would be simpler to fix them all in one go, but that might be a better option then. > > -- > With Best Regards, > Andy Shevchenko > > -- Link Mauve