Re: [PATCHv5 1 of 8] v4l2_subdev i2c: Add v4l2_i2c_new_subdev_board i2c helper function

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

 



Hi guys,

On Mon, Jun 08, 2009 at 05:19:22AM +0200, ext Douglas Schilling Landgraf wrote:
> Hi,
> 
> On Sun, 7 Jun 2009 22:29:14 -0300
> Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> wrote:
> 
> > Em Sun, 7 Jun 2009 08:40:08 +0200
> > Hans Verkuil <hverkuil@xxxxxxxxx> escreveu:
> > 
> > > On Saturday 06 June 2009 22:40:21 Eduardo Valentin wrote:
> > > > Hi Hans,
> > > >
> > > > On Sat, Jun 6, 2009 at 8:09 PM, Hans Verkuil <hverkuil@xxxxxxxxx>
> > > > wrote:
> > > > > On Saturday 06 June 2009 14:49:46 Hans Verkuil wrote:
> > > > > > On Saturday 06 June 2009 13:59:19 Hans Verkuil wrote:

<snip>

> > 
> > No please. We did already lots of change due to the i2c changes, and
> > there are still some occasional complaints at ML about regressions
> > that might be due to i2c changes.
> > 
> > Let's keep 2.6.31 clean, as previously agreed, without new KABI
> > changes. We should focus 2.6.31 on fixing any core issues that may
> > still have. Only with 2.6.30 we'll start to have feedbacks from
> > normal users.

<snip>

> > > >
> > > > I've cloned your tree and took a look at your code. Well, looks
> > > > like the proper way to do this change.
> > > > I didn't take this approach because it touchs other drivers.
> > > > However, concentrating the code  in only one
> > > > function is better. I also saw that you have fixed the kernel
> > > > version check in the v4l2_device_unregister
> > > > function. Great!
> > > >
> > > > I will resend my series without this patch. I will rebase it on
> > > > top of your subdev tree so the new api
> > > > can be used straight. Is that ok?
> > > 
> > > Yes, sure. Just be aware that there may be some small changes to my
> > > patch based on feedback I get. But it is a good test anyway of this
> > > API to see if it works well for you.
> > 
> > Eduardo,
> > 
> > Let's analyze and merge your changes using the current development
> > tree. If you think that Hans approach is better (I haven't analyzed
> > it yet), then it can later be converted to the new approach
> > 
> 
> I have talked with Eduardo during last week and if there is no
> objections, I am ready to request a pull from the current/last
> patches series.

Yes, my series is already in one of Douglas' trees and we have tested it.
However, in that series there is one patch which does partially what Hans is
proposing. Which is: add a way to pass platform info to i2c drivers, using
v4l2 i2c helper functions. They way it is done in this patch it does not affect
any other driver. Hans did also some re-factoring in existing i2c helper function,
besides adding new way to pass platform data.

If you agree we can use it for now and in next window we
change things to have them using the way Hans did (which is more complete).

What do you think?

> 
> Cheers,
> Douglas


Cheers,

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