Re: [PATCH v2 0/2] Improve drm_of_component_probe() and move rockchip to use it

2015-12-25 Thread Jean-Francois Moine
On Thu, 24 Dec 2015 12:36:10 + Russell King - ARM Linux wrote: > It seems that you're trying to work around a limitation in Linux by > modifying the hardware representation... Sorry to come back to this topic, but I think you are wrong. Looking at the imx6 DTs, the problem comes from the di

Re: [PATCH v5 3/4] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-12-08 Thread Jean-Francois Moine
On Tue, 8 Dec 2015 09:32:24 +0100 Maxime Ripard wrote: > If the H3 display block is done the same way than the A10 (and later) > one on this aspect, then the TCON has two channels with two different > streaming (or functional, you pick the name) clocks. The channel 0 is > usually used for RGB, th

Re: [PATCH] clk: sunxi: Extend the simple gates and handle the Allwinner H3

2015-12-08 Thread Jean-Francois Moine
On Tue, 8 Dec 2015 08:53:54 +0100 Maxime Ripard wrote: > Look, we all agreed on a solution that raised all objections, but > yours. > > I'm going to take Jens patch. OK. Good luck for the next SoCs! -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | h

Re: [PATCH v5 3/4] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-12-08 Thread Jean-Francois Moine
On Mon, 7 Dec 2015 19:44:30 +0100 Jens Kuske wrote: > >> + "bus_lcd0", "bus_lcd1", > >> "bus_deint", > > > "bus_tcon0", "bus_tcon1", "bus_deint", > > > > (the tcon1 clock is used by both lcd0 and lcd1, while > > the tcon0 clock is used for TV output f

Re: [PATCH] clk: sunxi: Extend the simple gates and handle the Allwinner H3

2015-12-07 Thread Jean-Francois Moine
On Mon, 7 Dec 2015 08:31:02 -0600 Rob Herring wrote: > On Sun, Dec 06, 2015 at 10:04:12AM +0100, Jean-Francois Moine wrote: > > The H3 has a clock gate definition similar to the other Allwinner SoCs, > > but with a different parent clock for each single gate. > > > &

Re: [PATCH v5 3/4] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-12-07 Thread Jean-Francois Moine
On Fri, 4 Dec 2015 22:24:42 +0100 Jens Kuske wrote: > The Allwinner H3 is a home entertainment system oriented SoC with > four Cortex-A7 cores and a Mali-400MP2 GPU. > > Signed-off-by: Jens Kuske > --- > arch/arm/boot/dts/sun8i-h3.dtsi | 497 > > 1 fi

[PATCH] clk: sunxi: Extend the simple gates and handle the Allwinner H3

2015-12-06 Thread Jean-Francois Moine
. Signed-off-by: Jean-Francois Moine --- This patch replaces a part of Jens Kuske's patch [PATCH v5 1/4] clk: sunxi: Add H3 clocks support --- Documentation/devicetree/bindings/clock/sunxi.txt | 25 +++ drivers/clk/sunxi/clk-simple-gates.c

Re: [PATCH v4 4/6] reset: sunxi: Add Allwinner H3 bus resets

2015-11-04 Thread Jean-Francois Moine
On Wed, 4 Nov 2015 08:30:14 -0800 Maxime Ripard wrote: > Hi Arnd, > > On Fri, Oct 30, 2015 at 09:27:03AM +0100, Arnd Bergmann wrote: > > On Tuesday 27 October 2015 17:50:24 Jens Kuske wrote: > > > > > > +static int sun8i_h3_bus_reset_xlate(struct reset_controller_dev *rcdev, > > > +

Re: [PATCH v4 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-30 Thread Jean-Francois Moine
On Tue, 27 Oct 2015 17:50:25 +0100 Jens Kuske wrote: > The Allwinner H3 is a home entertainment system oriented SoC with > four Cortex-A7 cores and a Mali-400MP2 GPU. > > Signed-off-by: Jens Kuske > --- > arch/arm/boot/dts/sun8i-h3.dtsi | 482 > > 1 fi

Re: [PATCH v4 0/6] ARM: sunxi: Introduce Allwinner H3 support

2015-10-27 Thread Jean-Francois Moine
On Tue, 27 Oct 2015 17:50:20 +0100 Jens Kuske wrote: > Changes since v3: > - add a clock driver specific to the bus gates instead of listing parents in > the DTSI > - skip the holes in the reset controller with of_xlate() Hi, Maxime, there we are :(. The H3 description is hardcoded and the nex

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-24 Thread Jean-Francois Moine
On Sat, 24 Oct 2015 09:13:28 +0200 Maxime Ripard wrote: > Or simply > > bus_gates { > clocks = <&ahb1>, <&ahb2>; > clock-indices = <5>, <6>, <8>, ... > clock-output-names = "bus_ce", "bus_dma", "bus_mmc0" > }; I don't understand: the apb1, apb2, ahb1 and ahb2 clocks may be pro

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-23 Thread Jean-Francois Moine
On Fri, 23 Oct 2015 20:14:06 +0200 Maxime Ripard wrote: > On Wed, Oct 21, 2015 at 06:20:27PM +0200, Jens Kuske wrote: > > + bus_gates: clk@01c20060 { > > + #clock-cells = <1>; > > + compatible = "allwinner,sun8i-h3-bus-gates-clk"; > > +

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Jean-Francois Moine
On Wed, 21 Oct 2015 18:20:27 +0200 Jens Kuske wrote: > The Allwinner H3 is a home entertainment system oriented SoC with > four Cortex-A7 cores and a Mali-400MP2 GPU. > > Signed-off-by: Jens Kuske > --- > arch/arm/boot/dts/sun8i-h3.dtsi | 499 > > 1 fi

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Jean-Francois Moine
On Thu, 22 Oct 2015 10:47:35 +0200 Maxime Ripard wrote: > Not really. The uart0 reset is the bit 16, in the reset register 4. > > 4 * 32 + 16 = 44. > > Not 112, but still not 208 either. The registers are numbered 1..5, then (4 - 1) * 32 + 16 = 112 -- Ken ar c'hentañ | ** Breizh

Re: [PATCH 5/6] ARM: dts: sunxi: Add Allwinner H3 DTSI

2015-10-22 Thread Jean-Francois Moine
On Thu, 22 Oct 2015 10:05:08 +0200 Maxime Ripard wrote: > > + uart0: serial@01c28000 { > > + compatible = "snps,dw-apb-uart"; > > + reg = <0x01c28000 0x400>; > > + interrupts = ; > > + reg-shift = <2>; > > +

Re: [PATCH 0/6] ARM: sunxi: Introduce Allwinner H3 support

2015-10-22 Thread Jean-Francois Moine
On Wed, 21 Oct 2015 21:18:45 +0200 Hans de Goede wrote: > Great to see that you've started working on this again. Last weekend I > ended up working on this too together with Reinder E.N. de Haan > > (added to the Cc). > > We took a slightly different approach for the gates clocks, see: > > ht

Re: [GIT PULL] On-demand device probing

2015-10-20 Thread Jean-Francois Moine
Sorry to enter this thread a bit late. About the number of probe deferred messages, I proposed a simple patch to reduce them: https://lkml.org/lkml/2013/8/20/218 I was wondering how many messages this patch could save... -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef

Re: [PATCH RFC v4 4/8] drm/i2c: tda998x: Add support of a DT graph of ports

2015-09-24 Thread Jean-Francois Moine
On Mon, 21 Sep 2015 10:19:18 -0500 Rob Herring wrote: > On 09/18/2015 06:06 AM, Jyri Sarha wrote: > > From: Jean-Francois Moine > > > > Two kinds of ports may be declared in a DT graph of ports: video and audio. > > This patch accepts the port value from a video po

Re: [PATCH v14 1/3] drm/i2c: tda998x: Add support of a DT graph of ports

2015-08-03 Thread Jean-Francois Moine
On Mon, 3 Aug 2015 17:56:17 +0300 Jyri Sarha wrote: > On 05/08/15 11:18, Jean-Francois Moine wrote: > > Two kinds of ports may be declared in a DT graph of ports: video and audio. > > This patch accepts the port value from a video port as an alternative > > to the video-p

Re: [PATCH v13 3/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-28 Thread Jean-Francois Moine
On Tue, 28 Jul 2015 14:53:58 +0100 Russell King - ARM Linux wrote: > On Tue, Jul 28, 2015 at 03:23:29PM +0200, Jean-Francois Moine wrote: > > The EDID arrives in the DRM connector when video starts. The built ELD > > may be stored either in the connector itself (default),

Re: [PATCH v13 3/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-28 Thread Jean-Francois Moine
On Tue, 28 Jul 2015 11:24:10 +0100 Mark Brown wrote: > On Tue, Jul 28, 2015 at 12:19:45PM +0200, Jean-Francois Moine wrote: > > Mark Brown wrote: > > > > > +int tda9998x_codec_register(struct device *dev, > > > > + str

[PATCH v14 0/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-28 Thread Jean-Francois Moine
move the CODEC to sound/soc (Mark Brown) - update the audio_sample_rate from the EDID (Andrew Jackson) v3: fix bad rate (Andrew Jackson) v2: check double stream start (Mark Brown) Jean-Francois Moine (3): drm/i2c: tda998x: Add support of a DT graph of ports drm/i2c: tda998x: Change drvdata for audio e

Re: [PATCH v13 3/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-28 Thread Jean-Francois Moine
On Mon, 20 Jul 2015 19:06:06 +0100 Mark Brown wrote: > On Fri, May 08, 2015 at 10:41:12AM +0200, Jean-Francois Moine wrote: > > > + > > + if (!priv->is_hdmi_sink > > +|| !encoder->crtc) > > + return NULL; > > That's weird inden

[PATCH v14 3/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-28 Thread Jean-Francois Moine
The tda998x CODEC maintains the audio constraints according to the HDMI device parameters (EDID) and sets dynamically the input ports in the TDA998x I2C driver on start/stop audio streaming. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 71

[PATCH v14 1/3] drm/i2c: tda998x: Add support of a DT graph of ports

2015-07-28 Thread Jean-Francois Moine
/tda998x.h prepares to the definition of a tda998x CODEC. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/drm/i2c/tda998x.txt| 51 drivers/gpu/drm/i2c/tda998x_drv.c | 90 +++--- include/sound/tda998x.h| 8

[PATCH v14 2/3] drm/i2c: tda998x: Change drvdata for audio extension

2015-07-28 Thread Jean-Francois Moine
encoder use. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index 0952eac..3504ef9 100644 --- a/drivers/gpu/drm/i2c

[PATCH v13 2/3] drm/i2c: tda998x: Change drvdata for audio extension

2015-07-20 Thread Jean-Francois Moine
encoder use. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index b0a730a..655ebb0 100644 --- a/drivers/gpu/drm/i2c

[PATCH v13 3/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-20 Thread Jean-Francois Moine
The tda998x CODEC maintains the audio constraints according to the HDMI device parameters (EDID) and sets dynamically the input ports in the TDA998x I2C driver on start/stop audio streaming. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 80

[PATCH v13 1/3] drm/i2c: tda998x: Add support of a DT graph of ports

2015-07-20 Thread Jean-Francois Moine
/tda998x.h prepares to the definition of a tda998x CODEC. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/drm/i2c/tda998x.txt| 51 drivers/gpu/drm/i2c/tda998x_drv.c | 90 +++--- include/sound/tda998x.h| 8

[PATCH v13 0/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-07-20 Thread Jean-Francois Moine
can from the CODEC to the TDA998x - move the CODEC to sound/soc (Mark Brown) - update the audio_sample_rate from the EDID (Andrew Jackson) v3: fix bad rate (Andrew Jackson) v2: check double stream start (Mark Brown) Jean-Francois Moine (3): drm/i2c: tda998x: Add support of a DT

[PATCH v12 6/6] ASoC: tda998x: add a codec to the HDMI transmitter

2015-05-08 Thread Jean-Francois Moine
The tda998x CODEC maintains the audio constraints according to the HDMI device parameters (EDID) and sets dynamically the input ports in the TDA998x I2C driver on start/stop audio streaming. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 80

[PATCH v12 1/6] drm/edid: add function to help find SADs

2015-05-08 Thread Jean-Francois Moine
From: Russell King - ARM Linux Add a function to find the start of the SADs in the ELD. This complements the helper to retrieve the SAD count. Signed-off-by: Jean-Francois Moine --- include/drm/drm_edid.h | 19 +++ 1 file changed, 19 insertions(+) diff --git a/include/drm

[PATCH v11 0/3] ASoC: tda998x: add a codec to the HDMI transmitter

2015-04-29 Thread Jean-Francois Moine
DEC to the TDA998x - move the CODEC to sound/soc (Mark Brown) - update the audio_sample_rate from the EDID (Andrew Jackson) v3: fix bad rate (Andrew Jackson) v2: check double stream start (Mark Brown) Jean-Francois Moine (3): drm/i2c: tda998x: Add support of a DT graph of ports

[PATCH v11 1/3] drm/i2c: tda998x: Add support of a DT graph of ports

2015-04-29 Thread Jean-Francois Moine
/tda998x.h prepares to the definition of a tda998x CODEC. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/drm/i2c/tda998x.txt| 51 + drivers/gpu/drm/i2c/tda998x_drv.c | 88 +++--- include/sound/tda998x.h| 8

Re: [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board

2015-02-06 Thread Jean-Francois Moine
On Fri, 06 Feb 2015 11:05:39 +0100 Sebastian Hesselbarth wrote: > >> We do have a driver for the framebuffer (armada_drm) and there is great > >> work from Russell King and others on the GPU and Decode engine (IIRC). > > > > Yes, but it has no DT support. Mine has. > > Yeah, it would be nice

Re: [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board

2015-02-05 Thread Jean-Francois Moine
On Thu, 05 Feb 2015 23:13:58 +0100 Sebastian Hesselbarth wrote: > > At this moment, I am trying to configure the framebuffer, but as Moine > > told me,it seems there is not video driver support for this board in > > DT... :( . > > Not quite true. Video is made up of at least 4 different device

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-02-03 Thread Jean-Francois Moine
On Tue, 3 Feb 2015 16:47:48 + Mark Brown wrote: > On Sat, Jan 24, 2015 at 08:30:27AM +0100, Jean-Francois Moine wrote: > > Mark Brown wrote: > > > On Fri, Jan 23, 2015 at 07:34:56PM +0100, Jean-Francois Moine wrote: > > > > > The simple card builder, &#x

Re: [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board

2015-01-30 Thread Jean-Francois Moine
On Fri, 30 Jan 2015 13:03:59 +0100 Sebastian Hesselbarth wrote: > Nope. The cm-a510 is just an add-on for a baseboard, it does not make > a working board. Just think of it as a feature-improved SoC. Well, I understand a bit, but I don't see clearly the physical system, nor how each part gets its

Re: [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board

2015-01-30 Thread Jean-Francois Moine
On Fri, 30 Jan 2015 12:00:16 +0100 Sebastian Hesselbarth wrote: > Adapting the .config and removing drivers is actually not an option. > IMHO, introducing DT was meant for a single multi-arch kernel that > can be shipped with common Linux distros. Therefore, DT is the place > you enable/disable a

Re: [PATCH] ARM: dts: mvebu: add ethernet to the cm-a510 board

2015-01-30 Thread Jean-Francois Moine
On Fri, 30 Jan 2015 10:44:07 +0100 Sebastian Hesselbarth wrote: > I had a closer look on the Compulab website of the SoM [1] and think > that we'll have to convert it to dove-cm-a510.dtsi and the baseboard > Gabriel is using (maybe SBC-A510 [2]). Sebastian, I don't understand why the A510 conta

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-26 Thread Jean-Francois Moine
On Mon, 26 Jan 2015 12:53:53 +0100 Lars-Peter Clausen wrote: > > - a generic sound node in the case of multi controllers or multi codec > >levels (after dt-card extension): > > > > sound { > > compatible = "linux,dt-card"; > > audio-root = <&audio1>; /* starting poi

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-24 Thread Jean-Francois Moine
On Sat, 24 Jan 2015 12:27:35 +0100 Lars-Peter Clausen wrote: > On 01/23/2015 07:34 PM, Jean-Francois Moine wrote: > > On Fri, 23 Jan 2015 14:56:04 +0100 > > Lars-Peter Clausen wrote: > > > >> On 01/23/2015 01:15 PM, Jean-Francois Moine wrote: > > There

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-23 Thread Jean-Francois Moine
On Fri, 23 Jan 2015 19:13:43 + Mark Brown wrote: > On Fri, Jan 23, 2015 at 07:34:56PM +0100, Jean-Francois Moine wrote: > > > A card builder is a device which > > - scans the graph of ports, > > - fills the struct snd_soc_card according to the links between t

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-23 Thread Jean-Francois Moine
On Fri, 23 Jan 2015 14:56:04 +0100 Lars-Peter Clausen wrote: > On 01/23/2015 01:15 PM, Jean-Francois Moine wrote: > [...] > > The DT should describe the hardware, and the simple-card mixes hardware > > and software. > > For example, the kirkwood controller may cre

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-23 Thread Jean-Francois Moine
On Thu, 22 Jan 2015 20:25:39 +0100 Lars-Peter Clausen wrote: > On 01/22/2015 09:07 AM, Jean-Francois Moine wrote: > > On Wed, 21 Jan 2015 21:14:07 +0100 > > Lars-Peter Clausen wrote: > > > >> [...] > >>> + card->dai_link->dai_fmt = > >>&

Re: [alsa-devel] [PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-22 Thread Jean-Francois Moine
On Wed, 21 Jan 2015 21:14:07 +0100 Lars-Peter Clausen wrote: > [...] > > + card->dai_link->dai_fmt = > > + snd_soc_of_parse_daifmt(of_cpu, "dt-audio-card,", > > + NULL, NULL) & > > + ~SND_SOC_DAIFMT_MASTER_MASK; > > > This one does

[PATCH v2 2/3] Documentation: of: Document audio graph bindings

2015-01-21 Thread Jean-Francois Moine
This patch adds a document describing common OF bindings for audio devices. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/sound/audio-interfaces.txt | 101 + 1 file changed, 101 insertions(+) create mode 100644 Documentation/devicetree/bindings/sound/audio

[PATCH v2 3/3] ASoC: add generic dt-card support

2015-01-21 Thread Jean-Francois Moine
LSA SoC DT based sound card support + * + * Copyright (C) 2015 Jean-Francois Moine + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + */ + +#include

[PATCH v2 1/3] ASoC: core: export snd_soc_get_dai_name

2015-01-21 Thread Jean-Francois Moine
snd_soc_get_dai_name() may be used to define a sound card with a different syntax from the one of the simple-card. Signed-off-by: Jean-Francois Moine --- include/sound/soc.h | 2 ++ sound/soc/soc-core.c | 5 +++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/include/sound

[PATCH v2 0/3] ASoC: add audio card creation from graph of ports in DT

2015-01-21 Thread Jean-Francois Moine
This patchset adds a way to build audio cards from the description of a graph of ports in a devicetree. v2: - move the soc-core and dt-card from the patchset "ASoC: tda998x: add a codec to the HDMI transmitter" - add DT documentation (Mark Brown) Jea

[PATCH v10 0/9] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-20 Thread Jean-Francois Moine
Mark Brown) - update the audio_sample_rate from the EDID (Andrew Jackson) v3: fix bad rate (Andrew Jackson) v2: check double stream start (Mark Brown) Jean-Francois Moine (9): ASoC: kirkwood: dynamically build the DAI array ASoC: kirkwood: check the DAI type from the DAI name ASoC: kirkwood: acce

[PATCH v10 3/9] ASoC: kirkwood: accept the DAI definitions from a graph of ports

2015-01-20 Thread Jean-Francois Moine
By default, both output ports 0 (I2S) and 1 (S/PDIF) are defined. A graph of port permits to define only the really connected ports of the board and to identify the remote ends. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/sound/mvebu-audio.txt | 30

[PATCH v10 2/9] ASoC: kirkwood: check the DAI type from the DAI name

2015-01-20 Thread Jean-Francois Moine
When the DAIs are created from a graph of ports, their types are not tied to their ID. Signed-off-by: Jean-Francois Moine --- sound/soc/kirkwood/kirkwood-i2s.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sound/soc/kirkwood/kirkwood-i2s.c b/sound/soc/kirkwood

[PATCH v10 1/9] ASoC: kirkwood: dynamically build the DAI array

2015-01-20 Thread Jean-Francois Moine
This patch prepares the driver to the creation of the DAIs from a graph of ports. Signed-off-by: Jean-Francois Moine --- sound/soc/kirkwood/kirkwood-i2s.c | 108 ++ sound/soc/kirkwood/kirkwood.h | 1 + 2 files changed, 40 insertions(+), 69 deletions

[PATCH v10 4/9] drm/i2c: tda998x: Add support of a DT graph of ports

2015-01-20 Thread Jean-Francois Moine
/tda998x.h prepares to the definition of a tda998x CODEC. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/drm/i2c/tda998x.txt| 51 + drivers/gpu/drm/i2c/tda998x_drv.c | 88 +++--- include/sound/tda998x.h| 8

[PATCH v10 6/9] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-20 Thread Jean-Francois Moine
The tda998x CODEC maintains the audio format and rate constraints according to the HDMI device parameters (EDID) and sets dynamically the input ports in the TDA998x I2C driver on start/stop audio streaming. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 124

[PATCH v10 7/9] drm/i2c: tda998x: set cts_n according to the sample width

2015-01-20 Thread Jean-Francois Moine
With I2S input, the CTS_N predivider depends on the audio sample width. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 21 ++--- include/sound/tda998x.h | 4 +++- sound/soc/codecs/tda998x.c| 5 +++-- 3 files changed, 24 insertions

[PATCH v10 9/9] ASoC: add generic dt-card support

2015-01-20 Thread Jean-Francois Moine
parameters found in the tree. Signed-off-by: Jean-Francois Moine --- sound/soc/generic/Kconfig | 2 + sound/soc/generic/Makefile | 2 + sound/soc/generic/dt-card.c | 275 3 files changed, 279 insertions(+) create mode 100644 sound/soc/generic/dt

[PATCH v10 8/9] ASoC: core: export snd_soc_get_dai_name

2015-01-20 Thread Jean-Francois Moine
snd_soc_get_dai_name() may be used to define a sound card with a different syntax from the one of the simple-card. Signed-off-by: Jean-Francois Moine --- include/sound/soc.h | 2 ++ sound/soc/soc-core.c | 5 +++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/include/sound

[PATCH v10 5/9] drm/i2c: tda998x: Change drvdata for audio extension

2015-01-20 Thread Jean-Francois Moine
encoder use. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index cf245ce..b35f35f 100644 --- a/drivers/gpu/drm/i2c

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-13 Thread Jean-Francois Moine
On Tue, 13 Jan 2015 19:54:15 + Russell King - ARM Linux wrote: > On Tue, Jan 13, 2015 at 09:41:01PM +0200, Jyri Sarha wrote: > > On 01/13/2015 09:26 PM, Russell King - ARM Linux wrote: > > >SCLK: _~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_~_ > > > WS: __~~~

Re: [PATCH RFC 0/3] Make "ti,tilcdc,slave" DT binding more sensible

2015-01-13 Thread Jean-Francois Moine
On Tue, 13 Jan 2015 19:12:25 +0200 Jyri Sarha wrote: > These patches are needed for Beaglebone-back HDMI audio. There is no > direct dependency between these patches and the other (dts and ASoC) > changes needed for the HDMI audio so these changes can be merged > independently. I also feel that t

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-13 Thread Jean-Francois Moine
On Tue, 13 Jan 2015 16:03:13 + Russell King - ARM Linux wrote: > On Tue, Jan 13, 2015 at 04:54:11PM +0100, Jean-Francois Moine wrote: > > 4 streams on 4 different APs (sources) should work, but 4 streams from > > a same source should be detailed. > > I'd like to k

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-13 Thread Jean-Francois Moine
On Tue, 13 Jan 2015 12:27:15 + Russell King - ARM Linux wrote: > On Tue, Jan 13, 2015 at 01:21:58PM +0100, Philipp Zabel wrote: > > I wonder if it wouldn't be nicer to have the AP# and type in the device > > tree and then calculate the register value from that in the driver. > > > > port

Re: [PATCH v9 3/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-13 Thread Jean-Francois Moine
On Fri, 09 Jan 2015 17:39:08 + Andrew Jackson wrote: > > + snd_pcm_hw_constraint_minmax(runtime, > > + SNDRV_PCM_HW_PARAM_CHANNELS, > > + 1, sad[SAD_MX_CHAN_I]); > > In the light of our discussions elsewhere [1], shouldn't t

Re: [PATCH v9 3/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-12 Thread Jean-Francois Moine
On Sun, 11 Jan 2015 23:03:14 +0200 Jyri Sarha wrote: > Some more comments based on my - finally successful - attempt to use > this code with BBB HDMI audio. > > On 01/07/2015 12:51 PM, Jean-Francois Moine wrote: > > This patch adds a CODEC to the NXP TDA998x HDMI transmit

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-12 Thread Jean-Francois Moine
On Mon, 12 Jan 2015 17:57:06 + Russell King - ARM Linux wrote: > I don't particularly like that - that makes the assumption that AP0 > always means I2S. What if a future chip decides to allow SPDIF on > AP0? Why should we need to re-invent the binding? > > IMHO, it would be much better to

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-12 Thread Jean-Francois Moine
On Mon, 12 Jan 2015 14:04:56 + Russell King - ARM Linux wrote: > On Mon, Jan 12, 2015 at 02:59:57PM +0100, Philipp Zabel wrote: > > Am Montag, den 12.01.2015, 12:25 + schrieb Russell King - ARM Linux: > > > It's not quite that simple, because the SPDIF AP pins are multiplexed > > > with t

Re: [alsa-devel] [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-10 Thread Jean-Francois Moine
On Fri, 9 Jan 2015 20:01:27 + Russell King - ARM Linux wrote: > > I proposed: > > > > audio-ports = <0x04>, <0x03>; > > audio-port-names = "spdif", "i2s"; > > > > Do you better like: > > > > audio-spdif-port = <0x04>; > > audio-i2s-port = <0x03>; > > Neither; we know tha

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-09 Thread Jean-Francois Moine
On Fri, 9 Jan 2015 14:57:41 + Russell King - ARM Linux wrote: > Well, if we look at the HDMI specs, there is a field in the audio info > frame (CA bits) which identifies the speaker allocation between the > HDMI PCM channels and the physical speakers. > > It describes a limited set - essenti

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-09 Thread Jean-Francois Moine
On Fri, 9 Jan 2015 11:45:29 + Russell King - ARM Linux wrote: > On Fri, Jan 09, 2015 at 12:30:36PM +0100, Jean-Francois Moine wrote: > > In my original version, the audio-ports are a bitmap of the pins, the > > bit 0 being the WS used for I2S. A fully wired tda998x would h

Re: [PATCH v9 3/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-09 Thread Jean-Francois Moine
On Fri, 9 Jan 2015 11:19:35 + Russell King - ARM Linux wrote: > Would it be more sensible to make SND_SOC_TDA998X depend on > DRM_I2C_NXP_TDA998X instead, maybe with a 'default y' - which is a > kinder way to have SND_SOC_TDA998X be enabled. If SND_SOC_TDA998X > doesn't have a prompt, then i

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-09 Thread Jean-Francois Moine
On Fri, 9 Jan 2015 12:13:04 +0200 Jyri Sarha wrote: > On 01/08/2015 06:42 PM, Jean-Francois Moine wrote: > > From Andrew's datasheet, the TDA998x's which are handled by the tda998x > > driver have only 4 input audio pins, the first two ones being either > > S/P

Re: [PATCH v9 3/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-09 Thread Jean-Francois Moine
On Fri, 9 Jan 2015 12:24:12 +0200 Jyri Sarha wrote: > > select SND_SOC_TDA998X if SND_SOC > > > > should work in all cases. > > > > I think that would still fail if DRM and TDA998x is built in and SND_SOC > is built as modules. A request_module() call before > tda9998x_codec_register() s

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-08 Thread Jean-Francois Moine
On Thu, 8 Jan 2015 16:53:41 +0200 Jyri Sarha wrote: > > + - audio-ports: must contain one or two values selecting the source > > + in the audio port. > > + The source type is given by the corresponding entry in > > + the audio-port-names property. > > + > > This binding does not allow m

Re: [PATCH v9 3/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-07 Thread Jean-Francois Moine
On Wed, 7 Jan 2015 15:41:38 + Russell King - ARM Linux wrote: > On Wed, Jan 07, 2015 at 03:10:47PM +, Andrew Jackson wrote: > > On 01/07/15 10:51, Jean-Francois Moine wrote: > > > diff --git a/drivers/gpu/drm/i2c/Kconfig b/drivers/gpu/drm/i2c/Kconfig > > > in

Re: [PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-07 Thread Jean-Francois Moine
On Wed, 07 Jan 2015 14:39:13 + Andrew Jackson wrote: > > + - audio-ports: must contain one or two values selecting the source > > + in the audio port. > > + The source type is given by the corresponding entry in > > + the audio-port-names property. > > I think that this entry might

[PATCH v9 3/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-07 Thread Jean-Francois Moine
. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/Kconfig | 1 + drivers/gpu/drm/i2c/tda998x_drv.c | 139 -- include/sound/tda998x.h | 23 + sound/soc/codecs/Kconfig | 4 + sound/soc/codecs/Makefile | 2 + sound

[PATCH v9 2/4] drm/i2c: tda998x: Change drvdata for audio extension

2015-01-07 Thread Jean-Francois Moine
encoder use. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c/tda998x_drv.c index 9d9b072..ce1478d 100644 --- a/drivers/gpu/drm/i2c

[PATCH v9 1/4] drm/i2c: tda998x: Add DT support for audio

2015-01-07 Thread Jean-Francois Moine
This patch permits the definition of the audio ports from the device tree. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/drm/i2c/tda998x.txt| 18 +++ drivers/gpu/drm/i2c/tda998x_drv.c | 60 ++ 2 files changed, 69 insertions(+), 9

[PATCH v9 4/4] drm/i2c: tda998x: set cts_n according to the sample width

2015-01-07 Thread Jean-Francois Moine
With I2S input, the CTS_N predivider depends on the sample width. Signed-off-by: Jean-Francois Moine --- drivers/gpu/drm/i2c/tda998x_drv.c | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c b/drivers/gpu/drm/i2c

[PATCH v9 0/4] ASoC: tda998x: add a codec to the HDMI transmitter

2015-01-07 Thread Jean-Francois Moine
ackson) v2: check double stream start (Mark Brown) Jean-Francois Moine (4): drm/i2c: tda998x: Add DT support for audio drm/i2c: tda998x: Change drvdata for audio extension ASoC: tda998x: add a codec to the HDMI transmitter drm/i2c: tda998x: set cts_n according to the sample width .../

[PATCH v6 3/3] ASoC: simple-card: Remove useless check

2015-01-01 Thread Jean-Francois Moine
The CPU and CODEC names are checked when getting the DAI names. There is no need to check them once more. Signed-off-by: Jean-Francois Moine --- sound/soc/generic/simple-card.c | 5 - 1 file changed, 5 deletions(-) diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple

[PATCH v6 0/3] ASoC: simple-card: Add multi-CODEC support

2015-01-01 Thread Jean-Francois Moine
ng example 2 v3: rebase on broonie git 2014/11/03 v2: accept list of CODECs in 'sound-dais' (Benoit Cousson) Happy New Year! Jean-Francois Moine (3): ASoC: simple-card: prepare multi-CODECs ASoC: simple-card: Add multi-CODECs in DT ASoC: simple-card: Remove useless check .../devicet

[PATCH v6 2/3] ASoC: simple-card: Add multi-CODECs in DT

2015-01-01 Thread Jean-Francois Moine
This patch allows to declare many CODECs per DAI link in the device tree. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/sound/simple-card.txt | 19 ++- sound/soc/generic/simple-card.c| 39 +- 2 files changed, 42 insertions

[PATCH v6 1/3] ASoC: simple-card: prepare multi-CODECs

2015-01-01 Thread Jean-Francois Moine
The function asoc_simple_card_sub_parse_of() is used to parse either the CPU or CODEC DAI. To prepare support of many CODECs in a DAI link, the parsing of the phandle and DAI name must be moved to the upper function asoc_simple_card_dai_link_of(). Signed-off-by: Jean-Francois Moine --- sound

Re: [RFC PATCH 0/6] Those patches is used for dw_hdmi audio.

2014-12-15 Thread Jean-Francois Moine
On Mon, 15 Dec 2014 12:00:13 + Russell King - ARM Linux wrote: > I really don't want to end up in another cocked up situation like > what happened with the Dove audio, where it became politically > impossible for the SolidRun platform to be properly supported by > mainline kernels. Hi Russel

[PATCH v5 2/4] ASoC: simple-card: Remove useless function argument

2014-11-25 Thread Jean-Francois Moine
The device node pointer 'cpu' is not used in the function asoc_simple_card_parse_daifmt(). Signed-off-by: Jean-Francois Moine --- sound/soc/generic/simple-card.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/sound/soc/generic/simple-card.c b/sound/soc/gene

[PATCH v5 1/4] ASoC: core: add multi-codec support in DT

2014-11-25 Thread Jean-Francois Moine
This patch exports a core function which handles the DT description of multi-codec links (as: "sound-dai = <&hdmi 0>, <&spdif_codec>;") and creates a CODEC component array in the DAI link. Signed-off-by: Jean-Francois Moine --- include/sound/soc.h | 3

[PATCH v5 3/4] ASoC: simple-card: add multi-CODECs in DT

2014-11-25 Thread Jean-Francois Moine
This patch allows many CODECs per link to be defined in the device tree. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/sound/simple-card.txt | 18 ++--- sound/soc/generic/simple-card.c| 81 -- 2 files changed, 51 insertions(+), 48

[PATCH v5 4/4] ASoC: simple-card: Remove useless check

2014-11-25 Thread Jean-Francois Moine
The CPU and CODEC names are checked when getting the DAI names. There is no need to check them once more. Signed-off-by: Jean-Francois Moine --- sound/soc/generic/simple-card.c | 5 - 1 file changed, 5 deletions(-) diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple

[PATCH v5 0/4] ASoC: simple-card: Add multi-CODEC support

2014-11-25 Thread Jean-Francois Moine
atch about simple-card.c - less code changes - remove the cast of pointers - less changes in the DT binding example 2 v3: rebase on broonie git 2014/11/03 v2: accept list of CODECs in 'sound-dais' (Benoit Cousson) Jean-Francois Moine (4): ASoC: core: add multi-codec

Re: [PATCH] ARM: dts: dove: add DT LCD controllers

2014-11-19 Thread Jean-Francois Moine
On Wed, 19 Nov 2014 11:32:29 + Russell King wrote: > + lcd1: lcd-controller@81 { > + compatible = "marvell,dove-lcd"; > + reg = <0x81 0x1000>; > + interrupts = <46>; > +

Re: [PATCH v4 3/4] ASoC: simple-card: add multi-CODECs in DT

2014-11-10 Thread Jean-Francois Moine
On Mon, 10 Nov 2014 09:39:57 +0100 Arnd Bergmann wrote: > On Sunday 09 November 2014 12:22:21 Jean-Francois Moine wrote: > > -- sound-dai: phandle and port of CPU/CODEC > > +either > > + - sound-dai : phandle and port

[PATCH v4 4/4] ASoC: simple-card: Remove useless check

2014-11-10 Thread Jean-Francois Moine
The CPU and CODEC names are checked when getting the DAI names. There is no need to check them once more. Signed-off-by: Jean-Francois Moine --- sound/soc/generic/simple-card.c | 5 - 1 file changed, 5 deletions(-) diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple

[PATCH v4 1/4] ASoC: core: add multi-codec support in DT

2014-11-10 Thread Jean-Francois Moine
SoC audio cards may have many CODECs per DAI link. This patch exports a core function handling such links in the DT (as: "sound-dais = <&hdmi 0>, <&spdif_codec>;") and creating a CODEC component array in the DAI link. Signed-off-by: Jean-Francois Moine --- includ

[PATCH v4 3/4] ASoC: simple-card: add multi-CODECs in DT

2014-11-10 Thread Jean-Francois Moine
This patch allows many CODECs per link to be defined in the device tree. Signed-off-by: Jean-Francois Moine --- .../devicetree/bindings/sound/simple-card.txt | 20 ++ sound/soc/generic/simple-card.c| 81 -- 2 files changed, 53 insertions(+), 48

[PATCH v4 2/4] ASoC: simple-card: Remove useless function argument

2014-11-10 Thread Jean-Francois Moine
The device node pointer 'cpu' is not used in the function asoc_simple_card_parse_daifmt(). Signed-off-by: Jean-Francois Moine --- sound/soc/generic/simple-card.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/sound/soc/generic/simple-card.c b/sound/soc/gene

[PATCH v4 0/4] ASoC: simple-card: Add multi-CODEC support

2014-11-10 Thread Jean-Francois Moine
pointers - less changes in the DT binding example 2 v3: rebase on broonie git 2014/11/03 v2: accept list of CODECs in 'sound-dais' (Benoit Cousson) Jean-Francois Moine (4): ASoC: core: add multi-codec support in DT ASoC: simple-card: Remove useless function argument ASoC: s

Re: [PATCH 1/2] of: Make const the device node pointers in of_clk_get and of_node_put

2014-11-09 Thread Jean-Francois Moine
On Sun, 9 Nov 2014 18:50:19 + Russell King - ARM Linux wrote: > What this then means is that it fundamentally undermines the idea of > storing the pointer to a device_node as a const pointer, as the device > node must always be modified when you're done with it (because it's a > ref-counted s

  1   2   3   >