From patchwork Wed Mar 16 19:16:41 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Olli Salonen X-Patchwork-Id: 33498 Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1agGwB-0006NT-28; Wed, 16 Mar 2016 19:16:55 +0000 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.76/mailfrontend-8) with esmtp id 1agGw8-00008o-mL; Wed, 16 Mar 2016 20:16:54 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934332AbcCPTQv (ORCPT + 1 other); Wed, 16 Mar 2016 15:16:51 -0400 Received: from mail-lf0-f44.google.com ([209.85.215.44]:35043 "EHLO mail-lf0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752138AbcCPTQu (ORCPT ); Wed, 16 Mar 2016 15:16:50 -0400 Received: by mail-lf0-f44.google.com with SMTP id l202so26683593lfl.2 for ; Wed, 16 Mar 2016 12:16:49 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=PLUTJphSBmLU57eqVwx54YZtLaIVHfuYg4qzMPTeN6I=; b=CKFkmiJdA9DbZ3z2Yltqg0MZjPVYsHo87i1vR6MDHBbTR0xQXLLPFQh6LA8ZkdddZi Omqw7MRaDSPaR2Vj7+PPooqCsHzovWhCUGnz9xtbwb3e1qtA6aUz83HSNMlVxemPz47u GyrEcubWey2mi+EwKGSXPttiwzV4HdiyUoV+T816oQhdt28ejXBYQhOu0Cxx+6Ekbw9F lJq2W+WLkBeU6CEbFgOVoWbHIHhc2vgAComK2gIZuUGss4cXrd+MMUPxBUiKyIS8VXAG cfG/jEeNEX4ERcJHg0W0zHxDS6opvslcanUN5o6ZLkZtuMfC6aXv7N6R+03YA5aT3XRQ 9k1Q== X-Gm-Message-State: AD7BkJLIr+/PjpOXt14RvHxRGmCsDHpKcZjXt+4gvkDecP4IdJGiqpPCoWYiOnJoQoC4Pw== X-Received: by 10.25.206.135 with SMTP id e129mr2150827lfg.46.1458155808515; Wed, 16 Mar 2016 12:16:48 -0700 (PDT) Received: from dl160.lan (178-55-1-76.bb.dnainternet.fi. [178.55.1.76]) by smtp.gmail.com with ESMTPSA id h6sm771742lbv.43.2016.03.16.12.16.47 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 16 Mar 2016 12:16:47 -0700 (PDT) From: Olli Salonen To: linux-media@vger.kernel.org Cc: Olli Salonen Subject: [PATCH] pctv452e: correct parameters for TechnoTrend TT S2-3600 Date: Wed, 16 Mar 2016 21:16:41 +0200 Message-Id: <1458155801-14534-1-git-send-email-olli.salonen@iki.fi> X-Mailer: git-send-email 1.9.1 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2016.3.16.190917 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_1500_1599 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_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __HTTPS_URI 0, __INT_PROD_TV 0, __MIME_TEXT_ONLY 0, __MULTIPLE_URI_TEXT 0, __SANE_MSGID 0, __SUBJ_ALPHA_NEGATE 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_IN_BODY 0, __URI_NS , __URI_WITH_PATH 0, __YOUTUBE_RCVD 0' 2008-02-25 Andre Weidemann added support for TT S2-3600 and noted that he still gets image distortions every now and then. It seems to be common knowledge in many projects that changing the USB parameters seems to help. OpenELEC has included this patch for a few years, for example. Nobody bothered to report the issue upstream though, it seems. References: https://github.com/OpenELEC/OpenELEC.tv/issues/1957 http://www.vdr-portal.de/board60-linux/board14-betriebssystem/board96-yavdr/p1033458-darstellungsproblem-bei-2-tt-3600-usb/#post1033458 (in German) I'd suggest we include the patch below. Any objections? Signed-off-by: Olli Salonen --- drivers/media/usb/dvb-usb/pctv452e.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/dvb-usb/pctv452e.c b/drivers/media/usb/dvb-usb/pctv452e.c index ec397c4..c05de1b 100644 --- a/drivers/media/usb/dvb-usb/pctv452e.c +++ b/drivers/media/usb/dvb-usb/pctv452e.c @@ -995,11 +995,11 @@ static struct dvb_usb_device_properties tt_connect_s2_3600_properties = { /* parameter for the MPEG2-data transfer */ .stream = { .type = USB_ISOC, - .count = 7, + .count = 4, .endpoint = 0x02, .u = { .isoc = { - .framesperurb = 4, + .framesperurb = 64, .framesize = 940, .interval = 1 }