* V4L2_PIX_FMT_SBGGR16 not in kernel @ 2008-04-24 14:12 Johan Hedlund 2008-04-24 14:31 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 11+ messages in thread From: Johan Hedlund @ 2008-04-24 14:12 UTC (permalink / raw) To: video4linux-list Hello I am working on a application that will use V4L2 on a linux-2.6-22 kernel. I am interested in using the format V4L2_PIX_FMT_SBGGR16 'BA82' for my captured images. But it seems like I only can find it the in the V4L2 specification and not in the mainline kernel. Is this format not a standard format that should be in the mainline kernel? Can I just add the definition to my code and use it or will it break something else? /Johan -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:12 V4L2_PIX_FMT_SBGGR16 not in kernel Johan Hedlund @ 2008-04-24 14:31 ` Mauro Carvalho Chehab 2008-04-24 14:35 ` Johan Hedlund 0 siblings, 1 reply; 11+ messages in thread From: Mauro Carvalho Chehab @ 2008-04-24 14:31 UTC (permalink / raw) To: Johan Hedlund; +Cc: video4linux-list On Thu, 24 Apr 2008 16:12:59 +0200 Johan Hedlund <johan.hedlund@enea.com> wrote: > Hello > > I am working on a application that will use V4L2 on a linux-2.6-22 > kernel. I am interested in using the format V4L2_PIX_FMT_SBGGR16 'BA82' > for my captured images. But it seems like I only can find it the in the > V4L2 specification and not in the mainline kernel. Is this format not a > standard format that should be in the mainline kernel? Can I just add > the definition to my code and use it or will it break something else? If it isn't at mainline kernel, there's no in-kernel drivers supporting it. It is better to use a format that is already supported by the drivers. Cheers, Mauro -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:31 ` Mauro Carvalho Chehab @ 2008-04-24 14:35 ` Johan Hedlund 2008-04-24 14:15 ` Steven Whitehouse 2008-04-24 16:33 ` Mauro Carvalho Chehab 0 siblings, 2 replies; 11+ messages in thread From: Johan Hedlund @ 2008-04-24 14:35 UTC (permalink / raw) To: Mauro Carvalho Chehab; +Cc: video4linux-list I am developing my own capture drivers for our own developed hardware based on a driver that does not exist in mainline. This driver used a uyuv format, but since we want to use the raw bayer format with 10-bit color resolution I need to change the format. /Johan On tor, 2008-04-24 at 11:31 -0300, Mauro Carvalho Chehab wrote: > On Thu, 24 Apr 2008 16:12:59 +0200 > Johan Hedlund <johan.hedlund@enea.com> wrote: > > > Hello > > > > I am working on a application that will use V4L2 on a linux-2.6-22 > > kernel. I am interested in using the format V4L2_PIX_FMT_SBGGR16 'BA82' > > for my captured images. But it seems like I only can find it the in the > > V4L2 specification and not in the mainline kernel. Is this format not a > > standard format that should be in the mainline kernel? Can I just add > > the definition to my code and use it or will it break something else? > > If it isn't at mainline kernel, there's no in-kernel drivers supporting it. It > is better to use a format that is already supported by the drivers. > > Cheers, > Mauro -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:35 ` Johan Hedlund @ 2008-04-24 14:15 ` Steven Whitehouse 2008-04-24 14:44 ` Mauro Carvalho Chehab 2008-04-24 16:33 ` Mauro Carvalho Chehab 1 sibling, 1 reply; 11+ messages in thread From: Steven Whitehouse @ 2008-04-24 14:15 UTC (permalink / raw) To: Johan Hedlund; +Cc: video4linux-list, Mauro Carvalho Chehab Hi, On Thu, Apr 24, 2008 at 04:35:35PM +0200, Johan Hedlund wrote: > I am developing my own capture drivers for our own developed hardware > based on a driver that does not exist in mainline. This driver used a > uyuv format, but since we want to use the raw bayer format with 10-bit > color resolution I need to change the format. > > /Johan > > Also I sent a patch for this a little while back... I thought it had been added, and I rather lost track of what has happend to it, Steve. -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:15 ` Steven Whitehouse @ 2008-04-24 14:44 ` Mauro Carvalho Chehab 2008-04-24 14:38 ` Steven Whitehouse 0 siblings, 1 reply; 11+ messages in thread From: Mauro Carvalho Chehab @ 2008-04-24 14:44 UTC (permalink / raw) To: Steven Whitehouse; +Cc: video4linux-list, Johan Hedlund On Thu, 24 Apr 2008 15:15:13 +0100 Steven Whitehouse <steve@chygwyn.com> wrote: > Hi, > > On Thu, Apr 24, 2008 at 04:35:35PM +0200, Johan Hedlund wrote: > > I am developing my own capture drivers for our own developed hardware > > based on a driver that does not exist in mainline. This driver used a > > uyuv format, but since we want to use the raw bayer format with 10-bit > > color resolution I need to change the format. > > > > /Johan > > > > > Also I sent a patch for this a little while back... I thought it had > been added, and I rather lost track of what has happend to it, Hi Steve, It seems that I lost your patches adding this new standard and adding a driver using it. Could you please send those to me again? Cheers, Mauro -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:44 ` Mauro Carvalho Chehab @ 2008-04-24 14:38 ` Steven Whitehouse 2008-04-24 16:29 ` Mauro Carvalho Chehab 2008-04-24 17:22 ` Guennadi Liakhovetski 0 siblings, 2 replies; 11+ messages in thread From: Steven Whitehouse @ 2008-04-24 14:38 UTC (permalink / raw) To: Mauro Carvalho Chehab; +Cc: video4linux-list, Johan Hedlund Hi, On Thu, Apr 24, 2008 at 11:44:24AM -0300, Mauro Carvalho Chehab wrote: > On Thu, 24 Apr 2008 15:15:13 +0100 > Steven Whitehouse <steve@chygwyn.com> wrote: > > > Hi, > > > > On Thu, Apr 24, 2008 at 04:35:35PM +0200, Johan Hedlund wrote: > > > I am developing my own capture drivers for our own developed hardware > > > based on a driver that does not exist in mainline. This driver used a > > > uyuv format, but since we want to use the raw bayer format with 10-bit > > > color resolution I need to change the format. > > > > > > /Johan > > > > > > > > Also I sent a patch for this a little while back... I thought it had > > been added, and I rather lost track of what has happend to it, > > Hi Steve, > > It seems that I lost your patches adding this new standard and adding a driver > using it. Could you please send those to me again? > > Cheers, > Mauro Btw, the V4L spec claims that it exists already: http://v4l2spec.bytesex.org/spec/r3796.htm Guennadi Liakhovetski submitted a driver which uses this too so thats why I'd thought it was already included. I hadn't looked to see what stage his driver was at recently. Somehow my patch got mangled up with some other stuff too, because I saw this thread: http://lists.zerezo.com/video4linux/msg21484.html but my original patch did nothing but add the two new pix formats so I've no idea why I was being credited with various extra changes as well, which were nothing to do with me. Btw, here was the original patch: http://www.archivum.info/video4linux-list@redhat.com/2008-01/msg00059.html I do have a (not yet submitted driver) which uses it, and the hold up has been basically that the various subsystems that I was using were in a state of flux. I hope that when the current merge window is complete I'll be able to take another look at it, and get it ready. I hadn't bothered to follow up on the patch since my driver wasn't ready yet, but nonetheless I didn't receive any feedback at the time to suggest that it was a prerequsite. There are a lot of sensors though that use this format. Pretty much all the Omnivision sensors and the Micron sensors all have 10 bit interfaces, even though some manufacturers only choose to use the upper 8 bits, so it will be something that it of general use to lots of people, Steve. -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:38 ` Steven Whitehouse @ 2008-04-24 16:29 ` Mauro Carvalho Chehab 2008-04-24 17:22 ` Guennadi Liakhovetski 1 sibling, 0 replies; 11+ messages in thread From: Mauro Carvalho Chehab @ 2008-04-24 16:29 UTC (permalink / raw) To: Steven Whitehouse; +Cc: video4linux-list, Johan Hedlund On Thu, 24 Apr 2008 15:38:25 +0100 Steven Whitehouse <steve@chygwyn.com> wrote: > Btw, the V4L spec claims that it exists already: > http://v4l2spec.bytesex.org/spec/r3796.htm > > Guennadi Liakhovetski submitted a driver which uses this too so > thats why I'd thought it was already included. I hadn't looked > to see what stage his driver was at recently. It isn't already at mainstream, but it will be soon. I'm finishing to prepare its submission. > I do have a (not yet submitted driver) which uses it, and the hold up > has been basically that the various subsystems that I was using were > in a state of flux. I hope that when the current merge window is > complete I'll be able to take another look at it, and get it ready. > > I hadn't bothered to follow up on the patch since my driver wasn't > ready yet, but nonetheless I didn't receive any feedback at the time > to suggest that it was a prerequsite. It makes no sense to change the API for something that is not used ;) > There are a lot of sensors though that use this format. Pretty much > all the Omnivision sensors and the Micron sensors all have 10 bit > interfaces, even though some manufacturers only choose to use the > upper 8 bits, so it will be something that it of general use to > lots of people, Good. Cheers, Mauro -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:38 ` Steven Whitehouse 2008-04-24 16:29 ` Mauro Carvalho Chehab @ 2008-04-24 17:22 ` Guennadi Liakhovetski 1 sibling, 0 replies; 11+ messages in thread From: Guennadi Liakhovetski @ 2008-04-24 17:22 UTC (permalink / raw) To: Steven Whitehouse; +Cc: video4linux-list, Johan Hedlund, Mauro Carvalho Chehab On Thu, 24 Apr 2008, Steven Whitehouse wrote: > Btw, the V4L spec claims that it exists already: > http://v4l2spec.bytesex.org/spec/r3796.htm > > Guennadi Liakhovetski submitted a driver which uses this too so > thats why I'd thought it was already included. I hadn't looked > to see what stage his driver was at recently. > > Somehow my patch got mangled up with some other stuff too, because > I saw this thread: > http://lists.zerezo.com/video4linux/msg21484.html > > but my original patch did nothing but add the two new pix formats > so I've no idea why I was being credited with various extra > changes as well, which were nothing to do with me. Right, there was a merge problem at some point, but it has been resolved, and now your patch in its original undamaged form in the v4l git tree waiting to be submitted into the mainline. Thanks Guennadi --- Guennadi Liakhovetski -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 14:35 ` Johan Hedlund 2008-04-24 14:15 ` Steven Whitehouse @ 2008-04-24 16:33 ` Mauro Carvalho Chehab 2008-04-24 17:25 ` Guennadi Liakhovetski 1 sibling, 1 reply; 11+ messages in thread From: Mauro Carvalho Chehab @ 2008-04-24 16:33 UTC (permalink / raw) To: Johan Hedlund; +Cc: video4linux-list, Guennadi Liakhovetski On Thu, 24 Apr 2008 16:35:35 +0200 Johan Hedlund <johan.hedlund@enea.com> wrote: > I am developing my own capture drivers for our own developed hardware > based on a driver that does not exist in mainline. This driver used a > uyuv format, but since we want to use the raw bayer format with 10-bit > color resolution I need to change the format. > > > I am interested in using the format V4L2_PIX_FMT_SBGGR16 'BA82' A format with this name will be added, but the fourcc code that Guennadi used is different from yours. Not sure what would be the proper one. This is what's currently defined: #define V4L2_PIX_FMT_SBGGR8 v4l2_fourcc('B','A','8','1') /* 8 BGBG.. GRGR.. */ #define V4L2_PIX_FMT_SBGGR16 v4l2_fourcc('B','Y','R','2') /* 16 BGBG.. GRGR.. */ Cheers, Mauro -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 16:33 ` Mauro Carvalho Chehab @ 2008-04-24 17:25 ` Guennadi Liakhovetski 2008-04-24 17:55 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 11+ messages in thread From: Guennadi Liakhovetski @ 2008-04-24 17:25 UTC (permalink / raw) To: Mauro Carvalho Chehab; +Cc: video4linux-list, Johan Hedlund On Thu, 24 Apr 2008, Mauro Carvalho Chehab wrote: > On Thu, 24 Apr 2008 16:35:35 +0200 > Johan Hedlund <johan.hedlund@enea.com> wrote: > > > I am developing my own capture drivers for our own developed hardware > > based on a driver that does not exist in mainline. This driver used a > > uyuv format, but since we want to use the raw bayer format with 10-bit > > color resolution I need to change the format. > > > > > I am interested in using the format V4L2_PIX_FMT_SBGGR16 'BA82' > > A format with this name will be added, but the fourcc code that Guennadi used > is different from yours. Not sure what would be the proper one. > > This is what's currently defined: > > #define V4L2_PIX_FMT_SBGGR8 v4l2_fourcc('B','A','8','1') /* 8 BGBG.. GRGR.. */ > #define V4L2_PIX_FMT_SBGGR16 v4l2_fourcc('B','Y','R','2') /* 16 BGBG.. GRGR.. */ As we have already seen in this thread, these defines actually originate from Steven Whitehouse and the BYR2 fourcc code agrees with http://www.fourcc.org/rgb.php Thanks Guennadi --- Guennadi Liakhovetski -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: V4L2_PIX_FMT_SBGGR16 not in kernel 2008-04-24 17:25 ` Guennadi Liakhovetski @ 2008-04-24 17:55 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 11+ messages in thread From: Mauro Carvalho Chehab @ 2008-04-24 17:55 UTC (permalink / raw) To: Guennadi Liakhovetski; +Cc: video4linux-list, Johan Hedlund On Thu, 24 Apr 2008 19:25:53 +0200 (CEST) Guennadi Liakhovetski <g.liakhovetski@gmx.de> wrote: > On Thu, 24 Apr 2008, Mauro Carvalho Chehab wrote: > > > On Thu, 24 Apr 2008 16:35:35 +0200 > > Johan Hedlund <johan.hedlund@enea.com> wrote: > > > > > I am developing my own capture drivers for our own developed hardware > > > based on a driver that does not exist in mainline. This driver used a > > > uyuv format, but since we want to use the raw bayer format with 10-bit > > > color resolution I need to change the format. > > > > > > > I am interested in using the format V4L2_PIX_FMT_SBGGR16 'BA82' > > > > A format with this name will be added, but the fourcc code that Guennadi used > > is different from yours. Not sure what would be the proper one. > > > > This is what's currently defined: > > > > #define V4L2_PIX_FMT_SBGGR8 v4l2_fourcc('B','A','8','1') /* 8 BGBG.. GRGR.. */ > > #define V4L2_PIX_FMT_SBGGR16 v4l2_fourcc('B','Y','R','2') /* 16 BGBG.. GRGR.. */ > > As we have already seen in this thread, these defines actually originate > from Steven Whitehouse and the BYR2 fourcc code agrees with > http://www.fourcc.org/rgb.php Argh... to much patches... Yes, the author is Steven. The patch is already applied at my -git and seems OK: http://git.kernel.org/?p=linux/kernel/git/mchehab/v4l-dvb.git;a=commitdiff;h=588cd02c512658f4756750c2aef46d473bd88663 I should be send in a few minutes the pull request for all those 452 patches pending merge. Cheers, Mauro -- video4linux-list mailing list Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe https://www.redhat.com/mailman/listinfo/video4linux-list ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2008-04-24 17:56 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-04-24 14:12 V4L2_PIX_FMT_SBGGR16 not in kernel Johan Hedlund 2008-04-24 14:31 ` Mauro Carvalho Chehab 2008-04-24 14:35 ` Johan Hedlund 2008-04-24 14:15 ` Steven Whitehouse 2008-04-24 14:44 ` Mauro Carvalho Chehab 2008-04-24 14:38 ` Steven Whitehouse 2008-04-24 16:29 ` Mauro Carvalho Chehab 2008-04-24 17:22 ` Guennadi Liakhovetski 2008-04-24 16:33 ` Mauro Carvalho Chehab 2008-04-24 17:25 ` Guennadi Liakhovetski 2008-04-24 17:55 ` Mauro Carvalho Chehab
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.