Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
Hi Shuah, On Thu, Sep 25, 2014 at 01:03:30PM -0600, Shuah Khan wrote: On 09/25/2014 12:45 PM, Shuah Khan wrote: ok now I know why the second path didn't apply. It depends on another change that added resume function 7ab1c07614b984778a808dc22f84b682fedefea1 You don't need the

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Thu, Sep 25, 2014 at 12:45:24PM -0600, Shuah Khan wrote: Revert is good. Just checked 3.16 and we are good on that. It needs to be reverted from 3.17 for sure. ok now I know why the second path didn't apply. It depends on another change that added resume function

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 10:00:30AM +0200, Johannes Stezenbach wrote: On Thu, Sep 25, 2014 at 12:45:24PM -0600, Shuah Khan wrote: Revert is good. Just checked 3.16 and we are good on that. It needs to be reverted from 3.17 for sure. ok now I know why the second path didn't apply. It

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Hi Johannes/Shuah, Em Fri, 26 Sep 2014 10:08:24 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: On Fri, Sep 26, 2014 at 10:00:30AM +0200, Johannes Stezenbach wrote: On Thu, Sep 25, 2014 at 12:45:24PM -0600, Shuah Khan wrote: Revert is good. Just checked 3.16 and we are good on

Re: [PATCH 17/17] smiapp: Decrease link frequency if media bus pixel format BPP requires

2014-09-26 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Wednesday 17 September 2014 23:45:41 Sakari Ailus wrote: Decrease the link frequency to the next lower if the user chooses a media bus code (BPP) cannot be achieved using the selected link frequency. Signed-off-by: Sakari Ailus

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
Hi Mauro, On Fri, Sep 26, 2014 at 07:14:11AM -0300, Mauro Carvalho Chehab wrote: I just pushed the pending patched and added a reverted patch for b89193e0b06f at the media_tree.git. Could you please use it to compile or, if you prefer to keep using 3.16, you can use the media_build.git[1]

Re: [PATCH 17/17] smiapp: Decrease link frequency if media bus pixel format BPP requires

2014-09-26 Thread Sakari Ailus
Hi Laurent, Thank you for your comments. On Fri, Sep 26, 2014 at 01:44:03PM +0300, Laurent Pinchart wrote: Hi Sakari, Thank you for the patch. On Wednesday 17 September 2014 23:45:41 Sakari Ailus wrote: Decrease the link frequency to the next lower if the user chooses a media bus code

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 13:07:27 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: Hi Mauro, On Fri, Sep 26, 2014 at 07:14:11AM -0300, Mauro Carvalho Chehab wrote: I just pushed the pending patched and added a reverted patch for b89193e0b06f at the media_tree.git. Could you please use

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 08:42:15AM -0300, Mauro Carvalho Chehab wrote: Could you please try this patch (untested): [media] drxk: load firmware again at resume No joy. I think you need to keep the firmware around for reuse after resume instead of requesting it again. [2.521597] PM: Image

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 08:42:15 -0300 Mauro Carvalho Chehab mche...@osg.samsung.com escreveu: Em Fri, 26 Sep 2014 13:07:27 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: Hi Mauro, On Fri, Sep 26, 2014 at 07:14:11AM -0300, Mauro Carvalho Chehab wrote: I just pushed the pending

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 14:02:33 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: On Fri, Sep 26, 2014 at 08:42:15AM -0300, Mauro Carvalho Chehab wrote: Could you please try this patch (untested): [media] drxk: load firmware again at resume No joy. I think you need to keep the

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 09:03:16AM -0300, Mauro Carvalho Chehab wrote: The patch I sent you (or some fixed version of it) is part of the solution, but this still bothers me: [3.776854] [813f974f] drxk_attach+0x546/0x656 [3.777675] [814c22a3]

Re: [PATCH 04/12] cx231xx: give each master i2c bus a seperate name

2014-09-26 Thread Antti Palosaari
On 09/26/2014 07:34 AM, Matthias Schwarzott wrote: On 25.09.2014 17:04, Antti Palosaari wrote: So this patch adds bus number to adapter name as postfix? cx231xx = cx231xx-1 Yes, it is attached, and the result looks like * cx231xx #0-0 * cx231xx #0-1 * cx231xx #0-2 I have no clear opinion

Re: [PATCH 02/12] cx231xx: use own i2c_client for eeprom access

2014-09-26 Thread Antti Palosaari
yes, of course :) If you has only one message, there is nothing to start again. Sending one message using i2c_transfer() means same than using i2c_master_send(), but the later one is just aimed to send single message and it cannot be used to send multiple messages at once. Code is correct,

[PATCH] [media] rc: fix hix5hd2 compile-test issue

2014-09-26 Thread Zhangfei Gao
Add dependence to solve build error in arch like ia64 error: implicit declaration of function 'readl_relaxed' 'writel_relaxed' Change CONFIG_PM to CONFIG_PM_SLEEP to solve warning: 'hix5hd2_ir_suspend' 'hix5hd2_ir_resume' defined but not used Reported-by: Fengguang Wu fengguang...@intel.com

