From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.bredband2.com ([83.219.192.166]:46309 "EHLO smtp.bredband2.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757128AbbCRVka (ORCPT ); Wed, 18 Mar 2015 17:40:30 -0400 Message-ID: <5509F0CC.6080104@southpole.se> Date: Wed, 18 Mar 2015 22:40:28 +0100 From: Benjamin Larsson MIME-Version: 1.0 To: Antti Palosaari CC: Linux Media Mailing List Subject: Re: [PATCH 10/10] mn88473: implement lock for all delivery systems References: <1426460275-3766-1-git-send-email-benjamin@southpole.se> <1426460275-3766-10-git-send-email-benjamin@southpole.se> <55074F74.2080000@iki.fi> <55075CD2.6060908@iki.fi> <55076302.807@southpole.se> <5509EA6B.9080805@iki.fi> In-Reply-To: <5509EA6B.9080805@iki.fi> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 03/18/2015 10:13 PM, Antti Palosaari wrote: > On 03/17/2015 01:10 AM, Benjamin Larsson wrote: >> On 03/16/2015 11:44 PM, Antti Palosaari wrote: >>> On 03/16/2015 11:47 PM, Antti Palosaari wrote: >>>> On 03/16/2015 12:57 AM, Benjamin Larsson wrote: >>>>> Signed-off-by: Benjamin Larsson >>>> >>>> Applied. >>> >>> I found this does not work at least for DVB-C. After playing with >>> modulator I find reg 0x85 on bank 1 is likely AGC. Its value is changed >>> according to RF level even modulation itself is turned off. >>> >>> I will likely remove that patch... It is a bit hard to find out lock >>> bits and it comes even harder without a modulator. Using typical tricks >>> to plug and unplug antenna, while dumping register values out is error >>> prone as you could not adjust signal strength nor change modulation >>> parameters causing wrong decision easily. >>> >>> regards >>> Antti >>> >> >> Indeed the logic was inverted. Will respin the patch. > > Any ETA for the new patch? > > regards > Antti > Sent and I tested DVB-C yesterday, it worked fine. MvH Benjamin Larsson