All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: James Courtier-Dutton <james.dutton@gmail.com>
Cc: alsa-devel@alsa-project.org
Subject: Re: RFC: snd_card_create() function
Date: Mon, 12 Jan 2009 16:25:50 +0100	[thread overview]
Message-ID: <s5htz84bl6p.wl%tiwai@suse.de> (raw)
In-Reply-To: <ad2655cb0901120706g1bfe019bk23316a288c08ee7b@mail.gmail.com>

At Mon, 12 Jan 2009 15:06:51 +0000,
James Courtier-Dutton wrote:
> 
> 2009/1/12 Takashi Iwai <tiwai@suse.de>:
> > At Mon, 12 Jan 2009 14:42:06 +0000,
> > James Courtier-Dutton wrote:
> >>
> >> Sounds good to me.
> >> One can keep the wrapper outside of the kernel, and only available in
> >> alsa-driver for those out of tree people.
> >
> > Yes, that's good.
> > I think it'd be better to do that after one kernel-cycle later for a
> > softer landing, though.
> >
> 
> How soft do we need this?

One kernel cycle is enough.

> The change is probably max of 3 lines of code per sound card that has
> an out of tree driver.
> The only one I am aware of is the xfi one. With the wrapper being more
> than 3 lines of code being put into the kernel only to be removed
> again seems a little un-necessary to me.
> For internal kernel changes, I don't think we should care about out of
> kernel drivers. None of the rest of the kernel developers go out of
> their way for anything out-of-mainline.

Well, my main concern isn't about out-of-kernel drivers like xfi, but
the drivers that will come from other trees.  The codes outside ALSA
tree, for example, V4L, can't be always controlled by us (suppose a
new V4L driver comes in for the next kernel).  And these are more or
less "mainline" development.

If you see the linux-next development, you'll find how the internal
API can be a pain among several tree merges.  A typical API change
(that has been often seen in driver-core area) introduces first a
wrapper while converting all in the present kernel, then kill it at
the next cycle.


Takashi

  reply	other threads:[~2009-01-12 15:25 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-12 14:35 RFC: snd_card_create() function Takashi Iwai
2009-01-12 14:42 ` James Courtier-Dutton
2009-01-12 14:47   ` Takashi Iwai
2009-01-12 15:06     ` James Courtier-Dutton
2009-01-12 15:25       ` Takashi Iwai [this message]
2009-01-14 15:14         ` James Courtier-Dutton
2009-01-14 15:31           ` Takashi Iwai
2009-01-12 16:08 ` Krzysztof Helt
2009-01-12 16:12   ` Takashi Iwai
2009-01-15 13:05 ` Takashi Iwai

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=s5htz84bl6p.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=james.dutton@gmail.com \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.