Re: [PATCH v3] rust: pwm: replace `core::mem::zeroed` with `pin_init::zeroed`

Francis Laniel <[email protected]>
Newsgroups org.kernel.linux.tools,org.kernel.vger.linux-kernel,org.kernel.vger.linux-pwm,org.kernel.vger.rust-for-linux
Message-ID <2849575.mvXUDI8C0e@pmachine>
Hi!

Le mardi 11 août 2026, 08:04:49 heure d’été d’Europe de l’Est Uwe Kleine-König a écrit :
> Hello,
> 
> On Wed, Jun 03, 2026 at 07:09:09PM +0300, Francis Laniel wrote:
> > All types in `bindings` implement `Zeroable` if they can, so use
> > `pin_init::zeroed` instead of relying on `unsafe` code.
> > 
> > If this ends up not compiling in the future, something in bindgen or on
> > the C side changed and is most likely incorrect.
> > 
> > Suggested-by: Benno Lossin <[email protected]>
> > Link: https://github.com/Rust-for-Linux/linux/issues/1189
> > Reviewed-by: Alexandre Courbot <[email protected]>
> > Acked-by: Michal Wilczynski <[email protected]>
> > Signed-off-by: Francis Laniel <[email protected]>
> 
> Applied to
> 
> 	https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next
> 
> for the 7.3 merge window.

Thank you for the merge!

> 
> b4 behaved strange on that patch:
> 
> 	$ b4 am -s -l -t -3 [email protected]
> 	Grabbing thread from lore.kernel.org/all/[email protected]/t.mbox.gz
> 	Analyzing 2 messages in the thread
> 	Looking for additional code-review trailers on lore.kernel.org
> 	Analyzing 23 code-review messages
> 	Checking attestation on all messages, may take a moment...
> 	---
> 	  ✓ [PATCH v3] rust: pwm: replace `core::mem::zeroed` with `pin_init::zeroed`
> 	    + Link: https://patch.msgid.link/[email protected]
> 	    + Signed-off-by: Uwe Kleine-König <[email protected]>
> 	  ---
> 	  ✓ Signed: DKIM/privacyrequired.com
> 	---
> 	Total patches: 1
> 	---
> 	NOTE: some trailers ignored due to from/email mismatches:
> 	    ! Trailer: Suggested-by: Benno Lossin <[email protected]>
> 	     Msg From: Miguel Ojeda <[email protected]>
> 	    ! Trailer: Acked-by: Michal Wilczynski <[email protected]>
> 	     Msg From: Miguel Ojeda <[email protected]>
> 	    ! Trailer: Signed-off-by: Francis Laniel <[email protected]>
> 	     Msg From: Miguel Ojeda <[email protected]>
> 	    ! Trailer: Reviewed-by: Alexandre Courbot <[email protected]>
> 	     Msg From: Miguel Ojeda <[email protected]>
> 	NOTE: Rerun with -S to apply them anyway
> 	Preparing fake-am for v3: rust: pwm: replace `core::mem::zeroed` with `pin_init::zeroed`
> 	  range: 2707b9df955d..81c3f2b030a1
> 	Preared a fake commit range for 3-way merge (2707b9df955d..81c3f2b030a1)
> 	---
> 	 Link: https://patch.msgid.link/[email protected]
> 	 Base: applies clean to current tree
> 	       git checkout -b v3_20260603_laniel_francis_privacyrequired_com HEAD
> 	       git am -3 ./v3_20260603_laniel_francis_rust_pwm_replace_core_mem_zeroed_with_pin_init_zeroed.mbx
> 
> I didn't research what went wrong there and the trailer made it into the
> patch anyhow and adding -S doesn't change the patch. I added the
> [email protected] mailing list to Cc:, maybe someone there wants to debug that.
> 
> Best regards and thanks for your contribution,
> Uwe
> 

Best regards,
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.