* [2.7.75] Misc compiler warnings
@ 2003-07-12 23:52 Christian Axelsson
2003-07-13 0:23 ` Christian Axelsson
0 siblings, 1 reply; 5+ messages in thread
From: Christian Axelsson @ 2003-07-12 23:52 UTC (permalink / raw)
To: linux-kernel
[-- Attachment #1: Type: text/plain, Size: 589 bytes --]
Here are some compiler warnings:
CC drivers/i2c/i2c-dev.o
drivers/i2c/i2c-dev.c: In function `show_dev':
drivers/i2c/i2c-dev.c:121: warning: unsigned int format, different type
arg (arg 3)
CC drivers/usb/core/file.o
drivers/usb/core/file.c: In function `show_dev':
drivers/usb/core/file.c:96: warning: unsigned int format, different type
arg (arg 3)
AS arch/i386/boot/setup.o
arch/i386/boot/setup.S: Assembler messages:
arch/i386/boot/setup.S:165: Warning: value 0x37ffffff truncated to
0x37ffffff
--
Christian Axelsson
smiler@lanil.mine.nu
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [2.7.75] Misc compiler warnings
2003-07-12 23:52 [2.7.75] Misc compiler warnings Christian Axelsson
@ 2003-07-13 0:23 ` Christian Axelsson
2003-07-13 4:08 ` Greg KH
0 siblings, 1 reply; 5+ messages in thread
From: Christian Axelsson @ 2003-07-13 0:23 UTC (permalink / raw)
To: linux-kernel
[-- Attachment #1: Type: text/plain, Size: 775 bytes --]
On Sun, 2003-07-13 at 01:52, Christian Axelsson wrote:
> Here are some compiler warnings:
>
> CC drivers/i2c/i2c-dev.o
> drivers/i2c/i2c-dev.c: In function `show_dev':
> drivers/i2c/i2c-dev.c:121: warning: unsigned int format, different type
> arg (arg 3)
>
> CC drivers/usb/core/file.o
> drivers/usb/core/file.c: In function `show_dev':
> drivers/usb/core/file.c:96: warning: unsigned int format, different type
> arg (arg 3)
>
> AS arch/i386/boot/setup.o
> arch/i386/boot/setup.S: Assembler messages:
> arch/i386/boot/setup.S:165: Warning: value 0x37ffffff truncated to
> 0x37ffffff
Ehm sorry, I should say that this is 2.5.75-mm1
On 2.5.75-vanilla only the AS message occour.
--
Christian Axelsson
smiler@lanil.mine.nu
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [2.7.75] Misc compiler warnings
2003-07-13 0:23 ` Christian Axelsson
@ 2003-07-13 4:08 ` Greg KH
2003-07-13 4:50 ` Andrew Morton
0 siblings, 1 reply; 5+ messages in thread
From: Greg KH @ 2003-07-13 4:08 UTC (permalink / raw)
To: Christian Axelsson; +Cc: linux-kernel
On Sun, Jul 13, 2003 at 02:23:29AM +0200, Christian Axelsson wrote:
> On Sun, 2003-07-13 at 01:52, Christian Axelsson wrote:
> > Here are some compiler warnings:
> >
> > CC drivers/i2c/i2c-dev.o
> > drivers/i2c/i2c-dev.c: In function `show_dev':
> > drivers/i2c/i2c-dev.c:121: warning: unsigned int format, different type
> > arg (arg 3)
> >
> > CC drivers/usb/core/file.o
> > drivers/usb/core/file.c: In function `show_dev':
> > drivers/usb/core/file.c:96: warning: unsigned int format, different type
> > arg (arg 3)
> >
> > AS arch/i386/boot/setup.o
> > arch/i386/boot/setup.S: Assembler messages:
> > arch/i386/boot/setup.S:165: Warning: value 0x37ffffff truncated to
> > 0x37ffffff
>
> Ehm sorry, I should say that this is 2.5.75-mm1
>
> On 2.5.75-vanilla only the AS message occour.
That is due to the size of dev_t being bigger in the -mm tree. When
that moves to the main kernel tree, I'll fix up the usb and i2c
warnings.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [2.7.75] Misc compiler warnings
2003-07-13 4:08 ` Greg KH
@ 2003-07-13 4:50 ` Andrew Morton
2003-07-13 5:29 ` Greg KH
0 siblings, 1 reply; 5+ messages in thread
From: Andrew Morton @ 2003-07-13 4:50 UTC (permalink / raw)
To: Greg KH; +Cc: smiler, linux-kernel
Greg KH <greg@kroah.com> wrote:
>
> On Sun, Jul 13, 2003 at 02:23:29AM +0200, Christian Axelsson wrote:
> > On Sun, 2003-07-13 at 01:52, Christian Axelsson wrote:
> > > Here are some compiler warnings:
> > >
> > > CC drivers/i2c/i2c-dev.o
> > > drivers/i2c/i2c-dev.c: In function `show_dev':
> > > drivers/i2c/i2c-dev.c:121: warning: unsigned int format, different type
> > > arg (arg 3)
> > >
> > > CC drivers/usb/core/file.o
> > > drivers/usb/core/file.c: In function `show_dev':
> > > drivers/usb/core/file.c:96: warning: unsigned int format, different type
> > > arg (arg 3)
> > >
> > > AS arch/i386/boot/setup.o
> > > arch/i386/boot/setup.S: Assembler messages:
> > > arch/i386/boot/setup.S:165: Warning: value 0x37ffffff truncated to
> > > 0x37ffffff
> >
> > Ehm sorry, I should say that this is 2.5.75-mm1
> >
> > On 2.5.75-vanilla only the AS message occour.
>
> That is due to the size of dev_t being bigger in the -mm tree. When
> that moves to the main kernel tree, I'll fix up the usb and i2c
> warnings.
>
No, these need to be fixed anyway. On ppc64 (at least), dev_t is `unsigned
long'.
So on that platform the code you have in there right now will always
generate zeroes or it will oops, if there are %s's further along.
I'm not sure what's the best fix really. Maybe casting to `unsigned long
long' and handling it with %llx.
I've fixed lots of these things but have a vague feeling I've been using
`unsigned long'. That's probably good enough most of the time.
Better would be just to not play around with dev_t's in-kernel in this
manner at all. Why are we doing it?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [2.7.75] Misc compiler warnings
2003-07-13 4:50 ` Andrew Morton
@ 2003-07-13 5:29 ` Greg KH
0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2003-07-13 5:29 UTC (permalink / raw)
To: Andrew Morton; +Cc: smiler, linux-kernel
On Sat, Jul 12, 2003 at 09:50:58PM -0700, Andrew Morton wrote:
>
> Better would be just to not play around with dev_t's in-kernel in this
> manner at all. Why are we doing it?
To export to userspace the dev_t assigned to a device. I should just
make a single function for this, but haven't gotten arround to it yet.
I'll fix these up to remove the warnings.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2003-07-13 5:14 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-07-12 23:52 [2.7.75] Misc compiler warnings Christian Axelsson
2003-07-13 0:23 ` Christian Axelsson
2003-07-13 4:08 ` Greg KH
2003-07-13 4:50 ` Andrew Morton
2003-07-13 5:29 ` Greg KH
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.