c are configured for I2S
* i don't see how existing off-tree sound cards with pcm512x codec could be
configured differently - it would not work
* tested explicitly, that there is no regression with Raspberry Pi +
sound card `sound/soc/bcm/hifiberry_dacplus.c`
Signed-off-by: Kirill Marinushkin
Set format from `set_fmt()` func instead of `hw_params()`, plus supportive
commits
Kirill Marinushkin (4):
ASoC: pcm512x: Fix not setting word length if DAIFMT_CBS_CFS
ASoC: pcm512x: Rearrange operations in `hw_params()`
ASoC: pcm512x: Move format check into `set_fmt()`
ASoC: pcm512x: Add
hecking
`SND_SOC_DAIFMT_MASTER_MASK`.
Tested with Raspberry Pi + sound card `hifiberry_dacplus` in CBS_CFS format
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Takashi Iwai
Cc: Liam Girdwood
Cc: Matthias Reichl
Cc: Kuninori Morimoto
Cc: Peter Ujfalusi
Cc: alsa-de...@alsa-project
vel/patch/
20201109212133.25869-1-kmarinush...@birdec.com/
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Takashi Iwai
Cc: Liam Girdwood
Cc: Matthias Reichl
Cc: Kuninori Morimoto
Cc: Peter Ujfalusi
Cc: alsa-de...@alsa-project.org
Cc: linux-kernel@vger.
This commit is a preparation for the next patch in the series.
It's goal is to make format check easy-to-move-out. Theoretically, more
butifications are possile in `hw_params()` func, but my intention in this
commit is to keep behaviour unchanged.
Signed-off-by: Kirill Marinushkin
Cc: Mark
Hello Peter,
On 11/12/2020 08:41 AM, Peter Ujfalusi wrote:
> Hi Kirill,
>
> On 11/11/2020 9.54, Kirill Marinushkin wrote:
>> Hello Peter,
>>
>> than you for your review!
>>
>>> The bus format and
>>>
>>>>switch
c are configured for I2S
* i don't see how existing off-tree sound cards with pcm512x codec could be
configured differently - it would not work
* tested explicitly, that there is no regression with Raspberry Pi +
sound card `sound/soc/bcm/hifiberry_dacplus.c`
Signed-off-by: Kirill Marinushkin
didn't know how to
use DSP_A and DSP_B. I will add them, and submit as patch v2
Best regards,
Kirill
On 11/10/2020 07:59 AM, Peter Ujfalusi wrote:
>
>
> On 09/11/2020 23.21, Kirill Marinushkin wrote:
>> Currently, pcm512x driver supports only I2S data format.
>> This
d for I2S
* i don't see how existing off-tree sound cards with pcm512x codec could be
configured differently - it would not work
* tested explicitly, that there is no regression with Raspberry Pi +
sound card `sound/soc/bcm/hifiberry_dacplus.c`
Signed-off-by: Kirill Marinushkin
Cc: Mark
Signed-off-by: Kirill Marinushkin
---
MAINTAINERS| 2 +-
sound/soc/codecs/pcm3060-i2c.c | 4 ++--
sound/soc/codecs/pcm3060-spi.c | 4 ++--
sound/soc/codecs/pcm3060.c | 4 ++--
sound/soc/codecs/pcm3060.h | 2 +-
5 files changed, 8 insertions(+), 8 deletions(-)
diff
On 06/11/19 19:49, Thomas Preston wrote:
> Add an I2C based codec driver for ST TDA7802 amplifier. By default, the
> amplifier supports 4 audio channels but can support up to 16 with
> multiple devices. Input is configurable for I2S or TDM.
>
> The unified device properties API is used to get boar
Softly reset registers values on module probe
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 8
1 file changed, 8 insertions(+)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index 6714aa8d9026..543cb86fd764 100644
--- a/sound/soc/codecs
for both ADC and DAC.
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 27 +++
sound/soc/codecs/pcm3060.h | 5 +
2 files changed, 32 insertions(+)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index 543cb86fd764..32b26f1c2282
>> First problem: I cannot compile the kernel with my version of compiler
>> Second problem: when I disable the feature - it cannot take effect,
>> because the parse-time error happens before `syncconfig`
On 12/08/18 13:42, Borislav Petkov wrote:
> Does this help?
>
> https://git.kernel.org/tip/
is not set
$ make
arch/x86/Makefile:224: *** You are building kernel with non-retpoline
compiler, please update your compiler.. Stop.
Fixes: 4cd24de3a098 ("x86/retpoline: Make CONFIG_RETPOLINE depend on compiler
support")
Signed-off-by: Kirill Marinushkin
Cc: Thomas Gleixner
Enable DAC/ADC only when playing/capturing
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 14 ++
sound/soc/codecs/pcm3060.h | 2 ++
2 files changed, 12 insertions(+), 4 deletions(-)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index
2018 08:12:53 +0100, Kirill Marinushkin wrote:
>> Output of pcm3060 codec may be configured as single-ended or differential
>>
>> Signed-off-by: Kirill Marinushkin
>> Cc: devicet...@vger.kernel.org
>> ---
>> Hello Mark,
>>
>> yesterday there was
Output of pcm3060 codec may be configured as single-ended or differential
Signed-off-by: Kirill Marinushkin
Cc: devicet...@vger.kernel.org
---
Hello Mark,
yesterday there was a misunderstanding: when I wrote you
> I think you forgot one patch in the series
you accidently applied the alre
maintainers to the CCs when replying
> to this mail.
>
> Thanks,
> Mark
>
> From b1cbde8fb287f4fd3493ca4167efee344c3e Mon Sep 17 00:00:00 2001
> From: Kirill Marinushkin
> Date: Mon, 12 Nov 2018 08:08:33 +0100
> Subject: [PATCH] ASoC: pcm3060: Add DT property for
nded
output")
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index 5f479aa61097..1dd851a7b43b 100644
--- a/sound/soc/codecs/pcm3060.c
++
Output of pcm3060 codec may be configured as single-ended or differential
Signed-off-by: Kirill Marinushkin
Cc: devicet...@vger.kernel.org
---
Documentation/devicetree/bindings/sound/pcm3060.txt | 6 ++
1 file changed, 6 insertions(+)
diff --git a/Documentation/devicetree/bindings/sound
DAC output may be differential (default) or single-ended.
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 14 ++
sound/soc/codecs/pcm3060.h | 1 +
2 files changed, 15 insertions(+)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index
Hello Mark,
On 11/05/18 12:20, Mark Brown wrote:
> On Mon, Oct 29, 2018 at 09:59:12AM +0100, Kirill Marinushkin wrote:
>> DAC may be switched between differential and single-ended output.
>
> Isn't this something that'd be better controlled by DT - it's usually
&
uot;)
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index f792ad637ae4..078a3b1fc064 100644
--- a/sound/soc/codecs/pcm3060.c
+++ b/sound/
DAC may be switched between differential and single-ended output.
Signed-off-by: Kirill Marinushkin
---
sound/soc/codecs/pcm3060.c | 9 +
sound/soc/codecs/pcm3060.h | 1 +
2 files changed, 10 insertions(+)
diff --git a/sound/soc/codecs/pcm3060.c b/sound/soc/codecs/pcm3060.c
index
Hello Takashi, Mike,
@Takashi
On 10/25/18 09:37, Takashi Iwai wrote:
> Well, in the API POV, it's nothing wrong to keep hwptr sticking while
> updating only delay value. It implies that the hardware chip doesn't
> provide the hwptr update.
Thank you for the clarification. Modifying `runtime->de
Hello Mike,
We are not on the same page. What you hear is not what I tell you.
Either you don't understand what happens in your commit, or I don't understand
what happens in the driver.
Hopefully somebody in the community can comment here.
On 10/24/18 21:54, Mike Brady wrote:
You modify the
Hello Mike,
On 10/24/18 10:20, Mike Brady wrote:
> Hi Kirill. Thanks for your comments.
>
>> On 22 Oct 2018, at 23:25, Kirill Marinushkin wrote:
>>
>> AFAIU, this patch is wrong. Please correct me, maybe I misunderstand
>> something.
>>
>>> The pro
Hello Mike,
AFAIU, this patch is wrong. Please correct me, maybe I misunderstand something.
> The problem that this patch seeks to resolve is that when userland asks for
> the delay
The userspace asks not for delay, but for the pointer.
You modify the function, which is called `snd_bcm2835_pcm_p
Hello Mike,
On 10/18/18 12:57, Mike Brady wrote:
> + ktime_t now = ktime_get();
> +
> + /* Give userspace better delay reporting by interpolating between GPU
> + * notifications, assuming audio speed is close enough to the clock
> + * used for ktime
> + */
> +
> + if ((k
Modified some if-statements to make them more clear
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: alsa-de...@alsa-project.org
Cc: linux-kernel@vger.kernel.org
---
sound/soc/codecs/pcm3060.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/sound/soc/codecs
Modified the complete file comments in C++ style, to make them look more
intentional
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: alsa-de...@alsa-project.org
Cc: linux-kernel@vger.kernel.org
---
sound/soc/codecs/pcm3060-i2c.c | 9 -
sound/soc/codecs/pcm3060-spi.c | 9
This commit adds support for TI PCM3060 CODEC.
The technical documentation is available at [1].
[1] http://ti.com/product/pcm3060
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Liam Girdwood
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: M R Swami Reddy
Cc: Vishwas A Deshpande
Cc: Kevin
Hello Mark,
I am developing sound support for a Linux-based device, with playback and
capture through the TI PCM3060 codec.
With the following patch, I propose to add the PCM3060 codec driver into
the kernel.
Best Regards,
Kirill
Kirill Marinushkin (1):
ASoC: pcm3060: Add codec driver
0x4c/0x6c [snd_pcm]
[<7f0e250c>] snd_open+0xa8/0x14c [snd]
[<802ce590>] chrdev_open+0xac/0x188
[<802c57b4>] do_dentry_open+0x10c/0x314
[<802c6ba8>] vfs_open+0x5c/0x88
[<802d9a68>] path_openat+0x368/0x944
[<802dacd4>] do_filp_open+0x70/0xc4
On 04/25/18 08:16, Greg Kroah-Hartman wrote:
> On Tue, Apr 24, 2018 at 09:57:29PM +0200, Kirill Marinushkin wrote:
>> In the current implementation, vchi_instance is inited during the first
>> call of bcm2835_audio_open_connection(), and is never freed. It causes a
>> memory
0x4c/0x6c [snd_pcm]
[<7f0e250c>] snd_open+0xa8/0x14c [snd]
[<802ce590>] chrdev_open+0xac/0x188
[<802c57b4>] do_dentry_open+0x10c/0x314
[<802c6ba8>] vfs_open+0x5c/0x88
[<802d9a68>] path_openat+0x368/0x944
[<802dacd4>] do_filp_open+0x70/0xc4
On 04/24/18 20:35, Andy Shevchenko wrote:
> On Tue, Apr 24, 2018 at 9:27 PM, Kirill Marinushkin
> wrote:
>
>> @Andy
>>
>>> AFAIR I gave you a tag and you again missed it.
>>> Before sending anything just check twice if all prerequisites are fulfilled.
>&g
ag and you again missed it.
> Before sending anything just check twice if all prerequisites are fulfilled.
I think you mix it up. This is a new patch, you didn't review it before.
> And yes, kbuild bot is right. You need to return known value.
Yes, that's right. I will fix it.
0x4c/0x6c [snd_pcm]
[<7f0e250c>] snd_open+0xa8/0x14c [snd]
[<802ce590>] chrdev_open+0xac/0x188
[<802c57b4>] do_dentry_open+0x10c/0x314
[<802c6ba8>] vfs_open+0x5c/0x88
[<802d9a68>] path_openat+0x368/0x944
[<802dacd4>] do_filp_open+0x70/0xc4
On 04/23/18 15:50, Greg Kroah-Hartman wrote:
> On Tue, Apr 17, 2018 at 07:00:28AM +0200, Kirill Marinushkin wrote:
>> In the current implementation, vchi_instance is inited during the first
>> call of bcm2835_audio_open_connection(), and is never freed. It causes a
>> memory
0x4c/0x6c [snd_pcm]
[<7f0e250c>] snd_open+0xa8/0x14c [snd]
[<802ce590>] chrdev_open+0xac/0x188
[<802c57b4>] do_dentry_open+0x10c/0x314
[<802c6ba8>] vfs_open+0x5c/0x88
[<802d9a68>] path_openat+0x368/0x944
[<802dacd4>] do_filp_open+0x70/0xc4
the corresponding patch for linux.
Signed-off-by: Kirill Marinushkin
Reviewed-by: Pierre-Louis Bossart
Tested-by: Pan Xiuli
Tested-by: Pierre-Louis Bossart
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: Mark Brown
Cc: Liam Girdwood
Cc: linux-kernel@vger.kernel.org
Cc: alsa-de...@alsa-project.org
ated
}
SectionLink."Codec" {
# used for binding to the physical link
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Reviewed-by: Pierre-Louis Bossart
This improves the coding style of this piece of code.
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Pierre-Louis Bossart
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: Pan Xiuli
Cc: Liam Girdwood
Cc: linux-kernel@vger.kernel.org
Cc: alsa-de...@alsa-project.org
---
sound/soc/soc
patch series,
please apply it to Linux.
Best Regards,
Kirill
Kirill Marinushkin (3):
ASoC: topology: Fix bclk and fsync inversion in set_link_hw_format()
ASoC: topology: Add missing clock gating parameter when parsing
hw_configs
ASoC: topology: Add definitions for mclk_dire
ated
}
SectionLink."Codec" {
# used for binding to the physical link
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Reviewed-by: Pierre-Louis Bossart
The values of bclk and fsync are inverted WRT the codec. But the existing
solution already works for Broadwell, see the alsa-lib config:
`alsa-lib/src/conf/topology/broadwell/broadwell.conf`
This commit provides the backwards-compatible solution to fix this misuse.
Signed-off-by: Kirill
Current comment makes not clear the direction of mclk. Previously, similar
description caused a misunderstanding for bclk_master and fsync_master.
This commit solves the potential confusion the same way it is solved for
bclk_master and fsync_master.
Signed-off-by: Kirill Marinushkin
Acked-by
On 04/03/18 19:21, Pierre-Louis Bossart wrote:
>
>
> On 04/03/2018 12:15 AM, Kirill Marinushkin wrote:
>> On 04/03/18 02:57, Pierre-Louis Bossart wrote:
>>>
>>> On 04/02/2018 04:17 PM, Kirill Marinushkin wrote:
>>>> Hello Pierre-Louis,
>>>&g
On 04/03/18 02:57, Pierre-Louis Bossart wrote:
>
>
> On 04/02/2018 04:17 PM, Kirill Marinushkin wrote:
>> Hello Pierre-Louis,
>>
>> I explicitly clarified with Takashi: to have this patch series merged, we
>> need a
>> tag "Reviewed-by" from you.
sa-lib, v3 2/2] ASoC: topology: Add missing clock gating
parameter when parsing hw_configs
On 03/27/18 22:56, Kirill Marinushkin wrote:
> Hello Jaroslav, Takashi, Mark,
>
> This patch series is a resend of [1] and [2], rebased on top of the latest
> head. It was logical to resend them togethe
the corresponding patch for linux.
Signed-off-by: Kirill Marinushkin
Tested-by: Pan Xiuli
Tested-by: Pierre-Louis Bossart
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: Mark Brown
Cc: Liam Girdwood
Cc: linux-kernel@vger.kernel.org
Cc: alsa-de...@alsa-project.org
---
include/sound/asoc.h
ated
}
SectionLink."Codec" {
# used for binding to the physical link
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Cc: Jaroslav Kysela
Cc: Tak
Below is the alsa-lib part of the patch-series
"ASoC: topology: Improve parsing hw_configs"
Kirill Marinushkin (2):
ASoC: topology: Fix bclk and fsync inversion in set_link_hw_format()
ASoC: topology: Add missing clock gating parameter when parsing
hw_configs
include/so
ated
}
SectionLink."Codec" {
# used for binding to the physical link
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Cc: Jaroslav Kysela
Cc: Tak
/10250485/
[2] https://patchwork.kernel.org/patch/10230611/
Kirill Marinushkin (2):
ASoC: topology: Fix bclk and fsync inversion in set_link_hw_format()
ASoC: topology: Add missing clock gating parameter when parsing
hw_configs
include/uapi/sound/asoc.h | 23 ---
sound/soc
The values of bclk and fsync are inverted WRT the codec. But the existing
solution already works for Broadwell, see the alsa-lib config:
`alsa-lib/src/conf/topology/broadwell/broadwell.conf`
This commit provides the backwards-compatible solution to fix this misuse.
Signed-off-by: Kirill
On 03/25/18 15:03, Andy Shevchenko wrote:
> On Sun, Mar 25, 2018 at 1:44 PM, Kirill Marinushkin
> wrote:
>> On 03/25/18 12:33, Andy Shevchenko wrote:
>>> On Fri, Mar 23, 2018 at 9:22 PM, Kirill Marinushkin
>>> wrote:
>>>> On 03/23/18 17:23, Andy Shev
On 03/25/18 12:33, Andy Shevchenko wrote:
> On Fri, Mar 23, 2018 at 9:22 PM, Kirill Marinushkin
> wrote:
>> On 03/23/18 17:23, Andy Shevchenko wrote:
>>> After addressing above, FWIW,
>>>
>>> Reviewed-by: Andy Shevchenko
> Seems you missed my tag in n
] [<802c7060>] (SyS_open) from [<801082c0>]
(ret_fast_syscall+0x0/0x28)
[ 139.306021] Code: e3c3303f e5932004 e2822001 e5832004 (e5943000)
[ 139.316265] ---[ end trace 7f3f7f6193b663ed ]---
[ 139.324956] note: aplay[463] exited with preempt_count 1
Signed-off-by: Kirill Marinushk
On 03/23/18 17:23, Andy Shevchenko wrote:
> On Thu, Mar 22, 2018 at 11:37 PM, Kirill Marinushkin
> wrote:
>
>
>> +static void snd_devm_release(struct device *dev)
>> +{
>> + struct bcm2835_chip *chip = dev_get_drvdata(dev);
>> +
>> + kfr
] [<802c7060>] (SyS_open) from [<801082c0>]
(ret_fast_syscall+0x0/0x28)
[ 139.306021] Code: e3c3303f e5932004 e2822001 e5832004 (e5943000)
[ 139.316265] ---[ end trace 7f3f7f6193b663ed ]---
[ 139.324956] note: aplay[463] exited with preempt_count 1
Signed-off-by: Kirill Marinushk
On 03/21/18 20:24, Andy Shevchenko wrote:
> On Wed, Mar 21, 2018 at 8:48 PM, Kirill Marinushkin
> wrote:
>> In the current implementation, `rmmod snd_bcm2835` does not release
>> resources properly. It causes an oops when trying to list sound devices.
>>
>> This comm
On 03/21/18 20:24, Andy Shevchenko wrote:
> On Wed, Mar 21, 2018 at 8:48 PM, Kirill Marinushkin
> wrote:
>> In the current implementation, `rmmod snd_bcm2835` does not release
>> resources properly. It causes an oops when trying to list sound devices.
>>
>> This comm
] [<802c7060>] (SyS_open) from [<801082c0>]
(ret_fast_syscall+0x0/0x28)
[ 139.306021] Code: e3c3303f e5932004 e2822001 e5832004 (e5943000)
[ 139.316265] ---[ end trace 7f3f7f6193b663ed ]---
[ 139.324956] note: aplay[463] exited with preempt_count 1
Signed-off-by: Kirill Marinushk
On 03/21/18 16:03, Andy Shevchenko wrote:
> On Thu, Mar 15, 2018 at 7:57 AM, Kirill Marinushkin
> wrote:
>> On 03/13/18 22:23, Andy Shevchenko wrote:
>>> On Tue, Mar 13, 2018 at 9:34 PM, Kirill Marinushkin
>>> wrote:
>>>> In the current implementa
unit inits w/o this error.
Fixes: 5e1ddb481776 ("UAPI: (Scripted) Disintegrate include/linux/usb")
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: Thomas Gleixner
Cc: Felipe Balbi
Cc: Philippe Ombredanne
Cc: Greg Kroah-Hartman
Cc: Rusl
On 03/13/18 22:23, Andy Shevchenko wrote:
> On Tue, Mar 13, 2018 at 9:34 PM, Kirill Marinushkin
> wrote:
>> In the current implementation, `rmmod snd_bcm2835` does not release
>> resources properly. It causes an oops when trying to list sound devices.
>>
>> This
] [<802c7060>] (SyS_open) from [<801082c0>]
(ret_fast_syscall+0x0/0x28)
[ 139.306021] Code: e3c3303f e5932004 e2822001 e5832004 (e5943000)
[ 139.316265] ---[ end trace 7f3f7f6193b663ed ]---
[ 139.324956] note: aplay[463] exited with preempt_count 1
Signed-off-by: Kirill Marinushk
On 03/06/18 13:14, Greg Kroah-Hartman wrote:
> On Mon, Mar 05, 2018 at 06:52:54AM +0100, Kirill Marinushkin wrote:
>> In the current implementation, `rmmod snd_bcm2835` does not release
>> resources properly. It causes an oops when trying to list sound devices.
>>
&g
139.306021] Code: e3c3303f e5932004 e2822001 e5832004 (e5943000)
[ 139.316265] ---[ end trace 7f3f7f6193b663ed ]---
[ 139.324956] note: aplay[463] exited with preempt_count 1
Signed-off-by: Kirill Marinushkin
Cc: Eric Anholt
Cc: Stefan Wahren
Cc: Greg Kroah-Hartman
the corresponding patch for alsa-lib.
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Takashi Iwai
Cc: Jaroslav Kysela
Cc: Pierre-Louis Bossart
Cc: Pan Xiuli
Cc: Liam Girdwood
Cc: linux-kernel@vger.kernel.org
Cc: alsa-de...@alsa-project.org
---
include/uapi/sound/asoc.h | 16
Firmware, etc.)
The solution includes 1 patch for linux + 1 patch for alsa-lib.
I will send them to this mailing thread.
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Pierre-Louis Bossart
Cc: Pan Xiuli
Cc: Liam Girdwood
Cc: linux-kernel@vger.kernel.org
Cc: alsa-de...@alsa-project.org
the corresponding patch for alsa-lib.
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Pierre-Louis Bossart
Cc: Pan Xiuli
Cc: Liam Girdwood
Cc: linux-kernel@vger.kernel.org
Cc: alsa-de...@alsa-project.org
---
include/uapi/sound/asoc.h | 16 ++--
sound/soc/soc-topology.c | 12
On 02/21/18 15:50, Andy Shevchenko wrote:
> On Tue, Feb 20, 2018 at 7:44 PM, Kirill Marinushkin
> wrote:
>> Clock gating parameter is a part of `dai_fmt`. It is supported by
>> `alsa-lib` when creating a topology binary file, but ignored by kernel
>> when lo
].
[1]. [alsa-lib,v2,1/2] topology: Fix parsing config with multiple
hw_configs
[2]. [alsa-lib,v2,2/2] ASoC: topology: Add defines for
snd_soc_tplg_hw_config.clock_gated
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Takashi Sakamoto
Cc: Takashi Iwai
Cc: Jaroslav Kysela
Cc: Liam
nk
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Cc: Mark Brown
Cc: Takashi Sakamoto
Cc: Takashi Iwai
Cc: Jaroslav Kysela
Cc: Liam Girdwood
Cc: alsa-de...@alsa-project.org
Cc:
used as
bool values due to the inverted logic. Therefore this commit adds the
defines for this field.
snd_soc_tplg_hw_config.clock_gated = 0 => no effect
snd_soc_tplg_hw_config.clock_gated = 1 => SND_SOC_DAIFMT_GATED
snd_soc_tplg_hw_config.clock_gated = 2 => SND_SOC_DAIFMT_CONT
On 02/19/18 07:05, Kirill Marinushkin wrote:
> Clock gating parameter is a part of `dai_fmt`. It is supported by
> `alsa-lib` when creating a topology binary file, but ignored by kernel
> when loading this topology file.
>
> After applying this commit, the clock gating parameter is
On 02/19/18 21:36, Kirill Marinushkin wrote:
> In kernel `soc-dai.h`, DAI clock gating is defined as following:
>
>
> \#define SND_SOC_DAIFMT_CONT (1 << 4) /* continuous clock */
> \#define SND_SOC_DAIFMT_GATED (0 << 4) /* clock is gated */
> ~~
On 02/20/18 14:45, Mark Brown wrote:
> On Tue, Feb 20, 2018 at 12:09:18PM +, Mark Brown wrote:
>> The patch
>>
>>ASoC: topology: Add missing clock gating parameter when parsing hw_configs
>>
>> has been applied to the asoc tree at
>>
>>https://git.kernel.org/pub/scm/linux/kernel/git/bro
nk
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Cc: Takashi Sakamoto
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Cc:
used as
bool values due to the inverted logic. Therefore this commit adds the
defines for this field.
snd_soc_tplg_hw_config.clock_gated = 0 => no effect
snd_soc_tplg_hw_config.clock_gated = 1 => SND_SOC_DAIFMT_GATED
snd_soc_tplg_hw_config.clock_gated = 2 => SND_SOC_DAIFMT_CONT
On 02/19/18 07:47, Takashi Sakamoto wrote:
> Hi,
>
> On Feb 19 2018 15:05, Kirill Marinushkin wrote:
>> In kernel `soc-dai.h`, DAI clock gating is defined as following:
>>
>>
>> \#define SND_SOC_DAIFMT_CONT (1 << 4) /* continuous clock */
>>
nk
id "0"
hw_configs [
"CodecHWConfig"
]
default_hw_conf_id "1"
}
Signed-off-by: Kirill Marinushkin
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Cc:
ig should
be inverted compared to the current logic:
clock_count = 1 => SND_SOC_DAIFMT_CONT
clock_count = 0 => SND_SOC_DAIFMT_GATED
Signed-off-by: Kirill Marinushkin
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: alsa-de...@alsa-project.org
Cc: linux-kernel@vger.kernel.org
---
include/uap
4557c305d4fc ("ASoC: Intel: Skylake: Add support for active suspend")
Signed-off-by: Kirill Marinushkin
Cc: Liam Girdwood
Cc: Mark Brown
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Cc: Vinod Koul
Cc: Guneshwor Singh
Cc: Naveen Manohar
Cc: Harsha Priya N
Cc: Pierre-Louis Bossart
Cc: alsa-de
On 02/13/18 06:04, Takashi Iwai wrote:
> On Mon, 12 Feb 2018 20:32:53 +0100,
> Kirill Marinushkin wrote:
>> Hello maintainers of ,
>>
>>
>> As discussed earlier in this thread, I propose this patch to stable.
>>
>> It fixes the issue, which exists s
Hello maintainers of ,
As discussed earlier in this thread, I propose this patch to stable.
It fixes the issue, which exists since v2.6.34.
Best Regards,
Kirill
On 02/12/18 09:08, Takashi Iwai wrote:
> On Thu, 08 Feb 2018 07:21:36 +0100,
> Kirill Marinushkin wrote:
>> On 02
On 02/07/18 06:45, Takashi Iwai wrote:
> On Mon, 29 Jan 2018 06:37:55 +0100,
> Kirill Marinushkin wrote:
>> The layout of the UAC2 Control request and response varies depending on
>> the request type. With the current implementation, only the Layout 2
>> Parameter Bloc
, Mid Control, Tremble Control), the response is parsed
incorrectly.
This commit:
* fixes the wLength field value in the request
* fixes parsing the range values from the response
Fixes: 23caaf19b11e ("ALSA: usb-mixer: Add support for Audio Class v2.0")
Signed-off-by: Kirill Marinushkin
Cc
u plan to continue using both ANSI X9.31 DRNG and DRBG - I agree with the
patch suggested by Artem Savkov.
If you plan to reduce using ANSI X9.31 DRNG and use DRBG more widely - I
suggest my patch.
Best Regards,
Kirill
---
FromKirill Marinushkin
Subject [PATCH v2] KEYS: fix big_key dependenc
> David, here's a bug report with reproducer that was sent to the
> oss-security mailing list for some unknown reason earlier today.
>
> Any ideas?
The problem is in the wrong RNG dependency.
I submitted a patch to fix the behaviour, you can find it by subject:
[PATCH] KEYS: fix big_key dependenc
> I can't say for certain that this is that actual cause as I haven't managed to
> reproduce it
Previously attached the configuration to reproduce the
issue, but it did not get into this thread. I reattach it here below.
---
#
# Automatically generated file; DO NOT EDIT.
# Linux/x86 4.7.0 Kerne
> May I suggest CRYPTO_DRBG?
Hello Ciao,
Thanks for your proposal. I submitted the PATCH v2 where I use
CRYPTO_RNG_DEFAULT the same way it is used in crypto/Kconfig.
It solves the issue, too. In addition, it selects CRYPTO_DRBG_MENU,
which is a more universal use-case, as I understood from your c
This patch fixes the following bug:
[oss-security] - panic at big_key_preparse #4.7-r6/rc7 & master
Signed-off-by: Kirill Marinushkin
---
security/keys/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/keys/Kconfig b/security/keys/Kconfig
index f82
Signed-off-by: Kirill Marinushkin
---
security/keys/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/keys/Kconfig b/security/keys/Kconfig
index f826e87..8213221 100644
--- a/security/keys/Kconfig
+++ b/security/keys/Kconfig
@@ -44,7 +44,7 @@ config BIG_KEYS
ead successfully.
---
commit 69ed34b303f87a1a53470dd37149ac1573d79da2
Author: Kirill Marinushkin
Date: Mon, 8 Aug 2016 23:19:32 +0200
KEYS: fix big_key dependency
Signed-off-by: Kirill Marinushkin
cc: David Howells
cc: Peter Hlavaty
cc: Greg KH
cc: sta...@vger.kernel.org
---
security/keys/Kconfi
> It's much more common to use
>
> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
This happened to be a wrong email thread and a wrong kernel tag;
I resent a proper patch in-reply-to a proper thread.
1 - 100 of 107 matches
Mail list logo
