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

2016-08-03 Thread Hiep Cao Minh
Hello Laurent-san, Thanks for your reply! 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 issue still happens at the Linux upstream version v4.7-rc2 Have you released the fixed patch? Please te

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 o

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 +++ > > 1 file changed, 3 insertions(+

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 sam

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 is

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 returns

[PATCH] drm: rcar-du: Link HDMI encoder with bridge

2016-08-03 Thread Laurent Pinchart
The conversion of the rcar-du driver from the I2C slave encoder to the DRM bridge API left the HDMI encoder's bridge pointer NULL, preventing the bridge from being handled automatically by the DRM core. Fix it. Fixes: 1d926114d8f4 ("drm: rcar-du: Remove i2c slave encoder interface for hdmi encode

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 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 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 [...] MBR, Sergei

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 th

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 >>> +++ b/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 --- a/drivers/me

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 a/drivers/media/

[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 b/drivers/net/ethernet/renesas/ravb_main.c index 867caf6..da

[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 eth0

[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 a

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

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

2016-08-03 Thread Hans Verkuil
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 a/drivers/medi

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 --- a/drivers/

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 b/drivers/media/i2c/adv718

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 b/drivers/media/i2c/adv7180.

Re: [PATCHv2 1/7] media: rcar-vin: make V4L2_FIELD_INTERLACED standard dependent

2016-08-03 Thread Sergei Shtylyov
On 08/02/2016 05:51 PM, Niklas Söderlund wrote: The field V4L2_FIELD_INTERLACED is standard dependent and should not unconditionally be equivalent to V4L2_FIELD_INTERLACED_TB. This patch adds a check to see if the video standard can be obtained and if it's a 60 Hz format. If the condition is me

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 >>> +++ b/drivers/media/i2c/adv7180

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 probe

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 adv7180_set_p

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

2016-08-03 Thread Sergei Shtylyov
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 probe time and when S_STD is called the driver will default to use V4L2_FIELD_INTERL

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