Re: [PATCH 3/7] ALSA: emu10k1: fix snd_emu1010_fpga_read() input masking for rev2 cards

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

 



On Fri, 21 Apr 2023 17:25:46 +0200,
Oswald Buddenhagen wrote:
> 
> On Fri, Apr 21, 2023 at 05:14:10PM +0200, Takashi Iwai wrote:
> > On Fri, 21 Apr 2023 16:10:02 +0200,
> > Oswald Buddenhagen wrote:
> >> 
> >> Signed-off-by: Oswald Buddenhagen <oswald.buddenhagen@xxxxxx>
> > 
> > No changelog is no good news.
> > Please give more information.
> > 
> i suppose this is a difference in philosophy. i generally refrain from
> covering in the commit message what comments in the patch already
> explain. otherwise it would be just redundant, and i don't like that
> very much.
> 
> anyway, i can add some speculation what problems this could have
> theoretically caused.

You didn't explain exactly *WHAT* you fixed, and that's the readers
may be interested more.  The summary says merely "fix
snd_emu1010_fpga_read() input masking for rev2 cards".  What was
broken beforehand and what was corrected by your patch?  The summary
isn't enough for explaining it, but you have more space here.


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