Small update: I've added Matt Gomboc's patch to this series as well:
http://patchwork.linuxtv.org/patch/17026/
Regards,
Hans
On Fri February 15 2013 09:46:59 Hans Verkuil wrote:
> This patch series cleans up the cx231xx driver based on v4l2-compliance
> reports.
>
> It is identical to
On Tue, 5 Mar 2013, Sachin Kamat wrote:
> module_platform_driver_probe() eliminates the boilerplate and simplifies
> the code.
>
> Signed-off-by: Sachin Kamat
Thanks, all 3 queued for 3.10
Regards
Guennadi
> ---
> drivers/media/platform/sh_veu.c | 13 +
> 1 files changed, 1 ins
Hi,
I have been working on LinuxDVB port, where the application wants to switch
dynamically from FRONT0 to DVR0 as DEMUX0 source, time and again.
The obvious way to handle this is to use DMX_SET_SOURCE which
connects/disconnects the FRONT0/DVR0 to DEMUX0.
Before implementing that, I would like
module_platform_driver_probe() eliminates the boilerplate and simplifies
the code.
Signed-off-by: Sachin Kamat
---
drivers/media/platform/sh_vou.c | 13 +
1 files changed, 1 insertions(+), 12 deletions(-)
diff --git a/drivers/media/platform/sh_vou.c b/drivers/media/platform/sh_vou
module_platform_driver_probe() eliminates the boilerplate and simplifies
the code.
Signed-off-by: Sachin Kamat
---
drivers/media/platform/soc_camera/mx1_camera.c | 13 +
1 files changed, 1 insertions(+), 12 deletions(-)
diff --git a/drivers/media/platform/soc_camera/mx1_camera.c
module_platform_driver_probe() eliminates the boilerplate and simplifies
the code.
Signed-off-by: Sachin Kamat
---
drivers/media/platform/sh_veu.c | 13 +
1 files changed, 1 insertions(+), 12 deletions(-)
diff --git a/drivers/media/platform/sh_veu.c b/drivers/media/platform/sh_veu
binV6uiUv4OS6.bin
Description: iso-8859-1
binQR42zVOSJh.bin
Description: iso-8859-1
Em Mon, 04 Mar 2013 22:24:43 +0100
Frank Schäfer escreveu:
> Am 04.03.2013 20:14, schrieb Mauro Carvalho Chehab:
> > 3) It doesn't properly address the real issue: a separate I2C
> > register is needed for bus B.
>
> Definitely. :(
> We talked about that at the beginning.
>
> I have spent
Em Mon, 04 Mar 2013 21:00:17 -0300
Alfredo Jesús Delaiti escreveu:
> Hi all
>
> El 04/03/13 16:42, Mauro Carvalho Chehab escribió:
> > Em Sun, 3 Mar 2013 13:40:51 -0300
> > Mauro Carvalho Chehab escreveu:
> >
> >> Em Sun, 03 Mar 2013 11:50:25 -0300
> >> Alfredo Jesús Delaiti escreveu:
> >>
> >
Hi all
El 04/03/13 16:42, Mauro Carvalho Chehab escribió:
Em Sun, 3 Mar 2013 13:40:51 -0300
Mauro Carvalho Chehab escreveu:
Em Sun, 03 Mar 2013 11:50:25 -0300
Alfredo Jesús Delaiti escreveu:
The new data replacement in mb86a20s
/*
* Initialization sequence: Use whatevere default value
Hi Mauro and others
El 03/03/13 13:15, Mauro Carvalho Chehab escribió:
Em Sun, 03 Mar 2013 11:50:25 -0300
Alfredo Jesús Delaiti escreveu:
Hi Mauro and others from the list
I searched for a plan B to get the data bus and after several
alternative plans that were available to me was to do a l
Hi Albert
A general comment first: I have no idea about this hardware, so, feel free
to ignore all my hardware-handling related comments. But just from looking
your handling of the pll1 clock does seem a bit fishy to me. You acquire
and release the clock in the generic mcam code, but only use i
Am 04.03.2013 22:31, schrieb Frank Schäfer:
> ...
> I don't expect this chip to appear in one of the devices with the
> currently supported generic IDs.
Hmm... maybe I should add:
it doesn't make sense to add the generic USB ID of this chip (eb1a:2765)
to the driver, because most of the devices u
Am 04.03.2013 21:23, schrieb Mauro Carvalho Chehab:
> Em Mon, 4 Mar 2013 17:20:05 -0300
> Mauro Carvalho Chehab escreveu:
>
>> Em Sun, 3 Mar 2013 20:40:57 +0100
>> Frank Schäfer escreveu:
...
>>> @@ -277,7 +386,9 @@ static int em28xx_i2c_xfer(struct i2c_adapter *i2c_adap,
>>>
Am 04.03.2013 20:14, schrieb Mauro Carvalho Chehab:
> Em Sun, 3 Mar 2013 20:37:39 +0100
> Frank Schäfer escreveu:
>
>> The current code first configures register EM28XX_R06_I2C_CLK, which includes
>> i2c speed, ack, wait and (on some devices) i2c bus selection.
>> The register value usually comes
m68k/sun3:
drivers/media/v4l2-core/videobuf2-dma-contig.c: In function ‘vb2_dc_mmap’:
drivers/media/v4l2-core/videobuf2-dma-contig.c:204: error: implicit declaration
of function ‘dma_mmap_coherent’
drivers/media/v4l2-core/videobuf2-dma-contig.c: In function
‘vb2_dc_get_base_sgt’:
drivers/media/v
Em Mon, 4 Mar 2013 17:20:05 -0300
Mauro Carvalho Chehab escreveu:
> Em Sun, 3 Mar 2013 20:40:57 +0100
> Frank Schäfer escreveu:
>
> > The webcam "SpeedLink VAD Laplace" (em2765 + ov2640) uses a special
> > algorithm
> > for i2c communication with the sensor, which is connected to a second i2c
Em Sun, 3 Mar 2013 20:40:57 +0100
Frank Schäfer escreveu:
> The webcam "SpeedLink VAD Laplace" (em2765 + ov2640) uses a special algorithm
> for i2c communication with the sensor, which is connected to a second i2c bus.
>
> We don't know yet how to find out which devices support/use it.
> It's v
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: Mon Mar 4 19:00:45 CET 2013
git branch: for_v3.9
git hash: d34c353cd40ff0e6e5ed3753e80f6aa7eaa2c298
gcc ve
Em Sun, 3 Mar 2013 13:40:51 -0300
Mauro Carvalho Chehab escreveu:
> Em Sun, 03 Mar 2013 11:50:25 -0300
> Alfredo Jesús Delaiti escreveu:
>
>
> > The new data replacement in mb86a20s
> >
> > /*
> > * Initialization sequence: Use whatevere default values that PV SBTVD
> > * does on its init
Now that some devices initialize register 0x2a with different
values, add the calculus formula, instead of hardcoding it.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-frontends/mb86a20s.c | 26 --
drivers/media/dvb-frontends/mb86a20s.h | 8 ++--
2 files
Up to now, the driver was simply assuming TV mode, 13 segs.
Implement the logic to control the ISDB operational mode.
Signed-off-by: Mauro Carvalho Chehab
---
drivers/media/dvb-frontends/mb86a20s.c | 74 +-
1 file changed, 63 insertions(+), 11 deletions(-)
diff -
Em Sun, 3 Mar 2013 20:37:39 +0100
Frank Schäfer escreveu:
> The current code first configures register EM28XX_R06_I2C_CLK, which includes
> i2c speed, ack, wait and (on some devices) i2c bus selection.
> The register value usually comes from the board definition.
> em28xx_i2c_register() is calle
Em Mon, 04 Mar 2013 19:27:12 +0100
Frank Schäfer escreveu:
> Am 04.03.2013 19:09, schrieb Mauro Carvalho Chehab:
> > Em Sun, 3 Mar 2013 20:37:34 +0100
> > Frank Schäfer escreveu:
> >
> >> Reduces the number of characters/lines, unifies the code and improves
> >> readability.
> > Had you actual
Am 04.03.2013 19:09, schrieb Mauro Carvalho Chehab:
> Em Sun, 3 Mar 2013 20:37:34 +0100
> Frank Schäfer escreveu:
>
>> Reduces the number of characters/lines, unifies the code and improves
>> readability.
> Had you actually test this patch? The reason why printk() is used on some
> places is bec
Em Sun, 3 Mar 2013 20:37:34 +0100
Frank Schäfer escreveu:
> Reduces the number of characters/lines, unifies the code and improves
> readability.
Had you actually test this patch? The reason why printk() is used on some
places is because dev->name is not available early.
That's said, it makes
Since commit 7543f344e9b06afe86b55a2620f5c11b38bd5642 ("[media] m920x:
factor out a m920x_write_seq() function") building m920x.o triggers this
GCC warning:
drivers/media/usb/dvb-usb/m920x.c: In function ‘m920x_probe’:
drivers/media/usb/dvb-usb/m920x.c:91:6: warning: ‘ret’ may be used
unin
From: Hans Verkuil
The dv_preset API is deprecated and is replaced by the much improved dv_timings
API. Remove the dv_preset support from this driver as this will allow us to
remove the dv_preset API altogether (s5p-tv being the last user of this code).
Signed-off-by: Hans Verkuil
Tested-by: To
From: Hans Verkuil
The dv_preset API is deprecated and is replaced by the much improved dv_timings
API. Remove the dv_preset support from this driver as this will allow us to
remove the dv_preset API altogether (s5p-tv being the last user of this code).
Signed-off-by: Hans Verkuil
Tested-by: To
From: Hans Verkuil
The dv_preset API is deprecated and is replaced by the much improved dv_timings
API. Remove the dv_preset support from this driver as this will allow us to
remove the dv_preset API altogether (s5p-tv being the last user of this code).
Signed-off-by: Hans Verkuil
Tested-by: To
From: Hans Verkuil
This just adds dv_timings support without modifying existing dv_preset
support.
Signed-off-by: Hans Verkuil
Tested-by: Tomasz Stanislawski
Cc: Kyungmin Park
---
drivers/media/platform/s5p-tv/mixer_video.c | 80 +++
1 file changed, 80 insertions(+)
From: Hans Verkuil
This just adds dv_timings support without modifying existing dv_preset
support.
Signed-off-by: Hans Verkuil
Tested-by: Tomasz Stanislawski
Cc: Kyungmin Park
---
drivers/media/platform/s5p-tv/hdmi_drv.c | 94 +-
1 file changed, 81 insertions(+)
Hi Tomasz,
Here is what I hope is the final patch series for this. I've incorporated
your suggestions and it's split off from the davinci/blackfin changes into
its own patch series to keep things better organized.
The changes since the previous version are:
- changed the order of the first three
From: Hans Verkuil
This just adds dv_timings support without modifying existing dv_preset
support, although I had to refactor a little bit in order to share
hdmiphy_find_conf() between the preset and timings code.
Signed-off-by: Hans Verkuil
Tested-by: Tomasz Stanislawski
Cc: Kyungmin Park
--
Hi Hans,
Thanks for the fix!
On Mon, Mar 4, 2013 at 4:49 PM, Hans Verkuil wrote:
> From: Hans Verkuil
>
> drivers/media/platform/davinci/dm644x_ccdc.c: In function
> ‘validate_ccdc_param’:
> drivers/media/platform/davinci/dm644x_ccdc.c:233:32: warning: comparison
> between ‘enum ccdc_gama_wid
Hi Hans,
Thanks for the patch!
On Mon, Mar 4, 2013 at 4:49 PM, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
Regards,
--Prabhakar Lad
> ---
> drivers/media/platform/davinci/dm355_ccdc.c | 10 +-
> drivers/media/platform/da
于 2013/3/4 17:30, Kamil Debski 写道:
Hi,
This problem is known to us and Marek is planning a fix. However, the
problem
proved to be quite difficult, so please be patient.
Best wishes,
Kamil,
Thanks for your information.
Is there any workaround method which can let me continue the MFC decoder
t
Em Sun, 3 Mar 2013 12:58:43 -0300
Mauro Carvalho Chehab escreveu:
> State 9 means TS started to be output, and it should be
> associated with FE_HAS_SYNC.
>
> The mb86a20scan get CNR statistics at state 7, when frame sync
> is obtained.
>
> As CNR may help to adjust the antenna, provide it ear
From: Hans Verkuil
drivers/media/platform/davinci/dm644x_ccdc.c: In function ‘validate_ccdc_param’:
drivers/media/platform/davinci/dm644x_ccdc.c:233:32: warning: comparison
between ‘enum ccdc_gama_width’ and ‘enum ccdc_data_size’ [-Wenum-compare]
It took a bit of work, see this thread of an ear
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
drivers/media/platform/davinci/dm355_ccdc.c | 10 +-
drivers/media/platform/davinci/dm355_ccdc_regs.h |2 +-
drivers/media/platform/davinci/isif.c|2 +-
drivers/media/platform/davinci/isif_regs.h |4
Hi all,
This is a second try at fixing the dm644x_ccdc compiler warning.
In addition it also fixes more 'gama' misspellings.
Prabhakar, can you look at this? I think this is a good solution
for this compiler warning.
Regards,
Hans
--
To unsubscribe from this list: send the line "unsubs
Hi Hans,
On Mon, Mar 4, 2013 at 3:06 PM, Hans Verkuil wrote:
> On Mon March 4 2013 10:29:26 Prabhakar Lad wrote:
>> Hi Hans,
>>
>> On Mon, Mar 4, 2013 at 2:35 PM, Hans Verkuil wrote:
>> > From: Hans Verkuil
>> >
>> > drivers/media/platform/davinci/dm644x_ccdc.c: In function
>> > ‘validate_ccdc
On Mon March 4 2013 10:29:26 Prabhakar Lad wrote:
> Hi Hans,
>
> On Mon, Mar 4, 2013 at 2:35 PM, Hans Verkuil wrote:
> > From: Hans Verkuil
> >
> > drivers/media/platform/davinci/dm644x_ccdc.c: In function
> > ‘validate_ccdc_param’:
> > drivers/media/platform/davinci/dm644x_ccdc.c:233:32: warni
Hi,
This problem is known to us and Marek is planning a fix. However, the
problem
proved to be quite difficult, so please be patient.
Best wishes,
--
Kamil Debski
Linux Platform Group
Samsung Poland R&D Center
> -Original Message-
> From: Lonsn [mailto:lonsn2...@gmail.com]
> Sent: Satu
Hi Hans,
On Mon, Mar 4, 2013 at 2:35 PM, Hans Verkuil wrote:
> From: Hans Verkuil
>
> drivers/media/platform/davinci/dm644x_ccdc.c: In function
> ‘validate_ccdc_param’:
> drivers/media/platform/davinci/dm644x_ccdc.c:233:32: warning: comparison
> between ‘enum ccdc_gama_width’ and ‘enum ccdc_da
Hi
On Mon, 4 Mar 2013, Sachin Kamat wrote:
> Use the newly introduced devm_ioremap_resource() instead of
> devm_request_and_ioremap() which provides more consistent error handling.
>
> Signed-off-by: Sachin Kamat
> Cc: Guennadi Liakhovetski
Thanks for the patches. I'll add all 4 to my 3.10 qu
Hi Hans,
Thanks for the patch!
On Mon, Mar 4, 2013 at 2:35 PM, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Since vpbe_display already provides a g_std op setting current_norm
> didn't do anything. Remove that code.
>
> Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
Regards,
--Prabhak
Hi Hans,
On Mon, Mar 4, 2013 at 2:35 PM, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
Regards,
--Prabhakar Lad
> ---
> drivers/media/platform/davinci/vpfe_capture.c | 47
> -
> 1 file changed, 7 insertions(+),
Hi Lonsn,
> From: Lonsn [mailto:lonsn2...@gmail.com]
> Sent: Saturday, March 02, 2013 5:00 AM
>
> Hi,
> I tested the MFC decode example v4l2_decode from
> http://git.infradead.org/users/kmpark/public-apps and meet some
> problems as following:
> # ./v4l2_decode -f /dev/video5 -m /dev/video9 -d /d
Hi,
> From: Lonsn [mailto:lonsn2...@gmail.com]
> Sent: Sunday, March 03, 2013 4:10 AM
>
> Which firmware should be used for S5PV210 for kernel 3.8?
> Here:
> https://git.kernel.org/cgit/linux/kernel/git/firmware/linux-
> firmware.git/commit/?id=fb5cda9c70277f633ca0c1e81b6fa7b13007bbf6
> It only s
Hi Hans,
Thanks for the patch!
On Mon, Mar 4, 2013 at 2:35 PM, Hans Verkuil wrote:
> From: Hans Verkuil
>
> Since vpfe_capture already provided a g_std op setting current_norm
> does not actually do anything. Remove it.
>
> Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
Regards,
--Pra
On Mon, Mar 04, 2013 at 01:45:21PM +0530, Sachin Kamat wrote:
> Use the newly introduced devm_ioremap_resource() instead of
> devm_request_and_ioremap() which provides more consistent error handling.
>
> devm_ioremap_resource() provides its own error messages; so all explicit
> error messages can
On Mon, Mar 04, 2013 at 01:45:20PM +0530, Sachin Kamat wrote:
> Use the newly introduced devm_ioremap_resource() instead of
> devm_request_and_ioremap() which provides more consistent error handling.
>
> devm_ioremap_resource() provides its own error messages; so all explicit
> error messages can
On Mon, Mar 04, 2013 at 01:45:19PM +0530, Sachin Kamat wrote:
> Use the newly introduced devm_ioremap_resource() instead of
> devm_request_and_ioremap() which provides more consistent error handling.
>
> Signed-off-by: Sachin Kamat
> ---
> drivers/media/platform/soc_camera/pxa_camera.c |8 ++
On Mon, Mar 04, 2013 at 01:45:18PM +0530, Sachin Kamat wrote:
> Use the newly introduced devm_ioremap_resource() instead of
> devm_request_and_ioremap() which provides more consistent error handling.
>
> Signed-off-by: Sachin Kamat
> Cc: Guennadi Liakhovetski
> ---
> drivers/media/platform/sh_v
From: Hans Verkuil
Signed-off-by: Hans Verkuil
---
drivers/media/platform/davinci/vpfe_capture.c | 47 -
1 file changed, 7 insertions(+), 40 deletions(-)
diff --git a/drivers/media/platform/davinci/vpfe_capture.c
b/drivers/media/platform/davinci/vpfe_capture.c
index
From: Hans Verkuil
Since vpfe_capture already provided a g_std op setting current_norm
does not actually do anything. Remove it.
Signed-off-by: Hans Verkuil
---
drivers/media/platform/davinci/vpfe_capture.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/media/platform/davinci/vp
From: Hans Verkuil
Since vpbe_display already provides a g_std op setting current_norm
didn't do anything. Remove that code.
Signed-off-by: Hans Verkuil
---
drivers/media/platform/davinci/vpbe_display.c | 10 +-
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/drivers/med
From: Hans Verkuil
The use of V4L2_IN/OUT_CAP_CUSTOM_TIMINGS is obsolete, use DV_TIMINGS instead.
Note that V4L2_IN/OUT_CAP_CUSTOM_TIMINGS is just a #define for
V4L2_IN/OUT_CAP_DV_TIMINGS.
At some point in the future these CUSTOM_TIMINGS defines might be removed.
Signed-off-by: Hans Verkuil
Ac
From: Hans Verkuil
Remove VPBE_ENC_DV_PRESET (the DV_PRESET API is no longer supported) and
VPBE_ENC_CUSTOM_TIMINGS is renamed to VPBE_ENC_DV_TIMINGS since the old
"CUSTOM_TIMINGS" name is deprecated in favor of "DV_TIMINGS".
Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
Acked-by: Sekha
From: Hans Verkuil
Finally remove the dv_preset support from this driver. Note that dv_preset
support was already removed from any bridge drivers that use this i2c
driver, so the dv_preset ops were no longer called and can be removed
safely.
Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
From: Hans Verkuil
The use of V4L2_OUT_CAP_CUSTOM_TIMINGS is deprecated, use DV_TIMINGS instead.
Note that V4L2_OUT_CAP_CUSTOM_TIMINGS is just a #define for
V4L2_OUT_CAP_DV_TIMINGS.
At some point in the future these CUSTOM_TIMINGS defines might be removed.
Signed-off-by: Hans Verkuil
Acked-by:
From: Hans Verkuil
drivers/media/platform/davinci/dm644x_ccdc.c: In function ‘validate_ccdc_param’:
drivers/media/platform/davinci/dm644x_ccdc.c:233:32: warning: comparison
between ‘enum ccdc_gama_width’ and ‘enum ccdc_data_size’ [-Wenum-compare]
Signed-off-by: Hans Verkuil
---
drivers/media/
From: Hans Verkuil
This removes some incorrect dv_preset references left over from copy-and-paste
errors.
Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
---
drivers/staging/media/davinci_vpfe/vpfe_video.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a
From: Hans Verkuil
In the functions tvp7002_mbus_fmt(), tvp7002_log_status and tvp7002_probe()
we should use the dv_timings data structures instead of dv_preset data
structures and functions.
This is the second step towards removing the deprecated preset support of this
driver.
Signed-off-by: H
From: Hans Verkuil
This is the first step towards removing the deprecated preset support of this
driver.
Signed-off-by: Hans Verkuil
Acked-by: Lad, Prabhakar
---
drivers/media/i2c/tvp7002.c | 90 +--
1 file changed, 45 insertions(+), 45 deletions(-)
Hi all,
This patch series is for the most part identical to the RFC patch series
posted earlier:
http://www.mail-archive.com/linux-media@vger.kernel.org/msg58762.html
The main changes are:
- dropped the Samsung-related patches, those will go through a separate
patch series.
- added patches 7-
Use the newly introduced devm_ioremap_resource() instead of
devm_request_and_ioremap() which provides more consistent error handling.
devm_ioremap_resource() provides its own error messages; so all explicit
error messages can be removed from the failure code paths.
Signed-off-by: Sachin Kamat
--
Use the newly introduced devm_ioremap_resource() instead of
devm_request_and_ioremap() which provides more consistent error handling.
devm_ioremap_resource() provides its own error messages; so all explicit
error messages can be removed from the failure code paths.
Signed-off-by: Sachin Kamat
--
Use the newly introduced devm_ioremap_resource() instead of
devm_request_and_ioremap() which provides more consistent error handling.
Signed-off-by: Sachin Kamat
---
drivers/media/platform/soc_camera/pxa_camera.c |8 +---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/dri
Use the newly introduced devm_ioremap_resource() instead of
devm_request_and_ioremap() which provides more consistent error handling.
Signed-off-by: Sachin Kamat
Cc: Guennadi Liakhovetski
---
drivers/media/platform/sh_veu.c |7 ---
1 files changed, 4 insertions(+), 3 deletions(-)
diff
72 matches
Mail list logo
