Hello Sergey, > From: Sergey Shtylyov, Sent: Thursday, October 5, 2023 3:47 AM > Subject: Re: [PATCH net 1/2] ravb: Fix dma_free_coherent() of desc_bat timing in ravb_remove() Thank you for your suggestion in other thread. I'll fix the subject. > On 10/4/23 12:12 PM, Yoshihiro Shimoda wrote: > > > The dma_free_coherent() in ravb_remove() should be called after > > How about: > > In ravb_remove(), dma_free_coherent() should be called after unregister_netdev(). I got it. I'll fix this. > > unregister_netdev(). Otherwise, this controller is possible to use > > the freed buffer. > > > > Fixes: c156633f1353 ("Renesas Ethernet AVB driver proper") > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> > > Reviewed-by: Sergey Shtylyov <s.shtylyov@xxxxxx> Thank you for your review! Best regards, Yoshihiro Shimoda > [...] > > MBR, Sergey