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: Sun Aug 10 04:00:27 CEST 2014
git branch: test
git hash: 0f3bf3dc1ca394a8385079a5653088672b65c5c4
gcc versi
There are still a few bugs that can happen when suspending and
a video stream is active. This patch series fix them. After
that, resume works fine, even it suspend happened while
streaming.
There is one remaining issue though: xc5000 firmware doesn't
load after resume.
What happens (on both analo
Now that xc5000_set_params() is also called during resume,
move the code that checks for the firmware to happen there.
This way, the firmware will be loaded either for analog or
digital TV when .resume callback is called.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/tuners/xc5000.c |
When DVB is streaming and suspend is called, it will call
au0828_stop_transport(), with will clean the streaming flag.
Due to that, stop_urb_transfer() will be called twice,
causing an oops.
So, we need another flag to be used at resume, telling it
to restart DVB.
While here, add a logic at stop
dev->dvb is always not null, as it is an area at the dev
memory. So, checking if (dev->dvb) is always true.
Instead of this stupid check, what the code wants to do is
to know if the DVB was successully registered.
Fix it by checking, instead, for dvb->frontend. It should
also be sure that this va
As we'll be adding a code to resume tuner operation, we
need to move the code that actually sets DTV on a separate
function, to be called by the resume code.
No functional changes, just code got moved.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/tuners/xc5000.c | 79 +
It doesn't make sense to handle an IR code given before
suspending after the device resume. So, turn off IR
int while suspending.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-input.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/media/usb/au0828/au
As the RC kthread can re-enable IR int, we should first
cancel the kthread and then disable IR int.
While here, remove a temporary debug printk.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-input.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
While several tuners have some sort of suspend/resume
implementation, this is currently mangled with an optional
.sleep callback that it is also used to put the device on
low power mode.
Not all drivers implement it, as returning the driver from
low power may require to re-load the firmware, with
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-dvb.c | 0
drivers/media/usb/au0828/au0828-input.c | 2 +-
2 files changed, 1 insertion(+), 1 deletion(-)
mode change 100755 => 100644 drivers/media/usb/au0828/au0828-dvb.c
diff --git a/drivers/media/usb/au0828/au0828-dv
Suspend/resume can be very tricky. That's the second attempt to fix
it with au0828.
With this patchset, suspend/resume to ram works fine if the device
is not being used. After resume, IR, digital TV and analog TV
keeps working.
On my tests, however, trying to suspend while watching TV caused
my t
As we need a function that reapply the last tuned radio,
in order to do resume, split the code that validates and
updates the internal priv struct from the ones that
actually set radio and TV.
A latter patch will add support for resume.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/tun
Instead of using printk(KERN_foo, use pr_foo() macros.
No functional changes.
Note: we should do the same for dprintk(), but that would
require to remove the dprintk levels. So, for now, let's
not touch on it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-cards.c | 1
At resume, we should restore the register contents. So,
reenable the bridge and GPIO settings.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-core.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/media/usb/au0828/au0828-core.c
b/drivers/media/usb/au0
xc5000_set_params() is a bad name for a function that
handles only digital TV. Rename it to xc5000_set_digital_params(),
and proper name the generic function that works for both
digital and analog.
No functional changes.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/tuners/xc5000.c | 1
The STREAM_ON state is used by s_format callback,
but the driver never sets it.
Fix it. This will also be needed in order to handle
suspend/resume ops.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-video.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers
From: Shuah Khan
Remove CONFIG_VIDEO_AU0828_RC scope around au0828_rc_register()
and au0828_rc_unregister() calls in au0828-core
Signed-off-by: Shuah Khan
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-core.c | 4
1 file changed, 4 deletions(-)
diff --git a/dri
No timers should be enabled during suspend. So,
stop them. At resume time, we should do the proper
initialization for it to keep working.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-core.c | 2 ++
drivers/media/usb/au0828/au0828-video.c | 59 +++
The scheduled work should be cancelled during suspend.
At resume time, we need to set the frontend again. So,
add such logic to the driver.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-core.c | 2 ++
drivers/media/usb/au0828/au0828-dvb.c | 31 ++
Suspend/resume conditions can be very tricky. Add some info
printk's to help tracking what's happening there.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828-core.c | 4
drivers/media/usb/au0828/au0828-dvb.c | 4
drivers/media/usb/au0828/au0828-input.c | 4 +
After xc5000 stops working, it waits for 5 seconds, waiting
for a new usage. Only after that it goes to low power mode.
If a suspend event happens before that, a work queue will
remain active, with causes suspend to crash.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/tuners/xc5000.c |
If a device suspends/hibertates with a station tuned, restore
the tuner station at resume.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/tuners/xc5000.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c
index 78695ed4549c..
Trying to make au0828 to suspend can do very bad things, as
the polling Kthread is not handled. We should disable it
during suspend, only re-enabling it at resume.
Still, analog and digital TV won't work, as we don't reinit
the settings at resume, but at least it won't hang.
Signed-off-by: Mauro
From: Shuah Khan
Define au0828_rc_*() stubs to avoid compile errors when
VIDEO_AU0828_RC is disabled and avoid the need to enclose
au0828_rc_*() in ifdef CONFIG_VIDEO_AU0828_RC in .c files.
Signed-off-by: Shuah Khan
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/usb/au0828/au0828.h |
Reviewed-by: Antti Palosaari
your code is easy to read!
Antti
On 08/08/2014 10:06 AM, Olli Salonen wrote:
Read MAC address from the EEPROM.
Signed-off-by: Olli Salonen
---
drivers/media/usb/dvb-usb/cxusb.c | 37 +
1 file changed, 37 insertions(+)
diff
Reviewed-by: Antti Palosaari
cxusb_ctrl_msg() uses USB buffers from the stack which is no-no. But it
is old mistake...
regards
Antti
On 08/08/2014 10:06 AM, Olli Salonen wrote:
TechnoTrend TT-connect CT2-4650 CI (0b48:3012) is an USB DVB-T2/C tuner with
the following components:
USB inte
Reviewed-by: Antti Palosaari
Antti
On 08/08/2014 10:06 AM, Olli Salonen wrote:
Driver for the CIMaX SP2 common interface chip. It is very much based on
the existing cimax2 driver for cx23885, but should be more reusable. The
product has been sold with name Atmel T90FJR as well and the data she
Reviewed-by: Antti Palosaari
Antti
On 08/08/2014 10:06 AM, Olli Salonen wrote:
Signed-off-by: Olli Salonen
---
drivers/media/dvb-core/dvb-usb-ids.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/dvb-core/dvb-usb-ids.h
b/drivers/media/dvb-core/dvb-usb-ids.h
index 5135a09
Looks fine!
regards
Antti
On 08/06/2014 07:38 AM, nibble.max wrote:
> add dvbsky rc keymaps.
>
> Signed-off-by: Nibble Max
> ---
> drivers/media/rc/keymaps/Makefile| 1 +
> drivers/media/rc/keymaps/rc-dvbsky.c | 78
>
> 2 files changed, 79 insertio
Moikka!
I reviewed that quickly. I noticed one real issue; USB control message
buffers. USB buffers are not allowed to taken from the stack as it does
not work every supported architecture. Easiest way is to put buffers to
state as state is allocated memory. In that case you have to take
accou
Looks fine
Acked-by: Antti Palosaari
Reviewed-by: Antti Palosaari
On 08/06/2014 07:34 AM, nibble.max wrote:
> change em28xx-dvb.c following the m88ds3103 config change
>
> Signed-off-by: Nibble Max
> ---
> drivers/media/usb/em28xx/em28xx-dvb.c | 4 +++-
> 1 file changed, 3 insertions(+), 1
On 08/08/2014 06:50 AM, nibble.max wrote:
Add ts clock and clock polarity, lnb set voltage.
+static int m88ds3103_set_voltage(struct dvb_frontend *fe,
+ fe_sec_voltage_t voltage)
+{
+ struct m88ds3103_priv *priv = fe->demodulator_priv;
+ u8 data;
+
+ dev_dbg(&priv->i2c-
I applied that patch too, but reverted register writes you removed /
moved to af9033 demod driver. Also, register write added for attach()
happens too late leaving tuner ops populated even it fails. That causes
surely kernel crash when tuner attach fails and DVB-core sees tuner
callbacks are po
I applied that too, but removed those register writes you added to
af9033 driver. Patch commit message was totally missing and I did a
*lot* of reverse-engineering in order to understand patch (actually all
these patches).
What I discovered, not sure if correct, but that patch updates IT9135 B
I applied that to my tree.
However, patch description could be nice. For example if that is same
firmware delivered with latest windows drivers...
regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord
AF903x/IT913x demod provides clock source(s). It seems that this
clock source is used for integrated RF tuner of IT913x. It is
enabled by default, but firmware disables it automatically when
suspend is requested (suspend_flag (0x004c) + trigger_ofsm
(0x)). Automatic disable behavior seems to be
From: Bimow Chen
That register is needed to program very first in order to operate
correctly.
[cr...@iki.fi: returned sequence back, removed sleep, moved reg
write earlier to prevent populating tuner ops in case of failure]
Signed-off-by: Bimow Chen
Signed-off-by: Antti Palosaari
---
drivers
Error on init leaves some internal divisor zero, which causes oops
later. Fix it by populating divisors even it fails.
Signed-off-by: Antti Palosaari
---
drivers/media/tuners/tuner_it913x.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/tuners/tuner_it913x.c
b/drivers/medi
Correct issues reported by checkpatch.pl.
Signed-off-by: Antti Palosaari
---
drivers/media/dvb-frontends/af9033.c | 30 ++
1 file changed, 18 insertions(+), 12 deletions(-)
diff --git a/drivers/media/dvb-frontends/af9033.c
b/drivers/media/dvb-frontends/af9033.c
inde
From: Bimow Chen
Update IT9135 BX tuner config 60 and 61 inittabs.
[cr...@iki.fi: removed two reg writes from driver init itself]
Signed-off-by: Bimow Chen
Signed-off-by: Antti Palosaari
---
drivers/media/dvb-frontends/af9033_priv.h | 20 +---
1 file changed, 9 insertions(+),
From: Malcolm Priestley
add the following IDs
USB_PID_PCTV_78E (0x025a) for PCTV 78e
USB_PID_PCTV_79E (0x0262) for PCTV 79e
For these it9135 devices.
Signed-off-by: Malcolm Priestley
Cc: Antti Palosaari
Cc: # v3.14+
Signed-off-by: Antti Palosaari
---
drivers/media/dvb-core/dvb-usb-ids.h |
Reverts commit 3ab25123373270152a9fae98e3c48ef1b2a878c0
[media] af9035: override tuner for AVerMedia A835B devices
Original commit itself is correct, but it was replaced by more
general solution (commit 1cbbf90d0406913ad4b44194b07f4f41bde84e54).
This old solution was committed by a accident and is
Old IT9135 AX needs a little bit different register settings for
sleep than newer IT9135 BX. This has been broken always, as power
management of the whole driver, but it started to be problem as I
fixed clock. Earlier clock was disabled very first on sleep and
rest of the commands were skipped by t
IT913x tuner driver disables own clock, provided by demod core, as
very a first operation when tuner is put on *sleep*. That likely
causes failure of all the rest commands on sleep sequence, which
leads situation where tuner is not actually on sleep, but consuming
a lot of power.
I measured 102mA
Some changes related to IT9135 chip versions. Mostly for better
sensitivity but also power management. I am considering to send
some of these to stable +3.15 too, but not sure yet...
These patches are available on Git tree also:
http://git.linuxtv.org/cgit.cgi/anttip/media_tree.git/log/?h=af9035
Correct issues reported by checkpatch.pl
Signed-off-by: Antti Palosaari
---
drivers/media/tuners/tuner_it913x.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/media/tuners/tuner_it913x.c
b/drivers/media/tuners/tuner_it913x.c
index 281d8c5..b92d599 100644
--- a/drivers/media/tun
Correct issues reported by checkpatch.pl.
Signed-off-by: Antti Palosaari
---
drivers/media/usb/dvb-usb-v2/af9035.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/dvb-usb-v2/af9035.c
b/drivers/media/usb/dvb-usb-v2/af9035.c
index 85f2c4b..f37cf7d 1006
From: Bimow Chen
IT9135 firmware update.
Signed-off-by: Bimow Chen
Signed-off-by: Antti Palosaari
---
Documentation/dvb/get_dvb_firmware | 24 +---
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/Documentation/dvb/get_dvb_firmware
b/Documentation/dvb/get_d
Integrated RF tuner of IT9135 is connected to demod clock source
named dyn0_clk. Enable that clock source in order to provide stable
clock early enough.
Cc: Bimow Chen
Signed-off-by: Antti Palosaari
---
drivers/media/usb/dvb-usb-v2/af9035.c | 9 ++---
1 file changed, 6 insertions(+), 3 dele
IT9135 RF tuner clock is coming from demodulator. We need enable it
early in demod init, before any tuner I/O. Currently it is enabled
by tuner driver itself, but it is too late and performance will be
reduced as some registers are not updated correctly. Clock is
disabled automatically when demod i
On 08/09/2014 07:36 PM, Guennadi Liakhovetski wrote:
> Hi Suman,
>
> On Sat, 9 Aug 2014, Suman Kumar wrote:
>
>> Fixes a coding style issue reported by checkpatch.pl
>
> Thanks for your patch. To my taste checkpatch.pl has unfortunately become
> too noisy with meaningless / unimportant warn
Hi Suman,
On Sat, 9 Aug 2014, Suman Kumar wrote:
> Fixes a coding style issue reported by checkpatch.pl
Thanks for your patch. To my taste checkpatch.pl has unfortunately become
too noisy with meaningless / unimportant warnings like this one. Is this
in CodingStyle? If not, my intention is
Fixes a coding style issue reported by checkpatch.pl
Signed-off-by: Suman Kumar
---
drivers/media/platform/soc_camera/soc_camera_platform.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c
b/drivers/media/platform/soc_camera/soc_c
Fixes a coding style issue reported by checkpatch.pl
Signed-off-by: Suman Kumar
---
drivers/media/platform/soc_camera/soc_camera_platform.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c
b/drivers/media/platform/soc_camera/soc_c
Fixes a coding style issue reported by checkpatch.pl
Signed-off-by: Suman Kumar
---
drivers/media/platform/soc_camera/soc_camera_platform.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/platform/soc_camera/soc_camera_platform.c
b/drivers/media/platform/soc_camera/soc_c
On 08/07/2014 06:36 PM, Frank Schäfer wrote:
>
> Am 07.08.2014 um 08:45 schrieb Hans Verkuil:
>> On 08/05/2014 05:18 PM, Frank Schäfer wrote:
>>> Hi Hans,
>>>
>>> Am 05.08.2014 um 09:00 schrieb Hans Verkuil:
Fix three compiler warnings:
drivers/media/usb/em28xx/em28xx-input.c: In fu
The correct field order in progressive mode is V4L2_FIELD_NONE, not
V4L2_FIELD_INTERLACED.
Cc:
Signed-off-by: Frank Schäfer
---
drivers/media/usb/em28xx/em28xx-video.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/media/usb/em28xx/em28xx-video.c
b/drivers/med
When a new video frame is started, the driver takes the next video buffer from
the list of active buffers and moves it to dev->usb_ctl.vid_buf /
dev->usb_ctl.vbi_buf
for further processing.
On streaming stop we currently only give back the pending buffers from the list
but not the ones which are
58 matches
Mail list logo
