On Mon, 24 Feb 2014 02:17:00 +
"li.xi...@freescale.com" wrote:
> > @@ -169,22 +164,26 @@ static int asoc_simple_card_parse_of(struct
> > device_node
> > *node,
> > /* CPU sub-node */
> > ret = -EINVAL;
> > np = of_get_child_by_name(node, "simple-audio-card,cpu");
> > - if (np)
On Mon, 24 Feb 2014 02:17:00 +
li.xi...@freescale.com li.xi...@freescale.com wrote:
@@ -169,22 +164,26 @@ static int asoc_simple_card_parse_of(struct
device_node
*node,
/* CPU sub-node */
ret = -EINVAL;
np = of_get_child_by_name(node, simple-audio-card,cpu);
- if
This patch series fixes a small problem about node device locks and also
extends the simple card driver to handle many DAI links as this exists
in the Cubox audio subsystem.
Jean-Francois Moine (4):
ASoC: simple-card: Fix device node locks
ASoC: simple-card: dynamically allocate the DAI link
Some device nodes stay locked and some other ones are not locked
while being used during the card lifetime.
Signed-off-by: Jean-Francois Moine
---
sound/soc/generic/simple-card.c | 51 +++--
1 file changed, 39 insertions(+), 12 deletions(-)
diff --git
There may be many couples of CPU/CODEC DAI links.
The example 2 is extracted from the Cubox DT.
Signed-off-by: Jean-Francois Moine
---
.../devicetree/bindings/sound/simple-card.txt | 34 +-
1 file changed, 33 insertions(+), 1 deletion(-)
diff --git a/Documentation
The DAI link array is hard-coded as a single CPU / CODEC DAIs link.
This patch allocates this array with the card definition and facilitates
handling more links.
Signed-off-by: Jean-Francois Moine
---
sound/soc/generic/simple-card.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions
Some simple audio cards may have many DAI links.
This patch makes them handled by the simple-card driver.
Signed-off-by: Jean-Francois Moine
---
sound/soc/generic/simple-card.c | 100 ++--
1 file changed, 66 insertions(+), 34 deletions(-)
diff --git a/sound
The DAI link array is hard-coded as a single CPU / CODEC DAIs link.
This patch allocates this array with the card definition and facilitates
handling more links.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/generic/simple-card.c | 7 ---
1 file changed, 4 insertions
Some simple audio cards may have many DAI links.
This patch makes them handled by the simple-card driver.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/generic/simple-card.c | 100 ++--
1 file changed, 66 insertions(+), 34 deletions(-)
diff
There may be many couples of CPU/CODEC DAI links.
The example 2 is extracted from the Cubox DT.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
.../devicetree/bindings/sound/simple-card.txt | 34 +-
1 file changed, 33 insertions(+), 1 deletion(-)
diff --git
Some device nodes stay locked and some other ones are not locked
while being used during the card lifetime.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/generic/simple-card.c | 51 +++--
1 file changed, 39 insertions(+), 12 deletions
This patch series fixes a small problem about node device locks and also
extends the simple card driver to handle many DAI links as this exists
in the Cubox audio subsystem.
Jean-Francois Moine (4):
ASoC: simple-card: Fix device node locks
ASoC: simple-card: dynamically allocate the DAI link
On Mon, 25 Nov 2013 19:47:04 +0100
Jean-Francois Moine wrote:
> At probe time, a clock device may not be ready when some other device
> wants to use it.
>
> This patch lets the functions clk_get/devm_clk_get return a probe defer
> when the clock is defined in the DT but not yet
On Mon, 25 Nov 2013 19:47:04 +0100
Jean-Francois Moine moin...@free.fr wrote:
At probe time, a clock device may not be ready when some other device
wants to use it.
This patch lets the functions clk_get/devm_clk_get return a probe defer
when the clock is defined in the DT but not yet
When the creation of the second i2c client was failing, the private
buffer was not freed.
This bug was introduced by the commit 6ae668cc19e8
'drm/i2c: tda998x: check the CEC device creation'
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 4 +++-
1 file changed, 3
When the creation of the second i2c client was failing, the private
buffer was not freed.
This bug was introduced by the commit 6ae668cc19e8
'drm/i2c: tda998x: check the CEC device creation'
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 4 +++-
1
At probe time, a clock device may not be ready when some other device
wants to use it.
This patch lets the functions clk_get/devm_clk_get return a probe defer
when the clock is defined in the DT but not yet available.
Signed-off-by: Jean-Francois Moine
---
resend v3:
- base kernel
At probe time, a clock device may not be ready when some other device
wants to use it.
This patch lets the functions clk_get/devm_clk_get return a probe defer
when the clock is defined in the DT but not yet available.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
resend v3
The functions snd_pcm_hw_constraint_mask() and
snd_pcm_hw_constraint_mask64() are needed to add audio constraints
to substreams at open/startup time.
This patch makes them available to ALSA modules.
Signed-off-by: Jean-Francois Moine
---
sound/core/pcm_lib.c | 2 ++
1 file changed, 2 insertions
The functions snd_pcm_hw_constraint_mask() and
snd_pcm_hw_constraint_mask64() are needed to add audio constraints
to substreams at open/startup time.
This patch makes them available to ALSA modules.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/core/pcm_lib.c | 2 ++
1 file
On Wed, 05 Feb 2014 10:19:22 +0100
Lars-Peter Clausen wrote:
> > So, in the CODEC, I don't see how I could update the parameters
> > dictated by the EDID otherwise in changing the DAI driver parameters.
> >
>
> The startup function is the right place. But instead of modifying the DAI
> use
On Tue, 4 Feb 2014 18:06:25 +
Mark Brown wrote:
> On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote:
>
> > + /* change the snd_soc_pcm_stream values of the driver */
> > + stream->rates = rate_mask;
> > + stream-&
On Tue, 4 Feb 2014 18:06:25 +
Mark Brown broo...@kernel.org wrote:
On Mon, Jan 27, 2014 at 09:48:54AM +0100, Jean-Francois Moine wrote:
+ /* change the snd_soc_pcm_stream values of the driver */
+ stream-rates = rate_mask;
+ stream-channels_max
On Wed, 05 Feb 2014 10:19:22 +0100
Lars-Peter Clausen l...@metafoo.de wrote:
So, in the CODEC, I don't see how I could update the parameters
dictated by the EDID otherwise in changing the DAI driver parameters.
The startup function is the right place. But instead of modifying the DAI
On Tue, 4 Feb 2014 18:12:13 +
Mark Brown wrote:
> On Sat, Feb 01, 2014 at 05:48:49PM +0100, Jean-Francois Moine wrote:
>
> > + - compatible: must be "nxp,tda998x-codec".
>
> It's not clear to me why there's a separate compatible here - as far as
> I can s
On Tue, 4 Feb 2014 17:54:10 +
Mark Brown wrote:
> On Tue, Feb 04, 2014 at 06:16:05PM +0100, Jean-Francois Moine wrote:
> > Mark Brown wrote:
>
> > > > + /* load the optional CODEC */
> > > > + of_platform_populate(np, NULL, NUL
On Tue, 4 Feb 2014 13:30:14 +
Mark Brown wrote:
> On Sun, Jan 26, 2014 at 07:45:36PM +0100, Jean-Francois Moine wrote:
>
> > + /* load the optional CODEC */
> > + of_platform_populate(np, NULL, NULL, >dev);
> > +
>
> Why is this using of_platform_p
On Tue, 4 Feb 2014 13:30:14 +
Mark Brown broo...@kernel.org wrote:
On Sun, Jan 26, 2014 at 07:45:36PM +0100, Jean-Francois Moine wrote:
+ /* load the optional CODEC */
+ of_platform_populate(np, NULL, NULL, client-dev);
+
Why is this using of_platform_populate()? That's
On Tue, 4 Feb 2014 17:54:10 +
Mark Brown broo...@kernel.org wrote:
On Tue, Feb 04, 2014 at 06:16:05PM +0100, Jean-Francois Moine wrote:
Mark Brown broo...@kernel.org wrote:
+ /* load the optional CODEC */
+ of_platform_populate(np, NULL, NULL, client-dev);
Why
On Tue, 4 Feb 2014 18:12:13 +
Mark Brown broo...@kernel.org wrote:
On Sat, Feb 01, 2014 at 05:48:49PM +0100, Jean-Francois Moine wrote:
+ - compatible: must be nxp,tda998x-codec.
It's not clear to me why there's a separate compatible here - as far as
I can see this can only appear
On Sun, 2 Feb 2014 19:15:05 +
Russell King - ARM Linux wrote:
> In which case, it may be better to reorder the remaining patches such
> that the DT changes are at the very end - which means we can still
> benefit from the rest of the patches if the DT solution remains an
> open question.
>
On Sun, 2 Feb 2014 18:04:34 +
Russell King - ARM Linux wrote:
> So, in summary, I'm pretty happy with this again - and it's all been
> tested here with no apparant detrimental effects. All committed and
> queued up here:
>
>
On Sun, 2 Feb 2014 18:23:49 +
Russell King - ARM Linux wrote:
> On Sun, Feb 02, 2014 at 07:06:06PM +0100, Jean-Francois Moine wrote:
> > On Sun, 2 Feb 2014 12:43:58 +
> > Russell King - ARM Linux wrote:
> >
> > > On Wed, Jan 29, 2014 at 10:01:22AM +0
On Sun, 2 Feb 2014 12:43:58 +
Russell King - ARM Linux wrote:
> On Wed, Jan 29, 2014 at 10:01:22AM +0100, Jean-Francois Moine wrote:
> > This patch set contains various extensions to the tda998x driver:
> >
> > - simplify the i2c read/write
> > - code cleanu
On Sun, 2 Feb 2014 16:23:09 +
Russell King - ARM Linux wrote:
> On Sat, Jan 25, 2014 at 06:14:42PM +0100, Jean-Francois Moine wrote:
> > This patch takes care of the write-only registers of the tda998x.
> >
> > The registers SOFTRESET, TBG_CNTRL_0 and TBG_CNTRL_1 have
On Sun, 2 Feb 2014 16:20:58 +
Russell King - ARM Linux wrote:
> On Sat, Jan 25, 2014 at 06:14:45PM +0100, Jean-Francois Moine wrote:
> > This patch adds more error checking inn I2C I/O functions.
> > In case of I/O error, this permits to avoid writing in bad controller
&
On Sun, 2 Feb 2014 16:20:58 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
On Sat, Jan 25, 2014 at 06:14:45PM +0100, Jean-Francois Moine wrote:
This patch adds more error checking inn I2C I/O functions.
In case of I/O error, this permits to avoid writing in bad controller
On Sun, 2 Feb 2014 16:23:09 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
On Sat, Jan 25, 2014 at 06:14:42PM +0100, Jean-Francois Moine wrote:
This patch takes care of the write-only registers of the tda998x.
The registers SOFTRESET, TBG_CNTRL_0 and TBG_CNTRL_1 have all
On Sun, 2 Feb 2014 12:43:58 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
On Wed, Jan 29, 2014 at 10:01:22AM +0100, Jean-Francois Moine wrote:
This patch set contains various extensions to the tda998x driver:
- simplify the i2c read/write
- code cleanup and fix some small
On Sun, 2 Feb 2014 18:23:49 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
On Sun, Feb 02, 2014 at 07:06:06PM +0100, Jean-Francois Moine wrote:
On Sun, 2 Feb 2014 12:43:58 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
On Wed, Jan 29, 2014 at 10:01:22AM
On Sun, 2 Feb 2014 18:04:34 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
So, in summary, I'm pretty happy with this again - and it's all been
tested here with no apparant detrimental effects. All committed and
queued up here:
On Sun, 2 Feb 2014 19:15:05 +
Russell King - ARM Linux li...@arm.linux.org.uk wrote:
In which case, it may be better to reorder the remaining patches such
that the DT changes are at the very end - which means we can still
benefit from the rest of the patches if the DT solution remains an
is now ignored.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 48 ---
include/drm/i2c/tda998x.h | 7 --
2 files changed, 50 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm
The supported audio parameters are described in the EDID which is
received by the HDMI transmitter from the connected screen.
Use these ones to adjust the audio stream parameters.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 15
include/drm/i2c/tda998x.h
In some boards, with I2S input, the NXP TDA998x HDMI transmitter did
not play audio streams with a sample width lower than S16_32.
This patch adjusts the CTS_N predivider according to the used sample
width.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 25
This patch adds the DT documentation of the NXP TDA998x CODEC.
Signed-off-by: Jean-Francois Moine
---
Documentation/devicetree/bindings/drm/i2c/tda998x.txt | 17 +
1 file changed, 17 insertions(+)
diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt
b
.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 4 +
sound/soc/codecs/Kconfig | 6 ++
sound/soc/codecs/Makefile | 2 +
sound/soc/codecs/tda998x.c| 216 ++
4 files changed, 228 insertions(+)
create
'
for the Beaglebone-Black board (Jyri Sarha)
Jean-Francois Moine (5):
drm/i2c: tda998x: add a function for dynamic audio input switch
ASoC: tda998x: add a codec driver for the TDA998x
ASoC: tda998x: add DT documentation of the tda998x CODEC
ASoC: tda998x: adjust the audio hw parameters
'
for the Beaglebone-Black board (Jyri Sarha)
Jean-Francois Moine (5):
drm/i2c: tda998x: add a function for dynamic audio input switch
ASoC: tda998x: add a codec driver for the TDA998x
ASoC: tda998x: add DT documentation of the tda998x CODEC
ASoC: tda998x: adjust the audio hw parameters
This patch adds the DT documentation of the NXP TDA998x CODEC.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
Documentation/devicetree/bindings/drm/i2c/tda998x.txt | 17 +
1 file changed, 17 insertions(+)
diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt
.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 4 +
sound/soc/codecs/Kconfig | 6 ++
sound/soc/codecs/Makefile | 2 +
sound/soc/codecs/tda998x.c| 216 ++
4 files changed, 228 insertions
The supported audio parameters are described in the EDID which is
received by the HDMI transmitter from the connected screen.
Use these ones to adjust the audio stream parameters.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 15
include/drm
In some boards, with I2S input, the NXP TDA998x HDMI transmitter did
not play audio streams with a sample width lower than S16_32.
This patch adjusts the CTS_N predivider according to the used sample
width.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c
is now ignored.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 48 ---
include/drm/i2c/tda998x.h | 7 --
2 files changed, 50 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b
The supported audio parameters are described in the EDID which is
received by the HDMI transmitter from the connected screen.
Use these ones to adjust the audio stream parameters.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 15
include/drm/i2c/tda998x.h
In some boards, with I2S input, the NXP TDA998x HDMI transmitter did
not play audio streams with a sample width lower than S16_32.
This patch adjusts the CTS_N predivider according to the used sample
width.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 25
This patch adds a CODEC driver for the NXP TDA998x HDMI transmitter.
The CODEC handles both I2S and S/PDIF input and does dynamic input
switch in the TDA998x I2C driver on audio streaming start/stop.
Signed-off-by: Jean-Francois Moine
---
sound/soc/codecs/Kconfig | 6 ++
sound/soc/codecs
This patch adds the DT documentation of the NXP TDA998x CODEC.
Signed-off-by: Jean-Francois Moine
---
Documentation/devicetree/bindings/sound/tda998x.txt | 16
1 file changed, 16 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/tda998x.txt
diff --git
'
for the Beaglebone-Black board (Jyri Sarha)
Jean-Francois Moine (5):
drm/i2c: tda998x: add a function for dynamic audio input switch
ASoC: tda998x: add a codec driver for TDA998x
ASoC: tda998x: add DT documentation
ASoC: tda998x: adjust the audio hw parameters from EDID
ASoC: tda998x: adjust
is now ignored.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 48 ---
include/drm/i2c/tda998x.h | 7 --
2 files changed, 50 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm
is now ignored.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 48 ---
include/drm/i2c/tda998x.h | 7 --
2 files changed, 50 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b
This patch adds a CODEC driver for the NXP TDA998x HDMI transmitter.
The CODEC handles both I2S and S/PDIF input and does dynamic input
switch in the TDA998x I2C driver on audio streaming start/stop.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
sound/soc/codecs/Kconfig | 6
This patch adds the DT documentation of the NXP TDA998x CODEC.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
Documentation/devicetree/bindings/sound/tda998x.txt | 16
1 file changed, 16 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound
'
for the Beaglebone-Black board (Jyri Sarha)
Jean-Francois Moine (5):
drm/i2c: tda998x: add a function for dynamic audio input switch
ASoC: tda998x: add a codec driver for TDA998x
ASoC: tda998x: add DT documentation
ASoC: tda998x: adjust the audio hw parameters from EDID
ASoC: tda998x: adjust
The supported audio parameters are described in the EDID which is
received by the HDMI transmitter from the connected screen.
Use these ones to adjust the audio stream parameters.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 15
include/drm
In some boards, with I2S input, the NXP TDA998x HDMI transmitter did
not play audio streams with a sample width lower than S16_32.
This patch adjusts the CTS_N predivider according to the used sample
width.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c
This patch adds more error checking inn I2C I/O functions.
In case of I/O error, this permits to avoid writing in bad controller
pages, a bad chipset detection or looping when getting the EDID.
Acked-by: Russell King
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu
This patch adds DT support to the tda998x.
Acked-by: Russell King
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm
This patch checks if the CEC device is well created at intialization
time.
Acked-by: Russell King
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers
The predivider division factor of the register PLL_SERIAL_2 is in the
range 0..3, the value 0 being used for a division by 1.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i2c
ca_i2s is only ever written to, but never read, so let's get rid of it.
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm
According to some tests on the Cubox (Marvell Armada 510 + TDA19988),
the S/PDIF input asks for a greater audio clock divider.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm
This patch sets the frequence as 'not indicated' instead of '48kHz'
and uses the asound values in the channel status definition.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm
This patch reduces the number of I2C exchanges by setting many bits in
one write and removing a useless write.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i2c
The picture aspect setting was zero, which is reserved.
A setting of Same As Picture makes more sense.
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b
This patch adds a definition of the values of the MUX_AP register and
simplifies the macro's defining the fields of the AIP_CLKSEL register.
This makes the format specific audio init sequence more readable.
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c
Signed-off-by: Jean-Francois Moine
---
.../devicetree/bindings/drm/i2c/tda998x.txt| 27 ++
1 file changed, 27 insertions(+)
create mode 100644 Documentation/devicetree/bindings/drm/i2c/tda998x.txt
diff --git a/Documentation/devicetree/bindings/drm/i2c/tda998x.txt
b
There is no need to enable/disable EDID read IRQ at each EDID block
read. This patch enables the IRQ at init time.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c
-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 113 +++---
1 file changed, 104 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm/i2c/tda998x_drv.c
index 5b0b6ca..7fab61a 100644
--- a/drivers/gpu/drm/i2c
This patch fixes the ENABLE_SPACE register, the value of which was
inverted.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm/i2c/tda998x_drv.c
index
The AIF has an uninitialized byte. This patch clears the whole buffer
before filling it.
Tested-by: Russell King
Acked-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm
This patch replaces hard coded values by hdmi constants.
Tested-by: Russell King
Acked-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
The cec i2c device is created in tda998x_encoder_init() when the DRM
driver starts.
This patch frees it when the DRM driver is unloaded.
Tested-by: Russell King
Acked-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 ++
1 file changed, 2 insertions
be fully set again to this value.
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 46 ---
1 file changed, 24 insertions(+), 22 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm/i2c/tda998x_drv.c
index
This patch:
- replaces ARRAY_SIZE() by sizeof() when a number of bytes is needed,
- adds a linefeed in an error message and
- removes an useless variable setting.
Acked-by: Russell King
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 13
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm/i2c/tda998x_drv.c
index 320b37f..fbd7937 100644
--- a/drivers/gpu/drm/i2c/tda998x_drv.c
This patch forces the page register to be set on the first I/O operation.
Tested-by: Russell King
Acked-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i2c
This patch prevents the system to be freezed at audio startup time,
replacing mdelay by msleep.
Tested-by: Russell King
Acked-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
register values
- add audio codec interface
Jean-Francois Moine (23):
drm/i2c: tda998x: simplify the i2c read/write functions
drm/i2c: tda998x: check more I/O errors
drm/i2c: tda998x: code cleanup
drm/i2c: tda998x: change probe message origin
drm/i2c: tda998x: don't freeze the system
This patch simplifies the i2c read/write functions and permits them to
be easily called in more contexts.
Acked-by: Russell King
Tested-by: Russell King
Signed-off-by: Jean-Francois Moine
---
drivers/gpu/drm/i2c/tda998x_drv.c | 322 +++---
1 file changed, 162
register values
- add audio codec interface
Jean-Francois Moine (23):
drm/i2c: tda998x: simplify the i2c read/write functions
drm/i2c: tda998x: check more I/O errors
drm/i2c: tda998x: code cleanup
drm/i2c: tda998x: change probe message origin
drm/i2c: tda998x: don't freeze the system
This patch simplifies the i2c read/write functions and permits them to
be easily called in more contexts.
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c
This patch forces the page register to be set on the first I/O operation.
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file changed, 1
This patch prevents the system to be freezed at audio startup time,
replacing mdelay by msleep.
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 19 ++-
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm/i2c/tda998x_drv.c
This patch:
- replaces ARRAY_SIZE() by sizeof() when a number of bytes is needed,
- adds a linefeed in an error message and
- removes an useless variable setting.
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois
The cec i2c device is created in tda998x_encoder_init() when the DRM
driver starts.
This patch frees it when the DRM driver is unloaded.
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
be fully set again to this value.
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 46 ---
1 file changed, 24 insertions(+), 22 deletions(-)
diff --git a/drivers/gpu/drm/i2c/tda998x_drv.c
b/drivers/gpu/drm/i2c
The AIF has an uninitialized byte. This patch clears the whole buffer
before filling it.
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 2 +-
1 file
This patch replaces hard coded values by hdmi constants.
Tested-by: Russell King rmk+ker...@arm.linux.org.uk
Acked-by: Russell King rmk+ker...@arm.linux.org.uk
Signed-off-by: Jean-Francois Moine moin...@free.fr
---
drivers/gpu/drm/i2c/tda998x_drv.c | 12 ++--
1 file changed, 6 insertions
501 - 600 of 992 matches
Mail list logo