[alsa-devel] [PATCH v3] ALSA: usb-audio: Fix max packet size calculation for USB audio

Ricard Wanderlof ricard.wanderlof at axis.com
Sun Oct 11 16:58:11 CEST 2015


On Sun, 11 Oct 2015, Takashi Iwai wrote:

> > The code as it stands has the following expression on line 613 to 
> > calculate the maximum isochronous packet size:
> > ...
> > size which triggered the if statement to start with.
> 
> The whole text should be put in the changelog.  That's what Clemens
> suggested implicitly.  There is no reason to hide such a precious
> information for later readers.

Yeah, I wasn't sure exactly what Clemens was hinting at.

You mean put the whole thing in the commit message? I can certainly do 
tha. It feels a bit excessive, to me it seemed that once the expression is 
correct a short commit message should suffice explaining what it does, but 
in order to convince potential reviewers (and provide a starting point for 
discussion) an explanation of the proposed change is needed in order to 
get the patch applied.

But I can certainly rework it and put it in the commit message instead.

/Ricard
-- 
Ricard Wolf Wanderlöf                           ricardw(at)axis.com
Axis Communications AB, Lund, Sweden            www.axis.com
Phone +46 46 272 2016                           Fax +46 46 13 61 30


More information about the Alsa-devel mailing list