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

Hi Sylwester,

On Tuesday 29 November 2011 18:19:09 Sylwester Nawrocki wrote:

On 11/29/2011 11:26 AM, Laurent Pinchart wrote:

quoted

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>
---

...

quoted

+	</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