Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-vpfe-cap

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sunday 21 June 2009 20:31:57 Mauro Carvalho Chehab wrote:
> Em Sun, 21 Jun 2009 20:20:29 +0200
>
> Hans Verkuil <hverkuil@xxxxxxxxx> escreveu:
> > On Sunday 21 June 2009 19:33:11 Mauro Carvalho Chehab wrote:
> > > Em Fri, 19 Jun 2009 14:39:14 +0200
> > >
> > > Hans Verkuil <hverkuil@xxxxxxxxx> 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:
> >
> > I'm assuming this can be corrected in a separate patch later? I don't
> > think this is a blocking issue.
>
> Yes, sure this is not blocking.
>
> BTW, please, _never_ send me an email to a public list C/C a
> subscribers-only list:
>
> Your mail to 'Davinci-linux-open-source' with the subject
>
>     Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-vpfe-cap
>
> Is being held until the list moderator can review it for approval.

I knew about this, but since this is the official submission of the davinci 
vpfe capture driver I could hardly exclude that list from this pull 
request! Perhaps next time I should use a BCC instead, that might be a 
reasonable compromise.

>
> > Muralidharan, can you take a look at this?

Yes, please.

Regards,

	Hans

> >
> > Regards,
> >
> > 	Hans
> >
> > > -/**
> > > +/*
> > >   * 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(-)



-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux