Re: [PATCH] ALSA: hda - add missing documentation to hdac_i915

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 24 Sep 2020 18:10:19 +0200,
Kai Vehmanen wrote:
> 
> Hey,
> 
> On Thu, 24 Sep 2020, Takashi Iwai wrote:
> 
> > On Thu, 24 Sep 2020 17:40:43 +0200, Kai Vehmanen wrote:
> > > Document missing function arguments and fix make W=1
> > > warning:
> > 
> > Do we really need to put this internal function in the API documents?
> > If not, the right fix would be to drop the kerneldoc marker instead.
> 
> I was puzzled at this as well. I took a look at some static internal
> functions and there were both with (e.g. snd_hdac_make_cmd()) and without 
> kerneldoc markers.

Some might have been exported functions in the past.

> But, but, granted, this is without any name prefix, so 
> maybe better to drop the kerneldoc. Let me send a v2.

OK, thanks.


Takashi



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux