From patchwork Sun May 22 08:16:34 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Jean-Francois Moine X-Patchwork-Id: 806342 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter2.kernel.org (8.14.4/8.14.3) with ESMTP id p4M8G9N1026350 for ; Sun, 22 May 2011 08:16:09 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752867Ab1EVIQH (ORCPT ); Sun, 22 May 2011 04:16:07 -0400 Received: from smtp5-g21.free.fr ([212.27.42.5]:43054 "EHLO smtp5-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752539Ab1EVIQF convert rfc822-to-8bit (ORCPT ); Sun, 22 May 2011 04:16:05 -0400 Received: from tele (unknown [82.245.201.222]) by smtp5-g21.free.fr (Postfix) with ESMTP id F197AD4821B for ; Sun, 22 May 2011 10:15:58 +0200 (CEST) Date: Sun, 22 May 2011 10:16:34 +0200 From: Jean-Francois Moine To: linux-media@vger.kernel.org Subject: [PATCH FOR 2.6.39] gspca - ov519: Fix a regression for ovfx2 webcams Message-ID: <20110522101634.33e10357@tele> X-Mailer: Claws Mail 3.7.9 (GTK+ 2.24.4; x86_64-pc-linux-gnu) Mime-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.6 (demeter2.kernel.org [140.211.167.43]); Sun, 22 May 2011 08:16:09 +0000 (UTC) By git commit c42cedbb658b, the bulk transfer size was changed to a lower value for resolutions != 1600x1200, but the image extraction routine still worked with the previous value, giving bad truncated images. Signed-off-by: Jean-François Moine --- drivers/media/video/gspca/ov519.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/media/video/gspca/ov519.c b/drivers/media/video/gspca/ov519.c index 36a46fc..5ac2f3c 100644 --- a/drivers/media/video/gspca/ov519.c +++ b/drivers/media/video/gspca/ov519.c @@ -4478,7 +4478,7 @@ static void ovfx2_pkt_scan(struct gspca_dev *gspca_dev, gspca_frame_add(gspca_dev, INTER_PACKET, data, len); /* A short read signals EOF */ - if (len < OVFX2_BULK_SIZE) { + if (len < gspca_dev->cam.bulk_size) { /* If the frame is short, and it is one of the first ones the sensor and bridge are still syncing, so drop it. */ if (sd->first_frame) {