All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean-Francois Moine <moinejf@free.fr>
To: Jyri Sarha <jsarha@ti.com>
Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
	dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter
Date: Mon, 27 Apr 2015 20:25:02 +0200	[thread overview]
Message-ID: <20150427202502.2573af29@armhf> (raw)
In-Reply-To: <553E1E99.9020101@ti.com>

On Mon, 27 Apr 2015 14:33:45 +0300
Jyri Sarha <jsarha@ti.com> wrote:

> Have you done anything about the tda998x audio support lately?
> 
> I was thinking of taking a shot at this now that I finally seem to have 
> some time for it. However, if you are just about to send another series 
> I'll wait for that first and see what makes the most sense after that.
> 
> My plan is to do something really simple first. No graph bindings or 
> anything fancy like that. Just a minimal dt-binding to for audio pin 
> configuration and use simple-card for the rest. I still try to make it 
> possible to support spdif and i2s at the same time, but I can not test 
> it as I do not have such HW.
> 
> I also try to make the ASoC part as generic as possible, so that it 
> could be reused by other HDMI encoders with spdif or i2s interface.

Hi Jyri,

I was busy on an other work, so, I did not advance on the tda998x.

If you look at the patch series, it contains 3 independant parts:
- dynamic building of the DAIs in the kirkwood audio controller,
- audio CODEC of the tda998x
- DT card based on audio graph

It seems that only the last part raised a problem.
Otherwise, the simple card works fine with the 2 first parts.

I am merging my patches in 4.1-rc1, and, as soon as I get an image on
my screen :), I will resubmit a patch series but about the tda998x
codec only.

Using i2s and s/pdif at the same time with the simple card asks for a
patch as the one I submitted in february 2014 (ASoC: simple-card: DT
fix and multi DAI links extension). As I am using my DT card, I will
not resubmit it. Eventually, the kirkwood patches are not critical.

-- 
Ken ar c'hentañ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

WARNING: multiple messages have this Message-ID (diff)
From: Jean-Francois Moine <moinejf@free.fr>
To: Jyri Sarha <jsarha@ti.com>
Cc: <alsa-devel@alsa-project.org>, <dri-devel@lists.freedesktop.org>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter
Date: Mon, 27 Apr 2015 20:25:02 +0200	[thread overview]
Message-ID: <20150427202502.2573af29@armhf> (raw)
In-Reply-To: <553E1E99.9020101@ti.com>

On Mon, 27 Apr 2015 14:33:45 +0300
Jyri Sarha <jsarha@ti.com> wrote:

> Have you done anything about the tda998x audio support lately?
> 
> I was thinking of taking a shot at this now that I finally seem to have 
> some time for it. However, if you are just about to send another series 
> I'll wait for that first and see what makes the most sense after that.
> 
> My plan is to do something really simple first. No graph bindings or 
> anything fancy like that. Just a minimal dt-binding to for audio pin 
> configuration and use simple-card for the rest. I still try to make it 
> possible to support spdif and i2s at the same time, but I can not test 
> it as I do not have such HW.
> 
> I also try to make the ASoC part as generic as possible, so that it 
> could be reused by other HDMI encoders with spdif or i2s interface.

Hi Jyri,

I was busy on an other work, so, I did not advance on the tda998x.

If you look at the patch series, it contains 3 independant parts:
- dynamic building of the DAIs in the kirkwood audio controller,
- audio CODEC of the tda998x
- DT card based on audio graph

It seems that only the last part raised a problem.
Otherwise, the simple card works fine with the 2 first parts.

I am merging my patches in 4.1-rc1, and, as soon as I get an image on
my screen :), I will resubmit a patch series but about the tda998x
codec only.

Using i2s and s/pdif at the same time with the simple card asks for a
patch as the one I submitted in february 2014 (ASoC: simple-card: DT
fix and multi DAI links extension). As I am using my DT card, I will
not resubmit it. Eventually, the kirkwood patches are not critical.

-- 
Ken ar c'hentañ	|	      ** Breizh ha Linux atav! **
Jef		|		http://moinejf.free.fr/

  reply	other threads:[~2015-04-27 18:25 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-20 19:35 [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter Jean-Francois Moine
2015-01-20 19:35 ` Jean-Francois Moine
2015-01-19 17:15 ` [PATCH v10 1/9] ASoC: kirkwood: dynamically build the DAI array Jean-Francois Moine
2015-01-19 17:15   ` Jean-Francois Moine
2015-01-19 17:38 ` [PATCH v10 2/9] ASoC: kirkwood: check the DAI type from the DAI name Jean-Francois Moine
2015-01-19 17:38   ` Jean-Francois Moine
2015-01-19 18:50 ` [PATCH v10 3/9] ASoC: kirkwood: accept the DAI definitions from a graph of ports Jean-Francois Moine
2015-01-19 18:50   ` Jean-Francois Moine
2015-01-20  9:27 ` [PATCH v10 4/9] drm/i2c: tda998x: Add support of a DT " Jean-Francois Moine
2015-01-20  9:27   ` Jean-Francois Moine
2015-01-20 11:16 ` [PATCH v10 5/9] drm/i2c: tda998x: Change drvdata for audio extension Jean-Francois Moine
2015-01-20 11:16   ` Jean-Francois Moine
2015-01-20 15:38 ` [PATCH v10 6/9] ASoC: tda998x: add a codec to the HDMI transmitter Jean-Francois Moine
2015-01-20 15:38   ` Jean-Francois Moine
2015-01-20 16:50 ` [PATCH v10 7/9] drm/i2c: tda998x: set cts_n according to the sample width Jean-Francois Moine
2015-01-20 16:50   ` Jean-Francois Moine
2015-01-20 18:47 ` [PATCH v10 8/9] ASoC: core: export snd_soc_get_dai_name Jean-Francois Moine
2015-01-20 18:47   ` Jean-Francois Moine
2015-01-20 19:16 ` [PATCH v10 9/9] ASoC: add generic dt-card support Jean-Francois Moine
2015-01-20 19:16   ` Jean-Francois Moine
     [not found]   ` <ddd1b299eca187ab2fa9b119da2ec8297ead1a66.1421782533.git.moinejf-GANU6spQydw@public.gmane.org>
2015-01-21 13:50     ` Mark Brown
2015-01-21 13:50       ` Mark Brown
2015-04-27 11:33 ` [PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter Jyri Sarha
2015-04-27 11:33   ` Jyri Sarha
2015-04-27 18:25   ` Jean-Francois Moine [this message]
2015-04-27 18:25     ` Jean-Francois Moine
2015-04-28  7:10     ` Jean-Francois Moine
2015-04-28  7:10       ` Jean-Francois Moine

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=20150427202502.2573af29@armhf \
    --to=moinejf@free.fr \
    --cc=alsa-devel@alsa-project.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jsarha@ti.com \
    --cc=linux-kernel@vger.kernel.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 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.