From: Pawel Osciak
This field is no longer used as MFC driver doesn't use vb2 alloc contexts
anymore.
Signed-off-by: Pawel Osciak
Signed-off-by: Kiran AVND
---
drivers/media/platform/s5p-mfc/s5p_mfc_common.h |3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/media/platform/s5p-m
From: Pawel Osciak
G_CTRL on V4L2_CID_MIN_BUFFERS_FOR_CAPTURE will fail if userspace happens to
query it after getting a resolution change event and before the codec has
a chance to parse the header and switch to an initialized state.
Signed-off-by: Pawel Osciak
Signed-off-by: Kiran AVND
---
From: Wu-Cheng Li
Use the requested size as the minimum bound, unless it's less than the
required hardware minimum. The bound align function will align to the
closest value but we do not want to adjust below the requested size.
Signed-off-by: Wu-Cheng Li
Signed-off-by: Kiran AVND
---
drivers/
While resolution change is detected by MFC, we flush out
older dpbs, send the resolution change event to application,
and then accept further queuing of new src buffers.
Sometimes, we error out during dpb flush because of lack of src
buffers. Since we have not started decoding new resolution yet,
From: Pawel Osciak
Interrupt result flags have to be cleared before a hardware job is run.
Otherwise, if they are cleared asynchronously, we may end up clearing them
after the interrupt for which we wanted to wait has already arrived, thus
overwriting the job results that we intended to wait for.
From: Arun Mankuzhi
If the software watchdog kicks in, we need to de-init MFC
before reloading firmware and re-intializing it again.
Signed-off-by: Arun Mankuzhi
Signed-off-by: Kiran AVND
---
drivers/media/platform/s5p-mfc/s5p_mfc.c |4
1 file changed, 4 insertions(+)
diff --git a/d
From: Arun Mankuzhi
>From MFC V8, the MFC wakeup sequence has changed.
MFC wakeup command has to be sent after the host receives
firmware load complete status from risc.
Signed-off-by: Arun Mankuzhi
Signed-off-by: Kiran AVND
---
drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c | 78 +++
Reset sequence for MFC V7 and V8 do not need RISC_ON
to be set to 0, while for MFC V6 it is still needed.
Also, remove a couple of register settings during Reset
which are not needed from V6 onwards.
Signed-off-by: Kiran AVND
---
drivers/media/platform/s5p-mfc/s5p_mfc_common.h |1 +
drivers
From: Pawel Osciak
If the software watchdog kicks in, the watchdog worker is not synchronized
with hardware interrupts and does not block other instances. It's possible
for it to clear the hw_lock, making other instances trigger a BUG() on
hw_lock checks. Since it's not fatal to clear the hw_lock
during reset sequence, it is advisable to follow the below
sequence, in order to avoid unexpected behavior from MFC
. set SFR 0x7110 MFC_BUS_RESET_CTRL 0x1
// wait for REQ_STATUS to be 1
. get SFR 0x7110 MFC_BUS_RESET_CTRL 0x3
// reset now
Signed-off-by: Kiran AVND
---
drivers/media/platform
From: Ilja Friedel
Timestamp i of a previously decoded buffer was overwritten for some
H.264 streams with timestamp i+1 of the next buffer. This happened when
encountering frame_type S5P_FIMV_DECODE_FRAME_SKIPPED, indicating no
new frame.
In most cases this wrong indexing might not have been not
From: Pawel Osciak
Handle REQBUFS(0) for CAPTURE queue as well. Also use the proper queue to call
it on for OUTPUT.
Signed-off-by: Pawel Osciak
Signed-off-by: Kiran AVND
---
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/drive
From: Prathyush K
The enter_suspend flag is set after we enter mfc suspend function but if
suspend fails after that due to any reason (like hardware timeout etc),
this flag must be cleared before returning an error. Otherwise, this
flag never gets cleared and the MFC suspend will always return an
Add V4L2_CID_MIN_BUFFERS_FOR_OUTPUT query for encoder.
Once mfc encoder state is HEAD_PARSED, which is sequence
header produced, dpb_count is avaialable. Let user space
query this value.
Signed-off-by: Kiran AVND
---
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 42 ++
Upstreaming the fixes which have gone in to Chrome OS tree for MFC driver.
Tested on MFCV8, MFCV7 and MFCV6 based Exynos5 based boards, peach-pi
(5800), peach-pit (5420) and snow (5250).
These are all independent fixes and hence posting them as a patchset.
Changes from v1:
1) Addressed all review
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 for that. I think name should be given when
Hi Antti,
I think that i2c_transfer sens no repeated start when sending only one
message per call.
At least the received eeprom content looks correct.
Regards
Matthias
On 25.09.2014 16:58, Antti Palosaari wrote:
> Reviewed-by: Antti Palosaari
>
> Please add commit description (why and how).
>
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: Fri Sep 26 04:00:16 CEST 2014
git branch: test
git hash: c0aaf696d45e2a72048a56441e81dad78659c698
gcc versi
[DVB driver for Earthsoft PT3 PCIE ISDB-S/T receiver]
Features & changes:
- 32bit + 64bit clean compile (removed math64.h)
- simpler & yet precise CNR calculation
- CNR @ 0.0001 dB (as stated in include/uapi/linux/dvb/frontend.h, not 1/1000
dB!)
- tuning enhancements (compared to PT1 DVB)
1. in
On Thu, Sep 25, 2014 at 03:24:47PM +0300, Laurent Pinchart wrote:
> If the requested clock rate passed to the XCLK set_rate or round_rate
> operation is 0, the driver will try to divide by 0. Fix this.
>
> Signed-off-by: Laurent Pinchart
Thanks!
Acked-by: Sakari Ailus
--
Sakari Ailus
e-mail:
Hello again,
I found this 2 source codes:
Debian 3.2 kernel module file: http://paste.debian.net/plain/123108
TeVii S482 official driver module file: http://paste.debian.net/plain/123109
And here is Igor commit to add S482 device to his tree:
https://bitbucket.org/liplianin/s2-liplianin-v39/comm
Thanks again,
I just write to Tevvi an email.
I notice that in the module information there is nothing about the
S482, and yes about S480:
# modinfo dvb_usb_dw2102
filename:
/lib/modules/3.2.0-4-amd64/kernel/drivers/media/dvb/dvb-usb/dvb-usb-dw2102.ko
license:GPL
version:0.1
desc
Hi Johannes and Mauro,
On 09/25/2014 12:45 PM, 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.
Mauro! Do you need anything from me for this revert?
Please let me know. Gotta be done soon I am guessing
with us so c
On 09/25/2014 12:17 PM, Johannes Stezenbach wrote:
> On Thu, Sep 25, 2014 at 11:40:45AM -0600, Shuah Khan wrote:
>>
>> Right. I introduced DVB_FE_DEVICE_RESUME code to resume
>> problems in drx39xxj driver. Because I had to make it not
>> toggle power on the fe for resume. In other words, for it
>>
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:
http://git.linuxtv.org/cgit.cgi/media_tree.
Thanks René,
I have more TeVii devices (S470 and S660) running in other machine
with no problems, just copying the firmware work out of the box.
Is really necessary to compile the Tevii driver? I have more DVB
device and want to use Debian kernels.
I try to compile it:
wget http://www.tevii.com
On Thu, Sep 25, 2014 at 11:40:45AM -0600, Shuah Khan wrote:
>
> Right. I introduced DVB_FE_DEVICE_RESUME code to resume
> problems in drx39xxj driver. Because I had to make it not
> toggle power on the fe for resume. In other words, for it
> to differentiate between disconnect and resume condition
Hi Johannes,
On 09/25/2014 11:36 AM, Johannes Stezenbach wrote:
> On Thu, Sep 25, 2014 at 06:01:34PM +0200, Johannes Stezenbach wrote:
> FWIW, there are six other xc5000 patches in the queue:
>
> http://git.linuxtv.org/cgit.cgi/media_tree.git/log/drivers/media/tuners/xc5000.c?h=devel-3.17-rc6
>
Am 25.09.2014 um 16:49 schrieb Dan Carpenter:
> On Thu, Sep 25, 2014 at 04:08:31PM +0200, Frank Schäfer wrote:
>>> ir = kzalloc(sizeof(*ir), GFP_KERNEL);
>>> + if (!ir)
>>> + return -ENOMEM;
>>> rc = rc_allocate_device();
>>> - if (!ir || !rc)
>>> + if (!rc)
>>>
On 09/25/2014 10:01 AM, Johannes Stezenbach wrote:
> Hi Shuah,
>
> On Thu, Sep 25, 2014 at 07:45:37AM -0600, Shuah Khan wrote:
>> On 09/25/2014 06:53 AM, Johannes Stezenbach wrote:
>>> ever since your patchset which implements suspend/resume
>>> for em28xx, hibernating the system breaks the Hauppa
On Thu, Sep 25, 2014 at 06:01:34PM +0200, Johannes Stezenbach wrote:
> Hi Shuah,
>
> On Thu, Sep 25, 2014 at 07:45:37AM -0600, Shuah Khan wrote:
> > On 09/25/2014 06:53 AM, Johannes Stezenbach wrote:
> > > ever since your patchset which implements suspend/resume
> > > for em28xx, hibernating the s
El 25/09/14 15:54, Frank Schäfer ha escrit:
>
> Am 23.09.2014 um 23:03 schrieb Luca Olivetti:
>> El 23/09/14 20:52, Frank Schäfer ha escrit:
>>
> This seems to be an ancient bug, which is known at least since 5 1/2
> years:
> https://lkml.org/lkml/2009/2/4/350
>> [...]
#if define
Em Thu, 25 Sep 2014 17:37:46 +0200
Julia Lawall escreveu:
> On Thu, 25 Sep 2014, Frank Schäfer wrote:
>
> > Hi Dan,
> >
> > Am 25.09.2014 um 13:39 schrieb Dan Carpenter:
> > > We call "kfree(ir->i2c_client);" in the error handling and that doesn't
> > > work if "ir" is NULL.
> > >
> > > Fixes: 7
Hi Shuah,
On Thu, Sep 25, 2014 at 07:45:37AM -0600, Shuah Khan wrote:
> On 09/25/2014 06:53 AM, Johannes Stezenbach wrote:
> > ever since your patchset which implements suspend/resume
> > for em28xx, hibernating the system breaks the Hauppauge WinTV HVR 930C
> > driver.
> > In v3.15.y and v3.16.y
Moikka
Could you look what system message log says?
Fedora uses nowadays journalctl command, but I think most of the others
just print to /var/log/messages or so. Maybe dmesg command works.
regards
Antti
On 09/25/2014 06:12 PM, Josu Lazkano wrote:
Hello all,
I want to use a new dual DVB-S2
On Thu, 25 Sep 2014, Frank Schäfer wrote:
> Hi Dan,
>
> Am 25.09.2014 um 13:39 schrieb Dan Carpenter:
> > We call "kfree(ir->i2c_client);" in the error handling and that doesn't
> > work if "ir" is NULL.
> >
> > Fixes: 78e719a5f30b ('[media] em28xx-input: i2c IR decoders: improve
> > i2c_client h
Reviewed-by: Antti Palosaari
Here is example naming those muxed adapters differently (as I mentioned
earlier patch), would make clear people who reads that code which
adapters are real and which are muxed. Now you do i2c scan for adapter
0, 1, 2, 3, without knowing 1 and 3 are actually mux se
Reviewed-by: Antti Palosaari
Antti
On 09/25/2014 08:08 AM, Matthias Schwarzott wrote:
All switching should be done by i2c mux adapters.
Drop explicit dont_use_port_3 flag.
Drop info message about switch.
Only the removed code in start_streaming is questionable:
It did switch the port_3 flag w
Reviewed-by: Antti Palosaari
I2C adapter naming is the thing here I ask you consider. After that
patch, you have 2 muxed I2C segments named I2C_1 and I2C_3. Real adapter
having these muxed adapter is I2C_1. So you reuse I2C_1 for muxed
adapter, which is possible as you don't need real adapter
I have to say I don't see very necessary to define function which
returns given adapter by adapter number.
*adapter = cx231xx_get_i2c_adap(I2C_1);
vs.
*adapter = dev->i2c_bus[I2C_1].i2c_adap;
not big difference. Anyhow, I didn't saw any bugs.
regards
Antti
On 09/25/2014 08:07 AM, Matthias Sch
So this patch adds bus number to adapter name as postfix?
"cx231xx" => "cx231xx-1"
I have no clear opinion for that. I think name should be given when
adapter is crated, not afterwards.
regards
Antti
On 09/25/2014 08:07 AM, Matthias Schwarzott wrote:
Signed-off-by: Matthias Schwarzott
---
Hello all,
I want to use a new dual DVB-S2 device, TeVii S480.
I am using Debian Wheezy with 3.2 kernel, I copy the firmware files:
# md5sum /lib/firmware/dvb-*
a32d17910c4f370073f9346e71d34b80 /lib/firmware/dvb-fe-ds3000.fw
2946e99fe3a4973ba905fcf59111cf40 /lib/firmware/dvb-usb-s660.fw
The d
On 09/25/2014 08:10 AM, Mauro Carvalho Chehab wrote:
> Hi Johannes and Shuah,
>
> Em Thu, 25 Sep 2014 07:45:37 -0600
> Shuah Khan escreveu:
>
>> Hi Johannes,
>>
>> On 09/25/2014 06:53 AM, Johannes Stezenbach wrote:
>>> Hi Shuah,
>>>
>>> ever since your patchset which implements suspend/resume
>>
Reviewed-by: Antti Palosaari
regards
Antti
On 09/25/2014 08:07 AM, Matthias Schwarzott wrote:
use already existing I2C_0 ... I2C_3
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-cards.c | 62 +++
drivers/media/usb/cx231xx/cx231xx.h
Reviewed-by: Antti Palosaari
Good catch, certainly pointless to add "dummy" I2C client per adapter.
Just wastes memory.
Missing patch description.
regards
Antti
On 09/25/2014 08:07 AM, Matthias Schwarzott wrote:
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-i2
Reviewed-by: Antti Palosaari
Please add commit description (why and how).
Some notes for further development:
It sends single messages, so you could (or even should) use
i2c_master_send/i2c_master_recv (i2c_transfer is aimed for sending
multiple messages using REPEATED START condition).
I a
tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 005f800508eb391480f463dad3d54e5b4ec67d57
commit: 7b34be71db533f3e0cf93d53cf62d036cdb5418a [media] use IS_ENABLED() macro
date: 1 year, 8 months ago
config: x86_64-randconfig-iv1-09252200 (attached as .config)
Reviewed-by: Antti Palosaari
Antti
On 09/25/2014 08:07 AM, Matthias Schwarzott wrote:
Signed-off-by: Matthias Schwarzott
---
drivers/media/usb/cx231xx/cx231xx-i2c.c | 17 +++--
drivers/media/usb/cx231xx/cx231xx.h | 2 +-
2 files changed, 12 insertions(+), 7 deletions(-)
On Thu, Sep 25, 2014 at 04:08:31PM +0200, Frank Schäfer wrote:
> > ir = kzalloc(sizeof(*ir), GFP_KERNEL);
> > + if (!ir)
> > + return -ENOMEM;
> > rc = rc_allocate_device();
> > - if (!ir || !rc)
> > + if (!rc)
> > goto error;
> >
> > /* record handles to o
(Resending because my crappy modem crashed while sending)
Am 25.09.2014 um 16:07 schrieb Mauro Carvalho Chehab:
> Em Thu, 25 Sep 2014 15:59:12 +0200
> Frank Schäfer escreveu:
>
>> Am 24.09.2014 um 01:18 schrieb Mauro Carvalho Chehab:
>>> Em Tue, 23 Sep 2014 21:32:02 +0200
>>> Frank Schäfer escre
Hi Johannes and Shuah,
Em Thu, 25 Sep 2014 07:45:37 -0600
Shuah Khan escreveu:
> Hi Johannes,
>
> On 09/25/2014 06:53 AM, Johannes Stezenbach wrote:
> > Hi Shuah,
> >
> > ever since your patchset which implements suspend/resume
> > for em28xx, hibernating the system breaks the Hauppauge WinTV
Hi Dan,
Am 25.09.2014 um 13:39 schrieb Dan Carpenter:
> We call "kfree(ir->i2c_client);" in the error handling and that doesn't
> work if "ir" is NULL.
>
> Fixes: 78e719a5f30b ('[media] em28xx-input: i2c IR decoders: improve
> i2c_client handling')
> Signed-off-by: Dan Carpenter
>
> diff --git a
Em Thu, 25 Sep 2014 15:59:12 +0200
Frank Schäfer escreveu:
>
> Am 24.09.2014 um 01:18 schrieb Mauro Carvalho Chehab:
> > Em Tue, 23 Sep 2014 21:32:02 +0200
> > Frank Schäfer escreveu:
> >
> >> Am 19.09.2014 um 18:02 schrieb Mauro Carvalho Chehab:
> >>> When both VBI and video are streaming, and
On 09/25/2014 05:40 AM, Dan Carpenter wrote:
> I moved the call to hybrid_tuner_release_state(priv) after
> "priv->firmware" dereference.
>
> Fixes: 5264a522a597 ('[media] media: tuner xc5000 - release firmwware from
> xc5000_release()')
> Signed-off-by: Dan Carpenter
>
> diff --git a/drivers/m
Am 24.09.2014 um 01:18 schrieb Mauro Carvalho Chehab:
> Em Tue, 23 Sep 2014 21:32:02 +0200
> Frank Schäfer escreveu:
>
>> Am 19.09.2014 um 18:02 schrieb Mauro Carvalho Chehab:
>>> When both VBI and video are streaming, and video stream is stopped,
>>> a subsequent trial to restart it will fail, b
Am 23.09.2014 um 23:03 schrieb Luca Olivetti:
> El 23/09/14 20:52, Frank Schäfer ha escrit:
>
This seems to be an ancient bug, which is known at least since 5 1/2 years:
https://lkml.org/lkml/2009/2/4/350
> [...]
>>> #if defined(CONFIG_MODULE) || defined(CONFIG_DVB_USB_AF9005_REMOTE)
>>
Hi Johannes,
On 09/25/2014 06:53 AM, Johannes Stezenbach wrote:
> Hi Shuah,
>
> ever since your patchset which implements suspend/resume
> for em28xx, hibernating the system breaks the Hauppauge WinTV HVR 930C driver.
> In v3.15.y and v3.16.y it throws a request_firmware warning
> during hibernat
Hi Shuah,
ever since your patchset which implements suspend/resume
for em28xx, hibernating the system breaks the Hauppauge WinTV HVR 930C driver.
In v3.15.y and v3.16.y it throws a request_firmware warning
during hibernate + resume, and the /dev/dvb/ device nodes disappears after
resume. In curre
Hi friend I am a banker in IDB BANK.I want to transfer an abandoned USD5.
5Million to your Bank account. 40/percent will be your share. No risk involved
but keep it as secret Thanks.Contact me for more details.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of
Hello.
On 9/23/2014 5:31 PM, Kamil Debski wrote:
From: linux-media-ow...@vger.kernel.org [mailto:linux-media-
ow...@vger.kernel.org] On Behalf Of Mikhail Ulyanov
Sent: Monday, August 25, 2014 2:30 PM
This patch contains driver for Renesas R-Car JPEG codec.
Cnanges since v1:
- s/g_fmt
I have a new Leadtek DTV dongle dual (usb 0413:6a05). With linux 3.16.3
the first adapter tunes and works fine, but the second tuner will only
tune to the one UHF multiplex in our area. The other five multiplexes
are all VHF and it won't lock on any of those.
Here's the kernel log from inserti
Em Mon, 22 Sep 2014 13:42:44 +0200
Hans Verkuil escreveu:
> Hi all,
>
> I have collected all suggested topics. If I missed any, then let me know.
>
> There seem to be three groups of topics: one has general topics, one is
> related to the compound controls, per frame configuration and Android,
If the requested clock rate passed to the XCLK set_rate or round_rate
operation is 0, the driver will try to divide by 0. Fix this.
Signed-off-by: Laurent Pinchart
---
drivers/media/platform/omap3isp/isp.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/platform/omap3isp/isp
We call "kfree(ir->i2c_client);" in the error handling and that doesn't
work if "ir" is NULL.
Fixes: 78e719a5f30b ('[media] em28xx-input: i2c IR decoders: improve i2c_client
handling')
Signed-off-by: Dan Carpenter
diff --git a/drivers/media/usb/em28xx/em28xx-input.c
b/drivers/media/usb/em28xx/
I moved the call to hybrid_tuner_release_state(priv) after
"priv->firmware" dereference.
Fixes: 5264a522a597 ('[media] media: tuner xc5000 - release firmwware from
xc5000_release()')
Signed-off-by: Dan Carpenter
diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c
index e4
On 25/09/14 00:27, Mauro Carvalho Chehab wrote:
> drivers/media//platform/s3c-camif/camif-capture.c: In function
> ‘camif_prepare_addr’:
> include/linux/dynamic_debug.h:64:16: warning: format ‘%x’ expects argument of
> type ‘unsigned int’, but argument 5 has type ‘dma_addr_t’ [-Wformat=]
> stat
On 25/09/14 03:24, Mauro Carvalho Chehab wrote:
> Got those warnings when compiling with gcc 4.9.1 for arm64:
>
> drivers/media/platform/exynos4-is/fimc-isp-video.c: In function
> ‘isp_video_capture_buffer_queue’:
> drivers/media/platform/exynos4-is/fimc-isp-video.c:221:4: warning: format
> ‘%x’
Hi Guennadi,
Am Samstag, den 20.09.2014, 09:24 +0200 schrieb Guennadi Liakhovetski:
> Hi Philippe,
>
> On Thu, 11 Sep 2014, Philipp Zabel wrote:
>
> > In preparation for a following patch, stop decrementing the endpoint node
> > refcount in the loop. This temporarily leaks a reference to the end
On 09/24/2014 05:57 PM, Shuah Khan wrote:
> On 09/24/2014 06:25 AM, Hans Verkuil wrote:
>> Hi Shuah,
>>
>>
>> Let's be realistic: if an application doesn't test for error codes,
>> then that's the problem of the application. Also, EBUSY will only be
>> returned if someone else is holding the tuner
tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: b94d525e58dc9638dd3f98094cb468bcfb262039
commit: 782d8b743aad7dfffa4c01e9e8b57fd35247d70a [media] dvb-frontends: use
IS_ENABLED
date: 1 year, 6 months ago
config: x86_64-randconfig-hsb1-09251420 (attached as
70 matches
Mail list logo
