Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1470575 > unrolled thread

[PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In

Started byDanny Milosavljevic <dannym@scratchpost.org>
First post2016-08-26 09:30 +0200
Last post2016-08-30 04:00 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In Danny Milosavljevic <dannym@scratchpost.org> - 2016-08-26 09:30 +0200
    [PATCH v9 2/2] Add mixer controls: Line-In, FM-In, Mic 2, Capture Source, Differential Line-In. Danny Milosavljevic <dannym@scratchpost.org> - 2016-08-26 09:30 +0200
    Re: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture  Source, Differential Line-In Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-08-27 14:10 +0200
      Re: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture  Source, Differential Line-In Danny Milosavljevic <dannym@scratchpost.org> - 2016-08-29 20:10 +0200
        Re: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture  Source, Differential Line-In Chen-Yu Tsai <wens@csie.org> - 2016-08-30 04:00 +0200

#1470575 — [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In

FromDanny Milosavljevic <dannym@scratchpost.org>
Date2016-08-26 09:30 +0200
Subject[PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In
Message-ID<sajKp-3YE-11@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

This patch adds support for some mixer controls:
- Line-In
- FM-In
- Mic 2
- Capture Source
- Differential Line-In

v9 changes compared to v8 are:
 - added Line Differential Capture Switch.
 - split Capture Source into Left Capture Select, Right Capture Select.
 - added Line Capture Volume.
 - rename "sun4i_codec_widgets" to "sun4i_codec_controls" for 
   consistency with the struct field it's used in.
 - rename "Line-In" to "Line".
 - rename "Power Amplifier Playback Volume" to "Headphone Playback Volume".

v8 changes compared to v7 are:
 - fixed the routes for line and mic capturing.

v7 changes compared to v6 are:
 - preparation for different A20, A10 controls is now in an extra patch.
 - all register definitions are now at the top.
 - sun7i-specific things (A20-specific things) are now less grouped-together.
 - rename "Power Amplifier Volume" to "Power Amplifier Playback Volume".

v6 changes compared to v5 are:
 - Mic preamplifier special cases for A20 and A10 now are now not icky:
   There are two different _widget arrays and the probe() function now 
   selects the right one to pass to snd_soc_register_codec() unmodified.
 - sun7i-specific things (A20-specific things) are now grouped together.

v5 changes compared to v4 are:
 - Mic preamplifier controls have more common names now.
 - Mic preamplifier scale has a 0 dB entry as well now, as documented in the 
   A20 user manual.
 - Mic preamplifier has special cases for A20 and A10 now.
 - Gain controls have "Gain" in the name now.

v4 changes compared to v3 are:
 - names of the input are not uppercase anymore.
 - bit index constants are now named as in the A20 user manual v1.4.
 - added Mic1-In, Mac2-In.
 - added Mic1 and Mic2 Pre-Amplifiers.

v3 changes compared to v2 are:
 - added DAPM routes.

v2 changes compared to v1 are:
 - moved Line-In and FM-In playback switches to their respective 
   sun4i_codec_*_mixer_controls.

v1 changes:
 - added linein, fmin output volumes and switches.

Danny Milosavljevic (2):
  ASoC: sun4i-codec: Distinguish sun4i from sun7i
  Add mixer controls: Line-In, FM-In, Mic 2, Capture Source,
    Differential Line-In.

 sound/soc/sunxi/sun4i-codec.c | 278 +++++++++++++++++++++++++++++++++++++++---
 1 file changed, 259 insertions(+), 19 deletions(-)

[toc] | [next] | [standalone]


#1470576 — [PATCH v9 2/2] Add mixer controls: Line-In, FM-In, Mic 2, Capture Source, Differential Line-In.

FromDanny Milosavljevic <dannym@scratchpost.org>
Date2016-08-26 09:30 +0200
Subject[PATCH v9 2/2] Add mixer controls: Line-In, FM-In, Mic 2, Capture Source, Differential Line-In.
Message-ID<sajKq-3YE-15@gated-at.bofh.it>
In reply to#1470575

[Multipart message — attachments visible in raw view] — view raw

Note: Mic1 Capture Volume is in a different register on A20 than on A10.
Note: Mic2 Capture Volume is in a different register on A20 than on A10.
---
 sound/soc/sunxi/sun4i-codec.c | 256 ++++++++++++++++++++++++++++++++++++++----
 1 file changed, 236 insertions(+), 20 deletions(-)

[toc] | [prev] | [next] | [standalone]


#1471158 — Re: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In

FromMaxime Ripard <maxime.ripard@free-electrons.com>
Date2016-08-27 14:10 +0200
SubjectRe: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In
Message-ID<saKAW-4mF-15@gated-at.bofh.it>
In reply to#1470575

[Multipart message — attachments visible in raw view] — view raw

Hi,

On Fri, Aug 26, 2016 at 09:22:28AM +0200, Danny Milosavljevic wrote:
> This patch adds support for some mixer controls:
> - Line-In
> - FM-In
> - Mic 2
> - Capture Source
> - Differential Line-In

Please send your patches using send-email.

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [prev] | [next] | [standalone]


#1472024 — Re: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In

FromDanny Milosavljevic <dannym@scratchpost.org>
Date2016-08-29 20:10 +0200
SubjectRe: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In
Message-ID<sbzap-2aw-11@gated-at.bofh.it>
In reply to#1471158
Hi Maxime,

On Sat, 27 Aug 2016 14:02:51 +0200
Maxime Ripard <maxime.ripard@free-electrons.com> wrote:

> Please send your patches using send-email.

If you are referring to "git send-email", I did send the patch series using that (with almost the same setup I use for many other projects). It probably barfed because my description enumeration contained a dash like "- Line In". Sigh.

Anyway, I'll send the E-Mail again.

Thanks,
   Danny

[toc] | [prev] | [next] | [standalone]


#1472195 — Re: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In

FromChen-Yu Tsai <wens@csie.org>
Date2016-08-30 04:00 +0200
SubjectRe: [PATCH v9 0/2] sun4i-codec: Add Line-In, FM-In, Mic 2, Capture Source, Differential Line-In
Message-ID<sbGvf-6E9-5@gated-at.bofh.it>
In reply to#1472024
On Tue, Aug 30, 2016 at 2:03 AM, Danny Milosavljevic
<dannym@scratchpost.org> wrote:
> Hi Maxime,
>
> On Sat, 27 Aug 2016 14:02:51 +0200
> Maxime Ripard <maxime.ripard@free-electrons.com> wrote:
>
>> Please send your patches using send-email.
>
> If you are referring to "git send-email", I did send the patch series using that (with almost the same setup I use for many other projects). It probably barfed because my description enumeration contained a dash like "- Line In". Sigh.

The problem is your patches are being treated as attachments.
Try using '--no-attach' with git format-patch, regenerate the patches
and check that they are not mulit-part before sending them again.

ChenYu

>
> Anyway, I'll send the E-Mail again.
>
> Thanks,
>    Danny

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web