mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Laurent Pinchart <laurent.pinchart@skynet.be>
Cc: video4linux-list@redhat.com,
	"v4l-dvb maintainer list" <v4l-dvb-maintainer@linuxtv.org>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	linux-kernel@vger.kernel.org,
	davinci-linux-open-source-bounces@linux.davincidsp.com,
	Mauro Carvalho Chehab <mchehab@infradead.org>
Subject: Re: [PULL] http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-ng
Date: Mon, 1 Dec 2008 13:51:55 +0100	[thread overview]
Message-ID: <200812011351.55323.hverkuil@xs4all.nl> (raw)
In-Reply-To: <200812011331.25676.laurent.pinchart@skynet.be>

On Monday 01 December 2008 13:31:25 Laurent Pinchart wrote:
> On Monday 01 December 2008, Hans Verkuil wrote:
> > Hi Mauro,
> >
> > Please pull from http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-ng for
> > the following:
> >
> > - v4l2: add v4l2_device and v4l2_subdev structs to the v4l2
> > framework. - v4l2-common: add i2c helper functions
> > - cs53l32a: convert to v4l2_subdev.
> > - cx25840: convert to v4l2_subdev.
> > - m52790: convert to v4l2_subdev.
> > - msp3400: convert to v4l2_subdev.
> > - saa7115: convert to v4l2_subdev.
> > - saa7127: convert to v4l2_subdev.
> > - saa717x: convert to v4l2_subdev.
> > - tuner: convert to v4l2_subdev.
> > - upd64031a: convert to v4l2_subdev.
> > - upd64083: convert to v4l2_subdev.
> > - vp27smpx: convert to v4l2_subdev.
> > - wm8739: convert to v4l2_subdev.
> > - wm8775: convert to v4l2_subdev.
> > - ivtv/ivtvfb: convert to v4l2_device/v4l2_subdev.
> >
> > All points raised in reviews are addressed so I think it is time to
> > get this merged so people can start to use it.
>
> Does linuxtv.org and Mercurial provide the necessary infrastructure
> to integrate those changes into the v4l-dvb repository while not
> pushing them upstream yet ? I'd like to see more people testing (and
> breaking and fixing :-)) your changes before they reach the mainline
> kernel.

That's basically why I want this to go into the v4l-dvb repository: this 
makes it easier for people to start working with it. It doesn't affect 
existing drivers, except for the i2c driver changes and those changes 
are just transforming a big switch to a set of functions. So I really 
consider this a pretty low-risk merge.

If someone is willing to do some testing with my tree in the next two 
weeks then I don't mind waiting, but it's been in development now from 
early September (if not earlier) and been reviewed several times. In 
addition, ivtv has been modified to work with it and that driver uses 
more sub-devices by far than any other driver.

I don't know what more I can do, to be honest.

Regards,

	Hans

>
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@skynet.be>
> > Reviewed-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
> > Reviewed-by: Andy Walls <awalls@radix.net>
> > Reviewed-by: David Brownell <david-b@pacbell.net>
> >
> > Once this is in I'll start on converting the other i2c drivers.
>
> Best regards,
>
> Laurent Pinchart



-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG

  reply	other threads:[~2008-12-01 12:52 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-12-01 11:46 Hans Verkuil
2008-12-01 12:27 ` Mauro Carvalho Chehab
2008-12-01 12:31   ` Hans Verkuil
2008-12-01 12:31 ` Laurent Pinchart
2008-12-01 12:51   ` Hans Verkuil [this message]
2008-12-01 13:29     ` Laurent Pinchart
2008-12-01 13:45       ` Hans Verkuil
2008-12-01 14:24         ` Laurent Pinchart
2008-12-01 14:47           ` Hans Verkuil
2008-12-02 17:01             ` Laurent Pinchart
2008-12-01 15:06           ` Mauro Carvalho Chehab
2008-12-01 18:07             ` Hans Verkuil
2008-12-02 16:58             ` Laurent Pinchart
2008-12-02 17:21               ` Mauro Carvalho Chehab
2008-12-02 17:26                 ` Hans Verkuil
2008-12-02  0:02         ` Andy Walls

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200812011351.55323.hverkuil@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=davinci-linux-open-source-bounces@linux.davincidsp.com \
    --cc=laurent.pinchart@skynet.be \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=v4l-dvb-maintainer@linuxtv.org \
    --cc=video4linux-list@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®