All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Jean Delvare <jdelvare@suse.de>
Cc: alsa-devel@alsa-project.org, Jaya Kumar <jayakumar.alsa@gmail.com>
Subject: Re: [PATCH] ALSA: cs553*: Fix dependencies
Date: Tue, 11 Mar 2014 10:38:05 +0100	[thread overview]
Message-ID: <s5hd2ht2hz6.wl%tiwai@suse.de> (raw)
In-Reply-To: <20140311103201.32b8d8da@endymion.delvare>

At Tue, 11 Mar 2014 10:32:01 +0100,
Jean Delvare wrote:
> 
> Hi Takashi,
> 
> On Tue, 11 Mar 2014 07:54:25 +0100, Takashi Iwai wrote:
> > At Mon, 10 Mar 2014 21:12:26 +0100,
> > Jean Delvare wrote:
> > > 
> > > The CS5530, CS5535 and CS5536 chipsets are companions of the Geode
> > > series of processors, which are 32-bit x86 processors. So the
> > > snd-cs5530 and snd-cs5535audio drivers are only needed on this
> > > architecture, except for build testing purpose.
> > > 
> > > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > > Cc: Jaya Kumar <jayakumar.alsa@gmail.com>
> > > Cc: Jaroslav Kysela <perex@perex.cz>
> > > Cc: Takashi Iwai <tiwai@suse.de>
> > 
> > Thanks, applied.
> 
> Hmm, we might have been a bit too quick, sorry. Looking at the
> defconfig files, it would seem that the CS5536 was also used on some
> MIPS machines and the CS3350 was also used on some PowerPC machines:
> 
> arch/mips/configs/lemote2f_defconfig:CONFIG_SND_CS5535AUDIO=m
> arch/powerpc/configs/ppc6xx_defconfig:CONFIG_SND_CS5530=m
> 
> So assuming these default configuration files are correct, the proper
> dependencies would be somewhat different. I'll investigate and send an
> updated or incremental patch, let me know what you prefer.

Ah, I vaguely remember of MIPS stuff, indeed.  I don't know of PPC,
but it's no surprise even if there are such.

So, yes, please send an incremental patch to cover these.


thanks,

Takashi

      reply	other threads:[~2014-03-11  9:38 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-10 20:12 [PATCH] ALSA: cs553*: Fix dependencies Jean Delvare
2014-03-11  6:54 ` Takashi Iwai
2014-03-11  9:32   ` Jean Delvare
2014-03-11  9:38     ` Takashi Iwai [this message]

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=s5hd2ht2hz6.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=jayakumar.alsa@gmail.com \
    --cc=jdelvare@suse.de \
    /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.