Re: [PATCH v2 3/5] [RFT] ARM: dts: wheat: Fix ADV7513 address usage

2018-02-12 Thread Lars-Peter Clausen
On 02/12/2018 07:11 PM, Kieran Bingham wrote: [...] > + /* > + * The adv75xx resets its addresses to defaults during low power power > + * mode. Because we have two ADV7513 devices on the same bus, we must > + * change both of them away from the defaults so that they do not > +

Re: [PATCH 2/2] drm: adv7511: Add support for i2c_new_secondary_device

2018-01-22 Thread Lars-Peter Clausen
On 01/22/2018 01:50 PM, Kieran Bingham wrote: > The ADV7511 has four 256-byte maps that can be accessed via the main I²C > ports. Each map has it own I²C address and acts as a standard slave > device on the I²C bus. > > Allow a device tree node to override the default addresses so that > address c

Re: [PATCH 1/4] Documentation: dt-bindings: iio: Add max961x

2017-02-24 Thread Lars-Peter Clausen
On 02/24/2017 04:22 PM, Geert Uytterhoeven wrote: > On Fri, Feb 24, 2017 at 4:05 PM, Jacopo Mondi > wrote: >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/iio/adc/max961x.txt >> @@ -0,0 +1,27 @@ >> +* Maxim max9611/max9612 current sense amplifier with 12-bits ADC interface >> + >> +Ma

Re: [PATCH] dmaengine: rcar: Fix release resources after interrupt process

2017-02-06 Thread Lars-Peter Clausen
On 02/06/2017 01:30 PM, Yoshihiro Shimoda wrote: > From: Hiroyuki Yokoyama > > This patch fixes the problem that occasionally released resources > before the end of interrupt processing. > > [ 58.156412] Unable to handle kernel NULL pointer dereference at virtual > address > [ 58.1

Re: [PATCH v4 1/2] iio: adc: Add Maxim MAX11100 driver

2017-01-15 Thread Lars-Peter Clausen
On 01/13/2017 10:50 AM, Jacopo Mondi wrote: > From: Jacopo Mondi > > Add iio driver for Maxim MAX11100 single-channel ADC. > > Signed-off-by: Jacopo Mondi > Tested-by: Marek Vasut > > --- > drivers/iio/adc/Kconfig| 9 +++ > drivers/iio/adc/Makefile | 1 + > drivers/iio/adc/max11100

Re: [alsa-devel] Question about struct snd_soc_dai() :: cpu_dai->codec

2016-08-04 Thread Lars-Peter Clausen
On 08/04/2016 04:38 AM, Kuninori Morimoto wrote: > > Hi Lars > >> I think moving forward we should get rid of the whole CPU/CODEC/platform >> concept. This is an outdated view of how the hardware looks like. When ASoC >> was initially introduce all hardware basically had a CPU side DAI, a CODEC >

Re: [alsa-devel] Question about struct snd_soc_dai() :: cpu_dai->codec

2016-08-03 Thread Lars-Peter Clausen
On 08/02/2016 08:47 AM, Kuninori Morimoto wrote: > > Hi Lars, Mark > > My previous mail was missing point... > In my opinion the flags are just as much a hack as the pointer. In an ideal setup the component does not need to know what type it is. The reason why we need this i

Re: [alsa-devel] How to post cleanup patches ?

2016-08-03 Thread Lars-Peter Clausen
On 08/03/2016 03:12 AM, Kuninori Morimoto wrote: > > Hi Mark > > Now codec driver and component driver has duplicated callback function, > and snd_soc_register_codec() just copied it to component side. > > int snd_soc_register_codec(xxx) { > ... > > if (codec_

Re: [alsa-devel] [PATCH 1/xx][sample] ASoC: add COMPONENT_FUNC() macro for snd_soc_component_driver

2016-08-03 Thread Lars-Peter Clausen
On 08/03/2016 03:13 AM, Kuninori Morimoto wrote: > snd_soc_component_driver requests some struct xxx, and int num_xxx. > To make initialize easy, this patch adds COMPONENT_FUNC() macro. > > Signed-off-by: Kuninori Morimoto > --- > include/sound/soc.h | 3 +++ > 1 file changed, 3 insertions(+) >

Re: [PATCHv2 7/7] [PATCHv5] media: adv7180: fix field type

2016-08-03 Thread Lars-Peter Clausen
On 08/03/2016 06:55 PM, Steve Longerbeam wrote: > On 08/03/2016 06:21 AM, Niklas Söderlund wrote: >> On 2016-08-02 17:00:07 +0200, Lars-Peter Clausen wrote: >>> [...] >>>> diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c >>>> index

Re: [PATCHv2 7/7] [PATCHv5] media: adv7180: fix field type

2016-08-03 Thread Lars-Peter Clausen
On 08/03/2016 04:42 PM, Ian Arkver wrote: > On 03/08/16 15:23, Lars-Peter Clausen wrote: >> On 08/03/2016 04:11 PM, Hans Verkuil wrote: >>> >>> On 08/03/2016 03:21 PM, Niklas Söderlund wrote: >>>> On 2016-08-02 17:00:07 +0200, Lars-Peter Clausen wrote: >&g

Re: [PATCHv2 7/7] [PATCHv5] media: adv7180: fix field type

2016-08-03 Thread Lars-Peter Clausen
On 08/03/2016 04:11 PM, Hans Verkuil wrote: > > > On 08/03/2016 03:21 PM, Niklas Söderlund wrote: >> On 2016-08-02 17:00:07 +0200, Lars-Peter Clausen wrote: >>> [...] >>>> diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c >>>>

Re: [PATCHv2 7/7] [PATCHv5] media: adv7180: fix field type

2016-08-03 Thread Lars-Peter Clausen
On 08/03/2016 04:23 PM, Lars-Peter Clausen wrote: > On 08/03/2016 04:11 PM, Hans Verkuil wrote: >> >> >> On 08/03/2016 03:21 PM, Niklas Söderlund wrote: >>> On 2016-08-02 17:00:07 +0200, Lars-Peter Clausen wrote: >>>> [...] >>>>> diff --git

Re: [PATCHv2 7/7] [PATCHv5] media: adv7180: fix field type

2016-08-02 Thread Lars-Peter Clausen
[...] > diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c > index a8b434b..c6fed71 100644 > --- a/drivers/media/i2c/adv7180.c > +++ b/drivers/media/i2c/adv7180.c > @@ -680,10 +680,13 @@ static int adv7180_set_pad_format(struct v4l2_subdev > *sd, > switch (format->format.

Re: [alsa-devel] Question about struct snd_soc_dai() :: cpu_dai->codec

2016-07-29 Thread Lars-Peter Clausen
On 07/29/2016 04:24 AM, Kuninori Morimoto wrote: > > Hi Lars > > Thank you for your feedback > >>> int snd_soc_runtime_set_dai_fmt(xxx) >>> { >>> ... >>> /* Flip the polarity for the "CPU" end of a CODEC<->CODEC link */ >>> if (cpu_dai->codec) { >>> ... >>> } >>>

