Re: [alsa-devel] [PATCH 3/xx][sample] ASoC: remove codec duplicated callback function

2016-08-02 Thread kbuild test robot
Hi Kuninori, [auto build test ERROR on asoc/for-next] [also build test ERROR on v4.7 next-20160802] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Kuninori-Morimoto/ASoC-remove-codec-duplicated

Re: [alsa-devel] [PATCH 2/xx][sample] ASoC: codec duplicated callback function goes to component on wm8978

2016-08-02 Thread kbuild test robot
Hi Kuninori, [auto build test ERROR on asoc/for-next] [also build test ERROR on v4.7 next-20160802] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Kuninori-Morimoto/ASoC-codec-duplicated

[PATCH 2/2 v2][resend] drm: bridge: add DesignWare HDMI I2S audio support

2016-08-02 Thread Kuninori Morimoto
From: Kuninori Morimoto Current dw-hdmi is supporting sound via AHB bus, but it has I2S audio feature too. This patch adds I2S audio support to dw-hdmi. This HDMI I2S is supported by using ALSA SoC common HDMI encoder driver. Tested-by: Jose Abreu

[PATCH 1/2 v2][resend] ASoC: hdmi-codec: enable multi probe for same device

2016-08-02 Thread Kuninori Morimoto
From: Kuninori Morimoto hdmi-codec driver is common HDMI sound driver, but it doesn't care about multi sound ports. For example, hdmi-codec driver is supporting 1 I2S and 1 SPDIF ports, so, we can't use this driver if HDMI has 2 or more I2S ports. And we would

[PATCH 0/2 v2][resend] DesignWare HDMI I2S suport

2016-08-02 Thread Kuninori Morimoto
Hi Archit, Mark Cc Thierry, Russell These are resend of v2 of DesignWare HDMI I2S support patches. (I added drm-bridge maintainer Archit on To:) It will use ALSA SoC hdmi-codec driver, but we can't use it as-is at this point. 1) patch tidyup hdmi-codec driver to enable dw-hdmi I2S support. 2)

[PATCH v2] ASoC: simple-card-utils: add missing MODULE_xxx()

2016-08-02 Thread Kuninori Morimoto
From: Kuninori Morimoto simple-card-utils might be used as module, but MODULE_xxx() information was missed. This patch adds it. Otherwise, we will have below error, and can't use it. Specil thanks to Kevin. > insmod simple-card-utils.ko simple_card_utils:

Re: [PATCH 1/3] drm: bridge: add DesignWare HDMI I2S audio support

2016-08-02 Thread Kuninori Morimoto
Hi Daniel > > > > Mark, Thierry, Daniel > > > > I wonder who can be maintainer for this patch ?? > > > > > > It's a DRM patch so I'd expect someone in the DRM subsystem. > > > > OK, I see. > > But, I will keep Cc to you for this patch-set. > > Archit Tajena is the maintainer of last resort

[PATCH 3/xx][sample] ASoC: remove codec duplicated callback function

2016-08-02 Thread Kuninori Morimoto
codec driver and component driver has duplicated callback functions, and codec side functions are just copied to component side when register timing. This was quick-hack, but no longer needed. This patch removes codec side duplicated callback function. Signed-off-by: Kuninori Morimoto

[PATCH 2/xx][sample] ASoC: codec duplicated callback function goes to component on wm8978

2016-08-02 Thread Kuninori Morimoto
codec driver and component driver has duplicated callback functions, and codec side functions are just copied to component side when register timing. This was quick-hack, but no longer needed. This patch moves these functions from codec driver to component driver. Signed-off-by: Kuninori Morimoto

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

2016-08-02 Thread Kuninori Morimoto
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(+) diff --git

How to post cleanup patches ?

