Re: [PULL] http://www.kernellabs.com/hg/~stoth/saa7164-merge

From: Michael Krufky
Date: Sun Sep 06 2009 - 11:12:11 EST


On Thu, Sep 3, 2009 at 11:35 PM, Steven Toth<stoth@xxxxxxxxxxxxxx> wrote:
> Hello Mauro,
>
> This patch series adds support for the NXP SAA7164 PCIe A/V bridge used by
> the Hauppauge HVR-2200 and HVR-2250 series of products. Support is limited
> to DVB-T / ATSC / QAM digital TV only. The driver has been in development
> (on and off) for around a year and the KernelLabs saa7164-stable tree (from
> which this patch set was prepared) has been in testing worldwide since
> approx May(?) 2009.
>
> The project page including links for firmware downloads, MythTV and Wiki
> instructions is here: http://www.kernellabs.com/blog/?page_id=17
>
> Two general observations with the tree:
>
> 1. The driver is a little verbose during initial module load, I need to trim
> a few lines of debug.
> 2. During 64bit compile I have one compile time warning to be addressed.
>
> Both of these will be resolved shortly and should not stop the driver being
> merged and made available to a much wider range of testers.
>
> So, please pull from http://www.kernellabs.com/hg/~stoth/saa7164-merge
>
>   -  Add the SAA7164 I2C bus identifier
>   -  SAA7164: Add support for the NXP SAA7164 silicon
>   -  SAA7164: Fix some 32/64bit compile time warnings
>   -  SAA7164: Adjust I/F's to the TDA10048 enabling DVB-T lock
>   -  SAA7164: Email address change
>   -  SAA7164: Remove volatiles for PCI writes (coding style violation)
>   -  SAA7164: Increase firmware load tolerance
>   -  SAA7164: OOPS avoidance during interrupt handling
>   -  SAA7164: Removed spurious I2C errors during driver load with DVB-T
> boards.
>   -  SAA7164: Fix the 88021 definition to work with production boards.
>   -  SAA7164: Fixed the missing eeprom parse on a specific board.
>   -  SAA7164: Fix IRQ related system hang when firmware is not found.
>   -  SAA7164: Fix i2c eeprom read errors during load (some boards).
>   -  SAA7164: Ensure we specify I/F's for all bandwidths
>   -  SAA7164: Added waitsecs module parameter
>   -  SAA7164: Cleanup a printk
>   -  SAA7164: Increase the firmware command timeout to avoid firmware
> errors.
>   -  SAA7164: Removed a duplicate call to address any PCI quirks.
>   -  SAA7164: IRQ / message timeout related change
>   -  SAA7164: Removed spurious debug
>   -  SAA7164: HVR2250 changes related to attach time tuner configuration
>   -  SAA7164: Remove meaningless if'0 code
>   -  SAA7164: Minor i2c assignment cleanup
>   -  SAA7164: Ensure the HVR-2200 second tuner is configured in slave mode.
>   -  SAA7164: Add support for a new HVR-2250 hardware revision
>
>  b/linux/Documentation/video4linux/CARDLIST.saa7164   |    8
>  b/linux/drivers/media/video/saa7164/Kconfig          |   19
>  b/linux/drivers/media/video/saa7164/Makefile         |   12
>  b/linux/drivers/media/video/saa7164/saa7164-api.c    |  778 +++++++++
>  b/linux/drivers/media/video/saa7164/saa7164-buffer.c |  162 ++
>  b/linux/drivers/media/video/saa7164/saa7164-bus.c    |  448 +++++
>  b/linux/drivers/media/video/saa7164/saa7164-cards.c  |  600 +++++++
>  b/linux/drivers/media/video/saa7164/saa7164-cmd.c    |  529 ++++++
>  b/linux/drivers/media/video/saa7164/saa7164-core.c   |  797 ++++++++++
>  b/linux/drivers/media/video/saa7164/saa7164-dvb.c    |  594 +++++++
>  b/linux/drivers/media/video/saa7164/saa7164-fw.c     |  632 +++++++
>  b/linux/drivers/media/video/saa7164/saa7164-i2c.c    |  202 ++
>  b/linux/drivers/media/video/saa7164/saa7164-reg.h    |  183 ++
>  b/linux/drivers/media/video/saa7164/saa7164-types.h  |  287 +++
>  b/linux/drivers/media/video/saa7164/saa7164.h        |  405 +++++
>  b/v4l/scripts/saa7164.pl                             |   57
>  linux/Documentation/video4linux/CARDLIST.saa7164     |    5
>  linux/drivers/media/video/Kconfig                    |    2
>  linux/drivers/media/video/Makefile                   |    1
>  linux/drivers/media/video/saa7164/saa7164-api.c      |    6
>  linux/drivers/media/video/saa7164/saa7164-buffer.c   |   15
>  linux/drivers/media/video/saa7164/saa7164-bus.c      |    6
>  linux/drivers/media/video/saa7164/saa7164-cards.c    |   74
>  linux/drivers/media/video/saa7164/saa7164-cmd.c      |   47
>  linux/drivers/media/video/saa7164/saa7164-core.c     |   98 -
>  linux/drivers/media/video/saa7164/saa7164-dvb.c      |   70
>  linux/drivers/media/video/saa7164/saa7164-fw.c       |   10
>  linux/drivers/media/video/saa7164/saa7164.h          |   18
>  linux/include/linux/i2c-id.h                         |    1
>  v4l/scripts/cardlist                                 |    3
>  v4l/scripts/fix_dvb_customise.pl                     |    3
>  v4l/versions.txt                                     |    1
>  32 files changed, 5956 insertions(+), 117 deletions(-)

After testing this tree on my machine, I have hit some minor build
issues, none of which should prevent Steve's tree from being merged,
although it would be extremely helpful to users for my additional
fixes to be merged when Steve's tree gets merged.

Mauro, After merging Steve's tree, please pull these additional
fixes, or simply pull Steve's changesets and mine together from my
tree. Please pull from:

http://kernellabs.com/hg/~mkrufky/saa7164

for the following fixes:

- make v4l/scripts/saa7164.pl executable
- saa7164: fix Kconfig: rename DVB_FE_CUSTOMIZE to MEDIA_TUNER_CUSTOMISE
- saa7164: fix Kconfig: remove HOTPLUG dependency
- enable VIDEO_SAA7164 for 2.6.24 and later
- saa7164: fix 64bit build warning

linux/drivers/media/video/saa7164/Kconfig | 3 +--
linux/drivers/media/video/saa7164/saa7164-core.c | 2 +-
v4l/versions.txt | 2 +-
3 files changed, 3 insertions(+), 4 deletions(-)

Kudos to Steve for the long and hard work spent on this saa7164 driver.

Cheers,

Mike Krufky
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/