linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* tm6000 video image
@ 2010-05-19 15:39 Stefan Ringel
  2010-05-19 15:49 ` Mauro Carvalho Chehab
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Ringel @ 2010-05-19 15:39 UTC (permalink / raw)
  To: Linux Media Mailing List; +Cc: Mauro Carvalho Chehab

Hi Mauro,

I have found what wrong is with video image. You generate video buffer
in function tm6000_isoc_copy, but that is not right. I move that in
function copy_multiplexed and copy_streams. And that works without this
http://www.stefan.ringel.de/pub/tm6000_image_10_05_2010.jpg (The lines
with little left shift) . Now, I generate a patch.


Stefan Ringel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: tm6000 video image
  2010-05-19 15:39 tm6000 video image Stefan Ringel
@ 2010-05-19 15:49 ` Mauro Carvalho Chehab
  2010-05-19 16:12   ` Stefan Ringel
  0 siblings, 1 reply; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2010-05-19 15:49 UTC (permalink / raw)
  To: Stefan Ringel; +Cc: Linux Media Mailing List

Stefan Ringel wrote:
> Hi Mauro,
> 
> I have found what wrong is with video image. 

Great!

> You generate video buffer
> in function tm6000_isoc_copy, but that is not right. I move that in
> function copy_multiplexed and copy_streams. And that works without this
> http://www.stefan.ringel.de/pub/tm6000_image_10_05_2010.jpg (The lines
> with little left shift) . 

Didn't work:

404: Not Found - www.stefan.ringel.de

> Now, I generate a patch.

Ok. It would be great to have this issue finally fixed.

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: tm6000 video image
  2010-05-19 15:49 ` Mauro Carvalho Chehab
@ 2010-05-19 16:12   ` Stefan Ringel
  2010-05-19 16:34     ` Stefan Ringel
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Ringel @ 2010-05-19 16:12 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: Linux Media Mailing List

Am 19.05.2010 17:49, schrieb Mauro Carvalho Chehab:
> Stefan Ringel wrote:
>   
>> Hi Mauro,
>>
>> I have found what wrong is with video image. 
>>     
> Great!
>
>   
>> You generate video buffer
>> in function tm6000_isoc_copy, but that is not right. I move that in
>> function copy_multiplexed and copy_streams. And that works without this
>> http://www.stefan.ringel.de/pub/tm6000_image_10_05_2010.jpg (The lines
>> with little left shift) . 
>>     
> Didn't work:
>
> 404: Not Found - www.stefan.ringel.de
>
>   
Sorry. A point  to much.

http://www.stefanringel.de/pub/tm6000_image_10_05_2010.jpg

>> Now, I generate a patch.
>>     
> Ok. It would be great to have this issue finally fixed.
>
> Cheers,
> Mauro
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>   


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: tm6000 video image
  2010-05-19 16:12   ` Stefan Ringel
@ 2010-05-19 16:34     ` Stefan Ringel
  2010-05-19 16:42       ` Mauro Carvalho Chehab
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Ringel @ 2010-05-19 16:34 UTC (permalink / raw)
  To: Mauro Carvalho Chehab; +Cc: Linux Media Mailing List

Mauro, I will little rewrite the function copy_streams and copy_packets,
I mean fusion these two function to one (copy_streams) and I have tested
both of them with shorter code -> works.

Stefan Ringel


Am 19.05.2010 18:12, schrieb Stefan Ringel:
> Am 19.05.2010 17:49, schrieb Mauro Carvalho Chehab:
>   
>> Stefan Ringel wrote:
>>   
>>     
>>> Hi Mauro,
>>>
>>> I have found what wrong is with video image. 
>>>     
>>>       
>> Great!
>>
>>   
>>     
>>> You generate video buffer
>>> in function tm6000_isoc_copy, but that is not right. I move that in
>>> function copy_multiplexed and copy_streams. And that works without this
>>> http://www.stefan.ringel.de/pub/tm6000_image_10_05_2010.jpg (The lines
>>> with little left shift) . 
>>>     
>>>       
>> Didn't work:
>>
>> 404: Not Found - www.stefan.ringel.de
>>
>>   
>>     
> Sorry. A point  to much.
>
> http://www.stefanringel.de/pub/tm6000_image_10_05_2010.jpg
>
>   
>>> Now, I generate a patch.
>>>     
>>>       
>> Ok. It would be great to have this issue finally fixed.
>>
>> Cheers,
>> Mauro
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>   
>>     
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>   


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: tm6000 video image
  2010-05-19 16:34     ` Stefan Ringel
@ 2010-05-19 16:42       ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 5+ messages in thread
From: Mauro Carvalho Chehab @ 2010-05-19 16:42 UTC (permalink / raw)
  To: Stefan Ringel; +Cc: Linux Media Mailing List

Stefan Ringel wrote:
> Mauro, I will little rewrite the function copy_streams and copy_packets,
> I mean fusion these two function to one (copy_streams) and I have tested
> both of them with shorter code -> works.

Go ahead.
> 
> Stefan Ringel
> 
> 
> Am 19.05.2010 18:12, schrieb Stefan Ringel:
>> Am 19.05.2010 17:49, schrieb Mauro Carvalho Chehab:
>>   
>>> Stefan Ringel wrote:
>>>   
>>>     
>>>> Hi Mauro,
>>>>
>>>> I have found what wrong is with video image. 
>>>>     
>>>>       
>>> Great!
>>>
>>>   
>>>     
>>>> You generate video buffer
>>>> in function tm6000_isoc_copy, but that is not right. I move that in
>>>> function copy_multiplexed and copy_streams. And that works without this
>>>> http://www.stefan.ringel.de/pub/tm6000_image_10_05_2010.jpg (The lines
>>>> with little left shift) . 
>>>>     
>>>>       
>>> Didn't work:
>>>
>>> 404: Not Found - www.stefan.ringel.de
>>>
>>>   
>>>     
>> Sorry. A point  to much.
>>
>> http://www.stefanringel.de/pub/tm6000_image_10_05_2010.jpg
>>
>>   
>>>> Now, I generate a patch.
>>>>     
>>>>       
>>> Ok. It would be great to have this issue finally fixed.
>>>
>>> Cheers,
>>> Mauro
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>   
>>>     
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>   
> 


-- 

Cheers,
Mauro

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2010-05-19 17:37 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-19 15:39 tm6000 video image Stefan Ringel
2010-05-19 15:49 ` Mauro Carvalho Chehab
2010-05-19 16:12   ` Stefan Ringel
2010-05-19 16:34     ` Stefan Ringel
2010-05-19 16:42       ` Mauro Carvalho Chehab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).