[linux-dvb] [hg:v4l-dvb] Proper Fix frontend DVBFE_ALGO_CUSTOM Search

[Manu Abraham]

On Sat, Nov 8, 2008 at 1:23 PM, Igor M. Liplianin <liplianin at tut.by> wrote:
> В сообщении от 30 October 2008 23:59:11 Igor M. Liplianin написал(а):
>> > The patch number 9480 was added via Manu Abraham <manu at linuxtv.org>
>> > to http://linuxtv.org/hg/v4l-dvb master development tree.
>> >
>> > Kernel patches in this development tree may be modified to be backward
>> > compatible with older kernels. Compatibility modifications will be
>> > removed before inclusion into the mainstream Kernel
>> >
>> > If anyone has any objections, please let us know by sending a message to:
>> >     v4l-dvb-maintainer at linuxtv.org
>> >
>> > ------
>> >
>> > From: Marco Schluessler  <marco at lordzodiac.de>
>> > Fix frontend DVBFE_ALGO_CUSTOM Search
>> >
>> > Issue: Currently, the DVBFE_ALGO_CUSTOM is called only at the first
>> > tuning. After the first call, the status is never set to force a new
>> > tuning.
>> >
>> >
>> > Signed-off-by: Marco Schluessler <marco at lordzodiac.de>
>> > Signed-off-by: Manu Abraham <manu at linuxtv.org>
>> >
>> >
>> > ---
>> >
>> >  linux/drivers/media/dvb/dvb-core/dvb_frontend.c |    3 +++
>> >  1 file changed, 3 insertions(+)
>> >
>> > diff -r d40d755b5163 -r 3c6a7759c267
>> > linux/drivers/media/dvb/dvb-core/dvb_frontend.c ---
>> > a/linux/drivers/media/dvb/dvb-core/dvb_frontend.c   Tue Oct 28 20:50:01
>> > 2008 +0400 +++ b/linux/drivers/media/dvb/dvb-core/dvb_frontend.c    Thu Oct
>> > 30 09:38:59 2008 +0400 @@ -1389,6 +1389,9 @@ int
>> > dtv_property_process_set(struct dvb_
>> >             dprintk("%s() Finalised property cache\n", __func__);
>> >             dtv_property_cache_submit(fe);
>> >
>> > +           /* Request the search algorithm to search */
>> > +           fepriv->algo_status |= DVBFE_ALGO_SEARCH_AGAIN;
>> > +
>> >             r |= dvb_frontend_ioctl_legacy(inode, file, FE_SET_FRONTEND,
>> >                     &fepriv->parameters);
>> >             break;
>> >
>> >
>> > ---
>>
>> Patch number 9480 needs to be unapplied. It is not resolving the issue.
>> Proper patch attached.
>
> Just to be clear - Marco Schluessler's patch not works for legacy applications, like szap.
>

Seems true,  will test this. Thought i had tested this.

Thanks,
Manu