On 12/19/2013 05:00 AM, Antti Palosaari wrote:
> Add new V4L device type VFL_TYPE_SDR for Software Defined Radio.
> It is registered as /dev/swradio0 (/dev/sdr0 was already reserved).
>
> Cc: Hans Verkuil <hverk...@xs4all.nl>
> Signed-off-by: Antti Palosaari <cr...@iki.fi>
> ---
> drivers/media/v4l2-core/v4l2-dev.c | 5 +++++
> include/media/v4l2-dev.h | 3 ++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/media/v4l2-core/v4l2-dev.c
> b/drivers/media/v4l2-core/v4l2-dev.c
> index 1cc1749..6a1e6a8 100644
> --- a/drivers/media/v4l2-core/v4l2-dev.c
> +++ b/drivers/media/v4l2-core/v4l2-dev.c
> @@ -767,6 +767,8 @@ static void determine_valid_ioctls(struct video_device
> *vdev)
> * %VFL_TYPE_RADIO - A radio card
> *
> * %VFL_TYPE_SUBDEV - A subdevice
> + *
> + * %VFL_TYPE_SDR - Software Defined Radio
> */
> int __video_register_device(struct video_device *vdev, int type, int nr,
> int warn_if_nr_in_use, struct module *owner)
> @@ -806,6 +808,9 @@ int __video_register_device(struct video_device *vdev,
> int type, int nr,
> case VFL_TYPE_SUBDEV:
> name_base = "v4l-subdev";
> break;
> + case VFL_TYPE_SDR:
> + name_base = "swradio";
I would add a small comment here explaining why "sdr" is a bad name to
use.
After that you can add my:
Acked-by: Hans Verkuil <hans.verk...@cisco.com>
Regards,
Hans
> + break;
> default:
> printk(KERN_ERR "%s called with unknown type: %d\n",
> __func__, type);
> diff --git a/include/media/v4l2-dev.h b/include/media/v4l2-dev.h
> index c768c9f..eec6e46 100644
> --- a/include/media/v4l2-dev.h
> +++ b/include/media/v4l2-dev.h
> @@ -24,7 +24,8 @@
> #define VFL_TYPE_VBI 1
> #define VFL_TYPE_RADIO 2
> #define VFL_TYPE_SUBDEV 3
> -#define VFL_TYPE_MAX 4
> +#define VFL_TYPE_SDR 4
> +#define VFL_TYPE_MAX 5
>
> /* Is this a receiver, transmitter or mem-to-mem? */
> /* Ignored for VFL_TYPE_SUBDEV. */
>
--
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
