Re: [GIT PULL FOR 3.10] DiBxxxx: fixes and improvements

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Olivier,

I have regenerated your patch against the most recent codebase, as
your patch no longer applies.  (There were changes merged on the 29th
of April that broke it.)

The new patch is attached, but I am reluctant to merge it, as I do not
have any description of the patch, and it lacks a sign-off.

Please test this patch and confirm that it produces the desired
effect, then resubmit with a short description and your sign-off.

Best regards,

Mike Krufky

On Sat, May 4, 2013 at 12:05 AM, Olivier GRENIE
<olivier.grenie@xxxxxxxxxx> wrote:
> Hello Mauro,
> can you apply the attached patch. This patch correct the proposed patch by Patrick for the dib807x. Sorry to not have seen it before.
>
> regards,
> Olivier
>
> ________________________________________
> From: Mauro Carvalho Chehab [mchehab@xxxxxxxxxx]
> Sent: Saturday, April 27, 2013 4:28 PM
> To: Patrick Boettcher
> Cc: linux-media@xxxxxxxxxxxxxxx; Olivier GRENIE; Patrick BOETTCHER
> Subject: Re: [GIT PULL FOR 3.10] DiBxxxx: fixes and improvements
>
> Hi Patrick,
>
> Em Mon, 22 Apr 2013 10:12:34 +0200
> Patrick Boettcher <pboettcher@xxxxxxxxxxxxxx> escreveu:
>
>> Hi Mauro,
>>
>> These patches contains some fixes and changes for the DiBcom demods and
>> SIPs.
>>
>> Please merge for 3.10 if possible.
>>
>>
>> The following changes since commit 60d509fa6a9c4653a86ad830e4c4b30360b23f0e:
>>
>>   Linux 3.9-rc8 (2013-04-21 14:38:45 -0700)
>>
>> are available in the git repository at:
>>
>>   git://git.linuxtv.org/pb/media_tree.git/ master
>
> Hmm... I suspect that there's something wrong with those changes.
>
> Testing it with a dib8076 usb stick seems that the code is worse than
> before, as it is now harder to get a lock here.
>
> With the previous code:
>
> INFO     Scanning frequency #1 725142857
> Carrier(0x03) Signal= 67.46% C/N= 0.00% UCB= 0 postBER= 0
> Viterbi(0x05) Signal= 67.08% C/N= 0.00% UCB= 0 postBER= 2097151
> Viterbi(0x07) Signal= 67.54% C/N= 0.25% UCB= 165 postBER= 0
> Sync   (0x0f) Signal= 67.06% C/N= 0.23% UCB= 151 postBER= 0
> Lock   (0x1f) Signal= 67.58% C/N= 0.24% UCB= 160 postBER= 338688
> Service #0 (60320) BAND HD channel 57.1.0
> Service #1 (60345) BAND 1SEG channel 57.1.1
>
> With the new code:
>
> INFO     Scanning frequency #1 725142857
>        (0x00) Signal= 68.80% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.78% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.69% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.82% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.29% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.27% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.28% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.27% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.55% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.50% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.43% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.65% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.75% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.29% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.28% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.25% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.43% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.46% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.43% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.90% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.50% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.28% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.22% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.22% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.43% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.41% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.41% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 68.96% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.42% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.24% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.22% C/N= 0.00% UCB= 0 postBER= 0
> RF     (0x01) Signal= 69.25% C/N= 0.00% UCB= 0 postBER= 0
>
> So, it seems that the changes broke something.
>
> Regards,
> Mauro

Attachment: 2013-05-04-040550.patch
Description: Binary data


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux