Re: [PATCH 0/5] soc-camera: convert to platform device

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

 



On Fri, 17 Apr 2009, Magnus Damm wrote:

> On Fri, Apr 17, 2009 at 4:51 PM, Guennadi Liakhovetski
> <g.liakhovetski@xxxxxx> wrote:
> > On Fri, 17 Apr 2009, Magnus Damm wrote:
> >> On Wed, Apr 15, 2009 at 9:17 PM, Guennadi Liakhovetski
> >> <g.liakhovetski@xxxxxx> wrote:
> >> > This patch series is a preparation for the v4l2-subdev conversion. Please,
> >> > review and test. My current patch-stack in the form of a
> >> > (manually-created) quilt-series is at
> >> > http://www.open-technology.de/download/20090415/ based on linux-next
> >> > history branch, commit ID in 0000-base file. Don't be surprised, that
> >> > patch-set also contains a few not directly related patches.
> >>
> >> Testing on Migo-R board with 2.6.30-rc2-git-something and the
> >> following cherry-picked patches:
> >>
> >> 0007-driver-core-fix-driver_match_device.patch
> >> 0033-soc-camera-host-driver-cleanup.patch
> >> 0034-soc-camera-remove-an-extra-device-generation-from-s.patch
> >> 0035-soc-camera-simplify-register-access-routines-in-mul.patch
> >> and part of 0036 (avoiding rejects, ap325 seems broken btw)
> >
> > Have I broken it or is it unrelated?
> 
> 2.6.30-rc seems broken on Migo-R. A quick check suggests the following:

Ok, before we come to Migo-R, what is with ap325? Have I broken it with 
this my series or is it a different problem?

> V4L/DVB (10141): OK
> V4L/DVB (10672): BAD
> V4L/DVB (11024): BAD

These seem to be pretty random snapshots... Are they all on Linus' master 
or on next or on v4l-dvb? You did pick up the

0007-driver-core-fix-driver_match_device.patch

above, so, you know about that problem (I am very much surprised in fact 
that patch is still not in the mainline, just sent a reminder...), and you 
did use it in all your tests with kernels past 
49b420a13ff95b449947181190b08367348e3e1b didn't you?

> OK means mplayer capture works as excepted with CEU and ov772x.
> BAD means failure to open() /dev/video0 in the case of CEU. vivi works fine.
> 
> Morimoto-san and/or Guennadi, do you see the same thing?

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
--
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