Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: ALSA development <alsa-devel@alsa-project.org>
Subject: Re: [PATCH] sound: Retire OSS
Date: Tue, 31 Oct 2017 11:15:31 +0100	[thread overview]
Message-ID: <s5h8tfrei7w.wl-tiwai@suse.de> (raw)
In-Reply-To: <s5hmv47ep8n.wl-tiwai@suse.de>

On Tue, 31 Oct 2017 08:43:52 +0100,
Takashi Iwai wrote:
> 
> On Mon, 30 Oct 2017 20:31:38 +0100,
> Jaroslav Kysela wrote:
> > 
> > Dne 30.10.2017 v 16:01 Takashi Iwai napsal(a):
> > > Hi,
> > > 
> > > below is a snippet of the patch I'm going to apply for finally
> > > dropping the OSS code.  It's been already disabled for a few kernel
> > > versions, and we didn't get any complains.  So, as discussed in the
> > > last audio mini-summit, we go forward now.
> > > 
> > > Since the whole patch just removing a bunch of files is too big to
> > > post (and useless), I attach only a part of it: namely, the changelog,
> > > the diffstat, and the files that are modified (but not deleted).
> > 
> > Reviewed-by: Jaroslav Kysela <perex@perex.cz>
> 
> Thanks, now I merged the patch to for-next branch.
> Let's see whether it goes well.

... and it had a bug, a leftover at sound/Makefile.
The following diff was applied.


Takashi

---
diff --git a/sound/Makefile b/sound/Makefile
index 6de45d2c32f7..e5794fdd3d79 100644
--- a/sound/Makefile
+++ b/sound/Makefile
@@ -2,8 +2,7 @@
 #
 
 obj-$(CONFIG_SOUND) += soundcore.o
-obj-$(CONFIG_SOUND_PRIME) += oss/
-obj-$(CONFIG_DMASOUND) += oss/
+obj-$(CONFIG_DMASOUND) += oss/dmasound/
 obj-$(CONFIG_SND) += core/ i2c/ drivers/ isa/ pci/ ppc/ arm/ sh/ synth/ usb/ \
 	firewire/ sparc/ spi/ parisc/ pcmcia/ mips/ soc/ atmel/ hda/ x86/
 obj-$(CONFIG_SND_AOA) += aoa/

  reply	other threads:[~2017-10-31 10:15 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-30 15:01 [PATCH] sound: Retire OSS Takashi Iwai
2017-10-30 19:31 ` Jaroslav Kysela
2017-10-31  7:43   ` Takashi Iwai
2017-10-31 10:15     ` Takashi Iwai [this message]
2017-10-31 10:29       ` Jaroslav Kysela

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=s5h8tfrei7w.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox