Re: [PATCH v4 2/3] v4l: Add V4L2_PIX_FMT_NV24 and V4L2_PIX_FMT_NV42 formats

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Sylwester,

On Tuesday 29 November 2011 18:19:09 Sylwester Nawrocki wrote:
> On 11/29/2011 11:26 AM, Laurent Pinchart wrote:
> > NV24 and NV42 are planar YCbCr 4:4:4 and YCrCb 4:4:4 formats with a
> > luma plane followed by an interleaved chroma plane.
> > 
> > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > ---
> 
> ...
> 
> > +	</example>
> > +      </refsect1>
> > +    </refentry>
> > +
> > 
> > +  <!--
> > +Local Variables:
> > +mode: sgml
> > +sgml-parent-document: "pixfmt.sgml"
> > +indent-tabs-mode: nil
> > +End:
> > +  -->
> 
> I think this comment chunk is redundant, it's just for emacs configuration.
> Every time I open the file containing litter like this I get a
> not-so-useful message, asking if I want to load the variables defined
> there or not.

I agree. I'll remove it. Thanks for pointing it out.

> I'm considering a patch cleaning up the Docbook from this emacs stuff, as I
> also made a mistake copying this comment when adding NV12M, NV12MT formats.
> 
> Looks like most of people are doing that, e.g. see
> 
> http://www.mail-archive.com/linux-media@vger.kernel.org/msg38637.html
> 
> ;)

-- 
Regards,

Laurent Pinchart