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

[Brandon Philips <brandon@xxxxxxxx>]

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.

Thanks,

	Brandon

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