Re: [PATCH 02/12] cx231xx: use own i2c_client for eeprom access

2014-09-26 Thread Antti Palosaari
I decided to check how eeprom I2C interface works from datasheet. Chip keeps internally current address value and read operations are done for that. Basically both reads works, as chip keeps count of current read address internally. That datasheet has nice figures from use cases:

Re: [PATCH] [media] rc: fix hix5hd2 compile-test issue

2014-09-26 Thread Russell King - ARM Linux
On Fri, Sep 26, 2014 at 08:57:30PM +0800, Zhangfei Gao wrote: Add dependence to solve build error in arch like ia64 error: implicit declaration of function 'readl_relaxed' 'writel_relaxed' Change CONFIG_PM to CONFIG_PM_SLEEP to solve warning: 'hix5hd2_ir_suspend' 'hix5hd2_ir_resume'

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 14:27:21 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: On Fri, Sep 26, 2014 at 09:03:16AM -0300, Mauro Carvalho Chehab wrote: The patch I sent you (or some fixed version of it) is part of the solution, but this still bothers me: [3.776854]

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 10:12:22AM -0300, Mauro Carvalho Chehab wrote: Try to add a WARN_ON or printk at em28xx_usb_resume(). It is called two times, once during hibernate and once during resume: root@debian:~# echo disk /sys/power/state [ 107.108149] PM: Syncing filesystems ... done. [

Re: [PATCH v2 1/6] V4L2: Add Renesas R-Car JPEG codec driver.

2014-09-26 Thread Laurent Pinchart
Hi Mikhail, Thank you for the patch. On Monday 25 August 2014 16:29:47 Mikhail Ulyanov wrote: This patch contains driver for Renesas R-Car JPEG codec. Cnanges since v1: - s/g_fmt function simplified - default format for queues added - dumb vidioc functions added to be in

Re: [PATCH] [media] rc: fix hix5hd2 compile-test issue

2014-09-26 Thread zhangfei
On 09/26/2014 09:12 PM, Russell King - ARM Linux wrote: On Fri, Sep 26, 2014 at 08:57:30PM +0800, Zhangfei Gao wrote: Add dependence to solve build error in arch like ia64 error: implicit declaration of function 'readl_relaxed' 'writel_relaxed' Change CONFIG_PM to CONFIG_PM_SLEEP to solve

Re: [PATCH 1/1] media: Set entity-links NULL in cleanup

2014-09-26 Thread Laurent Pinchart
Hi Sakari, On Wednesday 24 September 2014 12:17:45 Sakari Ailus wrote: Hi Laurent, Oops. this got buried in my inbox... Laurent Pinchart wrote: On Thursday 17 July 2014 14:53:49 Sakari Ailus wrote: On Thu, Jul 17, 2014 at 01:43:09PM +0200, Laurent Pinchart wrote: On Tuesday 27 May

Re: em28xx breaks after hibernate

2014-09-26 Thread Shuah Khan
On 09/26/2014 07:25 AM, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 10:12:22AM -0300, Mauro Carvalho Chehab wrote: Try to add a WARN_ON or printk at em28xx_usb_resume(). It is called two times, once during hibernate and once during resume: root@debian:~# echo disk /sys/power/state

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 03:25:13PM +0200, Johannes Stezenbach wrote: (snipped some irrelevant part of resume) Looking closer, I snipped too much: [1.646784] Freeing unused kernel memory: 1080K (8800018f2000 - 880001a0) Loading, please wait... [1.655328] busybox (67) used

Re: em28xx breaks after hibernate

2014-09-26 Thread Shuah Khan
On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 03:25:13PM +0200, Johannes Stezenbach wrote: (snipped some irrelevant part of resume) Looking closer, I snipped too much: [1.646784] Freeing unused kernel memory: 1080K (8800018f2000 - 880001a0)

Payment

2014-09-26 Thread Finance Department
Dear Recipient, You have been awarded the sum of 8,000,000.00 (Eight Million Pounds sterling) with reference number 77100146 by office of the ministry of finance UK.Send us your personal details to deliver your funds. Gloria Peter -- To unsubscribe from this list: send the line unsubscribe

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 08:30:32AM -0600, Shuah Khan wrote: On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: So, what is happening is that the em28xx driver still async initializes while the initramfs already has started resume. Thus the rootfs in not mounted and the firmware is not

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 05:06:02PM +0200, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 08:30:32AM -0600, Shuah Khan wrote: On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: So, what is happening is that the em28xx driver still async initializes while the initramfs already has

Re: em28xx breaks after hibernate

2014-09-26 Thread Shuah Khan
On 09/26/2014 09:06 AM, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 08:30:32AM -0600, Shuah Khan wrote: On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: So, what is happening is that the em28xx driver still async initializes while the initramfs already has started resume. Thus the

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 09:22:53 -0600 Shuah Khan shua...@osg.samsung.com escreveu: On 09/26/2014 09:06 AM, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 08:30:32AM -0600, Shuah Khan wrote: On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: So, what is happening is that the em28xx

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 09:22:53 -0600 Shuah Khan shua...@osg.samsung.com escreveu: On 09/26/2014 09:06 AM, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 08:30:32AM -0600, Shuah Khan wrote: On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: So, what is happening is that the em28xx

