[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	Envelope-to: mchehab@pedra Delivery-date: Thu, 24 Mar 2011 14:18:20 -0300 MIME-Version: 1.0 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 Sender: <mchehab@pedra>

Message ID

AANLkTi=hppcpARY1DOOJwK7kyKPe+2Q415jt8dNh8Z=-@mail.gmail.com (mailing list archive)

State

Superseded, archived

Headers

Envelope-to: mchehab@pedra
Delivery-date: Thu, 24 Mar 2011 14:18:20 -0300
MIME-Version: 1.0
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
Sender: <mchehab@pedra>

Pull-request

git://sol.kernellabs.com/dheitmueller/drx.git drxd

Message

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

Mauro Carvalho Chehab March 24, 2011, 5:34 p.m. UTC | #1

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

Devin Heitmueller March 24, 2011, 5:36 p.m. UTC | #2

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

Mauro Carvalho Chehab March 25, 2011, 3:50 p.m. UTC | #3

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

Mauro Carvalho Chehab April 2, 2011, 10:43 a.m. UTC | #4

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

Mauro Carvalho Chehab April 2, 2011, 10:47 p.m. UTC | #5

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