Re: [PATCH 0/6] Add frequency band information
Series looks good to me, ack series: Acked-by: Hans de Goede hdego...@redhat.com On 07/05/2012 12:25 PM, Hans Verkuil wrote: Hi Mauro, This should be the final patch series for adding multiband support to the kernel. This patch series assumes that this pull request was merged first: http://patchwork.linuxtv.org/patch/13180/ Changes since the previous RFC patch series: (See http://www.mail-archive.com/linux-media@vger.kernel.org/msg48549.html) - The name field was dropped. - A new modulation field was added that describes the possible modulation systems for that frequency band (currently only one modulation system can be supported per band). - Compat code was added to allow VIDIOC_ENUM_FREQ_BANDS to be used for existing drivers. A note regarding the cadet driver: I want to do a follow-up patch to this at a later date so that it uses the tea575x-tuner framework. But with these patches it will at least work correctly again. Regards, Hans -- 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
[RFC PATCH 0/6] Add frequency band information
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
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
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
