[PATCH] media: Drop superfluous PCM preallocation error checks

2019-02-06 Thread Takashi Iwai
snd_pcm_lib_preallocate_pages() and co always succeed, so the error check is simply redundant. Drop it. Signed-off-by: Takashi Iwai --- This is a piece I overlooked in the previous patchset: https://www.spinics.net/lists/alsa-devel/msg87223.html Media people, please give ACK if this is OK

Re: [alsa-devel] [PATCH v9 4/4] sound/usb: Use Media Controller API to share media resources

2018-12-19 Thread Takashi Iwai
nup is done in usb_audio_disconnect(). > > Signed-off-by: Shuah Khan Feel free to take my ack regarding the sound stuff: Reviewed-by: Takashi Iwai Thanks! Takashi

Re: DVB-S PCI card regression on 4.19 / 4.20

2018-11-27 Thread Takashi Iwai
On Tue, 27 Nov 2018 16:58:58 +0100, stakanov wrote: > > In data martedì 27 novembre 2018 13:49:46 CET, Mauro Carvalho Chehab ha > scritto: > > Hi Stakanov, > > > > Em Tue, 27 Nov 2018 11:02:57 +0100 > > > > stakanov escreveu: > > > In data luned

Re: DVB-S PCI card regression on 4.19 / 4.20

2018-11-26 Thread Takashi Iwai
On Fri, 23 Nov 2018 18:26:25 +0100, Mauro Carvalho Chehab wrote: > > Takashi, > > Could you please produce a Kernel for Stakanov to test > with the following patches: > > https://patchwork.linuxtv.org/patch/53044/ > https://patchwork.linuxtv.org/patch/53045/ > https://patchwork.linuxtv.org/patch

Re: DVB-S PCI card regression on 4.19 / 4.20

2018-11-20 Thread Takashi Iwai
On Tue, 20 Nov 2018 11:26:04 +0100, Mauro Carvalho Chehab wrote: > > Em Tue, 20 Nov 2018 10:54:22 +0100 > Takashi Iwai escreveu: > > > On Tue, 20 Nov 2018 10:51:59 +0100, > > stakanov wrote: > > > > > > In data martedì 20 novembre 2018 10:18:10 CE

Re: DVB-S PCI card regression on 4.19 / 4.20

2018-11-20 Thread Takashi Iwai
On Tue, 20 Nov 2018 10:51:59 +0100, stakanov wrote: > > In data martedì 20 novembre 2018 10:18:10 CET, Mauro Carvalho Chehab ha > scritto: > > Em Tue, 20 Nov 2018 09:07:57 +0100 > > > > stakanov escreveu: > > > In data martedì 20 novembre 2018 00:58:41 CET, Mauro Carvalho Chehab ha > > > > > >

Re: DVB-S PCI card regression on 4.19 / 4.20

2018-11-19 Thread Takashi Iwai
On Tue, 20 Nov 2018 00:19:54 +0100, stakanov wrote: > > In data martedì 20 novembre 2018 00:08:45 CET, Mauro Carvalho Chehab ha > scritto: > > uname -a > > > > > Linux silversurfer 4.20.0-rc3-1.g7e16618-default #1 SMP PREEMPT Mon Nov 19 > > > 18:54:15 UTC 2018 (7e16618) x86_64 x86_64 x86_64 GNU

Re: DVB-S PCI card regression on 4.19 / 4.20

2018-11-19 Thread Takashi Iwai
On Mon, 19 Nov 2018 18:53:26 +0100, Mauro Carvalho Chehab wrote: > > Could you ask the user to apply the enclosed patch and provide us > the results of those prints? OK, I built a test kernel in OBS home:tiwai:bsc1116374 repo. Now it's available at https://download.opensuse.org/repositories/ho

DVB-S PCI card regression on 4.19 / 4.20

2018-11-19 Thread Takashi Iwai
Hi, we've got a regression report on openSUSE Bugzilla regarding DVB-S PCI card: https://bugzilla.opensuse.org/show_bug.cgi?id=1116374 According to the reporter (Stakanov, Cc'ed), the card worked fine on 4.18.15, but since 4.19, it doesn't give any channels, sound nor picture, but only EPG is r

Re: [alsa-devel] [PATCH 4/8] sound: hpios: don't pass GFP_DMA32 to dma_alloc_coherent

2018-10-13 Thread Takashi Iwai
On Sat, 13 Oct 2018 18:35:40 +0200, Christoph Hellwig wrote: > > On Sat, Oct 13, 2018 at 06:18:28PM +0200, Takashi Iwai wrote: > > On Sat, 13 Oct 2018 17:17:03 +0200, > > Christoph Hellwig wrote: > > > > > > The DMA API does its own zone deci

Re: [alsa-devel] [PATCH 5/8] sound: intel/sst: don't pass GFP_DMA32 to dma_alloc_coherent

2018-10-13 Thread Takashi Iwai
On Sat, 13 Oct 2018 17:17:04 +0200, Christoph Hellwig wrote: > > The DMA API does its own zone decisions based on the coherent_dma_mask. > > Signed-off-by: Christoph Hellwig Reviewed-by: Takashi Iwai thanks, Takashi > --- > sound/soc/intel/common/sst-firmware.c | 2 +-

