On 02/03/2015 01:44 PM, Mark Brown wrote:
On Tue, Feb 03, 2015 at 08:54:57AM +0100, Manuel Lauss wrote:
+wm8731-mclk = devm_clk_get(spi-dev, mclk);
+if (IS_ERR(wm8731-mclk)) {
+wm8731-mclk = NULL;
+dev_warn(spi-dev, assuming static MCLK\n);
+}
This is broken for
On 02/03/2015 06:26 PM, Lars-Peter Clausen wrote:
On 02/03/2015 06:17 PM, Russell King - ARM Linux wrote:
On Tue, Feb 03, 2015 at 05:53:48PM +0100, Lars-Peter Clausen wrote:
On 02/03/2015 01:44 PM, Mark Brown wrote:
On Tue, Feb 03, 2015 at 08:54:57AM +0100, Manuel Lauss wrote:
+wm8731
On 02/03/2015 06:17 PM, Russell King - ARM Linux wrote:
On Tue, Feb 03, 2015 at 05:53:48PM +0100, Lars-Peter Clausen wrote:
On 02/03/2015 01:44 PM, Mark Brown wrote:
On Tue, Feb 03, 2015 at 08:54:57AM +0100, Manuel Lauss wrote:
+wm8731-mclk = devm_clk_get(spi-dev, mclk);
+if (IS_ERR
On 02/03/2015 11:17 AM, Zubair Lutfullah Kakakhel wrote:
[...]
V4 Changes
Removed clock binding because of pending work in clock tree. Will add
binding later. Rather than introduce a bad binding now and change later.
But this patch is introducing a bad binding. The part needs the clock to
On 02/01/2015 11:08 AM, Jean-Michel Hautbois wrote:
Looks mostly good, some things in addition to what Hans already said.
[...]
-
-static s32 adv_smbus_write_byte_data(struct adv7604_state *state,
-enum adv7604_page page, u8 command,
-
On 02/02/2015 12:48 AM, Guenter Roeck wrote:
[...]
static int regmap_i2c_write(void *context, const void *data, size_t count)
{
struct device *dev = context;
@@ -180,7 +216,10 @@ static const struct regmap_bus *regmap_get_i2c_bus(struct
i2c_client *i2c,
else if
On 02/02/2015 10:50 AM, Nicholas Mc Guire wrote:
On Mon, 02 Feb 2015, Lars-Peter Clausen wrote:
On 02/02/2015 09:37 AM, Nicholas Mc Guire wrote:
return type of wait_for_completion_timeout is unsigned long not int, rather
than introducing a new variable the wait_for_completion_timeout is moved
On 02/02/2015 09:37 AM, Nicholas Mc Guire wrote:
return type of wait_for_completion_timeout is unsigned long not int, rather
than introducing a new variable the wait_for_completion_timeout is moved
into the if condition as the return value is only used to detect timeout.
But the return value
On 02/02/2015 09:37 AM, Nicholas Mc Guire wrote:
return type of wait_for_completion_timeout is unsigned long not int, rather
than introducing a new variable the wait_for_completion_timeout is moved
into the if condition as the return value is only used to detect timeout.
But the return value
On 02/02/2015 10:50 AM, Nicholas Mc Guire wrote:
On Mon, 02 Feb 2015, Lars-Peter Clausen wrote:
On 02/02/2015 09:37 AM, Nicholas Mc Guire wrote:
return type of wait_for_completion_timeout is unsigned long not int, rather
than introducing a new variable the wait_for_completion_timeout is moved
On 02/02/2015 12:48 AM, Guenter Roeck wrote:
[...]
static int regmap_i2c_write(void *context, const void *data, size_t count)
{
struct device *dev = context;
@@ -180,7 +216,10 @@ static const struct regmap_bus *regmap_get_i2c_bus(struct
i2c_client *i2c,
else if
On 01/26/2015 03:53 PM, Zubair Lutfullah Kakakhel wrote:
Hi,
Here are a few simple patches for the jz4740.
For 1-3:
Acked-by: Lars-Peter Clausen
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More
On 01/26/2015 03:53 PM, Zubair Lutfullah Kakakhel wrote:
Hi,
Here are a few simple patches for the jz4740.
For 1-3:
Acked-by: Lars-Peter Clausen l...@metafoo.de
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
mentioning that this is the extension of
snd_pcm_hw_constraint_list() from points to ranges. Otherwise looks good.
Reviewed-by: Lars-Peter Clausen
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More
On 01/26/2015 05:42 PM, Mark Brown wrote:
On Mon, Jan 26, 2015 at 04:24:38PM +0100, Nicolas Ferre wrote:
Le 16/01/2015 02:17, Bo Shen a écrit :
Does this end up in the i2c_driver_id driver data or do we need some
extra code when devtype is assigned to check for an of_node and look at
the DT
On 01/24/2015 02:18 PM, Jean-Francois Moine wrote:
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
On 01/26/2015 12:30 PM, Zubair Lutfullah Kakakhel wrote:
On 26/01/15 10:40, Lars-Peter Clausen wrote:
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
As we are moving away from platform to DT, we cant rely on the board
file to do this now. So enable it here.
I don't understand
On 01/26/2015 12:32 PM, Zubair Lutfullah Kakakhel wrote:
On 26/01/15 10:37, Lars-Peter Clausen wrote:
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
[...]
+Required properties:
+- compatible : "ingenic,jz4740-i2s"
+- reg : I2S registers location and length
+- clocks : A
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
As we are moving away from platform to DT, we cant rely on the board
file to do this now. So enable it here.
I don't understand this changelog. The board file never did this. The driver
enables the clock in the startup() callback.
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
[...]
+Required properties:
+- compatible : "ingenic,jz4740-i2s"
+- reg : I2S registers location and length
+- clocks : AIC and I2S PLL clock specifiers.
+- clock-names: "aic" and "i2s"
We also need a handle to the DMA channels.
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
[...]
#define JZ_AIC_I2S_FMT_DISABLE_BIT_CLK BIT(12)
+#define JZ_AIC_I2S_FMT_DISABLE_BIT_ICLK BIT(13)
This looks like it slipped in by accident. Otherwise the patch looks good.
--
To unsubscribe from this list: send the line
On 01/26/2015 05:42 PM, Mark Brown wrote:
On Mon, Jan 26, 2015 at 04:24:38PM +0100, Nicolas Ferre wrote:
Le 16/01/2015 02:17, Bo Shen a écrit :
Does this end up in the i2c_driver_id driver data or do we need some
extra code when devtype is assigned to check for an of_node and look at
the DT
.
Might be worth mentioning that this is the extension of
snd_pcm_hw_constraint_list() from points to ranges. Otherwise looks good.
Reviewed-by: Lars-Peter Clausen l...@metafoo.de
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
[...]
#define JZ_AIC_I2S_FMT_DISABLE_BIT_CLK BIT(12)
+#define JZ_AIC_I2S_FMT_DISABLE_BIT_ICLK BIT(13)
This looks like it slipped in by accident. Otherwise the patch looks good.
--
To unsubscribe from this list: send the line
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
[...]
+Required properties:
+- compatible : ingenic,jz4740-i2s
+- reg : I2S registers location and length
+- clocks : AIC and I2S PLL clock specifiers.
+- clock-names: aic and i2s
We also need a handle to the DMA channels. Currently the
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
As we are moving away from platform to DT, we cant rely on the board
file to do this now. So enable it here.
I don't understand this changelog. The board file never did this. The driver
enables the clock in the startup() callback.
On 01/26/2015 12:32 PM, Zubair Lutfullah Kakakhel wrote:
On 26/01/15 10:37, Lars-Peter Clausen wrote:
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
[...]
+Required properties:
+- compatible : ingenic,jz4740-i2s
+- reg : I2S registers location and length
+- clocks : AIC and I2S PLL
On 01/26/2015 12:30 PM, Zubair Lutfullah Kakakhel wrote:
On 26/01/15 10:40, Lars-Peter Clausen wrote:
On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
As we are moving away from platform to DT, we cant rely on the board
file to do this now. So enable it here.
I don't understand
On 01/24/2015 02:18 PM, Jean-Francois Moine wrote:
On Sat, 24 Jan 2015 12:27:35 +0100
Lars-Peter Clausen l...@metafoo.de wrote:
On 01/23/2015 07:34 PM, Jean-Francois Moine wrote:
On Fri, 23 Jan 2015 14:56:04 +0100
Lars-Peter Clausen l...@metafoo.de wrote:
On 01/23/2015 01:15 PM, Jean
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:
[...]
The DT should describe the hardware, and the simple-card mixes hardware
and software.
For example, the kirkwood controller
On 01/23/2015 07:34 PM, Jean-Francois Moine wrote:
On Fri, 23 Jan 2015 14:56:04 +0100
Lars-Peter Clausen l...@metafoo.de 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
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 create 2 CPU DAIs. With the
simple-card, the DT contains a number to reference these DAIs (for
example,
On 01/23/2015 12:34 AM, Rasmus Villemoes wrote:
The intention is obviously to sign-extend a 12 bit quantity. But
because of C's promotion rules, the assignment is equivalent to "val16
&= 0xfff;". Use the proper API for this.
Signed-off-by: Rasmus Villemoes
Acked-by: Lars
is promoted to int before the inner
shift. sign_extend32 works equally well for 8 and 16 bits types, so
use that.
Signed-off-by: Rasmus Villemoes
Acked-by: Lars-Peter Clausen
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@
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 create 2 CPU DAIs. With the
simple-card, the DT contains a number to reference these DAIs (for
example,
shift. sign_extend32 works equally well for 8 and 16 bits types, so
use that.
Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk
Acked-by: Lars-Peter Clausen l...@metafoo.de
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord
-Peter Clausen l...@metafoo.de
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
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 =
+ snd_soc_of_parse_daifmt(of_cpu, "dt-audio-card,",
+
On 01/22/2015 09:07 AM, Jean-Francois Moine wrote:
On Wed, 21 Jan 2015 21:14:07 +0100
Lars-Peter Clausen l...@metafoo.de wrote:
[...]
+ card-dai_link-dai_fmt =
+ snd_soc_of_parse_daifmt(of_cpu, dt-audio-card,,
+ NULL, NULL
[...]
+ 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 not seem to be in the bindings documentation.
--
To unsubscribe
On 01/21/2015 04:53 PM, Ricardo Ribalda Delgado wrote:
If there is more than one spi device on a device tree, the second one
will fail to probe due to a duplicated bus_num.
> [...]
This should already be fixed. See commit 4b153a2137c5 ("spi: xilinx: Use
pdev->id instead of pdev->dev.id for
On 01/21/2015 04:53 PM, Ricardo Ribalda Delgado wrote:
If there is more than one spi device on a device tree, the second one
will fail to probe due to a duplicated bus_num.
[...]
This should already be fixed. See commit 4b153a2137c5 (spi: xilinx: Use
pdev-id instead of pdev-dev.id for the
[...]
+ 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 not seem to be in the bindings documentation.
--
To unsubscribe from
:367]: (warning) %d in format
string (no. 3)
requires 'int' but the argument type is 'unsigned int'.
[drivers/staging/iio/impedance-analyzer/ad5933.c:367]: (warning) %d in format
string (no. 4)
requires 'int' but the argument type is 'unsigned int'.
Signed-off-by: Asaf Vertz
Acked-by: Lars-Peter
Acked-by: Lars-Peter Clausen l...@metafoo.de
Thanks.
---
drivers/staging/iio/impedance-analyzer/ad5933.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c
b/drivers/staging/iio/impedance-analyzer/ad5933.c
index b6bd609
On 01/20/2015 08:12 AM, Greg Kroah-Hartman wrote:
On Tue, Jan 20, 2015 at 08:05:20AM +0100, Lars-Peter Clausen wrote:
On 01/20/2015 02:41 AM, Greg Kroah-Hartman wrote:
On Mon, Jan 19, 2015 at 11:04:27PM +, Russell King - ARM Linux wrote:
On Tue, Jan 20, 2015 at 03:01:42AM +0800, Greg
On 01/20/2015 02:41 AM, Greg Kroah-Hartman wrote:
On Mon, Jan 19, 2015 at 11:04:27PM +, Russell King - ARM Linux wrote:
On Tue, Jan 20, 2015 at 03:01:42AM +0800, Greg Kroah-Hartman wrote:
On Mon, Jan 19, 2015 at 07:55:56PM +0100, Wolfram Sang wrote:
diff --git a/drivers/i2c/i2c-core.c
On 01/19/2015 07:55 PM, Wolfram Sang wrote:
[...]
Let's get rid of this code before really nobody knows/understands
anymore what this was for and if it has a subtle use.
Getting rid of this is the right thing, cause it's just not how it should be
done, but unfortunately it is not as simple
On 01/20/2015 02:41 AM, Greg Kroah-Hartman wrote:
On Mon, Jan 19, 2015 at 11:04:27PM +, Russell King - ARM Linux wrote:
On Tue, Jan 20, 2015 at 03:01:42AM +0800, Greg Kroah-Hartman wrote:
On Mon, Jan 19, 2015 at 07:55:56PM +0100, Wolfram Sang wrote:
diff --git a/drivers/i2c/i2c-core.c
On 01/20/2015 08:12 AM, Greg Kroah-Hartman wrote:
On Tue, Jan 20, 2015 at 08:05:20AM +0100, Lars-Peter Clausen wrote:
On 01/20/2015 02:41 AM, Greg Kroah-Hartman wrote:
On Mon, Jan 19, 2015 at 11:04:27PM +, Russell King - ARM Linux wrote:
On Tue, Jan 20, 2015 at 03:01:42AM +0800, Greg
On 01/19/2015 07:55 PM, Wolfram Sang wrote:
[...]
Let's get rid of this code before really nobody knows/understands
anymore what this was for and if it has a subtle use.
Getting rid of this is the right thing, cause it's just not how it should be
done, but unfortunately it is not as simple
On 01/14/2015 01:02 PM, Mark Brown wrote:
[...]
I don't think that we need to prevent module unload when a stream is active.
From a framework point of view is not different from hot-unplug. I don't
see a reason why we'd jump through hoops to actively forbid removing the
module once it works
On 01/14/2015 09:47 AM, Takashi Iwai wrote:
At Wed, 14 Jan 2015 09:15:36 +0100,
Lars-Peter Clausen wrote:
On 01/14/2015 08:43 AM, Takashi Iwai wrote:
At Tue, 13 Jan 2015 21:54:12 +,
Mark Brown wrote:
On Tue, Jan 13, 2015 at 06:24:44PM +0100, Takashi Iwai wrote:
Wang, Jiada (ESD) wrote
On 01/14/2015 09:25 AM, jiwang wrote:
Hi
On 01/14/2015 05:15 PM, Lars-Peter Clausen wrote:
On 01/14/2015 08:43 AM, Takashi Iwai wrote:
At Tue, 13 Jan 2015 21:54:12 +,
Mark Brown wrote:
On Tue, Jan 13, 2015 at 06:24:44PM +0100, Takashi Iwai wrote:
Wang, Jiada (ESD) wrote:
I am using
On 01/14/2015 08:43 AM, Takashi Iwai wrote:
At Tue, 13 Jan 2015 21:54:12 +,
Mark Brown wrote:
On Tue, Jan 13, 2015 at 06:24:44PM +0100, Takashi Iwai wrote:
Wang, Jiada (ESD) wrote:
I am using i.MX6Q sabreSD board, which have imx_wm892 machine driver, wm8962
codec and SSI CPU DAI,
I
On 01/14/2015 08:43 AM, Takashi Iwai wrote:
At Tue, 13 Jan 2015 21:54:12 +,
Mark Brown wrote:
On Tue, Jan 13, 2015 at 06:24:44PM +0100, Takashi Iwai wrote:
Wang, Jiada (ESD) wrote:
I am using i.MX6Q sabreSD board, which have imx_wm892 machine driver, wm8962
codec and SSI CPU DAI,
I
On 01/14/2015 09:25 AM, jiwang wrote:
Hi
On 01/14/2015 05:15 PM, Lars-Peter Clausen wrote:
On 01/14/2015 08:43 AM, Takashi Iwai wrote:
At Tue, 13 Jan 2015 21:54:12 +,
Mark Brown wrote:
On Tue, Jan 13, 2015 at 06:24:44PM +0100, Takashi Iwai wrote:
Wang, Jiada (ESD) wrote:
I am using
On 01/14/2015 09:47 AM, Takashi Iwai wrote:
At Wed, 14 Jan 2015 09:15:36 +0100,
Lars-Peter Clausen wrote:
On 01/14/2015 08:43 AM, Takashi Iwai wrote:
At Tue, 13 Jan 2015 21:54:12 +,
Mark Brown wrote:
On Tue, Jan 13, 2015 at 06:24:44PM +0100, Takashi Iwai wrote:
Wang, Jiada (ESD) wrote
On 01/14/2015 01:02 PM, Mark Brown wrote:
[...]
I don't think that we need to prevent module unload when a stream is active.
From a framework point of view is not different from hot-unplug. I don't
see a reason why we'd jump through hoops to actively forbid removing the
module once it works
On 01/13/2015 10:34 AM, Bo Shen wrote:
[...]
- { "AIFOUTL", "Left", "ADCL" },
- { "AIFOUTL", "Right", "ADCR" },
- { "AIFOUTR", "Left", "ADCL" },
- { "AIFOUTR", "Right", "ADCR" },
+ { "AIFOUTL Mux", "Left", "ADCL" },
+ { "AIFOUTL Mux", "Right", "ADCR" },
+
On 01/13/2015 06:36 AM, Bo Shen wrote:
Hi Lars-Perter Clausen,
On 01/12/2015 05:30 PM, Lars-Peter Clausen wrote:
On 01/12/2015 04:32 AM, Bo Shen wrote:
[...]
diff --git a/sound/soc/codecs/wm8904.c b/sound/soc/codecs/wm8904.c
index 4d2d2b1..38582d7 100644
--- a/sound/soc/codecs/wm8904.c
+++ b
On 01/13/2015 06:36 AM, Bo Shen wrote:
Hi Lars-Perter Clausen,
On 01/12/2015 05:30 PM, Lars-Peter Clausen wrote:
On 01/12/2015 04:32 AM, Bo Shen wrote:
[...]
diff --git a/sound/soc/codecs/wm8904.c b/sound/soc/codecs/wm8904.c
index 4d2d2b1..38582d7 100644
--- a/sound/soc/codecs/wm8904.c
+++ b
On 01/13/2015 10:34 AM, Bo Shen wrote:
[...]
- { AIFOUTL, Left, ADCL },
- { AIFOUTL, Right, ADCR },
- { AIFOUTR, Left, ADCL },
- { AIFOUTR, Right, ADCR },
+ { AIFOUTL Mux, Left, ADCL },
+ { AIFOUTL Mux, Right, ADCR },
+ { AIFOUTR Mux, Left, ADCL },
+
On 01/12/2015 02:54 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 17:16 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 05:14 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:54 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:50 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:41
On 01/12/2015 04:32 AM, Bo Shen wrote:
[...]
diff --git a/sound/soc/codecs/wm8904.c b/sound/soc/codecs/wm8904.c
index 4d2d2b1..38582d7 100644
--- a/sound/soc/codecs/wm8904.c
+++ b/sound/soc/codecs/wm8904.c
@@ -1076,10 +1076,10 @@ static const struct snd_soc_dapm_route
adc_intercon[] = {
{
On 01/12/2015 04:32 AM, Bo Shen wrote:
[...]
diff --git a/sound/soc/codecs/wm8904.c b/sound/soc/codecs/wm8904.c
index 4d2d2b1..38582d7 100644
--- a/sound/soc/codecs/wm8904.c
+++ b/sound/soc/codecs/wm8904.c
@@ -1076,10 +1076,10 @@ static const struct snd_soc_dapm_route
adc_intercon[] = {
{
On 01/12/2015 02:54 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 17:16 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 05:14 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:54 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:50 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:41
On 01/08/2015 05:40 PM, Karol Wrona wrote:
iio kfifo can be used without trigger support so there is no need to build it.
Signed-off-by: Karol Wrona
Looks good, not sure why this was ever selected it in the first place.
Reviewed-by: Lars-Peter Clausen
---
drivers/iio/Kconfig |1
On 01/08/2015 05:40 PM, Karol Wrona wrote:
These ones helps to create and manage iio_kfifo buffer when
no-triggered buffer is used.
[...]
+int iio_notriggered_buffer_setup(struct iio_dev *indio_dev,
+const struct iio_buffer_setup_ops *setup_ops)
+
+{
+ int
On 01/08/2015 05:40 PM, Karol Wrona wrote:
These ones helps to create and manage iio_kfifo buffer when
no-triggered buffer is used.
[...]
+int iio_notriggered_buffer_setup(struct iio_dev *indio_dev,
+const struct iio_buffer_setup_ops *setup_ops)
+
+{
+ int
On 01/08/2015 05:40 PM, Karol Wrona wrote:
iio kfifo can be used without trigger support so there is no need to build it.
Signed-off-by: Karol Wrona k.wr...@samsung.com
Looks good, not sure why this was ever selected it in the first place.
Reviewed-by: Lars-Peter Clausen l...@metafoo.de
On 01/09/2015 05:14 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:54 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:50 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:41 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:38 PM, Ivan T. Ivanov wrote:
Instead of checking whether
On 01/09/2015 04:50 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:41 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:38 PM, Ivan T. Ivanov wrote:
Instead of checking whether provider module is still
loaded on every access to device just lock module to
memory when client get reference
On 01/09/2015 04:38 PM, Ivan T. Ivanov wrote:
Instead of checking whether provider module is still
loaded on every access to device just lock module to
memory when client get reference to provider device.
This has nothing to do with the module, it's about the device. In the Linux
device
On 01/09/2015 02:52 AM, Jianqun Xu wrote:
This patch makes snd_dmaengine_pcm_register with rockchip_dmaengine_pcm_config,
which configure the parameters of period and buffer match to rockchip DMAC.
===
without rockchip_dmaengine_pcm_config, and test with command -
aplay -D
On 01/09/2015 04:38 PM, Ivan T. Ivanov wrote:
Instead of checking whether provider module is still
loaded on every access to device just lock module to
memory when client get reference to provider device.
This has nothing to do with the module, it's about the device. In the Linux
device
On 01/09/2015 04:50 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:41 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:38 PM, Ivan T. Ivanov wrote:
Instead of checking whether provider module is still
loaded on every access to device just lock module to
memory when client get reference
On 01/09/2015 05:14 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:54 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:50 PM, Ivan T. Ivanov wrote:
On Fri, 2015-01-09 at 16:41 +0100, Lars-Peter Clausen wrote:
On 01/09/2015 04:38 PM, Ivan T. Ivanov wrote:
Instead of checking whether
On 01/09/2015 02:52 AM, Jianqun Xu wrote:
This patch makes snd_dmaengine_pcm_register with rockchip_dmaengine_pcm_config,
which configure the parameters of period and buffer match to rockchip DMAC.
===
without rockchip_dmaengine_pcm_config, and test with command -
aplay -D
Cameron
Signed-off-by: Karol Wrona
Looks good, thanks.
Reviewed-by: Lars-Peter Clausen
---
drivers/iio/industrialio-buffer.c |2 ++
include/linux/iio/iio.h |6 --
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/iio/industrialio-buffer.c
b
Cameron ji...@kernel.org
Signed-off-by: Karol Wrona k.wr...@samsung.com
Looks good, thanks.
Reviewed-by: Lars-Peter Clausen l...@metafoo.de
---
drivers/iio/industrialio-buffer.c |2 ++
include/linux/iio/iio.h |6 --
2 files changed, 6 insertions(+), 2 deletions(-)
diff
On 01/07/2015 05:13 PM, Arnd Bergmann wrote:
On Wednesday 07 January 2015 15:29:36 Lars-Peter Clausen wrote:
On 01/06/2015 02:48 PM, Arnd Bergmann wrote:
On Tuesday 06 January 2015 11:45:58 Lars-Peter Clausen wrote:
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic
On 01/06/2015 02:48 PM, Arnd Bergmann wrote:
On Tuesday 06 January 2015 11:45:58 Lars-Peter Clausen wrote:
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic of shmobile DMA today, jz4740 is the only
user of the slave_id field in dma_slave_config besides shmobile. This
use
On 01/06/2015 02:48 PM, Arnd Bergmann wrote:
On Tuesday 06 January 2015 11:45:58 Lars-Peter Clausen wrote:
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic of shmobile DMA today, jz4740 is the only
user of the slave_id field in dma_slave_config besides shmobile. This
use
On 01/07/2015 05:13 PM, Arnd Bergmann wrote:
On Wednesday 07 January 2015 15:29:36 Lars-Peter Clausen wrote:
On 01/06/2015 02:48 PM, Arnd Bergmann wrote:
On Tuesday 06 January 2015 11:45:58 Lars-Peter Clausen wrote:
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic
On 01/06/2015 01:47 PM, Måns Rullgård wrote:
Lars-Peter Clausen writes:
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic of shmobile DMA today, jz4740 is the only
user of the slave_id field in dma_slave_config besides shmobile. This
use is really incompatible
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic of shmobile DMA today, jz4740 is the only
user of the slave_id field in dma_slave_config besides shmobile. This
use is really incompatible with the way that other drivers use the
dmaengine API, so we should get rid of it.
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic of shmobile DMA today, jz4740 is the only
user of the slave_id field in dma_slave_config besides shmobile. This
use is really incompatible with the way that other drivers use the
dmaengine API, so we should get rid of it.
On 01/06/2015 01:47 PM, Måns Rullgård wrote:
Lars-Peter Clausen l...@metafoo.de writes:
On 01/05/2015 11:39 PM, Arnd Bergmann wrote:
As discussed on the topic of shmobile DMA today, jz4740 is the only
user of the slave_id field in dma_slave_config besides shmobile. This
use is really
Hi,
A few small comments inline.
On 01/05/2015 12:25 PM, Inha Song wrote:
--- /dev/null
+++ b/sound/soc/samsung/trats2_wm1811.c
@@ -0,0 +1,216 @@
[...]
+#include
+#include
+#include
+#include
+#include
+#include "i2s.h"
+#include "i2s-regs.h"
You probably don't need i2s-regs.h
Hi,
A few small comments inline.
On 01/05/2015 12:25 PM, Inha Song wrote:
--- /dev/null
+++ b/sound/soc/samsung/trats2_wm1811.c
@@ -0,0 +1,216 @@
[...]
+#include linux/of.h
+#include linux/module.h
+#include linux/clk.h
+#include sound/soc.h
+#include sound/pcm_params.h
+#include i2s.h
On 12/31/2014 05:20 PM, Andrew Jackson wrote:
From: Andrew Jackson
The probe routine was disabling the clock even
if the system was configured successfully. Add
a return statement to leave clocks enabled.
Signed-off-by: Andrew Jackson
Acked-by: Lars-Peter Clausen
Thanks, never noticed
-by: Lars-Peter Clausen l...@metafoo.de
Thanks, never noticed since the clock is shared with other peripherals.
---
Spotted while reviewing clock preparation
sound/soc/adi/axi-i2s.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/sound/soc/adi/axi-i2s.c b/sound/soc/adi
Adding V4L folks to Cc for more input.
On 12/08/2014 03:10 PM, Baluta, Teodora wrote:
Hello,
On Vi, 2014-12-05 at 02:15 +, Jonathan Cameron wrote:
On 04/12/14 13:00, Teodora Baluta wrote:
This patchset adds support for fingerprint sensors through the IIO interface.
This way userspace
On 12/09/2014 08:29 PM, Karol Wrona wrote:
iio kfifo allocate/free gained their devm_ wrappers.
Change-Id: I10c19ccd7c01491caf088b3629137425ddccd29c
Signed-off-by: Karol Wrona
Suggested-by: Jonathan Cameron
Looks good to me.
[...]
+struct iio_buffer *devm_iio_kfifo_allocate(struct device
On 12/09/2014 08:29 PM, Karol Wrona wrote:
indio_dev was unused in function body plus some small style fix - add new
lines after "if(sth) return sth" and before the last return statement.
Looks good, except for the missing updates for the users.
Doing some digging in the git history raveled
r anyway.
To detect if the source clock is fixed, the driver calls clk_round_rate
for two frequencies. If the results are equal, or if the call returns
an error, the driver assumes the clock is fixed.
Signed-off-by: Mike Looijmans
Acked-by: Lars-Peter Clausen
Thanks.
--
To unsubscribe from this
clock is fixed, the driver calls clk_round_rate
for two frequencies. If the results are equal, or if the call returns
an error, the driver assumes the clock is fixed.
Signed-off-by: Mike Looijmans mike.looijm...@topic.nl
Acked-by: Lars-Peter Clausen l...@metafoo.de
Thanks.
--
To unsubscribe
On 12/09/2014 08:29 PM, Karol Wrona wrote:
indio_dev was unused in function body plus some small style fix - add new
lines after if(sth) return sth and before the last return statement.
Looks good, except for the missing updates for the users.
Doing some digging in the git history raveled
On 12/09/2014 08:29 PM, Karol Wrona wrote:
iio kfifo allocate/free gained their devm_ wrappers.
Change-Id: I10c19ccd7c01491caf088b3629137425ddccd29c
Signed-off-by: Karol Wrona k.wr...@samsung.com
Suggested-by: Jonathan Cameron ji...@kernel.org
Looks good to me.
[...]
+struct iio_buffer
1001 - 1100 of 2555 matches
Mail list logo