On Sun, 2009-11-01 at 13:10 -0500, Brandon Jenkins wrote: > On Sun, Nov 1, 2009 at 7:37 AM, Andy Walls <awalls@xxxxxxxxx> wrote: > > On Sat, 2009-10-31 at 22:25 -0400, Brandon Jenkins wrote: > >> On Sat, Oct 31, 2009 at 8:41 PM, Andy Walls <awalls@xxxxxxxxx> wrote: > >> > On Sat, 2009-10-31 at 16:28 -0400, Devin Heitmueller wrote: > >> >> On Sat, Oct 31, 2009 at 4:16 PM, Andy Walls <awalls@xxxxxxxxx> wrote: > > Could you provide the panic to me? Off-list is fine. > > > > If I can't get this large buffer scheme to work for the general case to > > mainatin YUV frame alignment, I'll have to figure out what will likely > > be a much more complex scheme to ensure alignment is maintained in for > > YUV streams. :( > > > > Oh, well. > > > > Regards, > > Andy > > > > > > > Hi Andy, > > The panic happens upon reboot and it is only 1 line of text oddly shifted. > > Kernel panic - not syncing: DMA: Memory would be corrupted > > If I switch back to the current v4l-dvb drivers no issue. To switch > back I have to boot from a USB drive. Brandon, Eww. OK. Nevermind performing any more data collection. I'm going to use a new strategy (when I find the time). (Thinking out loud ...) Working under the assumptions that: 1. The encoder always returns 720 pixels worth of data per line (no matter what the scaling) 2. The software HM12 decoders can only deal with full, not partial, 16x8x2 UV macroblocks at 4:2:0, so scaled YUV height needs to be in multiples of 32 lines 3. The CX23418 actually can handle Memory Descriptor Lists (MDLs) with more than one buffer per list I'm going to use the MDLs to actually hold buffer lists with multiple buffers, where individual buffers are 720 * 32 * 3 / 2 = 33.75 kB each. That way I can build up buffer lists to hold precisely one frame of YUV data at a time, no matter what the scaling, and then know that if the cx18 driver misses an incoming MDL notification, the YUV frames will stay aligned. The 33.75 kB buffers should be no problem from a DMA perspective, compared to 607.5 kB buffers. The pain is that the cx18 driver right now has the hard-coded assumption that there is only one buffer per MDL. It will take a bit of effort to fix that assumption in the driver and generalize it to having 1 or more buffers per MDL. Anyway, thanks for the testing. Regards, Andy -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
