[RFC PATCH 0/6] Add frequency band information

2012-07-02 Thread Hans Verkuil
Hi all,

This patch series adds support for the new VIDIOC_ENUM_FREQ_BANDS ioctl that
tells userspace if a tuner supports multiple frequency bands.

This API is based on earlier attempts:

http://www.spinics.net/lists/linux-media/msg48391.html
http://www.spinics.net/lists/linux-media/msg48435.html

And an irc discussion:

http://linuxtv.org/irc/v4l/index.php?date=2012-06-26

This irc discussion also talked about adding rangelow/high to the 
v4l2_hw_freq_seek
struct, but I decided not to do that. The hwseek additions are independent to 
this
patch series, and I think it is best if Hans de Goede does that part so that 
that
API change can be made together with a driver that actually uses it.

In order to show how the new ioctl is used, this patch series adds support for 
it
to the very, very old radio-cadet driver.

Comments are welcome!

Regards,

Hans

PS: Mauro, I haven't been able to work on the radio profile yet, so that's not
included.

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


Re: [RFC PATCH 0/6] Add frequency band information

2012-07-02 Thread Hans de Goede

Hi,

Series looks good to me, ack series:
Acked-by: Hans de Goede hdego...@redhat.com

Regards,

Hans

On 07/02/2012 04:15 PM, Hans Verkuil wrote:

Hi all,

This patch series adds support for the new VIDIOC_ENUM_FREQ_BANDS ioctl that
tells userspace if a tuner supports multiple frequency bands.

This API is based on earlier attempts:

http://www.spinics.net/lists/linux-media/msg48391.html
http://www.spinics.net/lists/linux-media/msg48435.html

And an irc discussion:

http://linuxtv.org/irc/v4l/index.php?date=2012-06-26

This irc discussion also talked about adding rangelow/high to the 
v4l2_hw_freq_seek
struct, but I decided not to do that. The hwseek additions are independent to 
this
patch series, and I think it is best if Hans de Goede does that part so that 
that
API change can be made together with a driver that actually uses it.

In order to show how the new ioctl is used, this patch series adds support for 
it
to the very, very old radio-cadet driver.

Comments are welcome!

Regards,

Hans

PS: Mauro, I haven't been able to work on the radio profile yet, so that's not
included.



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


Re: [RFC PATCH 0/6] Add frequency band information

2012-07-02 Thread halli manjunatha
Hi,

This looks good to me

Acked-by: Manjunatha Halli manjunatha_ha...@ti.com

Regards
Manju

On Mon, Jul 2, 2012 at 9:42 AM, Hans de Goede hdego...@redhat.com wrote:

 Hi,

 Series looks good to me, ack series:
 Acked-by: Hans de Goede hdego...@redhat.com

 Regards,

 Hans


 On 07/02/2012 04:15 PM, Hans Verkuil wrote:

 Hi all,

 This patch series adds support for the new VIDIOC_ENUM_FREQ_BANDS ioctl that
 tells userspace if a tuner supports multiple frequency bands.

 This API is based on earlier attempts:

 http://www.spinics.net/lists/linux-media/msg48391.html
 http://www.spinics.net/lists/linux-media/msg48435.html

 And an irc discussion:

 http://linuxtv.org/irc/v4l/index.php?date=2012-06-26

 This irc discussion also talked about adding rangelow/high to the 
 v4l2_hw_freq_seek
 struct, but I decided not to do that. The hwseek additions are independent 
 to this
 patch series, and I think it is best if Hans de Goede does that part so that 
 that
 API change can be made together with a driver that actually uses it.

 In order to show how the new ioctl is used, this patch series adds support 
 for it
 to the very, very old radio-cadet driver.

 Comments are welcome!

 Regards,

 Hans

 PS: Mauro, I haven't been able to work on the radio profile yet, so that's 
 not
 included.





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