[linux-dvb] [PATCH 1/7] Adding empia base driver
mrechberger at gmail.com
Sat Nov 8 11:46:08 CET 2008
On Sat, Nov 8, 2008 at 11:42 AM, Markus Rechberger
<mrechberger at gmail.com> wrote:
> On Sat, Nov 8, 2008 at 11:37 AM, Mauro Carvalho Chehab
> <mchehab at infradead.org> wrote:
>> On Sat, 8 Nov 2008, Markus Rechberger wrote:
>>> As written earlier already I don't think that I didn't follow any
>>> rules here since I provided single
>>> patches at the very first beginning
>>> (this is all kernel code and only kernel code).
>>> That work didn't get attention and due a different decision of
>>> framework changes (which that codebase relied
>>> on) it all had to be rebased, I doubt that anyone
>>> would have reworked all that patch for patch. Instead it went into one
>>> repository and finally got modified to work again
>>> with the available framework rather than relying onto any such
>> One thing is to rebase a tree, another is to merge all patches into a big
>> one, not preserving the original authorships.
>> Development trees sometimes need rebase. This is done by popping all newer
>> patches from the tree, applying the upstream patches, and then pushing again
>> every individual patches, fixing the ones that don't apply well, but
>> preserving their authorships.
>> The modified patches should receive a special tag before the maintainer's
>> SOB, like:
>> [me at mymail: I did this to apply this patch]
>> as stated at the kernel docs.
>> This method will reduce a lot the risk of breaking improvements and other
>> fixes that happened upstream, and will properly preserve authorship of
>> individual patches.
>> If you were doing a rebase, your patches would likely be accepted.
let's link back to the only review done:
In my opinion it's pretty much hopeless trying to convert the current
em28xx driver into what you have. It's a huge amount of work that no
one wants to do and (in this case) with very little benefit.
More information about the linux-dvb