Em Fri, 19 Jun 2009 14:39:14 +0200
Hans Verkuil <hverk...@xs4all.nl> escreveu:

> Hi Mauro,
> 
> Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-vpfe-cap for 
> the following:
> 
> - tvp514x: Migration to sub-device framework

Hmm... the kernel-doc format is wrong on all function description comment
changes on tvp514x:

-/**
+/*
  * struct tvp514x_decoder - TVP5146/47 decoder object
- * @v4l2_int_device: Slave handle
- * @tvp514x_slave: Slave pointer which is used by @v4l2_int_device
+ * @sd: Subdevice Slave handle
  * @tvp514x_regs: copy of hw's regs with preset values.
  * @pdata: Board specific
- * @client: I2C client data
- * @id: Entry from I2C table
  * @ver: Chip version
- * @state: TVP5146/47 decoder state - detected or not-detected
+ * @streaming: TVP5146/47 decoder streaming - enabled or disabled.
  * @pix: Current pixel format
  * @num_fmts: Number of formats
  * @fmt_list: Format list
  * @current_std: Current standard
  * @num_stds: Number of standards
  * @std_list: Standards list
- * @route: input and output routing at chip level
+ * @input: Input routing at chip level
+ * @output: Output routing at chip level
  */

Please read Documentation/kernel-doc-nano-HOWTO.txt for the proper format.
Basically, it should be like this example:

/**
 * foobar() - short function description of foobar
 * @arg1:       Describe the first argument to foobar.
 * @arg2:       Describe the second argument to foobar.
 *              One can provide multiple line descriptions
 *              for arguments.
 *
 * A longer description, with more discussion of the function foobar()
 * that might be useful to those using or modifying it.  Begins with
 * empty comment line, and may include additional embedded empty
 * comment lines.
 *
 * The longer description can have multiple paragraphs.
 **/

> - v4l: vpfe capture bridge driver for DM355 and DM6446
> - vpfe_capture: add missing newlines and fix an incorrect error code.
> - v4l: ccdc hw device header file for vpfe capture
> - v4l: dm355 ccdc module for vpfe capture driver
> - v4l: dm644x ccdc module for vpfe capture driver
> - v4l: ccdc types used across ccdc modules for vpfe capture driver
> - v4l: common vpss module for video drivers
> - v4l: Makefile and config files for vpfe capture driver
> - v4l: davinci drivers should only be compiled for kernels >= 2.6.31.

I'll try to analyze those remaining patches later today. Unfortunately, I'm
very busy this weekend finishing some pending work.

> 
> Hopefully these changes can be merged into 2.6.31.
> 
> There are two arch/arm patches that need to be applied to the git tree. 
> These patches should be applied last.
> 
> They are:
> 
> http://patchwork.kernel.org/patch/30968/
> http://patchwork.kernel.org/patch/30974/
> 
> Note that I had to move patch 9 (vpss) in the original patch series before 
> patch 6 (the Makefile changes) to make sure everything would still compile.
> 
> Thanks,
> 
>         Hans
> 
> diffstat:
>  b/linux/drivers/media/video/davinci/Makefile           |    9
>  b/linux/drivers/media/video/davinci/ccdc_hw_device.h   |  110
>  b/linux/drivers/media/video/davinci/dm355_ccdc.c       | 1163 +++++++++
>  b/linux/drivers/media/video/davinci/dm355_ccdc_regs.h  |  310 ++
>  b/linux/drivers/media/video/davinci/dm644x_ccdc.c      |  878 ++++++
>  b/linux/drivers/media/video/davinci/dm644x_ccdc_regs.h |  145 +
>  b/linux/drivers/media/video/davinci/vpfe_capture.c     | 2136 
> +++++++++++++++++
>  b/linux/drivers/media/video/davinci/vpss.c             |  301 ++
>  b/linux/include/media/davinci/ccdc_types.h             |   43
>  b/linux/include/media/davinci/dm355_ccdc.h             |  336 ++
>  b/linux/include/media/davinci/dm644x_ccdc.h            |  184 +
>  b/linux/include/media/davinci/vpfe_capture.h           |  188 +
>  b/linux/include/media/davinci/vpfe_types.h             |   51
>  b/linux/include/media/davinci/vpss.h                   |   69
>  linux/drivers/media/video/Kconfig                      |   49
>  linux/drivers/media/video/Makefile                     |    2
>  linux/drivers/media/video/davinci/vpfe_capture.c       |   27
>  linux/drivers/media/video/tvp514x.c                    |  879 ++----
>  linux/drivers/media/video/tvp514x_regs.h               |   10
>  linux/include/media/tvp514x.h                          |    4
>  v4l/versions.txt                                       |    7
>  21 files changed, 6346 insertions(+), 555 deletions(-)
> 


-- 

Cheers,
Mauro
--
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

Reply via email to