Message ID | 20120515194331.77C519D401E@zog.reactivated.net (mailing list archive) |
---|---|
State | Changes Requested, archived |
Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1SUNem-0002gx-Nb for patchwork@linuxtv.org; Tue, 15 May 2012 21:43:40 +0200 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.75/mailfrontend-2) with esmtp for <patchwork@linuxtv.org> id 1SUNem-0000Y3-GO; Tue, 15 May 2012 21:43:40 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966600Ab2EOTnh (ORCPT <rfc822;patchwork@linuxtv.org>); Tue, 15 May 2012 15:43:37 -0400 Received: from mtaout01-winn.ispmail.ntl.com ([81.103.221.47]:20225 "EHLO mtaout01-winn.ispmail.ntl.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966000Ab2EOTnh (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 15 May 2012 15:43:37 -0400 Received: from aamtaout04-winn.ispmail.ntl.com ([81.103.221.35]) by mtaout01-winn.ispmail.ntl.com (InterMail vM.7.08.04.00 201-2186-134-20080326) with ESMTP id <20120515194334.IKKX10903.mtaout01-winn.ispmail.ntl.com@aamtaout04-winn.ispmail.ntl.com>; Tue, 15 May 2012 20:43:34 +0100 Received: from zog.reactivated.net ([86.14.215.141]) by aamtaout04-winn.ispmail.ntl.com (InterMail vG.3.00.04.00 201-2196-133-20080908) with ESMTP id <20120515194334.NNLO23925.aamtaout04-winn.ispmail.ntl.com@zog.reactivated.net>; Tue, 15 May 2012 20:43:34 +0100 Received: by zog.reactivated.net (Postfix, from userid 1000) id 77C519D401E; Tue, 15 May 2012 20:43:31 +0100 (BST) From: Daniel Drake <dsd@laptop.org> To: mchehab@infradead.org, corbet@lwn.net Cc: linux-media@vger.kernel.org Subject: [PATCH] mmp-camera: specify XO-1.75 clock speed Message-Id: <20120515194331.77C519D401E@zog.reactivated.net> Date: Tue, 15 May 2012 20:43:31 +0100 (BST) X-Cloudmark-Analysis: v=1.1 cv=R50lirqlHffDPPkwUlkuVa99MrvKdVWo//yz83qex8g= c=1 sm=0 a=BU0CKTnuwSQA:10 a=vJ1w_8FsMGIA:10 a=Op-mwl0xAAAA:8 a=y-LFh6KHSdXjpOUp5MwA:9 a=d4CUUju0HPYA:10 a=HpAAvcLHHh0Zw7uRqdWCyQ==:117 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 5.6.1.2065439, Antispam-Engine: 2.7.2.376379, Antispam-Data: 2012.5.15.193315 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_1400_1499 0, BODY_SIZE_2000_LESS 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __HAS_MSGID 0, __HAS_X_MAILING_LIST 0, __INT_PROD_COMP 0, __MIME_TEXT_ONLY 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS ' |
Commit Message
Daniel Drake
May 15, 2012, 7:43 p.m. UTC
For the ov7670 camera to return images at the requested frame rate,
it needs to make calculations based on the clock speed, which is
a completely external factor (depends on the wiring of the system).
On the XO-1.75, which is the only known mmp-camera user, the camera
is clocked at 48MHz.
Pass this information to the ov7670 driver, to fix an issue where
a framerate faster than the requested amount was being provided.
Signed-off-by: Daniel Drake <dsd@laptop.org>
---
drivers/media/video/marvell-ccic/mmp-driver.c | 7 +++++++
1 file changed, 7 insertions(+)
Jon, is it OK to assume that XO-1.75 is the only mmp-camera user?
Comments
On Tue, 15 May 2012 20:43:31 +0100 (BST)
Daniel Drake <dsd@laptop.org> wrote:
> Jon, is it OK to assume that XO-1.75 is the only mmp-camera user?
It's the only one I know of at the moment, certainly.
Even so, I think it would be a lot better to put this parameter into the
mmp_camera_platform_data structure instead of wiring it into the driver
source; it could then be set in olpc-xo-1-75.c with the other relevant
parameters. I won't oppose the inclusion of this patch, but...any chance
it could be done that way?
Thanks,
jon
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, May 16, 2012 at 3:12 PM, Jonathan Corbet <corbet@lwn.net> wrote: > On Tue, 15 May 2012 20:43:31 +0100 (BST) > Daniel Drake <dsd@laptop.org> wrote: > >> Jon, is it OK to assume that XO-1.75 is the only mmp-camera user? > > It's the only one I know of at the moment, certainly. > > Even so, I think it would be a lot better to put this parameter into the > mmp_camera_platform_data structure instead of wiring it into the driver > source; it could then be set in olpc-xo-1-75.c with the other relevant > parameters. I won't oppose the inclusion of this patch, but...any chance > it could be done that way? I'll look into it. Please put this patch on pause for now. Thanks Daniel -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Em 16-05-2012 18:15, Daniel Drake escreveu: > On Wed, May 16, 2012 at 3:12 PM, Jonathan Corbet <corbet@lwn.net> wrote: >> On Tue, 15 May 2012 20:43:31 +0100 (BST) >> Daniel Drake <dsd@laptop.org> wrote: >> >>> Jon, is it OK to assume that XO-1.75 is the only mmp-camera user? >> >> It's the only one I know of at the moment, certainly. >> >> Even so, I think it would be a lot better to put this parameter into the >> mmp_camera_platform_data structure instead of wiring it into the driver >> source; it could then be set in olpc-xo-1-75.c with the other relevant >> parameters. I won't oppose the inclusion of this patch, but...any chance >> it could be done that way? > > I'll look into it. Please put this patch on pause for now. I've marked this patch as "changes requested" at patchwork: http://patchwork.linuxtv.org/patch/11270/ I agree with Jon: adding such config stuff at platform data is better. Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/media/video/marvell-ccic/mmp-driver.c b/drivers/media/video/marvell-ccic/mmp-driver.c index c4c17fe..0ba49c7 100644 --- a/drivers/media/video/marvell-ccic/mmp-driver.c +++ b/drivers/media/video/marvell-ccic/mmp-driver.c @@ -188,6 +188,13 @@ static int mmpcam_probe(struct platform_device *pdev) mcam->chip_id = V4L2_IDENT_ARMADA610; mcam->buffer_mode = B_DMA_sg; spin_lock_init(&mcam->dev_lock); + + /* + * Set the clock speed for the XO-1.75; I don't believe this + * driver has ever run anywhere else. + */ + mcam->clock_speed = 48; + /* * Get our I/O memory. */