Re: [PATCHv4 0 of 8] FM Transmitter (si4713) and another changes

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

 



On Fri, May 29, 2009 at 08:03:59AM +0200, Valentin Eduardo (Nokia-D/Helsinki) wrote:
> Hi Douglas,
> 
> On Fri, May 29, 2009 at 07:36:20AM +0200, ext Douglas Schilling Landgraf wrote:
> > Hello Eduardo,
> > 
> > On Wed, 27 May 2009 12:35:47 +0300
> > Eduardo Valentin <eduardo.valentin@xxxxxxxxx> wrote:
> > 
> > > Hello all,
> > > 
> > >   I'm resending the FM transmitter driver and the proposed changes in
> > > v4l2 api files in order to cover the fmtx extended controls class.
> > > 
> > >   It is basically the same series of version #3. However I rewrote it
> > > to add the following comments:
> > > 
> > >   * Check kernel version for i2c helper function. Now the board data
> > > is passed not using i2c_board_info. This way all supported kernel
> > > versions can use the api. Besides that, the .s_config callback was
> > > added in core ops.
> > > 
> > >   * All patches are against v4l-dvb hg repository.
> > > 
> > >   Again, comments are welcome.
> > 
> > 
> > I have a comment, please check some headers to avoid errors. 
> > 
> > Instead of:
> > 
> > patch 05:
> > 
> > #include <media/linux/v4l2-device.h>
> > #include <media/linux/v4l2-ioctl.h>
> > #include <media/linux/v4l2-i2c-drv.h>
> > #include <media/linux/v4l2-subdev.h>
> > 
> > patch 06:
> > 
> > #include <media/linux/v4l2-device.h>
> > #include <media/linux/v4l2-common.h>
> > #include <media/linux/v4l2-ioctl.h>
> > 
> > Please use:
> > 
> > #include <media/v4l2-device.h>
> > #include <media/v4l2-ioctl.h>
> > #include <media/v4l2-i2c-drv.h>
> > #include <media/v4l2-subdev.h>
> > 
> > and
> > 
> > #include <media/v4l2-device.h>
> > #include <media/v4l2-common.h>
> > #include <media/v4l2-ioctl.h>
> 
> Right, I'll re send it.
> 
> Thanks for reviewing.

Actually this series is messed up. Sorry for that.
Resending.

> 
> > 
> > Cheers,
> > Douglas
> 
> -- 
> Eduardo Valentin

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