On Fri, Oct 27, 2023 at 06:20:08PM -0700, Justin Stitt wrote: > On Fri, Oct 27, 2023 at 4:43 PM Andrew Lunn <and...@lunn.ch> wrote: > > > > > +/** > > > + * ethtool_puts - Write string to ethtool string data > > > + * @data: Pointer to start of string to update > > > > Isn't it actually a pointer to a pointer to the start of string to > > update? > > I suppose.
Its kind of relevant because you increment by the length, which you can only do because it is **. I also find it a pretty strange API, so documenting it correctly is important for me. > > FWIW, I just copy-pasted the sprintf doc and tweaked: > /** > * ethtool_sprintf - Write formatted string to ethtool string data > * @data: Pointer to start of string to update So that need fixing as well. I will cook up a patch for this one. Andrew _______________________________________________ Intel-wired-lan mailing list Intel-wired-lan@osuosl.org https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
