From: "Alfredo Jesús Delaiti" <alfredodelaiti@netscape.net>
To: "Miroslav Slugeň" <thunder.m@email.cz>
Cc: "Miroslav Slugeň" <thunder.mmm@gmail.com>, linux-media@vger.kernel.org
Subject: Re: cx23885: Add basic analog radio support
Date: Tue, 22 Oct 2013 00:07:03 -0300 [thread overview]
Message-ID: <5265EBD7.3040406@netscape.net> (raw)
In-Reply-To: <5262AC1E.5030303@email.cz>
Hi Miroslav
El 19/10/13 12:58, Miroslav Slugeň escribió:
> Hi, i am not using those devices anymore for analog, we do change our
> devices to SDR and SW demodulation, but i am sending my latest patches
> all for kernel 3.2.XX
>
> Feel free to resubmit any kind of my code by your name ;)
>
> Miroslav Slugeň
> +420 724 825 885
>
Thanks you Miroslav, but is your code.
Regards,
Alfredo
> Alfredo Jesús Delaiti napsal(a):
>> Hi all
>>
>>
>> El 14/01/12 15:25, Miroslav Slugeň escribió:
>>> New version of patch, fixed video modes for DVR3200 tuners and working
>>> audio mux.
>>
>> I tested this patch (https://linuxtv.org/patch/9498/) with the latest
>> versions of git (September 28, 2013) with my TV card (Mygica X8507) and
>> it works.
>> I found some issue, although it may be through a bad implementation of
>> mine.
>>
>> Details of them:
>>
>> 1) Some warning when compiling
>>
>> ...
>> CC [M]
>> /home/alfredo/ISDB/Nuevo_Driver/git/media_build/v4l/cx23885-video.o
>> /home/alfredo/ISDB/Nuevo_Driver/git/media_build/v4l/cx23885-video.c:1910:8:
>>
>> : initialization from incompatible pointer type [enabled by default]
>> /home/alfredo/ISDB/Nuevo_Driver/git/media_build/v4l/cx23885-video.c:1910:8:
>>
>> warning: (near initialization for 'radio_ioctl_ops.vidioc_s_tuner')
>> [enabled by default]
>> /home/alfredo/ISDB/Nuevo_Driver/git/media_build/v4l/cx23885-video.c:1911:8:
>>
>> warning: initialization from incompatible pointer type [enabled by
>> default]
>> /home/alfredo/ISDB/Nuevo_Driver/git/media_build/v4l/cx23885-video.c:1911:8:
>>
>> warning: (near initialization for 'radio_ioctl_ops.vidioc_s_audio')
>> [enabled by default]
>> CC [M] /home/alfredo/ISDB/Nuevo_Driver/git/media_build/v4l/cx23885-vbi.o
>> ...
>>
>> --------------------------------------------------------
>> static const struct v4l2_ioctl_ops radio_ioctl_ops = {
>>
>> .vidioc_s_tuner = radio_s_tuner, /* line 1910 */
>> .vidioc_s_audio = radio_s_audio, /* line 1911 */
>>
>> --------------------------------------------------------
>>
>> 2)
>> No reports signal strength or stereo signal with KRadio. XC5000 neither
>> reported (modprobe xc5000 debug=1). Maybe a feature XC5000.
>> To listen in stereo, sometimes, you have to turn on the Digital TV then
>> Analog TV and then radio.
>>
>> 3)
>> To listen Analog TV I need changed to NTSC standard and then PAL-Nc (the
>> norm in my country is PAL-Nc). If I leave the tune in NTSC no problem
>> with sound.
>> The patch (https://linuxtv.org/patch/9505/) corrects the latter, if used
>> tvtime with xawtv not always.
>> If I see-Digital TV (ISDB-T), then so as to listen the radio I have
>> first put the TV-Analog, because I hear very low and a strong white
>> noise.
>> The latter is likely to be corrected by resetting the tuner, I have to
>> study it more.
>>
>> I put below attached the patch applied to the plate: X8507.
>>
>> Have you done any update of this patch?
>>
>> Thanks
next prev parent reply other threads:[~2013-10-22 3:07 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-14 18:25 cx23885: Add basic analog radio support Miroslav Slugeň
2012-02-28 17:16 ` Mauro Carvalho Chehab
2013-10-04 18:56 ` Alfredo Jesús Delaiti
2013-10-19 15:58 ` Miroslav Slugeň
2013-10-22 3:07 ` Alfredo Jesús Delaiti [this message]
2013-10-31 10:12 ` Mauro Carvalho Chehab
2013-11-04 17:18 ` Alfredo Jesús Delaiti
2013-11-13 14:52 ` [PATCH 1/2] cx23885 Radio Support [was: cx23885: Add basic analog radio support] Alfredo Jesús Delaiti
2013-12-20 9:54 ` Hans Verkuil
2013-12-22 2:36 ` Alfredo Jesús Delaiti
2013-11-13 15:04 ` [PATCH 2/2] X8502/8507 " Alfredo Jesús Delaiti
-- strict thread matches above, loose matches on Subject: below --
2011-12-17 0:35 cx23885: Add basic analog radio support Miroslav Slugeň
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5265EBD7.3040406@netscape.net \
--to=alfredodelaiti@netscape.net \
--cc=linux-media@vger.kernel.org \
--cc=thunder.m@email.cz \
--cc=thunder.mmm@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).