2016-08-02 Thread Kuninori Morimoto
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_drv->controls) {

Re: [PATCH 4/6] media: rcar-vin: add support for V4L2_FIELD_ALTERNATE

2016-08-02 Thread Hans Verkuil
On 08/02/2016 01:02 PM, Niklas Söderlund wrote: > On 2016-08-02 12:39:40 +0200, Hans Verkuil wrote: >> >> >> On 08/02/2016 12:32 PM, Niklas Söderlund wrote: >>> Hi Hans, >>> >>> Thanks for your feedback. >>> >>> On 2016-08-02 11:41:15 +0200, Hans Verkuil wrote: On 07/29/2016 07:40

[PATCHv2 0/7] Fix adv7180 and rcar-vin field handling

2016-08-02 Thread Niklas Söderlund
Hi, This series add V4L2_FIELD_ALTERNATE support to the rcar-vin driver and changes the field mode reported by adv7180 from V4L2_FIELD_INTERLACED to V4L2_FIELD_ALTERNATE. The change field mode reported by adv7180 was first done by Steve Longerbeam (https://lkml.org/lkml/2016/7/23/107), I have

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

2016-08-02 Thread Niklas Söderlund
From: Steve Longerbeam The ADV7180 and ADV7182 transmit whole fields, bottom field followed by top (or vice-versa, depending on detected video standard). So for chips that do not have support for explicitly setting the field mode, set the field mode to

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

2016-08-02 Thread Niklas Söderlund
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 frame. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 29

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

2016-08-02 Thread Niklas Söderlund
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 meet V4L2_FIELD_INTERLACED is treated as

[PATCHv2 6/7] media: adv7180: fill in mbus format in set_fmt

2016-08-02 Thread Niklas Söderlund
If the V4L2_SUBDEV_FORMAT_TRY is used in set_fmt the width, height etc would not be filled. Signed-off-by: Niklas Söderlund --- drivers/media/i2c/adv7180.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 1/3] drm: bridge: add DesignWare HDMI I2S audio support

2016-08-02 Thread Daniel Vetter
On Tue, Aug 02, 2016 at 12:47:46AM +, Kuninori Morimoto wrote: > > Hi Mark > > > > Mark, Thierry, Daniel > > > I wonder who can be maintainer for this patch ?? > > > > It's a DRM patch so I'd expect someone in the DRM subsystem. > > OK, I see. > But, I will keep Cc to you for this

Re: [PATCH 4/6] media: rcar-vin: add support for V4L2_FIELD_ALTERNATE

2016-08-02 Thread Niklas Söderlund
On 2016-08-02 12:39:40 +0200, Hans Verkuil wrote: > > > On 08/02/2016 12:32 PM, Niklas Söderlund wrote: > > Hi Hans, > > > > Thanks for your feedback. > > > > On 2016-08-02 11:41:15 +0200, Hans Verkuil wrote: > >> > >> > >> On 07/29/2016 07:40 PM, Niklas Söderlund wrote: > >>> The HW can

Re: [PATCH 4/6] media: rcar-vin: add support for V4L2_FIELD_ALTERNATE

2016-08-02 Thread Hans Verkuil
On 08/02/2016 12:32 PM, Niklas Söderlund wrote: > Hi Hans, > > Thanks for your feedback. > > On 2016-08-02 11:41:15 +0200, Hans Verkuil wrote: >> >> >> On 07/29/2016 07:40 PM, Niklas Söderlund wrote: >>> The HW can capture both ODD and EVEN fields in separate buffers so it's >>> possible to

Re: [PATCH 4/6] media: rcar-vin: add support for V4L2_FIELD_ALTERNATE

2016-08-02 Thread Niklas Söderlund
Hi Hans, Thanks for your feedback. On 2016-08-02 11:41:15 +0200, Hans Verkuil wrote: > > > On 07/29/2016 07:40 PM, Niklas Söderlund wrote: > > The HW can capture both ODD and EVEN fields in separate buffers so it's > > possible to support this field mode. > > > > Signed-off-by: Niklas

Re: [PATCH 4/6] media: rcar-vin: add support for V4L2_FIELD_ALTERNATE

2016-08-02 Thread Hans Verkuil
On 07/29/2016 07:40 PM, Niklas Söderlund wrote: > The HW can capture both ODD and EVEN fields in separate buffers so it's > possible to support this field mode. > > Signed-off-by: Niklas Söderlund > --- > drivers/media/platform/rcar-vin/rcar-dma.c | 26

Re: [PATCH 0/6] Fix adv7180 and rcar-vin field handling

2016-08-02 Thread Hans Verkuil
On 07/29/2016 07:40 PM, Niklas Söderlund wrote: > Hi, > > This series add V4L2_FIELD_ALTERNATE support to the rcar-vin driver and > changes the field mode reported by adv7180 from V4L2_FIELD_INTERLACED to > V4L2_FIELD_ALTERNATE. > > The change field mode reported by adv7180 was first done by

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

2016-08-02 Thread Kuninori Morimoto
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 in ASoC is because the framework has grown over

Re: [RFT PATCH v2 2/3] ARM: Fix *MIGHT_HAVE_PCI* config from being selected redundantly

2016-08-02 Thread Kishon Vijay Abraham I
Hi, On Monday 01 August 2016 04:42 PM, Kishon Vijay Abraham I wrote: > *MIGHT_HAVE_PCI* config is already selected in ARCH_MULTIPLATFORM. > Fix it from getting redundantly selected in all > ARCH_MULTIPLATFORM based machines. > > Signed-off-by: Kishon Vijay Abraham I > --- >

[PATCH 2/2 v2] drm: bridge: add DesignWare HDMI I2S audio support

2016-08-02 Thread Kuninori Morimoto
From: Kuninori Morimoto Current dw-hdmi is supporting sound via AHB bus, but it has I2S audio feature too. This patch adds I2S audio support to dw-hdmi. This HDMI I2S is supported by using ALSA SoC common HDMI encoder driver. Signed-off-by: Kuninori Morimoto

Re: [RFT PATCH v2 2/3] ARM: Fix *MIGHT_HAVE_PCI* config from being selected redundantly

2016-08-02 Thread Kishon Vijay Abraham I
On Tuesday 02 August 2016 12:40 AM, Robert Jarzmik wrote: > Kishon Vijay Abraham I writes: > >> *MIGHT_HAVE_PCI* config is already selected in ARCH_MULTIPLATFORM. >> Fix it from getting redundantly selected in all >> ARCH_MULTIPLATFORM based machines. > > If that's the purpose,

Re: [RFT PATCH v2 2/3] ARM: Fix *MIGHT_HAVE_PCI* config from being selected redundantly

2016-08-02 Thread Kishon Vijay Abraham I
Hi, On Monday 01 August 2016 08:54 PM, Russell King - ARM Linux wrote: > On Mon, Aug 01, 2016 at 04:42:29PM +0530, Kishon Vijay Abraham I wrote: >> *MIGHT_HAVE_PCI* config is already selected in ARCH_MULTIPLATFORM. >> Fix it from getting redundantly selected in all >> ARCH_MULTIPLATFORM based

[PATCH 0/2 v2] DesignWare HDMI I2S suport

2016-08-02 Thread Kuninori Morimoto
Hi Thierry, Russell, Mark These are v2 of DesignWare HDMI I2S support patches. It will use ALSA SoC hdmi-codec driver, but we can't use it as-is at this point. 1) patch tidyup hdmi-codec driver to enable dw-hdmi I2S support. 2) patch is based on it. Difficult is that these 2 patches are under