Re: [PATCH v4 0/3] nvmem: migrate to const void * in reg_write
Link Mauve <[email protected]>
| Newsgroups | org.infradead.lists.linux-amlogic,dev.linux.lists.asahi,dev.linux.lists.imx,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel,org.kernel.vger.rust-for-linux |
|---|---|
| Message-ID | <anmiAfteOQzPXRqX@desktop> |
On Mon, Aug 10, 2026 at 10:27:33AM +0300, Andy Shevchenko wrote: > On Sun, Jul 26, 2026 at 03:17:18PM +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. > > > > Thanks to the guidance of Andy Shevchenko, this now introduces a new > > callback and deprecates the existing one, with the goal of renaming the > > new one into the old one once no user remains in the kernel. > > LGTM now, FWIW, > Reviewed-by: Andy Shevchenko <[email protected]> > > But I think you also need to provide a road map of finishing this job (the best > is to provide a continuation in some of yours public Git trees to point out). I’ve just pushed these changes here, as usual just build-tested: https://github.com/linkmauve/linux/commits/nvmem-v6 I was planning on working on those after the 7.3 merge window if this series indeed goes into 7.3, but I was never worried this would be hard to do, it’s a pretty mechanical change. > > -- > With Best Regards, > Andy Shevchenko > > -- Link Mauve _______________________________________________ linux-amlogic mailing list [email protected] http://lists.infradead.org/mailman/listinfo/linux-amlogic