op 11-04-14 21:30, Thomas Hellstrom schreef:
Hi!
On 04/11/2014 08:09 PM, Maarten Lankhorst wrote:
op 11-04-14 12:11, Thomas Hellstrom schreef:
On 04/11/2014 11:24 AM, Maarten Lankhorst wrote:
op 11-04-14 10:38, Thomas Hellstrom schreef:
Hi, Maarten.
Here I believe we encounter a lot of
op 11-04-14 21:35, Thomas Hellstrom schreef:
On 04/11/2014 08:09 PM, Maarten Lankhorst wrote:
op 11-04-14 12:11, Thomas Hellstrom schreef:
On 04/11/2014 11:24 AM, Maarten Lankhorst wrote:
op 11-04-14 10:38, Thomas Hellstrom schreef:
Hi, Maarten.
Here I believe we encounter a lot of locking
On 04/14/2014 09:42 AM, Maarten Lankhorst wrote:
op 11-04-14 21:35, Thomas Hellstrom schreef:
On 04/11/2014 08:09 PM, Maarten Lankhorst wrote:
op 11-04-14 12:11, Thomas Hellstrom schreef:
On 04/11/2014 11:24 AM, Maarten Lankhorst wrote:
op 11-04-14 10:38, Thomas Hellstrom schreef:
Hi,
8-bit reads are needed in some cases; however the condition used was wrong.
Regular access (register width) was used if:
len == SMIAPP_REG_8BIT !only8
This causes 8-bit read access to be used always. The operator should be ||
instead: regular access can be used for 8-bit reads OR if
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/i2c/smiapp/smiapp-core.c
b/drivers/media/i2c/smiapp/smiapp-core.c
index 02041cc..3af8df8 100644
---
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-quirk.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/media/i2c/smiapp/smiapp-quirk.h
b/drivers/media/i2c/smiapp/smiapp-quirk.h
index 4f65c4e..96a253e 100644
---
This is more flexible. Quirk flags may be affected by configuration.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-core.c | 4 ++--
drivers/media/i2c/smiapp/smiapp-quirk.c | 7 ++-
drivers/media/i2c/smiapp/smiapp-quirk.h | 2 +-
3 files
Hi folks,
This is the second version of the smiapp and smiapp-pll quirk improvement
patchset. The first version can be found here:
URL:http://www.spinics.net/lists/linux-media/msg75538.html
Changes since v1:
- Fix a
For some sensors in some configurations the effective value of op clk div is
bits per pixel divided by two. The output clock is correctly calculated
whereas some of the rest of the clock tree uses higher clocks than
calculated. This also limits the bpp to even values if the number of lanes
is
Limits may exceed the value range of 32-bit unsigned integers. Thus use 64
bits for all of them.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.c | 72 +++---
drivers/media/i2c/smiapp-pll.h | 20 ++--
2 files
The quirk registers mechanism which allows register to have a static read
access value from the sensor specific quirks, is not used. Remove it. It is
to be replaced by a more generic register diversion quirk soon.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
It doesn't make sense to just copy the information to the PLL flags. Add a
new fields for the quirks to contain the PLL flags.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-core.c | 5 ++---
drivers/media/i2c/smiapp/smiapp-quirk.c | 2 +-
The actual pixel array pixel rate may be something else than vt_pix_clk_freq
on some implementations. Add a new field which contains the corrected value.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.c | 1 +
drivers/media/i2c/smiapp-pll.h | 1 +
2
Some sensors support odd pre-pll divisor.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.c | 39 ++-
drivers/media/i2c/smiapp-pll.h | 2 ++
2 files changed, 28 insertions(+), 13 deletions(-)
diff --git
No reason to keep this u8, really.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/i2c/smiapp-pll.h b/drivers/media/i2c/smiapp-pll.h
index a4a6498..5ce2b61 100644
---
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/smiapp/smiapp-core.c
b/drivers/media/i2c/smiapp/smiapp-core.c
index 6d940f0..284df17 100644
---
The sub-device names should be unique. Should two identical sensors be
present in the same media device they would be indistinguishable. The names
will change e.g. from vs6555 pixel array to vs6555 1-0010 pixel array.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
Some sensors have effectively the double pixel (and other clocks) compared
to calculations.
The frequency of the bus is also affected similarly so take this into
account when calculating pll_op_clock frequency.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
These values are unsigned, so use %u instead of %d.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.c | 94 +-
1 file changed, 47 insertions(+), 47 deletions(-)
diff --git a/drivers/media/i2c/smiapp-pll.c
The register address, width and flags are encoded as a 32-bit value. Add
macros for obtaining these separately. Use the macros in register access
functions.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-regs.c | 13 +++--
Add a quirk for diverting registers for on some sensors, even the standard
registers are not where they can be expected to be found. Add a quirk to
to help using such sensors.
smiapp_write_no_quirk() and smiapp_read_no_quirk() functions are provided
for the use of quirk implementations.
Limits may exceed the value range of 32 bit unsigned integers. Thus use 64
bits instead.
Use typed min/max/clamp macros. Debug printing changes as well.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-core.c | 30 --
SMIAPP_REG_ is the common prefix used in the driver for register related
definitions. Use it consistently.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp/smiapp-quirk.c| 2 +-
drivers/media/i2c/smiapp/smiapp-reg-defs.h | 8
op_pix_clk_div is directly assigned and not calculated. There's no need to
verify it.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/media/i2c/smiapp-pll.c
The PLL flags were not used correctly.
Signed-off-by: Sakari Ailus sakari.ai...@linux.intel.com
---
drivers/media/i2c/smiapp-pll.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/smiapp-pll.c b/drivers/media/i2c/smiapp-pll.c
index 2335529..ab5d9a3 100644
Hi Mauro,
Various fixes, most notable are the vb2 fixes and davinci improvements.
Regards,
Hans
The following changes since commit a83b93a7480441a47856dc9104bea970e84cda87:
[media] em28xx-dvb: fix PCTV 461e tuner I2C binding (2014-03-31 08:02:16
-0300)
are available in the git
On Apr 13, Alan Stern wrote:
On Sun, 13 Apr 2014, Sander Eikelenboom wrote:
Hi,
I'm hitting this warning on boot with a syntek usb video grabber, it's not
clear
to me if it's a driver issue of the stk1160 or a generic ehci issue.
It is a bug in the stk1160 driver.
Thanks for
On Mon, 14 Apr 2014, at 1:34, Laurent Pinchart wrote:
On Thursday 13 March 2014 22:08:36 William Manley wrote:
On 13/03/14 17:03, Laurent Pinchart wrote:
On Thursday 13 March 2014 10:48:20 Will Manley wrote:
On Thu, 13 Mar 2014, at 10:23, Laurent Pinchart wrote:
On Wednesday 12 March
Hi Mauro,
Various fixes, most notable are the vb2 fixes, saa7134 vb2 conversion and
davinci
improvements.
This pull request supersedes https://patchwork.linuxtv.org/patch/23123/ and
https://patchwork.linuxtv.org/patch/23596/. The saa7134 vb2 conversion patches
had to be rebased on top of the
On Apr 13, Sander Eikelenboom wrote:
I'm hitting this warning on boot with a syntek usb video grabber, it's not
clear
to me if it's a driver issue of the stk1160 or a generic ehci issue.
Can't reproduce the same warning easily here. Could you test the following
patch?
diff --git
Hi,
From: linux-media-ow...@vger.kernel.org [mailto:linux-media-
ow...@vger.kernel.org] On Behalf Of Mauro Carvalho Chehab
Sent: Wednesday, March 19, 2014 8:02 PM
As discussed on our IRC #v4l channels, most of the core developers will
be in San Jose - CA - USA for the Embedded Linux
Hi Jonathan,
This loop in ov7670_enum_framesizes():
for (i = 0; i n_win_sizes; i++) {
struct ov7670_win_size *win = info-devtype-win_sizes[index];
if (info-min_width win-width info-min_width)
continue;
if
Commit 75e2bdad8901a0b599e01a96229be922eef1e488 ov7670: allow
configuration of image size, clock speed, and I/O method uses a wrong
index to iterate an array. Apart from being wrong, it also uses an
unchecked value from user-space, which can cause access to unmapped
memory in the kernel, triggered
On Mon, 14 Apr 2014 14:50:15 +0200 (CEST)
Guennadi Liakhovetski g.liakhovet...@gmx.de wrote:
If any of the above if statements is true, it will
stay true forever, until the loop terminates. If that's intended, you
could at least use break immediately. If it's not - something else is
wrong
Hi Jacek,
Thanks for the update! Some comments below. I'll try to reply to the rest
during the coming days.
On Fri, Apr 11, 2014 at 04:56:52PM +0200, Jacek Anaszewski wrote:
Some LED devices support two operation modes - torch and
flash. This patch provides support for flash LED devices
in
This patch adds support for the Hauppauge ImpactVCB-e card to cx23885.
Tested with Composite input and S-Video.
While I do get audio it is very choppy. It is not clear whether that is
a general cx23885 driver problem or specific to this board. If it is specific
to the board, then I might have
Hi Hans,
While I do get audio it is very choppy. It is not clear whether that is
a general cx23885 driver problem or specific to this board. If it is specific
to the board, then I might have missed something.
Steven (Toth, not Cookson ;-) ), do you have an idea what it might be?
Nothing
From: Lad, Prabhakar prabhakar.cse...@gmail.com
this patch adds support to release the buffer by calling
vb2_buffer_done(), with state marked as VB2_BUF_STATE_QUEUED
if start_streaming() call back fails.
Signed-off-by: Lad, Prabhakar prabhakar.cse...@gmail.com
---
Adds support for limitation of maximal pixel clock of HDMI
signal. This feature is needed on boards that contains
lines or bridges with frequency limitations.
Signed-off-by: Tomasz Stanislawski t.stanisl...@samsung.com
---
.../devicetree/bindings/video/exynos_hdmi.txt |4
This patch eliminates redundant checks while retrieving HPD gpio from DT during
HDMI's probe().
Signed-off-by: Tomasz Stanislawski t.stanisl...@samsung.com
---
drivers/gpu/drm/exynos/exynos_hdmi.c | 13 -
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git
Hi everyone,
This patchset adds 4 fixes/updates to EXYNOS DRM driver for
HDMI subsystem.
All comments are welcome.
Regards,
Tomasz Stanislawski
Tomasz Stanislawski (4):
drm: exynos: hdmi: simplify extracting hpd-gpio from DT
drm: exynos: mixer: fix using usleep() in atomic context
drm:
This patch add proper compatibles for Mixer and HDMI chip
available on exynos4210 SoCs.
Signed-off-by: Tomasz Stanislawski t.stanisl...@samsung.com
---
drivers/gpu/drm/exynos/exynos_hdmi.c |3 +++
drivers/gpu/drm/exynos/exynos_mixer.c |3 +++
2 files changed, 6 insertions(+)
diff --git
This patch fixes calling usleep_range() after taking reg_slock
using spin_lock_irqsave(). The mdelay() is used instead.
Waiting in atomic context is not the best idea in general.
Hopefully, waiting occurs only when Video Processor fails
to reset correctly.
Signed-off-by: Tomasz Stanislawski
On 04/14/2014 05:00 PM, Tomasz Stanislawski wrote:
This patch eliminates redundant checks while retrieving HPD gpio from DT
during
HDMI's probe().
Signed-off-by: Tomasz Stanislawski t.stanisl...@samsung.com
---
drivers/gpu/drm/exynos/exynos_hdmi.c | 13 -
1 file changed, 4
On Apr 14, Ezequiel Garcia wrote:
On Apr 13, Sander Eikelenboom wrote:
I'm hitting this warning on boot with a syntek usb video grabber, it's not
clear
to me if it's a driver issue of the stk1160 or a generic ehci issue.
Can't reproduce the same warning easily here. Could you
Currently stk1160_read_reg() uses a stack-allocated char to get the
read control value. This is wrong because usb_control_msg() requires
a kmalloc-ed buffer, and a DMA-API warning is produced:
WARNING: CPU: 0 PID: 1376 at lib/dma-debug.c:1153 check_for_stack+0xa0/0x100()
ehci-pci :00:0a.0:
Hi Mauro,
This includes a compilation error fix for s5c73m3 driver related to
recently merged patch series moving the v4l2-of code and a memory
allocation bug fix for the Exynos FIMC driver.
The following changes since commit c9eaa447e77efe77b7fa4c953bd62de8297fd6c5:
Linux 3.15-rc1
Monday, April 14, 2014, 6:41:05 PM, you wrote:
Currently stk1160_read_reg() uses a stack-allocated char to get the
read control value. This is wrong because usb_control_msg() requires
a kmalloc-ed buffer, and a DMA-API warning is produced:
WARNING: CPU: 0 PID: 1376 at lib/dma-debug.c:1153
On Thu, Apr 10, 2014 at 2:18 PM, Guennadi Liakhovetski
g.liakhovet...@gmx.de wrote:
Hi Bryan,
On Tue, 8 Apr 2014, Bryan Wu wrote:
Thanks Josh, I think I will take you point and rework my patch again.
But I need Guennadi's review firstly, Guennadi, could you please help
to review it?
Ok,
On Mon, 14 Apr 2014, Ezequiel Garcia wrote:
Currently stk1160_read_reg() uses a stack-allocated char to get the
read control value. This is wrong because usb_control_msg() requires
a kmalloc-ed buffer, and a DMA-API warning is produced:
WARNING: CPU: 0 PID: 1376 at lib/dma-debug.c:1153
On 14/04/14 18:14, Bryan Wu wrote:
On Thu, Apr 10, 2014 at 2:18 PM, Guennadi Liakhovetski
g.liakhovet...@gmx.de wrote:
Hi Bryan,
On Tue, 8 Apr 2014, Bryan Wu wrote:
Thanks Josh, I think I will take you point and rework my patch again.
But I need Guennadi's review firstly, Guennadi, could you
On Apr 14, Alan Stern wrote:
On Mon, 14 Apr 2014, Ezequiel Garcia wrote:
Currently stk1160_read_reg() uses a stack-allocated char to get the
read control value. This is wrong because usb_control_msg() requires
a kmalloc-ed buffer, and a DMA-API warning is produced:
WARNING: CPU: 0
On Fri 11-04-14 08:58:59, Hans Verkuil wrote:
On 04/11/2014 12:18 AM, Jan Kara wrote:
On Thu 10-04-14 23:57:38, Jan Kara wrote:
On Thu 10-04-14 14:22:20, Hans Verkuil wrote:
On 04/10/14 14:15, Jan Kara wrote:
On Thu 10-04-14 13:07:42, Hans Verkuil wrote:
On 04/10/14 12:32, Jan Kara
On 04/10/2014 08:39 AM, Mauro Carvalho Chehab wrote:
Em Thu, 10 Apr 2014 12:46:53 +0100
One Thousand Gnomes gno...@lxorguk.ukuu.org.uk escreveu:
For example, some devices provide standard USB Audio Class, handled by
snd-usb-audio for the audio stream, while the video stream is handled
via a
Hi Tomasz,
On 04/15/2014 12:00 AM, Tomasz Stanislawski wrote:
This patch add proper compatibles for Mixer and HDMI chip
available on exynos4210 SoCs.
Signed-off-by: Tomasz Stanislawski t.stanisl...@samsung.com
---
drivers/gpu/drm/exynos/exynos_hdmi.c |3 +++
On Mon, 27 Jan 2014, Laurent Pinchart wrote:
Hi Thomas,
On Monday 27 January 2014 09:54:58 Thomas Pugliese wrote:
On Mon, 27 Jan 2014, Laurent Pinchart wrote:
On Friday 24 January 2014 15:17:28 Thomas Pugliese wrote:
Isochronous endpoints on devices with speed == USB_SPEED_WIRELESS
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: Tue Apr 15 04:00:26 CEST 2014
git branch: test
git hash: 277a163c83d7ba93fba1e8980d29a9f8bfcfba6c
gcc
My name is Dr.Paul Sawadogo from Burkina Faso please I have a cancer patient
in Coma who wants me to transfer her fortune worth 4.7 Million Dollars to
you.Please get back to me for more details.
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to
smatch says:
drivers/media/usb/s2255/s2255drv.c:2246 s2255_probe() warn:
possible memory leak of 'dev'
Signed-off-by: Daeseok Youn daeseok.y...@gmail.com
---
drivers/media/usb/s2255/s2255drv.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git
59 matches
Mail list logo