public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: John Banks <john.banks@noonanmedia.com>
To: Gert-Jan de Jonge <de_jonge@bysky.nl>
Cc: video4linux-list@redhat.com
Subject: Re: Any update on em28xx on igevp2 ?
Date: Thu, 18 Mar 2010 13:09:34 +0000	[thread overview]
Message-ID: <1268917774.3616.61.camel@chimpin> (raw)
In-Reply-To: <4BA21C12.6090609@bysky.nl>

On Thu, 2010-03-18 at 13:26 +0100, Gert-Jan de Jonge wrote:
> Hi Devin,
> 
> just some extra info:
> 
> I have tested the difference on the arm board compared to the pc:
>   int  height=576;
>   height >>= 576;
>   printf("%d\n", height);
> 
> on arm it gives a 0, on the pc it gives 576.
> on both i get a warning from the compiler, as in my test code the 576 is 
> hardcoded in stead of a variable
> warning: right shift count >= width of type ( which is logical ;) )
> 
> regards,
> Gert-Jan
> 
> 
> Gert-Jan de Jonge wrote:
> > Hi Devin,
> >
> > I am a big step further, I can now get video from the device.
> > At this moment I am looking at  the function em28xx_resolution_set on 
> > arm the height is set to 0 by the following lines:
> >
> >       if (!dev->progressive)
> >                height >>= norm_maxh(dev);
> >
> > I am not sure what it should do, should it really shift the height 
> > over the value of height ?
> > If I set the height to f.e. 576 ater this line, i can capture video ( 
> > it is 576 before this line )
> > should it shift by 1 if it is interlaced and the resolution is higher 
> > than the interlaced height ?
> >
> > regards,
> > Gert-Jan
> >

Hey,

Yeah this is how I fixed my problem also. Though mine was set as 0 but
144 on the pc. Strange since the resolution I then get out is certainly
not 144.

Hardcoding the values completely fixed the problem for me (as long as I
ignore the compiler warnings :P ).

Looking to find out why the arm gives 0 to fix the cause.


-- 
John Banks - Head of Engineering
Noonan Media Ltd 

www.noonanmedia.com 

MB: +44 779 62 64 707 
E: john.banks@noonanmedia.com

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2010-03-18 13:09 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-16 15:15 Any update on em28xx on igevp2 ? Gert-Jan de Jonge
2010-03-16 15:34 ` Devin Heitmueller
     [not found]   ` <4B9FA8FA.30204@saturnus.nl>
2010-03-16 15:54     ` Devin Heitmueller
     [not found]   ` <4BA2197E.4000305@saturnus.nl>
2010-03-18 12:26     ` Gert-Jan de Jonge
2010-03-18 13:09       ` John Banks [this message]
2010-03-18 14:26         ` Devin Heitmueller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1268917774.3616.61.camel@chimpin \
    --to=john.banks@noonanmedia.com \
    --cc=de_jonge@bysky.nl \
    --cc=video4linux-list@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox