On Tue, 02 Aug 2011 09:57:23 +0200 Kamil Debski wrote: > Hi, > > I am sorry, I did run "make htmldocs" and got no errors. So I have to be > doing something wrong building the docs. Could you tell me how do you build > the documentation? Knowing this I could check the next patch to make sure it > is error free.
I'm just running "make htmldocs". I expect the differences are in what versions of tools (or xml reference files etc.) we have installed. I'm using fedora 11 (which is fairly old). You have done what you should do, so I'll just ignore the warnings etc. (and move off of fedora 11 one day). Thanks. > Best wishes, > -- > Kamil Debski > Linux Platform Group > Samsung Poland R&D Center > > > > -----Original Message----- > > From: Randy Dunlap [mailto:rdun...@xenotime.net] > > Sent: 01 August 2011 19:24 > > To: Kamil Debski > > Cc: linux-media@vger.kernel.org; kyungmin.p...@samsung.com; > > jaeryul...@samsung.com; mche...@infradead.org > > Subject: Re: [PATCH] v4l2: Fix documentation of the codec device controls > > > > On Mon, 01 Aug 2011 16:51:39 +0200 Kamil Debski wrote: > > > > > Fixed missing ids of the codec controls description in the controls.xml > > file. > > > > > > Signed-off-by: Kamil Debski <k.deb...@samsung.com> > > > Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com> > > > Reported-by: Randy Dunlap <rdun...@xenotime.net> > > > --- > > > Hi, > > > > > > This patch fixes the problem with codec controls documentation reported by > > Randy > > > in the following email: > > > http://comments.gmane.org/gmane.linux.drivers.video-input- > > infrastructure/36288 > > > > > > Thanks for reporting those errors. > > > > > > Best wishes, > > > Kamil > > > > Hi, > > This patch eliminates some (3) of the reported errors, but not all of them. > > > > The fixed ones are: > > Error: no ID for constraint linkend: v4l2-mpeg-video-h264-level. > > Error: no ID for constraint linkend: v4l2-mpeg-video-h264-vui-sar-idc. > > Error: no ID for constraint linkend: v4l2-mpeg-video-mpeg4-level. > > > > but I still see these: > > Error: no ID for constraint linkend: v4l2-mpeg-video-header-mode. > > Error: no ID for constraint linkend: v4l2-mpeg-video-multi-slice-mode. > > Error: no ID for constraint linkend: v4l2-mpeg-video-h264-entropy-mode. > > Error: no ID for constraint linkend: v4l2-mpeg-video-h264-loop-filter-mode. > > Error: no ID for constraint linkend: v4l2-mpeg-video-h264-profile. > > Error: no ID for constraint linkend: v4l2-mpeg-video-mpeg4-profile. > > Error: no ID for constraint linkend: v4l2-mpeg-mfc51-video-frame-skip-mode. > > Error: no ID for constraint linkend: v4l2-mpeg-mfc51-video-force-frame-type. > > > > > > > --- > > > Documentation/DocBook/media/v4l/controls.xml | 24 ++++++++++++--------- > > --- > > > 1 files changed, 12 insertions(+), 12 deletions(-) > > > > > > diff --git a/Documentation/DocBook/media/v4l/controls.xml > > b/Documentation/DocBook/media/v4l/controls.xml > > > index 8516401..09d6872 100644 > > > --- a/Documentation/DocBook/media/v4l/controls.xml > > > +++ b/Documentation/DocBook/media/v4l/controls.xml > > > @@ -1455,7 +1455,7 @@ Applicable to the H264 encoder.</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-video-h264-vui-sar-idc"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_H264_VUI_SAR_IDC</constant>  > > ;</entry> > > > <entry>enum v4l2_mpeg_video_h264_vui_sar_idc</entry> > > > </row> > > > @@ -1561,7 +1561,7 @@ Applicable to the H264 encoder.</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-video-h264-level"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_H264_LEVEL</constant> </ent > > ry> > > > <entry>enum v4l2_mpeg_video_h264_level</entry> > > > </row> > > > @@ -1641,7 +1641,7 @@ Possible values are:</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-video-mpeg4-level"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_MPEG4_LEVEL</constant> </en > > try> > > > <entry>enum v4l2_mpeg_video_mpeg4_level</entry> > > > </row> > > > @@ -1689,7 +1689,7 @@ Possible values are:</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-h264-profile"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_H264_PROFILE</constant> </e > > ntry> > > > <entry>enum v4l2_mpeg_h264_profile</entry> > > > </row> > > > @@ -1774,7 +1774,7 @@ Possible values are:</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-mpeg4-profile"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_MPEG4_PROFILE</constant> </ > > entry> > > > <entry>enum v4l2_mpeg_mpeg4_profile</entry> > > > </row> > > > @@ -1820,7 +1820,7 @@ Applicable to the encoder. > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-multi-slice-mode"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_MULTI_SLICE_MODE</constant>  > > ;</entry> > > > <entry>enum v4l2_mpeg_multi_slice_mode</entry> > > > </row> > > > @@ -1868,7 +1868,7 @@ Applicable to the encoder.</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-h264-loop-filter-mode"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_H264_LOOP_FILTER_MODE</constant> > > </entry> > > > <entry>enum v4l2_mpeg_h264_loop_filter_mode</entry> > > > </row> > > > @@ -1913,9 +1913,9 @@ Applicable to the H264 encoder.</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-h264-entropy-mode"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_H264_ENTROPY_MODE</constant>&nbs > > p;</entry> > > > - <entry>enum v4l2_mpeg_h264_symbol_mode</entry> > > > + <entry>enum v4l2_mpeg_h264_entropy_mode</entry> > > > </row> > > > <row><entry spanname="descr">Entropy coding mode for H264 - > > CABAC/CAVALC. > > > Applicable to the H264 encoder. > > > @@ -2140,7 +2140,7 @@ previous frames. Applicable to the H264 > > encoder.</entry> > > > </row> > > > > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-header-mode"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_VIDEO_HEADER_MODE</constant> </en > > try> > > > <entry>enum v4l2_mpeg_header_mode</entry> > > > </row> > > > @@ -2320,7 +2320,7 @@ Valid only when H.264 and macroblock level RC is > > enabled (<constant>V4L2_CID_MPE > > > Applicable to the H264 encoder.</entry> > > > </row> > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-mfc51-frame-skip-mode"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_MFC51_VIDEO_FRAME_SKIP_MODE</constant> > > </entry> > > > <entry>enum v4l2_mpeg_mfc51_frame_skip_mode</entry> > > > </row> > > > @@ -2361,7 +2361,7 @@ the stream will meet tight bandwidth contraints. > > Applicable to encoders. > > > </entry> > > > </row> > > > <row><entry></entry></row> > > > - <row> > > > + <row id="v4l2-mpeg-mfc51-force-frame-type"> > > > <entry > > spanname="id"><constant>V4L2_CID_MPEG_MFC51_VIDEO_FORCE_FRAME_TYPE</constant > > > </entry> > > > <entry>enum v4l2_mpeg_mfc51_force_frame_type</entry> > > > </row> > > > -- > > > > > > --- > > ~Randy > > *** Remember to use Documentation/SubmitChecklist when testing your code *** > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code *** -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
