> > Seem that something in git-dvb (or mainline) broke this driver:
> >
> > drivers/media/video/stk11xx-core.c: In function 'show_contrast':
> > drivers/media/video/stk11xx-core.c:707: warning: initialization from
> > incompatible pointer type
> > drivers/media/video/stk11xx-core.c: In function
Andrew Morton napsal(a):
> On Sun, 26 Aug 2007 07:09:02 -0700 Jiri Slaby <[EMAIL PROTECTED]> wrote:
>
>> stk11xx, add a new webcam driver
>>
>> Adds support for stk1125, stk1135 and stkdcnew webcam built-in some
>> notebooks.
>
> Seem that something in git-dvb (or mainline) broke this driver:
>
On Sun, 26 Aug 2007 07:09:02 -0700 Jiri Slaby <[EMAIL PROTECTED]> wrote:
> stk11xx, add a new webcam driver
>
> Adds support for stk1125, stk1135 and stkdcnew webcam built-in some
> notebooks.
Seem that something in git-dvb (or mainline) broke this driver:
drivers/media/video/stk11xx-core.c: In
Jiri Slaby wrote:
+ /* From 80x60 to 640x480 */
+ const u8 values_1_204[] = {
+ 0x12, 0x11, 0x3b, 0x6a, 0x13, 0x10, 0x00, 0x01, 0x02, 0x13,
+ 0x39, 0x38, 0x37, 0x35, 0x0e, 0x12, 0x04, 0x0c, 0x0d, 0x17,
+ 0x18, 0x32, 0x19, 0x1a, 0x03, 0x1b, 0x1
Andrew Morton napsal(a):
> On Tue, 28 Aug 2007 07:33:08 +0200 Jiri Slaby <[EMAIL PROTECTED]> wrote:
>
>>> Again, ARRAY_SIZE() would be clearer here.
>> No, this is only do this 16 times, no corresponding table :).
>
> OK, poorly chosen example. But there are lots of others, like:
Yes, you menti
On Tue, 28 Aug 2007 07:33:08 +0200 Jiri Slaby <[EMAIL PROTECTED]> wrote:
> > Again, ARRAY_SIZE() would be clearer here.
>
> No, this is only do this 16 times, no corresponding table :).
OK, poorly chosen example. But there are lots of others, like:
+
+ for (i = 0; i < 59; i++) {
+
Andrew Morton napsal(a):
> On Sun, 26 Aug 2007 07:09:02 -0700
> Jiri Slaby <[EMAIL PROTECTED]> wrote:
> + retok = stk11xx_check_device(dev, 500);
> + if (retok != 1) {
> + dev_err(&dev->udev->dev, "load microcode fail\n");
> + return -
On Sun, 26 Aug 2007 07:09:02 -0700
Jiri Slaby <[EMAIL PROTECTED]> wrote:
> Hi,
>
> is it possible to have this driver in the -mm tree for testing purposes until
> v4l library will be developped and image resize with bayer->rgb conversion
> will be moved there? (Then, I'll push it through v4l peop
Oliver Neukum napsal(a):
> Am Freitag, 15. Juni 2007 schrieb Jiri Slaby:
>> +struct stk11xx_frame_buf {
>> + void *data;
>> + volatile int filled;
>> + struct stk11xx_frame_buf *next;
>> +};
>> +
>
> What's the "volatile" doing here?
removed (s/volatile/unsigned/), thanks,
--
h
Thierry Merle napsal(a):
> Le vendredi 15 juin 2007 à 23:08 +0200, Jiri Slaby a écrit :
>> Mauro Carvalho Chehab wrote:
>>> Em Sáb, 2007-06-02 às 11:00 +0200, Thierry Merle escreveu:
Nevertheless, I will start to specify the framework.
The helper daemon would link to the v4l2-apps/lib.
Le vendredi 15 juin 2007 à 23:08 +0200, Jiri Slaby a écrit :
> Mauro Carvalho Chehab wrote:
> > Em Sáb, 2007-06-02 às 11:00 +0200, Thierry Merle escreveu:
> > >
> > > Mauro Carvalho Chehab a écrit :
> > > >>> This seems to be an interesting approach.
> > > >>>
> > > >>>
> > > >>>
> >
Em Sáb, 2007-06-02 às 11:00 +0200, Thierry Merle escreveu:
>
> Mauro Carvalho Chehab a écrit :
> >>> This seems to be an interesting approach.
> >>>
> >>>
> >>>
> >> Interesting but impossible to do for ioctl calls.
> >> When the application does a ioctl(fd_of_mnt_video0,VIDIOC_G_FMT,&ar
Mauro Carvalho Chehab a écrit :
This seems to be an interesting approach.
Interesting but impossible to do for ioctl calls.
When the application does a ioctl(fd_of_mnt_video0,VIDIOC_G_FMT,&arg)
for example, there is no way for the userspace helper to catch this ioctl.
The applicati
> > This seems to be an interesting approach.
> >
> >
> Interesting but impossible to do for ioctl calls.
> When the application does a ioctl(fd_of_mnt_video0,VIDIOC_G_FMT,&arg)
> for example, there is no way for the userspace helper to catch this ioctl.
> The application could only open/read
> >> + * Copyright (C) Nicolas VIVIEN
> >
> > It would be interesting to have Nicolas SOB as well, if possible.
>
> I don't think he ever knows about this version of the driver. I got his GPL
> driver, cleaned up -- coding style, v4l1 and v4l2 ioctl conversion to v4l2
> functions, some bug fixes
Mauro Carvalho Chehab a écrit :
Em Ter, 2007-05-29 às 21:04 +0200, Thierry Merle escreveu:
Mauro Carvalho Chehab a écrit :
Hi Mauro and Markus,
Just to summ up what I understood we need:
What do we need in userspace, only for v4l (dvb is not concerned):
- colorspace tran
Mauro Carvalho Chehab napsal(a):
> Hi Jiri,
Hi.
> I have some comments for your driver.
Well, that was exactly what the code needed, yet another eyes.
>> + * Copyright (C) Nicolas VIVIEN
>
> It would be interesting to have Nicolas SOB as well, if possible.
I don't think he ever knows about th
Em Ter, 2007-05-29 às 21:04 +0200, Thierry Merle escreveu:
>
> Mauro Carvalho Chehab a écrit :
> >
> >
> >>> Hi Mauro and Markus,
> >>> Just to summ up what I understood we need:
> >>>
> >>> What do we need in userspace, only for v4l (dvb is not concerned):
> >>> - colorspace translations
>
Mauro Carvalho Chehab a écrit :
Hi Mauro and Markus,
Just to summ up what I understood we need:
What do we need in userspace, only for v4l (dvb is not concerned):
- colorspace translations
- filters that be done in hardware if the selected hardware can,
otherwise software plugin
- decom
> > Hi Mauro and Markus,
> > Just to summ up what I understood we need:
> >
> > What do we need in userspace, only for v4l (dvb is not concerned):
> > - colorspace translations
> > - filters that be done in hardware if the selected hardware can,
> > otherwise software plugin
> > - decompression
Thierry Merle a écrit :
Mauro Carvalho Chehab a écrit :
As Luca pointed, if we add conversion for one driver, we should add
for
the rest.
Instead, it would be better if Jiri sends the decoding and the
rescaling
stuff as a patch to v4l2-apps/lib, starting the API decoding library.
Once we h
Mauro Carvalho Chehab a écrit :
As Luca pointed, if we add conversion for one driver, we should add for
the rest.
Instead, it would be better if Jiri sends the decoding and the rescaling
stuff as a patch to v4l2-apps/lib, starting the API decoding library.
Once we have a library, we can ask the
> > As Luca pointed, if we add conversion for one driver, we should add for
> > the rest.
> >
> > Instead, it would be better if Jiri sends the decoding and the rescaling
> > stuff as a patch to v4l2-apps/lib, starting the API decoding library.
> > Once we have a library, we can ask the userspace
On 5/28/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
Em Seg, 2007-05-28 às 17:14 +0200, Markus Rechberger escreveu:
> > > +/*
> > > + * Bayer conversion
> > > + */
> >
> > We don't do format conversions in kernel. Instead, you should return a
> > proper Bayer Fourcc format (like V4L2_PIX
Em Seg, 2007-05-28 às 17:14 +0200, Markus Rechberger escreveu:
> > > +/*
> > > + * Bayer conversion
> > > + */
> >
> > We don't do format conversions in kernel. Instead, you should return a
> > proper Bayer Fourcc format (like V4L2_PIX_FMT_SBGGR8).
> >
>
> It's ok in his case since most userspace
On 5/28/07, Luca Risolia <[EMAIL PROTECTED]> wrote:
On Monday 28 May 2007 17:14:51 Markus Rechberger wrote:
> On 5/28/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
> > We don't do format conversions in kernel. Instead, you should return a
> > proper Bayer Fourcc format (like V4L2_PIX_FMT_S
On Monday 28 May 2007 17:14:51 Markus Rechberger wrote:
> On 5/28/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
> > We don't do format conversions in kernel. Instead, you should return a
> > proper Bayer Fourcc format (like V4L2_PIX_FMT_SBGGR8).
>
> It's ok in his case since most userspace a
On 5/24/07, Jiri Slaby <[EMAIL PROTECTED]> wrote:
On 5/24/07, Markus Rechberger <[EMAIL PROTECTED]> wrote:
> Hi Jiri,
>
> On 5/24/07, Jiri Slaby <[EMAIL PROTECTED]> wrote:
> > Well, no objections on v4l list, try to merge it. Any further comments
will
> > be
> > appreciated.
> >
> > --
> >
> > st
On 5/28/07, Mauro Carvalho Chehab <[EMAIL PROTECTED]> wrote:
Hi Jiri,
I have some comments for your driver.
> + * Copyright (C) Nicolas VIVIEN
It would be interesting to have Nicolas SOB as well, if possible.
> +
> +#ifndef CONFIG_STK11XX_DEBUG_STREAM
> +#define CONFIG_STK11XX_DEBUG_STREAM
Hi Jiri,
I have some comments for your driver.
> + * Copyright (C) Nicolas VIVIEN
It would be interesting to have Nicolas SOB as well, if possible.
> +
> +#ifndef CONFIG_STK11XX_DEBUG_STREAM
> +#define CONFIG_STK11XX_DEBUG_STREAM 0
> +#endif
> +
> +#if CONFIG_STK11XX_DEBUG_STREAM
I would inst
Jiri Slaby wrote:
> On 5/25/07, Stefan Richter <[EMAIL PROTECTED]> wrote:
>> Ismail Dönmez wrote:
>> > On Thursday 24 May 2007 20:38:05 Diego Calleja wrote:
>> >> Maybe this is a too picky requeriment, but IMO it would be nice if the
>> >> module would be called "camera_stk11xx", or had any other p
On 5/25/07, Stefan Richter <[EMAIL PROTECTED]> wrote:
Jiri Slaby wrote:
> On 5/24/07, Markus Rechberger <[EMAIL PROTECTED]> wrote:
>> On 5/24/07, Jiri Slaby <[EMAIL PROTECTED]> wrote:
> [...]
>> > +static int stk1125_camera_asleep(struct stk11xx *dev)
>> > +{
>> > + int value;
>> > +
>> > +
Jiri Slaby wrote:
> On 5/24/07, Markus Rechberger <[EMAIL PROTECTED]> wrote:
>> On 5/24/07, Jiri Slaby <[EMAIL PROTECTED]> wrote:
> [...]
>> > +static int stk1125_camera_asleep(struct stk11xx *dev)
>> > +{
>> > + int value;
>> > +
>> > + stk11xx_read_registry(dev, 0x0104, &value);
>> > +
Ismail Dönmez wrote:
> On Thursday 24 May 2007 20:38:05 Diego Calleja wrote:
>> El Thu, 24 May 2007 16:01:33 +0200 (CEST), Jiri Slaby <[EMAIL PROTECTED]>
> escribió:
>> > +config USB_STK11XX
>> > + tristate "STK11XX based webcams"
>> > + depends on VIDEO_V4L2
>> > + ---help---
>> > +This wi
On 5/24/07, Markus Rechberger <[EMAIL PROTECTED]> wrote:
Hi Jiri,
On 5/24/07, Jiri Slaby <[EMAIL PROTECTED]> wrote:
> Well, no objections on v4l list, try to merge it. Any further comments will
> be
> appreciated.
>
> --
>
> stk11xx, add a new webcam driver
[...]
> +static int stk1125_camera_a
On Thursday 24 May 2007 20:38:05 Diego Calleja wrote:
> El Thu, 24 May 2007 16:01:33 +0200 (CEST), Jiri Slaby <[EMAIL PROTECTED]>
escribió:
> > +config USB_STK11XX
> > + tristate "STK11XX based webcams"
> > + depends on VIDEO_V4L2
> > + ---help---
> > + This will add support for Syntek w
El Thu, 24 May 2007 16:01:33 +0200 (CEST), Jiri Slaby <[EMAIL PROTECTED]>
escribió:
> +config USB_STK11XX
> + tristate "STK11XX based webcams"
> + depends on VIDEO_V4L2
> + ---help---
> + This will add support for Syntek webcams such as dc1125 and stk1135.
> +
> + If you c
On 5/24/07, Markus Rechberger <[EMAIL PROTECTED]> wrote:
Hi Jiri,
On 5/24/07, Jiri Slaby <[EMAIL PROTECTED]> wrote:
> Well, no objections on v4l list, try to merge it. Any further comments will
> be
> appreciated.
>
> --
>
> stk11xx, add a new webcam driver
>
> Cc: Mauro Carvalho Chehab <[EMAIL
38 matches
Mail list logo
