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 majord...@vger.kernel.org
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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
