From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp.alphaone-tech.com ([174.133.83.186]:60487 "EHLO tyler.alphaone-tech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757484Ab2DXXJ3 (ORCPT ); Tue, 24 Apr 2012 19:09:29 -0400 Message-ID: <4F9726A7.4060208@non-elite.com> Date: Tue, 24 Apr 2012 17:18:15 -0500 From: Bob W MIME-Version: 1.0 To: Konstantin Dimitrov CC: =?ISO-8859-1?Q?Christian_Pr=E4hauser?= , linux-media@vger.kernel.org, Marek Ochaba Subject: Re: DVB-S2 multistream support References: <4EF67721.9050102@unixsol.org> <4EF6DD91.2030800@iki.fi> <4EF6F84C.3000307@redhat.com> <4EF7066C.4070806@redhat.com> <85A7A8FC-150C-4463-B09C-85EED6F851A8@cosy.sbg.ac.at> <78E6697C-BD32-4062-BC2C-A5F7D0CBD79C@cosy.sbg.ac.at> <4F5F6C2D.1080206@non-elite.com> <2FB3FA12-CAFE-4558-9935-1AF14D44DFA9@cosy.sbg.ac.at> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi all, in playing with these patches, I am noticing the bbframes are only partially being delivered by the hardware. I am only getting the first 188 bytes of the frame. I think it is related to the programming of the saa716xx. In this function: saa716x_dma_start the params stuct is filled out, with 188 in the samples and pitch elements.. does anyone know a little more about this saa716x and how to program it correctly to receive more data to account for the big bbframes? thanks Bob On 3/19/2012 10:19 AM, Konstantin Dimitrov wrote: > hello Christian, > > first of all thank you for the great work. so, i have few follow-ups about it: > > 1. since your current patch contains patches that are kernel-specific > changes (i.e. some general changes to the V4L code for the kernel you > use), changes related to the BBFrame-support and hardware-specific > changes (i.e. related to the TBS 6925 card, which at the moment is the > only affordable hardware at least i know that can handle BBFrames) in > order for more easy use in different kernels as well for the purpose > of others to review the changes related to the BBFrame-support in V4L > 'dvb-core' i split your patch to 3 separate patches (they are attached > to this email): > > - 01-bb-dmx.patch : it's supposed to contains all changes you made to > the demux part of V4L 'dvb-core' - i didn't make any further changes - > just collected your original changes in the patch; it should apply > clean to all recent kernel released in the last several months (i > believe at least in the last 6 months), because no changes in this > parts of 'dvb-core' were made recently > > - 02-bb-fe.patch : those are changes for BBFrame-support related to > the frontend - i separated those changes and i didn't include them in > '01-bb-dmx.patch', because recently a lot of changes were made to that > part of V4L. so, this patch will apply clean to > "tbs-linux-drivers_v120206" and to other V4L trees most probably it > will require to manually apply the changes to the respective files, > but since it changes less than 10 lines in only 3 files that's just > fine for manual patching > > - 03-bb-tbs.patch : those are all changes specific to the TBS 6925 > hardware. additionally to your changes i defined "tsout" module > parameter to both 'stv090x' and 'saa716x_tbs-dvb' - that's convenient > to change between TS and BB mode just with reloading the modules with > rmmod/modprobe > > 2. so i used the above 3 patches with "tbs-linux-drivers_v120206" and > after i applied them: > > # cd linux-tbs-drivers > # patch -p1 < ../01-bb-dmx.patch > # patch -p1 < ../02-bb-fe.patch > # patch -p1 < ../03-bb-tbs.patch > > the driver builds successfully, but i'm not sure if i didn't miss > something from your original patch. also, i put file called > 'tbs6925.conf' in /etc/modprobe.d with lines in it: > > options stv090x tsout=0 > options saa716x_tbs-dvb tsout=0 > > for easy enable/disable of the BB mode with just reloading the modules > in order to make the testing easier. > > 3. so on the test i did every few seconds i get the following errors: > > _dvb_dmx_swfilter_bbframe: invalid use of reserved TS/GS value 1 > > _dvb_dmx_swfilter_bbframe: invalid data field length (length 0, left 10) > > and hex-dump of the BBFrame data. is that supposed to happen? please, > can you confirm if it happens or not in your environment with your > original patch or i messed-up something when i prepared the 3 patches > from point 1. so, that request is in relation to my seconds request - > please, review the 3 patches from point 1 and confirm they are correct > and i didn't miss anything - if they are correct patch 01 and 02 can > be used for review of the code. > > many thanks, > konstantin >