Re: [PATCH v10 3/4] V4L2: WL1273 FM Radio: Controls for the FM radio.

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

 



On Fri, 2010-09-24 at 15:09 +0200, ext Hans Verkuil wrote:
> > This driver implements V4L2 controls for the Texas Instruments
> > WL1273 FM Radio.
> >
> > Signed-off-by: Matti J. Aaltonen <matti.j.aaltonen@xxxxxxxxx>
> > ---
> >  drivers/media/radio/Kconfig        |   15 +
> >  drivers/media/radio/Makefile       |    1 +
> >  drivers/media/radio/radio-wl1273.c | 1859
> > ++++++++++++++++++++++++++++++++++++
> >  drivers/mfd/Kconfig                |    5 +
> >  drivers/mfd/Makefile               |    2 +
> >  5 files changed, 1882 insertions(+), 0 deletions(-)
> >  create mode 100644 drivers/media/radio/radio-wl1273.c
> >
> > +static int wl1273_fm_vidioc_querycap(struct file *file, void *priv,
> > +				     struct v4l2_capability *capability)
> > +{
> > +	struct wl1273_device *radio = video_get_drvdata(video_devdata(file));
> > +
> > +	dev_dbg(radio->dev, "%s\n", __func__);
> > +
> > +	strlcpy(capability->driver, WL1273_FM_DRIVER_NAME,
> > +		sizeof(capability->driver));
> > +	strlcpy(capability->card, "Texas Instruments Wl1273 FM Radio",
> > +		sizeof(capability->card));
> > +	strlcpy(capability->bus_info, "I2C", sizeof(capability->bus_info));
> > +
> > +	capability->capabilities = V4L2_CAP_HW_FREQ_SEEK |
> > +		V4L2_CAP_TUNER | V4L2_CAP_RADIO | V4L2_CAP_AUDIO |
> > +		V4L2_CAP_RDS_CAPTURE | V4L2_CAP_MODULATOR |
> > +		V4L2_CAP_RDS_OUTPUT | V4L2_TUNER_CAP_RDS_BLOCK_IO;
> > +
> > +	return 0;
> > +}
> 
> V4L2_TUNER_CAP_RDS_BLOCK_IO is a tuner/modulator capability! Not a
> querycap capability! It's added at the wrong place.

Alright... I'll fix that in v11...

Thanks,
Matti

> 
> Regards,
> 
>          Hans
> 


--
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