Re: Question about struct snd_soc_dai() :: cpu_dai->codec

2016-07-29 Thread Lars-Peter Clausen
On 07/29/2016 02:30 AM, Takashi Sakamoto wrote: > Lars, > > On Jul 29 2016 05:33, Lars-Peter Clausen wrote: >> Hotplug is something that always pops up sooner or later. E.g. if someone >> puts a ASoC supported CODEC on a hot-pluggable device (maybe USB) we >> don't

Re: [alsa-devel] Question about struct snd_soc_dai() :: cpu_dai->codec

2016-07-28 Thread Lars-Peter Clausen
On 07/28/2016 10:42 PM, Takashi Iwai wrote: > On Thu, 28 Jul 2016 22:33:31 +0200, > Lars-Peter Clausen wrote: >> >> On 07/28/2016 05:46 AM, Vinod Koul wrote: >>> On Wed, Jul 27, 2016 at 10:22:41PM +0200, Takashi Iwai wrote: >>>> On Wed, 27 Jul 201

Re: [alsa-devel] Question about struct snd_soc_dai() :: cpu_dai->codec

2016-07-28 Thread Lars-Peter Clausen
On 07/28/2016 05:46 AM, Vinod Koul wrote: > On Wed, Jul 27, 2016 at 10:22:41PM +0200, Takashi Iwai wrote: >> On Wed, 27 Jul 2016 20:22:33 +0200, >> Mark Brown wrote: >>> >>> On Wed, Jul 27, 2016 at 08:11:49PM +0200, Takashi Iwai wrote: >>> I'm wondering whether it's a better option to block th

Re: [alsa-devel] Question about struct snd_soc_dai() :: cpu_dai->codec

2016-07-28 Thread Lars-Peter Clausen
On 07/26/2016 07:41 AM, Kuninori Morimoto wrote: > > Hi ALSA SoC > > My current headache is ALSA SoC's each modules (= Card/Codec/CPU/Platform) > doesn't care about "unbind/rmmod". > For example, if someone unbinded/rmmoded "Codec", Card or other modules > doesn't know about it. Thus, user can co

Re: [PATCH] adv7604: fix SPA register location for ADV7612

2016-01-19 Thread Lars-Peter Clausen
On 01/19/2016 01:17 PM, Ulrich Hecht wrote: > SPA location LSB register is at 0x70. > > Signed-off-by: Ulrich Hecht > --- > drivers/media/i2c/adv7604.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/media/i2c/adv7604.c b/drivers/media/i2c/adv7604.c > index 3787f81..f78d36c