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
next prev parent 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