Re: [PATCH v12 0/3] TI WL1273 FM Radio driver...

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Matti,

I've acked patches 1 and 2 and added some comments to patch 3.

Two other patches are needed, though: the new BLOCK_IO and CONTROLS
capabilities also need to be set in existing RDS drivers:

video/saa6588.c
radio/radio-cadet.c
radio/si4713-i2c.c
radio/si470x/radio-si470x-common.c

The other patch is for v4l2-ctl.cpp in the v4l-utils repository where
support for these new caps needs to be added as well.

This second patch can be provided once the other patches are merged, although
it would be nice if you can have it earlier.

Regards,

	Hans

On Thursday, October 07, 2010 15:16:10 Matti J. Aaltonen wrote:
> Hello Mauro, Hans and others.
> 
> I haven't gotten any comments to the latest patch set. The audio part
> of the driver has already been accepted so I'm now trying to apply a
> similar approach as with the codec. I've abstracted out the physical
> control layer from the driver, it could use I2c or UART but the driver
> now has only read and write calls (and a couple of other calls). Also
> the driver doesn't export anything and it doesn't expose the FM radio
> bands it uses internally to the outsize world.
> 
> Cheers,
> Matti
> 
> 
> Matti J. Aaltonen (3):
>   V4L2: Add seek spacing and RDS CAP bits.
>   V4L2: WL1273 FM Radio: Controls for the FM radio.
>   Documentation: v4l: Add hw_seek spacing and two TUNER_RDS_CAP flags.
> 
>  Documentation/DocBook/v4l/dev-rds.xml              |   10 +-
>  .../DocBook/v4l/vidioc-s-hw-freq-seek.xml          |   10 +-
>  drivers/media/radio/Kconfig                        |   16 +
>  drivers/media/radio/Makefile                       |    1 +
>  drivers/media/radio/radio-wl1273.c                 | 1848 ++++++++++++++++++++
>  include/linux/videodev2.h                          |    5 +-
>  6 files changed, 1886 insertions(+), 4 deletions(-)
>  create mode 100644 drivers/media/radio/radio-wl1273.c
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG, part of Cisco
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux