In the near future, the driver core is going to not allow direct access
to the driver_data pointer in struct device. Instead, the functions
dev_get_drvdata() and dev_set_drvdata() should be used. These functions
have been around since the beginning, so are backwards compatible with
all older kern
Em Mon, 15 Jun 2009 17:13:28 +0200
Richard Röjfors escreveu:
> This is an initial driver for Analog Devices ADV7180 Video Decoder.
>
> So far it only supports query standard.
Hmm... it seems too preliminar for merging. Also, as this is an i2c ancillary
driver, it would be interesting if you co
On Tue, Jun 16, 2009 at 1:01 AM, Karicheri,
Muralidharan wrote:
>>> +
>>> +struct v4l2_subdev_bus {
>>> + enum v4l2_subdev_bus_type type;
>>> + u8 width;
>>> + /* 0 - active low, 1 - active high */
>>> + unsigned pol_vsync:1;
>>> + /* 0 - active low, 1 - active high */
Em Sun, 14 Jun 2009 20:30:18 -0400
Devin Heitmueller escreveu:
> Hello Mauro,
>
> Please pull from
> http://kernellabs.com/hg/~dheitmueller/em28xx-no-audio for the
> following:
>
> - em28xx: don't create audio device if not supported
>
> This should correct the problems you noticed with the pr
Hi
Next code for implement Philips FM1216MK5.
1. Implement get_stereo function.
2. Add correct data byte for FM radio mode.
diff -r bff77ec33116 linux/drivers/media/common/tuners/tuner-simple.c
--- a/linux/drivers/media/common/tuners/tuner-simple.c Thu Jun 11 18:44:23
2009 -0300
+++ b/linux/dr
Em Mon, 15 Jun 2009 23:35:59 +0200
Hans Verkuil escreveu:
> > By looking on this patch, it is obfuscating the function even more.
> > Creating more
> > confusion on it, due to some reasons:
> >
> > 1) The name "kernel number" doesn't seem much appropriate. Any number used
> > in
> > "kernel" c
Hi,
Am Montag, den 15.06.2009, 23:35 +0200 schrieb Hans Verkuil:
> On Monday 15 June 2009 21:48:32 Mauro Carvalho Chehab wrote:
> > Hi Hans,
> >
> > Em Mon, 15 Jun 2009 13:27:42 +0200
> > Hans Verkuil escreveu:
> >
> > > On Sunday 14 June 2009 12:15:34 Hans Verkuil wrote:
> > > > On Sunday 14 J
On Tuesday 16 June 2009 00:24:34 Karicheri, Muralidharan wrote:
> Hans,
>
> Please see my response below.
> >A general remark: currently you link your inputs directly to a subdev.
> > This approach has two disadvantages:
> >
> >1) It doesn't work if there are no subdevs at all (e.g. because
> >
On Monday 15 June 2009 23:47:07 Karicheri, Muralidharan wrote:
> Hans,
>
> I am not clear about some of your comments. Please see below with a [MK]
> prefix.
>
> >> + /* set if client specific interface param is available */
> >> + if (subdev->pdata) {
> >> + /* each client will have
Thomas Kernen wrote:
Hello to all,
I'm currently testing a TT-S1500 budget card with the TT budget CI
adapter with vl4 tree and kernel 2.6.28.
When I modprobe budget_ci, the CI adapter seems to be detected but not
registered in /dev/dvb/adapter3/ca0 as I would have expected it to be.
Inst
Hans,
Please see my response below.
>> +/* { plus irq }, */
>> /* { I2C_BOARD_INFO("tlv320aic3x", 0x1b), }, */
>
>Huh? What's this? I only know the tlv320aic23b and that's an audio driver.
>
[MK] Thanks to David for answering this.
>> -/* { I2C_BOARD_INFO("tvp5146", 0x5d), }, */
>>
Hans,
I will do my best to push v3 of this patch this week.
Murali Karicheri
Software Design Engineer
Texas Instruments Inc.
Germantown, MD 20874
Phone : 301-515-3736
email: m-kariche...@ti.com
>-Original Message-
>From: Hans Verkuil [mailto:hverk...@xs4all.nl]
>Sent: Monday, June 15, 20
Hans,
I am not clear about some of your comments. Please see below with a [MK] prefix.
>> +static int debug;
>> +static u32 numbuffers = 3;
>> +static u32 bufsize = (720 * 576 * 2);
>> +
>> +module_param(numbuffers, uint, S_IRUGO);
>> +module_param(bufsize, uint, S_IRUGO);
>> +module_param(debug,
On Monday 15 June 2009 20:44:15 Karicheri, Muralidharan wrote:
> Hans,
>
> That is great!
>
> Once I change the code based on the comments, do you think there is a
> chance to get the driver to be merged to 2.6.31 (If so, how soon should I
> be ready with the next version, v3 of the patch) or do we
On Monday 15 June 2009 21:48:32 Mauro Carvalho Chehab wrote:
> Hi Hans,
>
> Em Mon, 15 Jun 2009 13:27:42 +0200
> Hans Verkuil escreveu:
>
> > On Sunday 14 June 2009 12:15:34 Hans Verkuil wrote:
> > > On Sunday 14 June 2009 11:50:51 Hans Verkuil wrote:
> > > > Hi Mauro,
> > > >
> > > > Please pu
On Monday 15 June 2009 21:51:13 Mauro Carvalho Chehab wrote:
> Em Mon, 15 Jun 2009 16:02:40 +0200
> Hans Verkuil escreveu:
>
> >
> > The sticking point for me is that warning since for cx18/ivtv it is OK if
> > you
> > get something else then you specified (since it is a starting index meant to
On 06/15/2009 03:01 AM, Erik de Castro Lopo wrote:
On Sat, 13 Jun 2009 18:12:10 +1000
Hans de Goede wrote:
Getting ovfx2 support into the mainline kernel sounds like a good idea!
I'm not such a big fan of merging the driver as is though, as it does
its own buffer management (and ioctl handl
Hello to all,
I'm currently testing a TT-S1500 budget card with the TT budget CI
adapter with vl4 tree and kernel 2.6.28.
When I modprobe budget_ci, the CI adapter seems to be detected but not
registered in /dev/dvb/adapter3/ca0 as I would have expected it to be.
Instead I see the followin
Em Mon, 15 Jun 2009 16:02:40 +0200
Hans Verkuil escreveu:
>
> The sticking point for me is that warning since for cx18/ivtv it is OK if you
> get something else then you specified (since it is a starting index meant to
> distinguish mpeg encoders from raw video inputs, from mpeg decoders, etc.).
Hi Hans,
Em Mon, 15 Jun 2009 13:27:42 +0200
Hans Verkuil escreveu:
> On Sunday 14 June 2009 12:15:34 Hans Verkuil wrote:
> > On Sunday 14 June 2009 11:50:51 Hans Verkuil wrote:
> > > Hi Mauro,
> > >
> > > Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-misc for the
> > > following
Hei Jan,
On 06/10/2009 09:21 AM, Jan Nikitenko wrote:
This patch fixes stack corruption bug present in dump_regs function of
zl10353 and qt1010 drivers:
the buffer buf is one byte smaller than required - there is 4 chars
for address prefix, 16*3 chars for dump of 16 eeprom bytes per line
and 1 b
Hans,
That is great!
Once I change the code based on the comments, do you think there is a chance to
get the driver to be merged to 2.6.31 (If so, how soon should I be ready with
the next version, v3 of the patch) or do we need to wait until 2.6.32?
Regards,
Murali Karicheri
Software Design En
This message is generated daily by a cron job that builds v4l-dvb for
the kernels and architectures in the list below.
Results of the daily build of v4l-dvb:
date:Mon Jun 15 19:00:04 CEST 2009
path:http://www.linuxtv.org/hg/v4l-dvb
changeset: 11975:144d8d0cebc5
gcc version: gcc
On Monday 15 June 2009 18:24:53 Karicheri, Muralidharan wrote:
> Hans,
>
> Thanks for your review. I will get back to you if I need more
> information on your comments.
>
> You have reviewed the following patches in this series...
> 1,7,8,10
Those were the only ones I had any comments on. So I'm
=
>dm644x_clear_wbl_overflow;
>> + else
>> + return -ENODEV;
>
>Do you need clean up procedure if you return error here? I mean -
>calls to release_mem_region, release_mem_region, etc
>
Oops! I need to add that. Thanks.
>> + spin_lock_init(&oper_cfg.vpss_lock);
>> +
>> + /* set the default image parameters in the device */
>> + ret = vpfe_config_image_format(vpfe_dev,
>> + &vpfe_standards[vpfe_dev-
>>std_index].std_id);
>> + if (ret)
>> + goto unlock_out;
>
>Why you check ret value and go to label b
Hans,
Thanks for your review. I will get back to you if I need more
information on your comments.
You have reviewed the following patches in this series...
1,7,8,10
Following are part of this series which requires review as well.
[PATCH 2/10 - v2] ccdc hw device header file for vpfe capture
[PA
The supported formats count must be set to 0 after debug output
right before the second pass.
Signed-off-by: Stefan Herbrechtsmeier
diff --git a/linux/drivers/media/video/soc_camera.c
b/linux/drivers/media/video/soc_camera.c
--- a/linux/drivers/media/video/soc_camera.c
+++ b/linux/drivers/media
On Sun, 2009-06-07 at 09:21 +0800, Figo.zhang wrote:
> On Sun, 2009-06-07 at 09:16 +0800, Figo.zhang wrote:
> > On Wed, 2009-06-03 at 10:01 +0800, Figo.zhang wrote:
> > > add poiter check for videobuf_queue_core_init().
> > >
> > > any guys who write a v4l driver, pass a NULL pointer or a non-inin
hi Mauro,
is it ok for this patch?
Best Regards,
Figo.zhang
On Sat, 2009-06-06 at 17:16 +0800, Figo.zhang wrote:
> vfree() does it's own NULL checking, no need for explicit check before
> calling it.
>
> Signed-off-by: Figo.zhang
> ---
> drivers/media/video/zr364xx.c |6 --
> 1 file
>> +
>> +struct v4l2_subdev_bus {
>> + enum v4l2_subdev_bus_type type;
>> + u8 width;
>> + /* 0 - active low, 1 - active high */
>> + unsigned pol_vsync:1;
>> + /* 0 - active low, 1 - active high */
>> + unsigned pol_hsync:1;
>> + /* 0 - low to high , 1 - h
Just wanted to thank Douglas Landgraf for the help he provided me with getting
the em28xx driver working with the Kworld 315U tuner device (digital only at
this time). As time permits, I hope to get the analog inputs and tuner to work
too (if anyone want to help please let me know).
Thanks,
This is an initial driver for Analog Devices ADV7180 Video Decoder.
So far it only supports query standard.
Signed-off-by: Richard Röjfors
---
Index: linux-2.6.30-rc7/drivers/media/video/adv7180.c
===
--- linux-2.6.30-rc7/drivers/me
hermann pitton wrote:
[snip]
The most undiscovered configurations seem to be such ones about antenna
inputs and their switching. Again according to Hartmut, and he did not
know exactly what is going on here, some for us and him at this point
unknown checksums are used to derive even that informa
From: Jonathan Cameron
OV7670 soc-camera driver. Merge of drivers from Jonathan Corbet,
Darius Augulis and Jonathan Cameron
Signed-off-by: Jonathan Cameron
---
This is my first cut at a merge of the various ov7670 drivers to work
with Guennadi Liakhovetski's work on moving soc-camera over to v
On Monday 15 June 2009 15:44:21 Mauro Carvalho Chehab wrote:
> Em Mon, 15 Jun 2009 13:25:28 +0200
> Hans Verkuil escreveu:
>
> > Hi all,
> >
> > While looking at the video_register_device changes that broke ov511 I
> > realized
> > that the video_register_device_index function is never called f
Em Mon, 15 Jun 2009 13:25:28 +0200
Hans Verkuil escreveu:
> Hi all,
>
> While looking at the video_register_device changes that broke ov511 I realized
> that the video_register_device_index function is never called from drivers. It
> will always assign a default index number. I also don't see a
hi Mauro,
is it ok for this v2?
Best Regards,
Figo.zhang
> 2009/6/1 Figo.zhang
> bttv-driver.c,cx23885-video.c,cx88-video.c: poll method lose
> race condition for capture video.
>
> In v2, use the clear logic.Thanks to Trent Piepho's help.
>
>
Hi Mauro,
On Mon, 15 Jun 2009, Mauro Carvalho Chehab wrote:
Em Thu, 11 Jun 2009 15:35:14 -0700 (PDT)
Trent Piepho escreveu:
On Thu, 11 Jun 2009, Patrick Boettcher wrote:
On Wed, 27 May 2009, Trent Piepho wrote:
On Tue, 26 May 2009, Patrick Boettcher wrote:
Does this patch to fix these prob
Em Thu, 11 Jun 2009 15:35:14 -0700 (PDT)
Trent Piepho escreveu:
> On Thu, 11 Jun 2009, Patrick Boettcher wrote:
> > On Wed, 27 May 2009, Trent Piepho wrote:
> > > On Tue, 26 May 2009, Patrick Boettcher wrote:
> > > Does this patch to fix these problems look ok?
> >
> > In fact, everything looks c
On Sunday 14 June 2009 12:15:34 Hans Verkuil wrote:
> On Sunday 14 June 2009 11:50:51 Hans Verkuil wrote:
> > Hi Mauro,
> >
> > Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-misc for the
> > following:
> >
> > - ivtv/cx18: fix regression: class controls are no longer seen
> > - v4
Hi all,
While looking at the video_register_device changes that broke ov511 I realized
that the video_register_device_index function is never called from drivers. It
will always assign a default index number. I also don't see a good use-case
for giving it an explicit index. My proposal is to remov
On Friday 12 June 2009 18:51:03 Robert Krakora wrote:
> From: Robert Krakora
>
> Fix for no return value check of uvc_ctrl_set() which calls
> mutex_lock_interruptible().
>
> Priority: normal
>
> Signed-off-by: Robert Krakora
Acked-by: Laurent Pinchart
I'd appreciate if you would at least CC m
On Friday 12 June 2009 18:53:29 Robert Krakora wrote:
> From: Robert Krakora
>
> Added two webcams with 'No FID' quirk.
>
> Priority: normal
>
> Signed-off-by: Robert Krakora
>
> diff -r bff77ec33116 linux/drivers/media/video/uvc/uvc_driver.c
> --- a/linux/drivers/media/video/uvc/uvc_driver.c
Hi Hans,
On Sun, 14 Jun 2009, Hans Verkuil wrote:
Hi Patrick,
The daily build reports these warnings for dib3000mc.c and dib7000p.c:
/marune/build/v4l-dvb-master/v4l/dib3000mc.c: In
function 'dib3000mc_i2c_enumeration':
/marune/build/v4l-dvb-master/v4l/dib3000mc.c:863: warning: the frame size
Hans Verkuil wrote:
On Sunday 14 June 2009 14:44:53 Hans Verkuil wrote:
On Sunday 14 June 2009 12:14:38 Hans Verkuil wrote:
On Wednesday 06 May 2009 20:31:33 hvaib...@ti.com wrote:
From: Vaibhav Hiremath
This patch converts TVP514x driver to sub-device framework
from V4L2-int framework.
No
46 matches
Mail list logo
