* Wei Yang <richard.weiyang@xxxxxxxxx> [240830 10:36]: > On Fri, Aug 30, 2024 at 09:30:37AM -0400, Liam R. Howlett wrote: > >* Wei Yang <richard.weiyang@xxxxxxxxx> [240830 04:47]: > >> On Fri, Aug 30, 2024 at 12:12:14AM -0400, Liam R. Howlett wrote: > >> >* Wei Yang <richard.weiyang@xxxxxxxxx> [240829 22:57]: > >> >> This patch tries to cleanup some interface description: > >> >> > >> >> * function name mismatch > >> >> * parameter name mismatch > >> >> * parameter all end up with ':' > >> >> * not prefix '*' if parameter is a pointer > >> > > >> >Why not use * if the parameter is a pointer? > >> > >> This tries to comply with kernel doc format. > >> > >> If you prefix a '*' in kenrel doc, ./scripts/kernel-doc would report a > >> warning. > > > >Thanks. I wanted those so that it was overly obvious when they were > >used wrong but I guess they have to go. > > > > Not fully get your point. > > You are fine to keep this change, or prefer me to drop it in next version? I am fine with the changes. Apologies for not being clear. Thanks, Liam