[GIT,PULL,FOR,v3.10] au0828 driver overhaul
Message ID | 201303221722.24739.hverkuil@xs4all.nl (mailing list archive) |
---|---|
State | Superseded, 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 1UJ4jj-0008Uq-1o; Fri, 22 Mar 2013 17:22:35 +0100 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-3) with esmtp id 1UJ4ji-00061J-E4; Fri, 22 Mar 2013 17:22:34 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933936Ab3CVQWc (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 22 Mar 2013 12:22:32 -0400 Received: from smtp-vbr10.xs4all.nl ([194.109.24.30]:1308 "EHLO smtp-vbr10.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933862Ab3CVQWb (ORCPT <rfc822;linux-media@vger.kernel.org>); Fri, 22 Mar 2013 12:22:31 -0400 Received: from alastor.dyndns.org (166.80-203-20.nextgentel.com [80.203.20.166] (may be forged)) (authenticated bits=0) by smtp-vbr10.xs4all.nl (8.13.8/8.13.8) with ESMTP id r2MGMQpO054487 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Fri, 22 Mar 2013 17:22:28 +0100 (CET) (envelope-from hverkuil@xs4all.nl) Received: from durdane.localnet (marune.xs4all.nl [80.101.105.217]) (Authenticated sender: hans) by alastor.dyndns.org (Postfix) with ESMTPSA id 5E11511E0194; Fri, 22 Mar 2013 17:22:26 +0100 (CET) From: Hans Verkuil <hverkuil@xs4all.nl> To: linux-media@vger.kernel.org Subject: [GIT PULL FOR v3.10] au0828 driver overhaul Date: Fri, 22 Mar 2013 17:22:24 +0100 User-Agent: KMail/1.13.7 (Linux/3.9.0-rc1-durdane; KDE/4.8.4; x86_64; ; ) Cc: Devin Heitmueller <dheitmueller@kernellabs.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Message-Id: <201303221722.24739.hverkuil@xs4all.nl> X-Virus-Scanned: by XS4ALL Virus Scanner 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: 2013.3.22.161217 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_2000_2999 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, __CT 0, __CTE 0, __CT_TEXT_PLAIN 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILING_LIST 0, __MIME_TEXT_ONLY 0, __MIME_VERSION 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS , __USER_AGENT 0' |
Pull-request
git://linuxtv.org/hverkuil/media_tree.git au0828bMessage
Hans Verkuil
March 22, 2013, 4:22 p.m. UTC
Hi all, This pull request converts the au0828/au8522 drivers to the latest frameworks, except for vb2 as usual. Tested with a WinTV aero generously donated by Hauppauge some time ago. I also did a lot of fixes in the disconnect handling and setting up the right routing/std information at the right time. It works fine with qv4l2, but there is still a bug causing tvtime to fail. That's caused by commit e58071f024aa337b7ce41682578b33895b024f8b, applied August last year, that broke g_tuner: after that 'signal' would always be 0 and tvtime expects signal to be non-zero for a valid frequency. The signal field is set by the au8522, but g_tuner is only called for the tuner (well, also for au8522 but since the i2c gate is set for the tuner that won't do anything). I have a patch for that but I want to convert that to using an i2c mux instead. For the time being I'd like to get this merged since at least it is in a lot better shape. Note: this pull request sits on top of this 'const' pull request: http://patchwork.linuxtv.org/patch/17568/ Regards, Hans The following changes since commit 8bf1a5a826d06a9b6f65b3e8dffb9be59d8937c3: v4l2-ioctl: add precision when printing names. (2013-03-22 11:59:21 +0100) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git au0828b for you to fetch changes up to 9b216a590115ea8aac389b9bb9248b7adce25f7f: au0828: improve firmware loading & locking. (2013-03-22 17:13:57 +0100) ---------------------------------------------------------------- Hans Verkuil (15): au8522_decoder: convert to the control framework. au0828: fix querycap. au0828: frequency handling fixes. au0828: fix intendation coding style issue. au0828: fix audio input handling. au0828: convert to the control framework. au0828: add prio, control event and log_status support au0828: add try_fmt_vbi support, zero vbi.reserved, pix.priv. au0828: replace deprecated current_norm by g_std. au8522_decoder: remove obsolete control ops. au0828: fix disconnect sequence. au0828: simplify i2c_gate_ctrl. au0828: don't change global state information on open(). au0828: fix initial video routing. au0828: improve firmware loading & locking. drivers/media/dvb-frontends/au8522_decoder.c | 123 ++++++++------------------ drivers/media/dvb-frontends/au8522_priv.h | 6 +- drivers/media/usb/au0828/au0828-core.c | 61 +++++++++---- drivers/media/usb/au0828/au0828-video.c | 286 ++++++++++++++++++++++++++++++++++++------------------------ drivers/media/usb/au0828/au0828.h | 7 ++ 5 files changed, 260 insertions(+), 223 deletions(-) -- 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
Comments
On Fri March 22 2013 17:22:24 Hans Verkuil wrote: > Hi all, > > This pull request converts the au0828/au8522 drivers to the latest frameworks, > except for vb2 as usual. > > Tested with a WinTV aero generously donated by Hauppauge some time ago. > > I also did a lot of fixes in the disconnect handling and setting up the > right routing/std information at the right time. > > It works fine with qv4l2, but there is still a bug causing tvtime to fail. > That's caused by commit e58071f024aa337b7ce41682578b33895b024f8b, applied > August last year, that broke g_tuner: after that 'signal' would always be 0 > and tvtime expects signal to be non-zero for a valid frequency. The signal > field is set by the au8522, but g_tuner is only called for the tuner (well, > also for au8522 but since the i2c gate is set for the tuner that won't do > anything). > > I have a patch for that but I want to convert that to using an i2c mux instead. > > For the time being I'd like to get this merged since at least it is in a > lot better shape. > > Note: this pull request sits on top of this 'const' pull request: > > http://patchwork.linuxtv.org/patch/17568/ > > Regards, > > Hans Oops, missed a compiler warning relating to the const changes. Nacked-by: Hans Verkuil <hans.verkuil@xs4all.nl> I'll post a new, fixed, version. Regards, Hans > > The following changes since commit 8bf1a5a826d06a9b6f65b3e8dffb9be59d8937c3: > > v4l2-ioctl: add precision when printing names. (2013-03-22 11:59:21 +0100) > > are available in the git repository at: > > git://linuxtv.org/hverkuil/media_tree.git au0828b > > for you to fetch changes up to 9b216a590115ea8aac389b9bb9248b7adce25f7f: > > au0828: improve firmware loading & locking. (2013-03-22 17:13:57 +0100) > > ---------------------------------------------------------------- > Hans Verkuil (15): > au8522_decoder: convert to the control framework. > au0828: fix querycap. > au0828: frequency handling fixes. > au0828: fix intendation coding style issue. > au0828: fix audio input handling. > au0828: convert to the control framework. > au0828: add prio, control event and log_status support > au0828: add try_fmt_vbi support, zero vbi.reserved, pix.priv. > au0828: replace deprecated current_norm by g_std. > au8522_decoder: remove obsolete control ops. > au0828: fix disconnect sequence. > au0828: simplify i2c_gate_ctrl. > au0828: don't change global state information on open(). > au0828: fix initial video routing. > au0828: improve firmware loading & locking. > > drivers/media/dvb-frontends/au8522_decoder.c | 123 ++++++++------------------ > drivers/media/dvb-frontends/au8522_priv.h | 6 +- > drivers/media/usb/au0828/au0828-core.c | 61 +++++++++---- > drivers/media/usb/au0828/au0828-video.c | 286 ++++++++++++++++++++++++++++++++++++------------------------ > drivers/media/usb/au0828/au0828.h | 7 ++ > 5 files changed, 260 insertions(+), 223 deletions(-) > -- > 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 > -- 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