LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Jan Dittmer <jdi@l4x.org>
To: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: lirc@bartelmus.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] lirc: remove backwards compatibility macro obfuscation (Was: Free Linux Driver Development!)
Date: Fri, 02 Feb 2007 13:51:35 +0100	[thread overview]
Message-ID: <45C333D7.3000205@l4x.org> (raw)
In-Reply-To: <84144f020702020332wee0aa69v19d09768f5643779@mail.gmail.com>

Pekka Enberg wrote:
> On 2/2/07, Jan Dittmer <jdi@l4x.org> wrote:
>> Pekka, it would be better if you could sort out most of the
>> basic issues with lirc directly with the developers of lirc
>> and then prepare a complete patch series and post that to
>> lkml. Incrementally adding one driver after another. Posting
>> patches to non-existent sources to lkml is pointless. First
>> create a discussion base, please.
> 
> What discussion base? You need to get rid of the cruft anyway and now
> you have patch to do that. I am not volunteering to sort out _all_ the
> issues.

I really really welcome your efforts - no doubt.

I wanted to express that the patches you posted are simply not
suitable for lkml discussion as there was no full patchset for
lirc for review posted, prior to your patches.

Normal process for new features (and lirc is a new feature so
far lkml is concerned) is like (as I understand it):
  1. post full patchset
  2. duck
  3. receive criticism & patches
  4. integrate results from 3
  5. goto 1
You started at 3. It would be better to start with the whole
picture at 1.

I hope I made myself clearer now.

Jan

  reply	other threads:[~2007-02-02 12:51 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-02  8:39 Pekka J Enberg
2007-02-02 11:04 ` Vincent Vanackere
2007-02-04  3:04   ` [PATCH] lirc: remove backwards compatibility macro obfuscation Christoph Bartelmus
2007-02-02 11:10 ` [PATCH] lirc: remove backwards compatibility macro obfuscation (Was: Free Linux Driver Development!) Jan Dittmer
2007-02-02 11:32   ` Pekka Enberg
2007-02-02 12:51     ` Jan Dittmer [this message]
2007-02-04  3:03 ` [PATCH] lirc: remove backwards compatibility macro obfuscation Christoph Bartelmus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=45C333D7.3000205@l4x.org \
    --to=jdi@l4x.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lirc@bartelmus.de \
    --cc=penberg@cs.helsinki.fi \
    --subject='Re: [PATCH] lirc: remove backwards compatibility macro obfuscation (Was: Free Linux Driver Development'\!')' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).