Re: [alsa-devel] [PATCH 4/8] sound: hpios: don't pass GFP_DMA32 to dma_alloc_coherent

2018-10-13 Thread Takashi Iwai
On Sat, 13 Oct 2018 17:17:03 +0200, Christoph Hellwig wrote: > > The DMA API does its own zone decisions based on the coherent_dma_mask. > > Signed-off-by: Christoph Hellwig Reviewed-by: Takashi Iwai Would you like to take this as a series, or shall I take individually throug

Re: [RFC PATCH] sound, media: array_find() can be static

2018-04-22 Thread Takashi Iwai
On Sun, 22 Apr 2018 20:05:03 +0200, kbuild test robot wrote: > > > Fixes: dbd775375e7d ("sound, media: allow building ISA drivers it with > COMPILE_TEST") It's not actually a bug that was introduced by the commit, but it just made appearing clearer. So I dropped this tag. > Signed-off-by: Fen

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-22 Thread Takashi Iwai
On Sun, 22 Apr 2018 20:05:03 +0200, kbuild test robot wrote: > > Hi Mauro, > > I love your patch! Perhaps something to improve: > > [auto build test WARNING on linuxtv-media/master] > [also build test WARNING on v4.17-rc1 next-20180420] > [if your patch is applied to the wrong git tree, please d

Re: [PATCH] sound, isapnp: allow building more drivers with COMPILE_TEST

2018-04-20 Thread Takashi Iwai
> > So, remove such dependencies when COMPILE_TEST. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Takashi Iwai thanks, Takashi > --- > drivers/pnp/isapnp/Kconfig | 2 +- > sound/isa/Kconfig | 6 +++--- > 2 files changed, 4 insertions(+), 4 delet

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Takashi Iwai
On Fri, 20 Apr 2018 15:01:22 +0200, Mauro Carvalho Chehab wrote: > > Em Fri, 20 Apr 2018 09:51:29 -0300 > Mauro Carvalho Chehab escreveu: > > > Em Fri, 20 Apr 2018 14:37:46 +0200 > > Takashi Iwai escreveu: > > > > > On Fri, 20 Apr 2018 14:32:15 +

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Takashi Iwai
On Fri, 20 Apr 2018 14:51:29 +0200, Mauro Carvalho Chehab wrote: > > Em Fri, 20 Apr 2018 14:37:46 +0200 > Takashi Iwai escreveu: > > > On Fri, 20 Apr 2018 14:32:15 +0200, > > Mauro Carvalho Chehab wrote: > > > > > > All sound drivers that don'

Re: [PATCH 3/4] sound, media: allow building ISA drivers it with COMPILE_TEST

2018-04-20 Thread Takashi Iwai
On Fri, 20 Apr 2018 14:32:15 +0200, Mauro Carvalho Chehab wrote: > > All sound drivers that don't depend on PNP can be safelly > build with COMPILE_TEST, as ISA provides function stubs to > be used for such purposes. > > As a side effect, with this change, the radio-miropcm20 > can now be built o

Re: [alsa-devel] [trivial PATCH] treewide: Align function definition open/close braces

2017-12-18 Thread Takashi Iwai
| 6 +++--- > kernel/trace/trace_printk.c | 4 ++-- > lib/raid6/sse2.c | 14 +++--- > sound/soc/fsl/fsl_dma.c | 2 +- For sound bits, Acked-by: Takashi Iwai thanks, Takashi

Re: [alsa-devel] [PATCH] ALSA: hda - Fix applying MSI dual-codec mobo quirk

2017-06-01 Thread Takashi Iwai
On Thu, 01 Jun 2017 22:58:24 +0200, Takashi Iwai wrote: > > The previous commit [63691587f7b0: ALSA: hda - Apply dual-codec quirk > for MSI Z270-Gaming mobo] attempted to apply the existing dual-codec > quirk for a MSI mobo. But it turned out that this isn't applied > pro

[PATCH v2 10/27] ALSA: hdsp: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/rme9652/hdsp.c | 67 1 file changed, 50 insertions(+), 17 deletions(-) diff

[PATCH v2 23/27] ALSA: pcm: Direct in-kernel read/write support

2017-06-01 Thread Takashi Iwai
pport is really simple because of the code refactoring until now. Signed-off-by: Takashi Iwai --- include/sound/pcm.h | 38 +- sound/core/pcm_lib.c | 26 +- 2 files changed, 58 insertions(+), 6 deletions(-) diff --git a/include/sound/

[PATCH v2 17/27] ALSA: pcm: Check PCM state by a common helper function

2017-06-01 Thread Takashi Iwai
Signed-off-by: Takashi Iwai --- sound/core/pcm_lib.c | 81 +++- 1 file changed, 29 insertions(+), 52 deletions(-) diff --git a/sound/core/pcm_lib.c b/sound/core/pcm_lib.c index 0db8d4e0fca2..e4f5c43b6448 100644 --- a/sound/core/pcm_lib.c +++ b

[PATCH v2 03/27] ALSA: dummy: Convert to new PCM copy ops

2017-06-01 Thread Takashi Iwai
It's a dummy ops, so just replacing it. Signed-off-by: Takashi Iwai --- sound/drivers/dummy.c | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/sound/drivers/dummy.c b/sound/drivers/dummy.c index 172dacd925f5..dd5ed037adf2 100644 --- a/sound/dr

[PATCH v2 02/27] ALSA: pcm: Introduce copy_user, copy_kernel and fill_silence ops

2017-06-01 Thread Takashi Iwai
unify the implementations of both interleaved and non-interleaved cases, as we'll see in the later patch. As of this stage, copy_kernel ops isn't referred yet, but only copy_user is used. Signed-off-by: Takashi Iwai --- include/sound/pcm.h | 7 + sound/

[PATCH v2 15/27] [media] solo6x10: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. The device supports only 1 channel and 8bit sample, so it's always bytes=frames, and we need no conversion of unit in the callback. Also, it's a capture stream, thus no silence is needed. Signed-off-by: Takashi Iwai --- dri

[PATCH] ALSA: hda - Fix applying MSI dual-codec mobo quirk

2017-06-01 Thread Takashi Iwai
tries just because they were put in the wrong position, although we have a list ordered by PCI SSID numbers. This patch fixes it by rearranging the unordered entries. Fixes: 63691587f7b0 ("ALSA: hda - Apply dual-codec quirk for MSI Z270-Gaming mobo") Reported-by: Rudolf Schmidt Signed-

[PATCH v2 19/27] ALSA: pcm: Call directly the common read/write helpers

2017-06-01 Thread Takashi Iwai
Make snd_pcm_lib_read() and *_write() static inline functions that call the common helper functions directly. This reduces a slight amount of codes, and at the same time, it's a preparation for the further cleanups / fixes. Signed-off-by: Takashi Iwai --- include/sound/pcm.h

[PATCH v2 14/27] ASoC: blackfin: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. In AC97 and I2S-TDM mode, we need to convert back to frames, but otherwise the conversion is pretty straightforward. Signed-off-by: Takashi Iwai --- sound/soc/blackfin/bf5xx-ac97-pcm.c | 27 +++ sound/soc

[PATCH v2 08/27] ALSA: rme96: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. The conversion is straightforward with standard helper functions, and now we can drop the bytes <-> frames conversions in callbacks. Signed-off-by: Takashi Iwai --- sound/pci/rme96.

[PATCH v2 04/27] ALSA: es1938: Convert to the new PCM copy ops

2017-06-01 Thread Takashi Iwai
Replace the copy ops with the new copy_user and copy_kernel ops. It's used only for a capture stream (for some hardware workaround), thus we need no silence operation. Signed-off-by: Takashi Iwai --- sound/pci/es1938.c | 33 + 1 file changed, 25 insertions(

[PATCH v2 00/27] Revised full patchset for PCM in-kernel copy support

2017-06-01 Thread Takashi Iwai
. I'm Cc'ing the media and the USB people since it touches solo6x10 and usb-gadget drivers. The previous ACK was dropped as each patch was rewritten again. Sorry for the doubly patch-review labours. thanks, Takashi === Takashi Iwai (26): ALSA: pcm: Introduce copy_user, copy_

[PATCH v2 12/27] ALSA: sb: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. For avoiding the code redundancy, slightly hackish macros are introduced. Signed-off-by: Takashi Iwai --- sound/isa/sb/emu8000_pcm.c | 190 ++--- 1 file changed, 109 insertions(+), 81 deletions

[PATCH v2 11/27] ALSA: gus: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. For simplifying the code a bit, two local helpers are introduced here: get_bpos() and playback_copy_ack(). Signed-off-by: Takashi Iwai --- sound/isa/gus/gus_pcm.c | 97 ++--- 1 file changed

[PATCH v2 13/27] ALSA: sh: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. Fixed also the user-space buffer copy with the proper copy_from_user*() variant. Signed-off-by: Takashi Iwai --- sound/sh/sh_dac_audio.c | 54 +++-- 1 file changed, 30 insertions(+), 24

[PATCH v2 24/27] usb: gadget: u_uac1: Kill set_fs() usage

2017-06-01 Thread Takashi Iwai
With the new API to perform the in-kernel buffer copy, we can get rid of set_fs() usage in this driver, finally. Signed-off-by: Takashi Iwai --- drivers/usb/gadget/function/u_uac1.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/usb/gadget/function/u_uac1.c b

[PATCH v2 05/27] ALSA: nm256: Convert to new PCM copy ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new ops. The conversion is straightforward with standard helper functions, and now we can drop the bytes <-> frames conversions in callbacks. Signed-off-by: Takashi Iwai --- sound/pci/nm256/nm256.

[PATCH v2 20/27] ALSA: pcm: More unification of PCM transfer codes

2017-06-01 Thread Takashi Iwai
specific copy ops or it's the standard read/write. Another bonus by this change is that we now guarantee the silencing behavior when NULL buffer is passed to write helpers. It'll simplify some codes later. Signed-off-by: Takashi Iwai --- sound/core/pcm_l

[PATCH v2 18/27] ALSA: pcm: Shuffle codes

2017-06-01 Thread Takashi Iwai
Just shuffle the codes, without any change otherwise. Signed-off-by: Takashi Iwai --- sound/core/pcm_lib.c | 212 +-- 1 file changed, 106 insertions(+), 106 deletions(-) diff --git a/sound/core/pcm_lib.c b/sound/core/pcm_lib.c index e4f5c43b6448

[PATCH v2 27/27] ALSA: doc: Update copy_user, copy_kernel and fill_silence PCM ops

2017-06-01 Thread Takashi Iwai
Signed-off-by: Takashi Iwai --- .../sound/kernel-api/writing-an-alsa-driver.rst| 111 ++--- 1 file changed, 76 insertions(+), 35 deletions(-) diff --git a/Documentation/sound/kernel-api/writing-an-alsa-driver.rst b/Documentation/sound/kernel-api/writing-an-alsa-driver.rst

[PATCH v2 22/27] ALSA: pcm: Simplify snd_pcm_playback_silence()

2017-06-01 Thread Takashi Iwai
Use the existing silence helper codes for simplification. Signed-off-by: Takashi Iwai --- sound/core/pcm_lib.c | 50 -- 1 file changed, 20 insertions(+), 30 deletions(-) diff --git a/sound/core/pcm_lib.c b/sound/core/pcm_lib.c index f15460eaf8b5

[PATCH v2 25/27] ALSA: pcm: Kill set_fs() in PCM OSS layer

2017-06-01 Thread Takashi Iwai
: Takashi Iwai --- sound/core/oss/pcm_oss.c | 77 1 file changed, 12 insertions(+), 65 deletions(-) diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c index e306f05ce51d..2d6a825cfe88 100644 --- a/sound/core/oss/pcm_oss.c +++ b/sound/core

[PATCH v2 26/27] ALSA: pcm: Build OSS writev/readv helpers conditionally

2017-06-01 Thread Takashi Iwai
() directly instead. Signed-off-by: Takashi Iwai --- sound/core/oss/io.c | 4 ++-- sound/core/oss/pcm_oss.c| 12 ++-- sound/core/oss/pcm_plugin.h | 6 ++ 3 files changed, 10 insertions(+), 12 deletions(-) diff --git a/sound/core/oss/io.c b/sound/core/oss/io.c index

[PATCH v2 16/27] ALSA: pcm: Drop the old copy and silence ops

2017-06-01 Thread Takashi Iwai
Now that all users of old copy and silence ops have been converted to the new PCM ops, the old stuff can be retired and go away. Signed-off-by: Takashi Iwai --- include/sound/pcm.h | 5 - sound/core/pcm_lib.c | 38 +- sound/soc/soc-pcm.c | 2 -- 3

[PATCH v2 07/27] ALSA: rme32: Convert to the new PCM copy ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new ops. The conversion is straightforward with standard helper functions, and now we can drop the bytes <-> frames conversions in callbacks. Signed-off-by: Takashi Iwai --- sound/pci/rme32.

[PATCH v2 06/27] ALSA: korg1212: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. Although we can refactor this messy code, at this time, the changes are kept as small as possible. Let's clean up later. Signed-off-by: Takashi Iwai --- sound/pci/korg1212/korg1212.c | 112

[PATCH v2 09/27] ALSA: rme9652: Convert to the new PCM ops

2017-06-01 Thread Takashi Iwai
Replace the copy and the silence ops with the new PCM ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/rme9652/rme9652.c | 71 ++--- 1 file changed, 54 insertions(+), 17 deletions(-) diff

[PATCH v2 21/27] ALSA: pcm: Unify read/write loop

2017-06-01 Thread Takashi Iwai
Both __snd_pcm_lib_read() and __snd_pcm_write() functions have almost the same code to loop over samples. For simplification, this patch unifies both as the single helper, __snd_pcm_lib_xfer(). Other than that, there should be no functional change by this patch. Signed-off-by: Takashi Iwai

[PATCH 16/16] ALSA: pcm: Drop the old copy and silence ops

2017-05-21 Thread Takashi Iwai
Now that all users of old copy and silence ops have been converted to the new copy_silence ops, the old stuff can be retired and go away. Signed-off-by: Takashi Iwai --- include/sound/pcm.h | 5 - sound/core/pcm_lib.c | 38 +- sound/soc/soc-pcm.c | 2

[PATCH 14/16] ASoC: blackfin: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
take in_kernel into account on this architecture, so the conversion is easy otherwise. Signed-off-by: Takashi Iwai --- sound/soc/blackfin/bf5xx-ac97-pcm.c | 6 ++--- sound/soc/blackfin/bf5xx-ac97.c | 18 ++- sound/soc/blackfin/bf5xx-i2s-pcm.c

[PATCH 15/16] [media] solo6x10: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. It's a capture stream, thus no silence is needed. Signed-off-by: Takashi Iwai --- drivers/media/pci/solo6x10/solo6x10-g723.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/media/pci/sol

[PATCH 01/16] ALSA: pcm: Introduce copy_silence PCM ops

2017-05-21 Thread Takashi Iwai
OSS code for the on-the-fly conversion. As this patch stands, only in_kernel=false is passed. The actual usage of in_kernel=true will be introduced later. Signed-off-by: Takashi Iwai --- include/sound/pcm.h | 3 +++ sound/core/pcm_lib.c | 74

[PATCH 12/16] ALSA: sb: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. We could reduce the redundant silence code by that. Signed-off-by: Takashi Iwai --- sound/isa/sb/emu8000_pcm.c | 99 ++ 1 file changed, 30 insertions(+), 69 deletions(-) diff --git a/sound

[PATCH 04/16] ALSA: es1938: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. It's used only for a capture stream (for some hardware workaround), thus we need no silence operation but only to add the in_kernel memcpy() handling. Signed-off-by: Takashi Iwai --- sound/pci/es1938.c | 11 +++ 1

[PATCH 10/16] ALSA: hdsp: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/rme9652/hdsp.c | 44 1 file changed, 20 insertions(+), 24 deletions(-) diff

[PATCH 02/16] ALSA: Update document about copy_silence PCM ops

2017-05-21 Thread Takashi Iwai
Signed-off-by: Takashi Iwai --- .../sound/kernel-api/writing-an-alsa-driver.rst| 110 - 1 file changed, 63 insertions(+), 47 deletions(-) diff --git a/Documentation/sound/kernel-api/writing-an-alsa-driver.rst b/Documentation/sound/kernel-api/writing-an-alsa-driver.rst

[PATCH 08/16] ALSA: rme96: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/rme96.c | 52 ++-- 1 file changed, 26 insertions(+), 26 deletions(-) diff

[PATCH 06/16] ALSA: nm256: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/nm256/nm256.c | 35 ++- 1 file changed, 14 insertions(+), 21 deletions(-) diff --git a/sound

[PATCH 09/16] ALSA: rme9652: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/rme9652/rme9652.c | 46 - 1 file changed, 20 insertions(+), 26 deletions(-) diff

[PATCH 05/16] ALSA: korg1212: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The redundant function calls are reduced and the copy/silence are handled directly in callback functions now. Signed-off-by: Takashi Iwai --- sound/pci/korg1212/korg1212.c | 128 -- 1 file

[PATCH 13/16] ALSA: sh: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. A straightforward conversion with standard helper functions. Signed-off-by: Takashi Iwai --- sound/sh/sh_dac_audio.c | 40 +--- 1 file changed, 9 insertions(+), 31 deletions(-) diff --git a/sound

[PATCH 07/16] ALSA: rme32: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/pci/rme32.c | 49 - 1 file changed, 24 insertions(+), 25 deletions(-) diff --git

[PATCH 11/16] ALSA: gus: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
Replace the copy and the silence ops with the new merged ops. The conversion is straightforward with standard helper functions. Signed-off-by: Takashi Iwai --- sound/isa/gus/gus_pcm.c | 43 +-- 1 file changed, 9 insertions(+), 34 deletions(-) diff --git

[PATCH 03/16] ALSA: dummy: Convert to copy_silence ops

2017-05-21 Thread Takashi Iwai
It's a dummy ops, so just replacing it. Signed-off-by: Takashi Iwai --- sound/drivers/dummy.c | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/sound/drivers/dummy.c b/sound/drivers/dummy.c index 172dacd925f5..68519689a9ea 100644 --- a/sound/drivers/du

[PATCH 00/16] ALSA: Convert to new copy_silence PCM ops

2017-05-21 Thread Takashi Iwai
uffer copy, so that the PCM drivers can pass the buffer in kernel-space later directly without set_fs() hackery. The latest codes are found in topic/kill-set_fs branch of sound git tree. The media people are Cc'ed for solo6x10 changes. Takashi === Takashi Iwai (16): ALSA: pcm:

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

2016-12-06 Thread Takashi Iwai
On Tue, 06 Dec 2016 19:41:37 +0100, Shuah Khan wrote: > > Hi Takashi, > > On 12/05/2016 11:50 PM, Takashi Iwai wrote: > > On Wed, 30 Nov 2016 23:01:16 +0100, > > Shuah Khan wrote: > >> > >> --- a/sound/usb/card.c > >> +++ b/sound/usb/car

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

2016-12-05 Thread Takashi Iwai
On Mon, 05 Dec 2016 23:44:30 +0100, Shuah Khan wrote: > > On 11/30/2016 03:01 PM, Shuah Khan wrote: > > Change ALSA driver to use Media Controller API to share media resources > > with DVB, and V4L2 drivers on a AU0828 media device. > > > > Media Controller specific initialization is done after s

Re: [PATCH v6 3/3] sound/usb: Use Media Controller API to share media resources

2016-12-05 Thread Takashi Iwai
On Wed, 30 Nov 2016 23:01:16 +0100, Shuah Khan wrote: > > --- a/sound/usb/card.c > +++ b/sound/usb/card.c (snip) > @@ -616,6 +617,11 @@ static int usb_audio_probe(struct usb_interface *intf, > if (err < 0) > goto __error; > > + if (quirk && quirk->media_device) { > +

Re: [RFC][PATCH 0/2] ALSA: control: export all of TLV related macros to user land

2016-09-11 Thread Takashi Iwai
On Sun, 11 Sep 2016 05:06:41 +0200, Takashi Sakamoto wrote: > > On Sep 10 2016 22:41, Takashi Iwai wrote: > > On Sat, 10 Sep 2016 09:25:31 +0200, > > Takashi Sakamoto wrote: > >> > >> On Sep 10 2016 15:44, Takashi Iwai wrote: > >>> On Sat, 10 Sep 2

Re: [RFC][PATCH 0/2] ALSA: control: export all of TLV related macros to user land

2016-09-10 Thread Takashi Iwai
On Sat, 10 Sep 2016 09:25:31 +0200, Takashi Sakamoto wrote: > > On Sep 10 2016 15:44, Takashi Iwai wrote: > > On Sat, 10 Sep 2016 06:50:14 +0200, > > Takashi Sakamoto wrote: > >> > >> Hi, > >> > >> Currently, TLV related protocol is not sh

Re: [RFC][PATCH 0/2] ALSA: control: export all of TLV related macros to user land

2016-09-09 Thread Takashi Iwai
On Sat, 10 Sep 2016 06:50:14 +0200, Takashi Sakamoto wrote: > > Hi, > > Currently, TLV related protocol is not shared to user land. This is not > good in a point of application interfaces, because application developers > can't realize the protocol just to see UAPI headers. > > For this purpose,

Re: [alsa-devel] [very-RFC 0/8] TSN driver for the kernel

2016-06-20 Thread Takashi Iwai
On Tue, 21 Jun 2016 08:38:57 +0200, Richard Cochran wrote: > > On Tue, Jun 21, 2016 at 07:54:32AM +0200, Takashi Iwai wrote: > > > I still would appreciate an answer to my other questions, though... > > > > Currently HD-audio (both ASoC and legacy ones) are the onl

Re: [alsa-devel] [very-RFC 0/8] TSN driver for the kernel

2016-06-20 Thread Takashi Iwai
On Mon, 20 Jun 2016 17:21:26 +0200, Richard Cochran wrote: > > On Mon, Jun 20, 2016 at 02:31:48PM +0200, Richard Cochran wrote: > > Where is this "audio_time" program of which you speak? > > Never mind, found it in alsa-lib. > > I still would appreciate an answer to my other questions, though...

Re: [RFC PATCH v2 0/5] Media Device Allocator API

2016-04-04 Thread Takashi Iwai
On Tue, 05 Apr 2016 05:35:55 +0200, Shuah Khan wrote: > > There are known problems with media device life time management. When media > device is released while an media ioctl is in progress, ioctls fail with > use-after-free errors and kernel hangs in some cases. > > Media Device can be in any t

Re: [alsa-devel] [PATCH] sound/usb: Fix memory leak in media_snd_stream_delete() during unbind

2016-03-19 Thread Takashi Iwai
unpoison_shadow+0x36/0x50 > [ 1445.086890] [] ? kasan_unpoison_shadow+0x36/0x50 > [ 1445.086895] [] ? kasan_kmalloc+0x5e/0x70 > > Signed-off-by: Shuah Khan Mauro, please take it through your tree. Acked-by: Takashi Iwai thanks, Takashi > --- > sound/usb/media.c | 2 +- >

Re: [PATCH] media: add GFP flag to media_*() that could get called in atomic context

2016-03-14 Thread Takashi Iwai
On Mon, 14 Mar 2016 11:13:58 +0100, Mauro Carvalho Chehab wrote: > > Em Mon, 14 Mar 2016 09:22:37 +0200 > Sakari Ailus escreveu: > > > Hi Shuah, > > > > On Sat, Mar 12, 2016 at 06:48:09PM -0700, Shuah Khan wrote: > > > Add GFP flags to media_create_pad_link(), media_create_intf_link(), > > > me

Re: [PATCH 26/31] sound/usb: Update ALSA driver to use Managed Media Controller API

2016-01-07 Thread Takashi Iwai
On Wed, 06 Jan 2016 22:05:35 +0100, Shuah Khan wrote: > > diff --git a/sound/usb/Makefile b/sound/usb/Makefile > index 2d2d122..665fdd9 100644 > --- a/sound/usb/Makefile > +++ b/sound/usb/Makefile > @@ -2,6 +2,18 @@ > # Makefile for ALSA > # > > +# Media Controller > +ifeq ($(CONFIG_MEDIA_CONT

Re: [alsa-devel] [PATCH MC Next Gen] sound/usb: Fix out of bounds access in media_entity_init()

2015-12-07 Thread Takashi Iwai
On Sat, 05 Dec 2015 01:00:29 +0100, Shuah Khan wrote: > > Fix the out of bounds access in media_entity_init() found > by KASan. This is a result of media_mixer_init() failing > to allocate memory for all 3 of its pads before calling > media_entity_init(). Fix it to allocate memory for the > right

Re: [PATCH] media, sound: tea575x: constify snd_tea575x_ops structures

2015-11-22 Thread Takashi Iwai
On Sun, 22 Nov 2015 11:32:53 +0100, Julia Lawall wrote: > > The snd_tea575x_ops structures are never modified, so declare them as > const. > > Done with the help of Coccinelle. > > Signed-off-by: Julia Lawall Reviewed-by: Takashi Iwai Feel free to take via media tre

Re: [PATCH MC Next Gen v2 2/3] sound/usb: Create media mixer function and control interface entities

2015-11-03 Thread Takashi Iwai
On Tue, 03 Nov 2015 17:06:45 +0100, Shuah Khan wrote: > > On 10/25/2015 03:37 PM, Shuah Khan wrote: > > On 10/22/2015 01:16 AM, Takashi Iwai wrote: > >> On Wed, 21 Oct 2015 01:25:15 +0200, > >> Shuah Khan wrote: > >>> > >>> Add support for

Re: [PATCH MC Next Gen v2 2/3] sound/usb: Create media mixer function and control interface entities

2015-10-22 Thread Takashi Iwai
On Wed, 21 Oct 2015 01:25:15 +0200, Shuah Khan wrote: > > Add support for creating MEDIA_ENT_F_AUDIO_MIXER entity for > each mixer and a MEDIA_INTF_T_ALSA_CONTROL control interface > entity that links to mixer entities. MEDIA_INTF_T_ALSA_CONTROL > entity corresponds to the control device for the c

[PATCH] [media] c8sectpfe: Remove select on CONFIG_FW_LOADER_USER_HELPER_FALLBACK

2015-10-18 Thread Takashi Iwai
selects by itself. Let's remove it. Fixes: 850a3f7d5911 ('[media] c8sectpfe: Add Kconfig and Makefile for the driver') Signed-off-by: Takashi Iwai --- drivers/media/platform/sti/c8sectpfe/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/platform/sti/c8sec

Re: Linux 4.2 ALSA snd-usb-audio inconsistent lock state warn in PCM nonatomic mode

2015-08-31 Thread Takashi Iwai
On Tue, 01 Sep 2015 00:48:30 +0200, Shuah Khan wrote: > > Hi Takashi, > > I am seeing the following inconsistent lock state warning when PCM > is run in nonatomic mode. This is on 4.2.0 and with the following > change to force PCM on nonatomic mode: > > diff --git a/sound/usb/stream.c b/sound/us

Re: [PATCH 7/7] sound/usb: Update ALSA driver to use Managed Media Controller API

2015-07-20 Thread Takashi Iwai
On Mon, 20 Jul 2015 10:47:46 +0200, Dan Carpenter wrote: > > On Tue, Jul 14, 2015 at 06:34:06PM -0600, Shuah Khan wrote: > > + ret = media_entity_setup_link(link, flags); > > + if (ret) { > > + dev_err(mctl->media_dev->dev, > > + "Cou

Re: [PATCH 2/2] sound/usb: Update ALSA driver to use media controller API

2015-05-27 Thread Takashi Iwai
At Fri, 8 May 2015 13:31:31 -0600, Shuah Khan wrote: > > Change ALSA driver to use media controller API to share tuner > with DVB and V4L2 drivers that control AU0828 media device. > Media device is created based on a newly added field value > in the struct snd_usb_audio_quirk. Using this approac

Re: DVB suspend/resume regression on 3.19

2015-02-16 Thread Takashi Iwai
At Fri, 13 Feb 2015 16:12:40 +0100, Takashi Iwai wrote: > > At Fri, 13 Feb 2015 12:41:25 -0200, > Mauro Carvalho Chehab wrote: > > > > Em Fri, 13 Feb 2015 15:02:42 +0100 > > Takashi Iwai escreveu: > > > > > At Mon, 09 Feb 2015 11:59:07 +0100, >

Re: DVB suspend/resume regression on 3.19

2015-02-13 Thread Takashi Iwai
At Fri, 13 Feb 2015 12:41:25 -0200, Mauro Carvalho Chehab wrote: > > Em Fri, 13 Feb 2015 15:02:42 +0100 > Takashi Iwai escreveu: > > > At Mon, 09 Feb 2015 11:59:07 +0100, > > Takashi Iwai wrote: > > > > > > Hi, > > > > > > we&#x

Re: DVB suspend/resume regression on 3.19

2015-02-13 Thread Takashi Iwai
At Mon, 09 Feb 2015 11:59:07 +0100, Takashi Iwai wrote: > > Hi, > > we've got a bug report about the suspend/resume regression of DVB > device with 3.19. The symptom is VLC doesn't work after S3 or S4 > resume. strace shows that /dev/dvb/adaptor0/dvr return

DVB suspend/resume regression on 3.19

2015-02-09 Thread Takashi Iwai
Hi, we've got a bug report about the suspend/resume regression of DVB device with 3.19. The symptom is VLC doesn't work after S3 or S4 resume. strace shows that /dev/dvb/adaptor0/dvr returns -ENODEV. The reporter confirmed that 3.18 works fine, so the regression must be in 3.19. There is a rel

Re: [PATCH 1/2] [media] sound: simplify au0828 quirk table

2014-10-30 Thread Takashi Iwai
At Thu, 30 Oct 2014 08:53:04 -0200, Mauro Carvalho Chehab wrote: > > From: Mauro Carvalho Chehab > > Add a macro to simplify au0828 quirk table. That makes easier > to check it against the USB IDs at drivers/media/usb/au0828-card.c > > Cc: sta...@vger.kernel.org > Signed-off-by: Mauro Carvalho

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-26 Thread Takashi Iwai
At Sat, 25 Oct 2014 11:41:15 -0200, Mauro Carvalho Chehab wrote: > > (re-sending from my third e-mail - somehow, the two emails I have at > Samsung didn't seem to be delivering to vger.kernel.org today) > > Em Wed, 22 Oct 2014 14:26:41 -0500 > Pierre-Louis Bossart escreveu: > > > On 10/21/14,

[PATCH] [media] uvc: Fix destruction order in uvc_delete()

2014-10-24 Thread Takashi Iwai
rsively, so the later device_del() can't find its sysfs. The fix is simply rearrange the call orders in uvc_delete() so that the child is removed before the parent. Bugzilla: https://bugzilla.suse.com/show_bug.cgi?id=897736 Reported-and-tested-by: Martin Pluskal Cc: Signed-off-by: Takash

[PATCH] DocBook: Reduce noise from make cleandocs

2014-10-21 Thread Takashi Iwai
I've got a harmless warning when running make cleandocs on an already cleaned tree: Documentation/DocBook/media/Makefile:28: recipe for target 'cleanmediadocs' failed make[1]: [cleanmediadocs] Error 1 (ignored) Suppress this by passing -f to rm. Signed-off-b

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-21 Thread Takashi Iwai
At Tue, 21 Oct 2014 12:08:59 -0400, Devin Heitmueller wrote: > > > Sorry, I'm not convinced by that. If the device has to be controlled > > exclusively, the right position is the open/close. Otherwise, the > > program cannot know when it becomes inaccessible out of sudden during > > its operatio

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-21 Thread Takashi Iwai
At Tue, 21 Oct 2014 17:42:51 +0200, Hans Verkuil wrote: > > > > On 10/16/2014 04:48 PM, Takashi Iwai wrote: > > At Thu, 16 Oct 2014 08:39:14 -0600, > > Shuah Khan wrote: > >> > >> On 10/16/2014 08:16 AM, Takashi Iwai wrote: > >>> At T

Re: [PATCH v2 1/6] media: add media token device resource framework

2014-10-21 Thread Takashi Iwai
At Tue, 21 Oct 2014 13:58:59 +0200, Hans Verkuil wrote: > > > > On 10/21/2014 01:51 PM, Takashi Iwai wrote: > > At Tue, 21 Oct 2014 12:46:03 +0200, > > Hans Verkuil wrote: > >> > >> Hi Shuah, > >> > >> As promised, here is my review fo

Re: [PATCH v2 1/6] media: add media token device resource framework

2014-10-21 Thread Takashi Iwai
At Tue, 21 Oct 2014 12:46:03 +0200, Hans Verkuil wrote: > > Hi Shuah, > > As promised, here is my review for this patch series. > > On 10/14/2014 04:58 PM, Shuah Khan wrote: > > Add media token device resource framework to allow sharing > > resources such as tuner, dma, audio etc. across media d

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-19 Thread Takashi Iwai
At Sat, 18 Oct 2014 20:49:58 +0200, Mauro Carvalho Chehab wrote: > > Em Thu, 16 Oct 2014 08:59:29 -0600 > Shuah Khan escreveu: > > > On 10/16/2014 08:48 AM, Takashi Iwai wrote: > > > At Thu, 16 Oct 2014 08:39:14 -0600, > > > Shuah Khan wrote: > > >

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Takashi Iwai
At Thu, 16 Oct 2014 08:39:14 -0600, Shuah Khan wrote: > > On 10/16/2014 08:16 AM, Takashi Iwai wrote: > > At Thu, 16 Oct 2014 08:10:52 -0600, > > Shuah Khan wrote: > >> > >> On 10/16/2014 08:01 AM, Takashi Iwai wrote: > >>> At Thu, 16

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Takashi Iwai
At Thu, 16 Oct 2014 08:10:52 -0600, Shuah Khan wrote: > > On 10/16/2014 08:01 AM, Takashi Iwai wrote: > > At Thu, 16 Oct 2014 07:10:37 -0600, > > Shuah Khan wrote: > >> > >> On 10/16/2014 06:00 AM, Lars-Peter Clausen wrote: > >&

  1   2   >