Thread (14 messages) 14 messages, 2 authors, 2012-05-03

Re: [PATCH 3/8] Sometimes the ISDN chip only controls the D-channel

From: David Miller <davem@davemloft.net>
Date: 2012-05-03 06:51:07

From: Karsten Keil <redacted>
Date: Thu, 03 May 2012 08:31:57 +0200
I did put the additional PCM infrastructure in this series, because
the approval test was done with it in place.
I did plan the update of the low level drivers in a separate patchset
from the beginning.

What do you prefer, adding the driver part now, as additional patch, or
removing this additional infrastruckture part and submit it in a later
series ?
I feel like I'm talking to a wall.

A patch should do one, and only one thing.  It should not have
changes which are unrelated to that one thing.

What part of this is so hard to understand?

To make matters worse, you didn't even make a mention of those
unrelated changes in your commit message.

So that patch was bogus on at least two counts.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help