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: 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.