From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Guy Martin <gmsoft@tuxicoman.be>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: Xawtv sparc 64bit fix
Date: Sun, 25 Apr 2010 13:55:44 -0300 [thread overview]
Message-ID: <4BD47410.9000006@redhat.com> (raw)
In-Reply-To: <20100423170316.12e01bfc@borg.bxl.tuxicoman.be>
Guy Martin wrote:
>
> Hi,
>
> Here is an old patch of mine which I tried to submit in 2006 but never
> got it. I didn't really know who was xawtv's maintainer at that time.
>
>
>
> The calculation to compute the 64bit alignement in struct-dump.c is
> plain wrong. The alignment has to be computed with a structure
> containing a char and then a 64bit integer and then substract the
> pointer of the 64bit int to the one of the char.
>
> This fix v4l-info doing a Bus Error on sparc with structs containing
> 64 bit integer following a non 64bit field aligned on a 8 byte boundary
> like v4l2_standard.
>
>
> Signed-off-by: Guy Martin <gmsoft@tuxicoman.be>
I tried to compile it (x86_64 arch) and your patch produced two warnings:
../structs/struct-dump.c: In function ‘print_struct’:
../structs/struct-dump.c:48: warning: cast from pointer to integer of different size
../structs/struct-dump.c:48: warning: cast from pointer to integer of different size
Could you please fix it?
>
>
> Regards,
> Guy
>
--
Cheers,
Mauro
next prev parent reply other threads:[~2010-04-25 16:56 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-23 15:03 Xawtv sparc 64bit fix Guy Martin
2010-04-25 16:55 ` Mauro Carvalho Chehab [this message]
2010-04-26 16:52 ` Guy Martin
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=4BD47410.9000006@redhat.com \
--to=mchehab@redhat.com \
--cc=gmsoft@tuxicoman.be \
--cc=linux-media@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.