Hi Mauro On Sat, 19 Sep 2009, Mauro Carvalho Chehab wrote: > Em Sat, 19 Sep 2009 01:06:02 -0300 > Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> escreveu: > > > Hi Guennadi, > > > > I'm about to send our pull request. > > > > While doing my last checks, I noticed a difference between our tree and > > upstream. I'm not sure what happens. Could you please check? > > > > The enclosed patch is the diff from upstream to -hg. > > Ok, I discovered the cause of the conflict: > git patch 6d1386c6b8db54ac8d94c01194e0c27cd538532b were applied before the > soc_camera conversion to v4l dev/subdev. > > I've applied the patch on our development tree. Still, we have a few diffs, > probably meaning that I solved it at the wrong way at git. No, please, don't change anything in our trees. Pual should have pushed his tree after v4l to Linus, but he has done it before. The idea is we should push our tree as is and then solve the conflict on merge. That should be easy. But if you start patching the v4l tree, that can make things much more complicated. BTW, your patch below is not the correct fix. Thanks Guennadi > > Please let me know what would be the proper way to fix it: by keeping > clk_enable/clk_disable (so reverting part of Magnus changes), > or by using, instead pm_runtime_get_sync/pm_runtime_put_sync. > > I guess the latter is the proper fix, but, as both use API's that are sh > specific, the better is if you could point me the right way. > > Cheers, > Mauro. > > diff -upr oldtree/drivers/media/video/sh_mobile_ceu_camera.c /home/v4l/tokernel/wrk/linux-next/drivers/media/video/sh_mobile_ceu_camera.c > --- oldtree/drivers/media/video/sh_mobile_ceu_camera.c 2009-09-19 09:08:13.000000000 -0300 > +++ /home/v4l/tokernel/wrk/linux-next/drivers/media/video/sh_mobile_ceu_camera.c 2009-09-19 01:35:28.000000000 -0300 > @@ -404,7 +404,7 @@ static int sh_mobile_ceu_add_device(stru > "SuperH Mobile CEU driver attached to camera %d\n", > icd->devnum); > > - pm_runtime_get_sync(ici->dev); > + clk_enable(pcdev->clk); > > ceu_write(pcdev, CAPSR, 1 << 16); /* reset */ > while (ceu_read(pcdev, CSTSR) & 1) > @@ -438,7 +438,7 @@ static void sh_mobile_ceu_remove_device( > } > spin_unlock_irqrestore(&pcdev->lock, flags); > > - pm_runtime_put_sync(ici->dev); > + clk_disable(pcdev->clk); > > dev_info(icd->dev.parent, > "SuperH Mobile CEU driver detached from camera %d\n", > > > > > > > Cheers, > Mauro > --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- 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
