Re: [PATCH] LinuxPPS (with new syscalls API) - new version

From: David Woodhouse
Date: Sun Jul 01 2007 - 08:03:15 EST


On Sat, 2007-06-30 at 19:13 +0200, Rodolfo Giometti wrote:
> Maybe I can define a special struct for exchanging time data as:
>
> struct pps_timedata_s {
> long sec;
> long nsec;
> }

Seems reasonable enough in principle -- but whatever you do, don't use
"long" for it. That would definitely need different behaviour for 32-bit
vs. 64-bit. Use explicitly sized types such as uint32_t or uint64_t.

--
dwmw2

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/