Re: [PATCH 1/8] staging: android: binder: replace explicit size types

From: Greg KH
Date: Sat Apr 06 2013 - 02:00:21 EST


On Fri, Apr 05, 2013 at 04:17:47PM -0700, Arve Hjønnevåg wrote:
> On Thu, Apr 4, 2013 at 5:32 AM, Serban Constantinescu
> <serban.constantinescu@xxxxxxx> wrote:
> >
> > Since the binder driver uses both uint32_t and unsigned int any further
> > kernel changes will be difficult to read. This patch fixes the inconsistent
> > types usage.
> >
>
> Would it make more sense to only change the types that need to be
> larger on a 64 bit system?

I agree. You are also changing the type from explicit to "unexplicit",
the exact opposite from what I would be expecting to see here. Why?
--
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/