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

2016-08-03 Thread Takashi Sakamoto
On Jul 30 2016 07:08, Mark Brown wrote: But I think it's logically difficult to manage state of sound card; e.g. disconnect. When one sound card instance consists of instances of several 'DAI', 'Codecs' and 'Components' (this 'component' is not in ALSA core contexts[1]) and we try to unload one

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

2016-08-03 Thread Kuninori Morimoto
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 > side DAI and a DMA. The DMA was connected to the CPU DAI

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

2016-08-03 Thread Kuninori Morimoto
Hi Lars Thank you for your feedback > > 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 +++

Re: How to post cleanup patches ?

2016-08-03 Thread Kuninori Morimoto
Hi Mark > > But my concern is that you hate big-patch-set, and it is almost 70 patches. > > Can I post these patches to ML ? or should I use git pull request ? or I > > shouldn't post ? > > A very repetitive patch series which does the same thing to multiple > drivers (so is essentially the

Re: Rcar Gen2: DU: Bootlogs Error and show image error

2016-08-03 Thread Laurent Pinchart
Hello Hiep-san, (Dropping the linux-sh mailing list and adding linux-renesas-soc) On Tuesday 28 Jun 2016 10:19:20 Hiep Cao Minh wrote: > Hi Laurent-san > > Sorry to bother you again! > I would like to confirm the fixed patch for this issue on Lager. > I have retested it on Lager(Rcar-H2), The

spi-rspi mixes DMA and PIO transfers causing PIO transfer to fail.

2016-08-03 Thread Daniel Palmer
I got DMA working on the RZ/A1H and started to notice this: spi_master spi0: receive timeout What seems to be happening is that there is a DMA transfer for the SPI bus happening so the interrupts are masked. A small transfer that is smaller than the fifo size comes along so __rspi_can_dma

Re: [PATCH] pinctrl: sh-pfc: r8a7792: add VIN pin groups

2016-08-03 Thread Sergei Shtylyov
Hello. On 07/23/2016 01:24 AM, Sergei Shtylyov wrote: Add VIN[0-5] pin groups to the R8A7794 PFC driver. Oops, R8A7792, of course. Geert, have you merged this one or not yet? Signed-off-by: Sergei Shtylyov [...] MBR, Sergei

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

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

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 +++ >

Re: [PATCHv2 4/7] media: rcar-vin: fix height for TOP and BOTTOM fields

2016-08-03 Thread Sergei Shtylyov
On 08/02/2016 05:51 PM, Niklas Söderlund wrote: The height used for V4L2_FIELD_TOP and V4L2_FIELD_BOTTOM where wrong. The frames only contain one filed so the height should be half of the s/filed/field/. frame. Signed-off-by: Niklas Söderlund

Re: The failure summary report for linux stable v4.7

2016-08-03 Thread Laurent Pinchart
Hello Hiep-san, On Wednesday 03 Aug 2016 20:27:07 Hiep Cao Minh wrote: > Hi all, > > We would like to report the summary of the failure to ML, starting from > Linux stable v4.7. > > Before, we only reported each issue to the person in charge at the time > of issue detection. > > Now we think

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

2016-08-03 Thread Niklas Söderlund
On 2016-08-03 10:14:45 -0700, Steve Longerbeam wrote: > On 08/03/2016 09:58 AM, Lars-Peter Clausen wrote: > > 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

Re: [PATCH 0/3] DesignWare HDMI I2S suport

2016-08-03 Thread Mark Brown
On Tue, Aug 02, 2016 at 05:25:38AM +, Kuninori Morimoto wrote: > I understand that 1) patch will be handled by drm side maintainer. > And 2) was already accepted by you. > I believe maintainer of 3) patch is you ?? Yes. signature.asc Description: PGP signature

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

2016-08-03 Thread Steve Longerbeam
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 a8b434b..c6fed71 100644 >>> --- a/drivers/media/i2c/adv7180.c >>> +++

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

2016-08-03 Thread Steve Longerbeam
On 08/03/2016 09:58 AM, Lars-Peter Clausen wrote: > 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

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 a8b434b..c6fed71 100644 ---

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: > [...] >> diff --git

[PATCH 1/2] ravb: use SET_RUNTIME_PM_OPS macro

2016-08-03 Thread Niklas Söderlund
Use macro to define the runtime PM operations. Signed-off-by: Niklas Söderlund --- drivers/net/ethernet/renesas/ravb_main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/ethernet/renesas/ravb_main.c

[PATCH 0/2] ravb: add sleep PM suspend/resume support

2016-08-03 Thread Niklas Söderlund
Hi, This series adds sleep PM suspend/resume support to the ravb driver. It is based on v4.7 and tested on Salvator-X. I used NFS root filesystem and was able to suspend/resume and keep the system working. I also tested suspend/resume with a initramfs when the interface was down (ifconfig

[PATCH 2/2] ravb: add sleep PM suspend/resume support

2016-08-03 Thread Niklas Söderlund
The interface would not function after the system had been woken up after have been suspended (echo mem > /sys/power/state) cycle. The reason for this is that all device registers have been reset to its default values. This patch adds sleep suspend and resume functions that detached the interface

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

2016-08-03 Thread Ian Arkver
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: [...] 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: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 index a8b434b..c6fed71 100644 ---

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

2016-08-03 Thread Hans Verkuil
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 a/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 a/drivers/media/i2c/adv7180.c

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

2016-08-03 Thread Hans Verkuil
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 >>> index a8b434b..c6fed71 100644 >>> --- a/drivers/media/i2c/adv7180.c >>> +++

Re: [PATCHv2 5/7] media: rcar-vin: add support for V4L2_FIELD_ALTERNATE

2016-08-03 Thread Niklas Söderlund
On 2016-08-03 16:22:22 +0300, Sergei Shtylyov wrote: > Hello. > > On 08/02/2016 05:51 PM, Niklas Söderlund wrote: > > > The HW can capture both ODD and EVEN fields in separate buffers so it's > > possible to support V4L2_FIELD_ALTERNATE. This patch add support for > > this mode. > > > > At

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

2016-08-03 Thread Niklas Söderlund
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 a8b434b..c6fed71 100644 > > --- a/drivers/media/i2c/adv7180.c > > +++ b/drivers/media/i2c/adv7180.c > > @@ -680,10 +680,13 @@ static int

Re: How to post cleanup patches ?

2016-08-03 Thread Mark Brown
On Wed, Aug 03, 2016 at 01:12:25AM +, Kuninori Morimoto wrote: > But my concern is that you hate big-patch-set, and it is almost 70 patches. > Can I post these patches to ML ? or should I use git pull request ? or I > shouldn't post ? A very repetitive patch series which does the same thing