All of lore.kernel.org
 help / color / mirror / Atom feed
From: Antti Palosaari <crope@iki.fi>
To: Benjamin Larsson <benjamin@southpole.se>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH 1/1] mn88473: implement lock for all delivery systems
Date: Sun, 22 Mar 2015 13:30:04 +0200	[thread overview]
Message-ID: <550EA7BC.60206@iki.fi> (raw)
In-Reply-To: <550D455F.4050500@southpole.se>

On 03/21/2015 12:18 PM, Benjamin Larsson wrote:
> On 03/21/2015 12:25 AM, Antti Palosaari wrote:
>> On 03/21/2015 01:13 AM, Benjamin Larsson wrote:
>>> On 03/19/2015 03:44 PM, Antti Palosaari wrote:
>>>> Bad news. It does lock for DVB-C now, but DVB-T nor DVB-T2 does not
>>>> lock.
>>>>
>>>> regards
>>>> Antti
>>>
>>> I'm getting tired :/. Had the time to test now and the checks is
>>> supposed to be negated.
>>>
>>> if (utmp & 0xA0) { -> if (!(utmp & 0xA0))
>>>
>>> But as stock dvbv5-scan crashes on ubuntu 14.04 and I can't unload the
>>> mn88473 module I will confirm this when I have an actual working version
>>> of dvbv5-scan and Ubuntu.
>>
>> You could also use w_scan. Or install latest dvbv5-scan from git - it
>> works even without install by running from compile directory.
>
> Ok, will try that later.
>
> Anyway, I tried the attached patch and I was able to get a lock after a
> reboot from windows with the windows driver initializing the stick. If I
> replugged the stick I get no signal and if I try the rtl2832 demod it
> reports the same. So I think that my signal is just to weak for the
> kernel r820t driver. So it would be nice if you could test this patch
> and see if it works.

Now it works. Next time I really expect you will test your patches 
somehow before sending. Now I tested 3 different patch versions, find 2 
first to be broken and last one working. It took around 2 hours of my time.

Patch applied.

Antti


>
>
>>
>> regards
>> Antti
>>
>
> MvH
> Benjamin Larsson

-- 
http://palosaari.fi/

  reply	other threads:[~2015-03-22 11:30 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-18 21:37 [PATCH 1/1] mn88473: implement lock for all delivery systems Benjamin Larsson
2015-03-19 14:44 ` Antti Palosaari
2015-03-20 23:13   ` Benjamin Larsson
2015-03-20 23:25     ` Antti Palosaari
2015-03-21 10:18       ` Benjamin Larsson
2015-03-22 11:30         ` Antti Palosaari [this message]
2015-03-22 16:02           ` Benjamin Larsson
2015-03-22 19:53             ` Antti Palosaari

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=550EA7BC.60206@iki.fi \
    --to=crope@iki.fi \
    --cc=benjamin@southpole.se \
    --cc=linux-media@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.