Re: em28xx breaks after hibernate

2014-09-26 Thread Johannes Stezenbach
On Fri, Sep 26, 2014 at 09:22:53AM -0600, Shuah Khan wrote: [ 20.212162] usb 1-1: reset high-speed USB device number 2 using ehci-pci [ 20.503868] em2884 #0: Resuming extensions [ 20.505275] em2884 #0: Resuming video extensionem2884 #0: Resuming DVB extension [ 20.533513]

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 17:22:28 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: On Fri, Sep 26, 2014 at 05:06:02PM +0200, Johannes Stezenbach wrote: On Fri, Sep 26, 2014 at 08:30:32AM -0600, Shuah Khan wrote: On 09/26/2014 08:25 AM, Johannes Stezenbach wrote: So, what is

Re: em28xx breaks after hibernate

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 17:42:27 +0200 Johannes Stezenbach j...@linuxtv.org escreveu: On Fri, Sep 26, 2014 at 09:22:53AM -0600, Shuah Khan wrote: [ 20.212162] usb 1-1: reset high-speed USB device number 2 using ehci-pci [ 20.503868] em2884 #0: Resuming extensions [ 20.505275]

Re: linux-next: Tree for Sep 26 (media/pci/pt3)

2014-09-26 Thread Randy Dunlap
On 09/26/14 04:10, Stephen Rothwell wrote: Hi all, There will be no linux-next release on Monday. This has not been a good day :-( Changes since 20140925: on x86_64: when CONFIG_MODULES is not enabled: ../drivers/media/pci/pt3/pt3.c: In function 'pt3_attach_fe':

[PATCH] [media] Documentation: FE_SET_PROPERTY requires R/W

2014-09-26 Thread Mauro Carvalho Chehab
FE_SET_PROPERTY requires to open the devnode on R/W mode, or otherwise it will fail. Document it. Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com --- Documentation/DocBook/media/dvb/dvbproperty.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: linux-next: Tree for Sep 26 (media/pci/pt3)

2014-09-26 Thread Mauro Carvalho Chehab
Em Fri, 26 Sep 2014 10:01:47 -0700 Randy Dunlap rdun...@infradead.org escreveu: On 09/26/14 04:10, Stephen Rothwell wrote: Hi all, There will be no linux-next release on Monday. This has not been a good day :-( Changes since 20140925: on x86_64: when CONFIG_MODULES is not

[PATCH] media, platform, LLVMLinux: Remove nested function from ti-vpe

2014-09-26 Thread Behan Webster
Replace the use of nested functions where a normal function will suffice. Nested functions are not liked by upstream kernel developers in general. Their use breaks the use of clang as a compiler, and doesn't make the code any better. This code now works for both gcc and clang. Signed-off-by:

[PATCH] pt3: fix DTV FE I2C driver load error paths

2014-09-26 Thread Antti Palosaari
Get rid of 'module_is_live' usage. on x86_64: when CONFIG_MODULES is not enabled: ../drivers/media/pci/pt3/pt3.c: In function 'pt3_attach_fe': ../drivers/media/pci/pt3/pt3.c:433:6: error: implicit declaration of function 'module_is_live' [-Werror=implicit-function-declaration] Reported-by:

Re: problem with second tuner on Leadtek DTV dongle dual

2014-09-26 Thread Hamish Moffatt
On 26/09/14 04:44, Antti Palosaari wrote: Moikka Performance issues are fixed recently (at least I hope so), but it will took some time in order to get fixes in stable. Unfortunately I don't have any IT9135 BX (ver 2 chip) dual device to test like yours... Could you install that kernel tree:

Re: problem with second tuner on Leadtek DTV dongle dual

2014-09-26 Thread Antti Palosaari
On 09/27/2014 04:49 AM, Hamish Moffatt wrote: On 26/09/14 04:44, Antti Palosaari wrote: Moikka Performance issues are fixed recently (at least I hope so), but it will took some time in order to get fixes in stable. Unfortunately I don't have any IT9135 BX (ver 2 chip) dual device to test like

cron job: media_tree daily build: ERRORS

2014-09-26 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sat Sep 27 04:00:18 CEST 2014 git branch: test git hash: 214635f94dc3e4069b05817e5d55b58784ba8971 gcc

Re: problem with second tuner on Leadtek DTV dongle dual

2014-09-26 Thread Hamish Moffatt
On 27 September 2014 12:02:14 PM AEST, Antti Palosaari cr...@iki.fi wrote: On 09/27/2014 04:49 AM, Hamish Moffatt wrote: On 26/09/14 04:44, Antti Palosaari wrote: Moikka Performance issues are fixed recently (at least I hope so), but it will took some time in order to get fixes in stable.