Web lists-archives.org

Re: NACK NACK! [PATCH] Add two new fourcc codes for 16bpp formats

On Tue, 5 Feb 2008 00:00:38 -0800
Brandon Philips <brandon@xxxxxxxx> wrote:

> On 08:16 Tue 05 Feb 2008, Guennadi Liakhovetski wrote:
> > On Mon, 4 Feb 2008, Brandon Philips wrote:
> > 
> > > On 15:31 Thu 31 Jan 2008, Guennadi Liakhovetski wrote:
> > > > From: Steven Whitehouse <steve@xxxxxxxxxxx>
> > > > 
> > > > This adds two new fourcc codes (as per info at fourcc.org)
> > > > for 16bpp mono and 16bpp Bayer formats.
> > > 
> > > This patch was merged in the following commit:
> > >  http://linuxtv.org/hg/v4l-dvb/rev/d002378ff8c2
> > > 
> > > I have a number of issues:
> > >  
> > > - Why was V4L2_CID_AUTOEXPOSURE added!  I am working to get an auto
> > >   exposure control into the spec but this was merged without discussion.
> > >   Please remove this and wait for my patch.
> > > 
> > > - Why was a SoC config option added with this commit?
> > > 
> > > - mailimport changes in this commit too!  Why is mailimport running
> > >   sudo!?! 
> > > 
> > > A mistake was obviously made here.
> > 
> > Yes, strange. In the original patch
> > 
> > http://marc.info/?l=linux-video&m=120179045830566&w=2
> > 
> > it was still ok.
> 
> Yea, it must have been something on Mauro's end.

Yes. It was a silly mistake from my side... I'll revert soon. I'm currently
backporting kernel changes.

I should be using a separate tree for testing newer changesets. Unfortunately,
mercurial spends a large amount of disk space when you fork a tree, since it
doesn't support versioning, and the 160Gb disk on my notebook is almost full
with lots and lots of mercurial branches :( So, sometimes, bad things happen.

Maybe we've took the wrong direction when we've decided to select
mercurial. It were better and easier to use, on that time, but the -git
improvements happened too fast.

Cheers,
Mauro

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