[GIT,PULL] HVR-900 R2 and PCTV 330e DVB support
Message ID | AANLkTi=hppcpARY1DOOJwK7kyKPe+2Q415jt8dNh8Z=-@mail.gmail.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Return-path: <mchehab@pedra> Envelope-to: mchehab@pedra Delivery-date: Thu, 24 Mar 2011 14:18:20 -0300 Received: from mchehab by pedra with local (Exim 4.72) (envelope-from <mchehab@pedra>) id 1Q2oAu-0001g2-DD for mchehab@pedra; Thu, 24 Mar 2011 14:18:20 -0300 Received: from casper.infradead.org [85.118.1.10] by pedra with IMAP (fetchmail-6.3.17) for <mchehab@localhost> (single-drop); Thu, 24 Mar 2011 14:18:20 -0300 (BRT) Received: from vger.kernel.org ([209.132.180.67]) by casper.infradead.org with esmtp (Exim 4.72 #1 (Red Hat Linux)) id 1Q2o9x-00061j-Aa; Thu, 24 Mar 2011 17:17:21 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757108Ab1CXRFh (ORCPT <rfc822; kmpark@infradead.org> + 1 other); Thu, 24 Mar 2011 13:05:37 -0400 Received: from mail-ew0-f46.google.com ([209.85.215.46]:49075 "EHLO mail-ew0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751263Ab1CXRFg (ORCPT <rfc822;linux-media@vger.kernel.org>); Thu, 24 Mar 2011 13:05:36 -0400 Received: by ewy4 with SMTP id 4so135192ewy.19 for <linux-media@vger.kernel.org>; Thu, 24 Mar 2011 10:05:35 -0700 (PDT) MIME-Version: 1.0 Received: by 10.213.21.206 with SMTP id k14mr176826ebb.13.1300986335187; Thu, 24 Mar 2011 10:05:35 -0700 (PDT) Received: by 10.213.8.69 with HTTP; Thu, 24 Mar 2011 10:05:35 -0700 (PDT) Date: Thu, 24 Mar 2011 13:05:35 -0400 Message-ID: <AANLkTi=hppcpARY1DOOJwK7kyKPe+2Q415jt8dNh8Z=-@mail.gmail.com> Subject: [GIT PULL] HVR-900 R2 and PCTV 330e DVB support From: Devin Heitmueller <dheitmueller@kernellabs.com> To: Linux Media Mailing List <linux-media@vger.kernel.org> Content-Type: text/plain; charset=ISO-8859-1 Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org Sender: <mchehab@pedra> |
Pull-request
git://sol.kernellabs.com/dheitmueller/drx.git drxdMessage
Devin Heitmueller
March 24, 2011, 5:05 p.m. UTC
This patch series finally merges in Ralph Metzler's drx-d driver and brings up the PCTV 330e and HVR-900R2. The patches have been tested for quite some time by users on the Kernel Labs blog, and they have been quite happy with them. The firmware required can be found here: http://kernellabs.com/firmware/drxd/ The following changes since commit 41f3becb7bef489f9e8c35284dd88a1ff59b190c: [media] V4L DocBook: update V4L2 version (2011-03-11 18:09:02 -0300) are available in the git repository at: git://sol.kernellabs.com/dheitmueller/drx.git drxd Devin Heitmueller (12): drx: add initial drx-d driver drxd: add driver to Makefile and Kconfig drxd: provide ability to control rs byte em28xx: enable support for the drx-d on the HVR-900 R2 drxd: provide ability to disable the i2c gate control function em28xx: fix GPIO problem with HVR-900R2 getting out of sync with drx-d em28xx: include model number for PCTV 330e em28xx: add digital support for PCTV 330e drxd: move firmware to binary blob em28xx: remove "not validated" flag for PCTV 330e em28xx: add remote control support for PCTV 330e drxd: Run lindent across sources Documentation/video4linux/CARDLIST.em28xx | 2 +- drivers/media/dvb/frontends/Kconfig | 11 + drivers/media/dvb/frontends/Makefile | 2 + drivers/media/dvb/frontends/drxd.h | 61 + drivers/media/dvb/frontends/drxd_firm.c | 929 ++ drivers/media/dvb/frontends/drxd_firm.h | 118 + drivers/media/dvb/frontends/drxd_hard.c | 2806 ++++++ drivers/media/dvb/frontends/drxd_map_firm.h |12694 +++++++++++++++++++++++++++ drivers/media/video/em28xx/em28xx-cards.c | 21 +- drivers/media/video/em28xx/em28xx-dvb.c | 22 +- drivers/media/video/em28xx/em28xx.h | 2 +- 11 files changed, 16649 insertions(+), 19 deletions(-) create mode 100644 drivers/media/dvb/frontends/drxd.h create mode 100644 drivers/media/dvb/frontends/drxd_firm.c create mode 100644 drivers/media/dvb/frontends/drxd_firm.h create mode 100644 drivers/media/dvb/frontends/drxd_hard.c create mode 100644 drivers/media/dvb/frontends/drxd_map_firm.h
Comments
Em 24-03-2011 14:05, Devin Heitmueller escreveu: > This patch series finally merges in Ralph Metzler's drx-d driver and > brings up the PCTV 330e and > HVR-900R2. The patches have been tested for quite some time by users > on the Kernel Labs blog, > and they have been quite happy with them. > > The firmware required can be found here: > > http://kernellabs.com/firmware/drxd/ > > The following changes since commit 41f3becb7bef489f9e8c35284dd88a1ff59b190c: > > [media] V4L DocBook: update V4L2 version (2011-03-11 18:09:02 -0300) > > are available in the git repository at: > git://sol.kernellabs.com/dheitmueller/drx.git drxd The pull request went fine. I'll be handling the series right now. One quick note for your next pull requests: Please don't add: Priority: normal Meta-tag on git patches. All patches are handled by default as normal patches. If you want to send me fixes, please use a separate pull request. Also, if a patch is meant to be sent also to stable kernels, just add: Cc: stable@kernel.org And upstream stable team will take care on it, when the patch arrives Linus tree. > > Devin Heitmueller (12): > drx: add initial drx-d driver > drxd: add driver to Makefile and Kconfig > drxd: provide ability to control rs byte > em28xx: enable support for the drx-d on the HVR-900 R2 > drxd: provide ability to disable the i2c gate control function > em28xx: fix GPIO problem with HVR-900R2 getting out of sync with drx-d > em28xx: include model number for PCTV 330e > em28xx: add digital support for PCTV 330e > drxd: move firmware to binary blob > em28xx: remove "not validated" flag for PCTV 330e > em28xx: add remote control support for PCTV 330e > drxd: Run lindent across sources > > Documentation/video4linux/CARDLIST.em28xx | 2 +- > drivers/media/dvb/frontends/Kconfig | 11 + > drivers/media/dvb/frontends/Makefile | 2 + > drivers/media/dvb/frontends/drxd.h | 61 + > drivers/media/dvb/frontends/drxd_firm.c | 929 ++ > drivers/media/dvb/frontends/drxd_firm.h | 118 + > drivers/media/dvb/frontends/drxd_hard.c | 2806 ++++++ > drivers/media/dvb/frontends/drxd_map_firm.h |12694 +++++++++++++++++++++++++++ > drivers/media/video/em28xx/em28xx-cards.c | 21 +- > drivers/media/video/em28xx/em28xx-dvb.c | 22 +- > drivers/media/video/em28xx/em28xx.h | 2 +- > 11 files changed, 16649 insertions(+), 19 deletions(-) > create mode 100644 drivers/media/dvb/frontends/drxd.h > create mode 100644 drivers/media/dvb/frontends/drxd_firm.c > create mode 100644 drivers/media/dvb/frontends/drxd_firm.h > create mode 100644 drivers/media/dvb/frontends/drxd_hard.c > create mode 100644 drivers/media/dvb/frontends/drxd_map_firm.h > > -- 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 Thu, Mar 24, 2011 at 1:34 PM, Mauro Carvalho Chehab <mchehab@redhat.com> wrote: > One quick note for your next pull requests: Please don't add: > > Priority: normal Sure, no problem. I actually knew that but just forgot to strip the priority tags from the patches when I converted the tree from hg to git. Thanks, Devin
Hi Devin, Em 24-03-2011 14:05, Devin Heitmueller escreveu: > This patch series finally merges in Ralph Metzler's drx-d driver and > brings up the PCTV 330e and > HVR-900R2. The patches have been tested for quite some time by users > on the Kernel Labs blog, > and they have been quite happy with them. > > The firmware required can be found here: > > http://kernellabs.com/firmware/drxd/ > > The following changes since commit 41f3becb7bef489f9e8c35284dd88a1ff59b190c: > > [media] V4L DocBook: update V4L2 version (2011-03-11 18:09:02 -0300) > > are available in the git repository at: > git://sol.kernellabs.com/dheitmueller/drx.git drxd > > Devin Heitmueller (12): > drx: add initial drx-d driver > drxd: add driver to Makefile and Kconfig > drxd: provide ability to control rs byte > em28xx: enable support for the drx-d on the HVR-900 R2 > drxd: provide ability to disable the i2c gate control function > em28xx: fix GPIO problem with HVR-900R2 getting out of sync with drx-d > em28xx: include model number for PCTV 330e > em28xx: add digital support for PCTV 330e > drxd: move firmware to binary blob > em28xx: remove "not validated" flag for PCTV 330e > em28xx: add remote control support for PCTV 330e > drxd: Run lindent across sources Still lots of CodingStyle issues, but they could be easily cleaned by a few scripting. I've cleaned them and added at my experimental tree: http://git.linuxtv.org/mchehab/experimental.git?a=shortlog;h=refs/heads/drxd It compiles fine, and I don't think that any of the changes would break DRX-D, but, in any case, it would be great if you could double check. I noticed just one issue with the drxd driver: it is still using a semaphore instead of a mutex: + struct semaphore mutex; ... +static int HI_CfgCommand(struct drxd_state *state) +{ + int status = 0; + + down(&state->mutex); It should be doing: s/struct semaphore/struct mutex/ s/down/mutex_lock/ s/up/mutex_unlock/ s/sema_init/mutex_init/ at the places it occur. I've added a patch for it at the end of the series. Could you please double check if everything is ok, for me to move this upstream? Thanks! 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
Hi Devin, Em 25-03-2011 13:09, Devin Heitmueller escreveu: > On Fri, Mar 25, 2011 at 11:50 AM, Mauro Carvalho Chehab > <mchehab@redhat.com> wrote: >> I've added a patch for it at the end of the series. >> >> Could you please double check if everything is ok, for me to move this upstream? >> >> Thanks! >> Mauro >> > > Sure, I will find some time this weekend to try out your tree. Any news? Thanks! 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
Em 02-04-2011 18:01, Devin Heitmueller escreveu: > On Sat, Apr 2, 2011 at 6:43 AM, Mauro Carvalho Chehab > <mchehab@redhat.com> wrote: >>> Sure, I will find some time this weekend to try out your tree. >> >> Any news? > > Sorry for the delay - I had taken apart my DVB-T generator setup and > hadn't gotten around to getting it set back up until now. > > I tried your patches - they don't appear to cause any breakage. > > Tested-by: Devin Heitmueller <dheitmueller@kernellabs.com> Thanks for the test. Merged it on my main tree. > > Thanks, > > Devin > -- 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