All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <j.w.r.degoede@hhs.nl>
To: "Erik Andrén" <erik.andren@gmail.com>
Cc: video4linux-list@redhat.com, sqcam-devel@lists.sourceforge.net
Subject: Re: Advice wanted on producing an in kernel sq905 driver
Date: Tue, 18 Nov 2008 13:52:13 +0100	[thread overview]
Message-ID: <4922BA7D.7010303@hhs.nl> (raw)
In-Reply-To: <62e5edd40811180338q2527233aw37b91734465f6b49@mail.gmail.com>

Erik Andrén wrote:
> 2008/11/18 Hans de Goede <j.w.r.degoede@hhs.nl>:
>> Erik Andrén wrote:
>>
>> <snip>
>>
>>>> Correct, there is nothing special you need to do for that, just pass
>>>> frames
>>>> with the raw bayer data to userspace and set the pixelformat to one of:
>>>> V4L2_PIX_FMT_SBGGR8 /* v4l2_fourcc('B', 'A', '8', '1'), 8 bit BGBG..
>>>> GRGR..
>>>> */
>>>> V4L2_PIX_FMT_SGBRG8 /* v4l2_fourcc('G', 'B', 'R', 'G'), 8 bit GBGB..
>>>> RGRG..
>>>> */
>>>> V4L2_PIX_FMT_SGRBG8 /* v4l2_fourcc('G','R','B','G'), 8 bit GRGR.. BGBG..
>>>> */
>>>> V4L2_PIX_FMT_SRGGB8 /* v4l2_fourcc('R','G','G','B'), 8 bit RGRG.. GBGB..
>>>> */
>>>>
>>>> Note the last 2 currently are only defined internally in libv4l and not
>>>> in
>>>> linux/videodev2.h as no drivers use them yet, but if you need one of them
>>>> adding it to linux/videodev2.h is fine.
>>> I'm currently developing a driver where I need the two lower ones in
>>> order to get a correct bayer decoding.
>>> Would it possible to add them into the linux/videodev2.h?
>>>
>>> I can send a patch tonight if required.
>>>
>> It is usual for such patches to be submitted together with the driver using
>> the new defines. Just be sure you define
>> V4L2_PIX_FMT_SGRBG8 as v4l2_fourcc('G','R','B','G')
>> and
>> V4L2_PIX_FMT_SRGGB8 as v4l2_fourcc('R','G','G','B')
>>
>> You can try sending a patch to Mauro as preperation for your driver, but I'm
>> not sure he will take such a patch, he did not accept it from me in the past
>> as no drivers were using them, maybe with a driver on the horizon he will
>> accept such a patch.
>>
> 
> I'm currently in the process of porting the old quickcam driver [1] to
> the gspca framework sofar I have working video using the vv6410
> sensor. Next step is to get the hdcs and pb0100 sensors up to speed.
> The driver will be tracked in
> http://linuxtv.org/hg/~eandren/gspca-stv06xx/ (nothing pushed yet)
> 

Good! and <ugh> I've been working on the same, I'm nowhere near as far as you 
though, can you please put a tarbal or whatever somewhere, then I can see if 
there is anything in my version worth salvaging to improve your driver in some 
points.

I've access to an pb0100 using cam.

Regards,

Hans

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2008-11-18 12:47 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-17 22:53 Advice wanted on producing an in kernel sq905 driver Adam Baker
2008-11-18 10:00 ` Hans de Goede
2008-11-18 10:00   ` Erik Andrén
2008-11-18 11:03     ` Hans de Goede
2008-11-18 11:38       ` Erik Andrén
2008-11-18 12:52         ` Hans de Goede [this message]
2008-11-18 13:08           ` Erik Andrén
2008-11-18 14:12           ` Antonio Ospite

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4922BA7D.7010303@hhs.nl \
    --to=j.w.r.degoede@hhs.nl \
    --cc=erik.andren@gmail.com \
    --cc=sqcam-devel@lists.sourceforge.net \
    --cc=video4linux-list@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.