[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20081202101359.GB4645@gundam.enneenne.com>
Date: Tue, 2 Dec 2008 11:13:59 +0100
From: Rodolfo Giometti <giometti@...eenne.com>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: Randy Dunlap <randy.dunlap@...cle.com>,
linux-kernel@...r.kernel.org
Subject: Re: mmotm 2008-11-26-17-00 uploaded (hwmon / dev_attr_name)
On Mon, Dec 01, 2008 at 10:37:41AM -0800, Andrew Morton wrote:
> I'll sort it out.
Ok. :)
> > > drivers/pps/clients/pps-ldisc.c: In function 'pps_tty_dcd_change':
> > > drivers/pps/clients/pps-ldisc.c:32: warning: cast from pointer to integer of different size
> > > drivers/pps/clients/pps-ldisc.c: In function 'pps_tty_open':
> > > drivers/pps/clients/pps-ldisc.c:77: warning: cast to pointer from integer of different size
> > > drivers/pps/clients/pps-ldisc.c:82: warning: cast from pointer to integer of different size
> > > drivers/pps/clients/pps-ldisc.c: In function 'pps_tty_close':
> > > drivers/pps/clients/pps-ldisc.c:91: warning: cast from pointer to integer of different size
> > >
> > >
> > > For the ->disc_data warnings you could perhaps just make all the
> > > relevant scalar types `long', not `int'.
> >
> > But doing like this will breaks 32bit archs... =:-o
>
> I don't think so. sizeof(long) == sizeof(int) on 32-bit.
But "%d" != "%ld". :)
However this is a minor issue... I'm going to propose a patch ASAP.
Ciao,
Rodolfo
--
GNU/Linux Solutions e-mail: giometti@...eenne.com
Linux Device Driver giometti@...ux.it
Embedded Systems phone: +39 349 2432127
UNIX programming skype: rodolfo.giometti
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists