Commit fbe79a17fddb7f0b11aa15b9c93e9a4a26165ed8

Authored by Masahiro Yamada
Committed by Tom Rini
1 parent 61f06b143e

m68k: define __kernel_size_t as unsinged int again

Commit ddc94378d changed the definition of __kernel_size_t
from unsigned int to unsigned long.

It is true that it fixed warnings on some crosstools
but it increased warnings on the others.

The problem is that we cannot see consistency in terms of
the typedef of __kernel_size_t on M68K architecture.

However, I'd like to suggest to have __kernel_size_t to be
unsigned int again.

Rationale:

[1] Linux Kernel defines __kernel_size_t on M68K as unsigned int.
    Let's stick to the Linux's way.

[2] We want to build boards with popular pre-built toolchains,
    not the one locally-built by indivisuals.
    I think m68-linux-gcc which can be downloaded from www.kernel.org
    is the candidate for our _recommended_ toolchains.

With this patch, all the m68k boards can be built without any warnings.

Give it a try with the following crosstools:

https://www.kernel.org/pub/tools/crosstool/files/bin/x86_64/4.6.3/
x86_64-gcc-4.6.3-nolibc_m68k-linux.tar.xz

or

https://www.kernel.org/pub/tools/crosstool/files/bin/x86_64/4.9.0/
x86_64-gcc-4.9.0-nolibc_m68k-linux.tar.xz

(The latter is newer.)

Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Jason Jin <Jason.jin@freescale.com>

Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff

arch/m68k/include/asm/posix_types.h
... ... @@ -15,7 +15,7 @@
15 15 typedef int __kernel_pid_t;
16 16 typedef unsigned int __kernel_uid_t;
17 17 typedef unsigned int __kernel_gid_t;
18   -typedef unsigned long __kernel_size_t;
  18 +typedef unsigned int __kernel_size_t;
19 19 typedef int __kernel_ssize_t;
20 20 typedef long __kernel_ptrdiff_t;
21 21 typedef long __kernel_time_t;