Em Tue, 20 Aug 2013 18:39:33 +0200 Frank Schäfer <fschaefer.oss@xxxxxxxxxxxxxx> escreveu: > Am 20.08.2013 17:31, schrieb Mauro Carvalho Chehab: > > Em Tue, 20 Aug 2013 15:38:57 +0200 > > Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> escreveu: > > > >> Hi Mauro, > >> > >> On Sunday 18 August 2013 12:20:08 Mauro Carvalho Chehab wrote: > >>> Em Sun, 18 Aug 2013 13:40:25 +0200 Frank Schäfer escreveu: > >>>> Am 17.08.2013 12:51, schrieb Guennadi Liakhovetski: > >>>>> Hi Frank, > >>>>> As I mentioned on the list, I'm currently on a holiday, so, replying > >>>>> briefly. > >>>> Sorry, I missed that (can't read all mails on the list). > >>>> > >>>>> Since em28xx is a USB device, I conclude, that it's supplying clock to > >>>>> its components including the ov2640 sensor. So, yes, I think the driver > >>>>> should export a V4L2 clock. > >>>> Ok, so it's mandatory on purpose ? > >>>> I'll take a deeper into the v4l2-clk code and the > >>>> em28xx/ov2640/soc-camera interaction this week. > >>>> Have a nice holiday ! > >>> commit 9aea470b399d797e88be08985c489855759c6c60 > >>> Author: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> > >>> Date: Fri Dec 21 13:01:55 2012 -0300 > >>> > >>> [media] soc-camera: switch I2C subdevice drivers to use v4l2-clk > >>> > >>> Instead of centrally enabling and disabling subdevice master clocks in > >>> soc-camera core, let subdevice drivers do that themselves, using the > >>> V4L2 clock API and soc-camera convenience wrappers. > >>> > >>> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> > >>> Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > >>> Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > >>> Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > >>> > >>> (c/c the ones that acked with this broken changeset) > >>> > >>> We need to fix it ASAP or to revert the ov2640 changes, as some em28xx > >>> cameras are currently broken on 3.10. > >>> > >>> I'll also reject other ports to the async API if the drivers are > >>> used outside an embedded driver, as no PC driver currently defines > >>> any clock source. The same applies to regulators. > >>> > >>> Guennadi, > >>> > >>> Next time, please check if the i2c drivers are used outside soc_camera > >>> and apply the fixes where needed, as no regressions are allowed. > >> We definitely need to check all users of our sensor drivers when making such a > >> change. Mistakes happen, so let's fix them. > >> > >> Guennadi is on holidays until the end of this week. Would that be too late to > >> fix the issue (given that 3.10 is already broken) ? > > Well, it is simple: we should either revert the patch(es) that broke it or > > someone should fix it at em28xx. If nobody could fix it, I'll just revert > > the patches that broke it and ask -stable to do the same. > > > > Btw, 3.10 is a long term stable, so, it is not too late for fixes there. > AFAICS, 3.10 should be fine. > It should be possible to fix em28xx before Linus releases 3.11, but what > about other drivers ? > It seems like a v4l2-clock has been made mandatory for all sensor > drivers (not only ov2640). > I don't know if there are any other users of these drivers apart from > soc_camera and em28xx... ? Currently, gspca doesn't use the sensors drivers (nor uvc). So, very few places drivers are actually affected. I think only em28xx is affected by ov2640 changes. > > >> The fix shouldn't be too > >> complex, registering a dummy V4L2 clock in the em28xx driver should be enough. > >> v4l2-clk.c should provide a helper function to do so as that will be a pretty > >> common operation. > > Ok, but this doesn't solve one issue: who would do it and when. > I can spend some time on em28xx tomorrow evening. > > Regards, > Frank > > > > > Cheers, > > Mauro > -- Cheers, Mauro -- 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