Eric Lee / smarc-uboot

24 Jul, 2013

1 commit

24 Dec, 2011

4 commits

  • 7ddd4475a   post/lib_powerpc/multi.c: fix stack overflow error ... Browse Code »

    The code and comment disagreed: the comment claimed that r6...r31
    were copied, and consequently the arrays for "src" and "dst" were
    declared with 26 entries, but the actual code ("lmw r5,0(r3)" and
    "stmw r5,0(r4)") copied _27_ words (r5 through r31), which resulted
    in false "POST cpu Error at multi test" messages.

    Fix the comment and the array sizes.

    Signed-off-by: Wolfgang Denk
    Cc: Anatolij Gustschin
    Cc: Stefan Roese
    Cc: Kumar Gala
    Cc: Kim Phillips
    Cc: Andy Fleming
    Acked-by: Anatolij Gustschin
    Tested-by: Anatolij Gustschin

    Wolfgang Denk
     
  • 38081ff7a   post/lib_powerpc/multi.c: code cleanup ... Browse Code »

    Clean up and document the code:

    - get rid of unneeded code block
    - add comment which code is generated

    Signed-off-by: Wolfgang Denk
    Cc: Anatolij Gustschin
    Cc: Stefan Roese
    Cc: Kumar Gala
    Cc: Kim Phillips
    Cc: Andy Fleming
    Acked-by: Anatolij Gustschin

    Wolfgang Denk
     
  • a63aec54a   post/lib_powerpc/multi.c: make checkpatch clean ... Browse Code »

    Signed-off-by: Wolfgang Denk
    Cc: Anatolij Gustschin
    Cc: Stefan Roese
    Cc: Kumar Gala
    Cc: Kim Phillips
    Cc: Andy Fleming
    Acked-by: Anatolij Gustschin

    Wolfgang Denk
     
  • 139e1875d   PPC: fix "Warning: FOO uses hard float, BAR uses soft float". ... Browse Code »

    It appears that with recent versions of GCC the explicit
    "-mhard-float" command line option takes precedence over the
    ``asm(".gnu_attribute 4, 2");'' in the source file, so this no longer
    helps to avoid the warnings we get when linking code that uses FP
    instructions with other code that was built using soft-float.

    We can remove the ".gnu_attribute" (which appears to carry no other
    information, at least so far) from the object files, but we also have
    to make sure we don't pull in the __gcc_qsub() and __gcc_qmul()
    functions from the standard libgcc, as these would again "infect" our
    linking. We copy this code from:
    gcc-4.2.2/gcc/config/rs6000/darwin-ldouble.c
    This old version was chosen because it was still available under a
    compatible license (GCC v2+). The file was stripped down to the
    needed parts, and reformatted so it passes checkpatch with only one
    warning (do not add new typedefs).

    Signed-off-by: Wolfgang Denk
    Cc: Kumar Gala
    Cc: Stefan Roese
    Cc: Andy Fleming
    Cc: Kim Phillips
    Tested-by: Stefan Roese
    Tested-by: Anatolij Gustschin

    Wolfgang Denk
     

17 Nov, 2011

1 commit

26 Jul, 2011

1 commit

26 Jan, 2011

1 commit

  • e009cdeb6   powerpc: Fix FPU post related link warnings ... Browse Code »

    If we built POST on PPC's that didn't enable CONFIG_SYS_POST_FPU we'd
    get the following warning with newer toolchains:

    powerpc-linux-gnu-ld: Warning: lib_powerpc/fpu/libpostpowerpcfpu.o
    uses hard float, libpost.o uses soft float

    We actually worked around this sometime ago with the following commit:

    commit ce82ff05388b5ddafdf6082ef0776cce72c40b1c
    Author: Yuri Tikhonov
    Date: Sat Dec 20 14:54:21 2008 +0300

    FPU POST: fix warnings when building with 2.18 binutils

    However, this only took into effect if CONFIG_SYS_POST_FPU was enabled.
    We can simply move the GNU_FPOST_ATTR out of the CONFIG_SYS_POST_FPU
    ifdef block to address the issue.

    Signed-off-by: Kumar Gala

    Kumar Gala
     

18 Nov, 2010

1 commit

  • 6d8962e81   Switch from archive libraries to partial linking ... Browse Code »

    Before this commit, weak symbols were not overridden by non-weak symbols
    found in archive libraries when linking with recent versions of
    binutils. As stated in the System V ABI, "the link editor does not
    extract archive members to resolve undefined weak symbols".

    This commit changes all Makefiles to use partial linking (ld -r) instead
    of creating library archives, which forces all symbols to participate in
    linking, allowing non-weak symbols to override weak symbols as intended.
    This approach is also used by Linux, from which the gmake function
    cmd_link_o_target (defined in config.mk and used in all Makefiles) is
    inspired.

    The name of each former library archive is preserved except for
    extensions which change from ".a" to ".o". This commit updates
    references accordingly where needed, in particular in some linker
    scripts.

    This commit reveals board configurations that exclude some features but
    include source files that depend these disabled features in the build,
    resulting in undefined symbols. Known such cases include:
    - disabling CMD_NET but not CMD_NFS;
    - enabling CONFIG_OF_LIBFDT but not CONFIG_QE.

    Signed-off-by: Sebastien Carlier

    Sebastien Carlier
     

22 Apr, 2010

1 commit

  • a47a12bec   Move arch/ppc to arch/powerpc ... Browse Code »

    As discussed on the list, move "arch/ppc" to "arch/powerpc" to
    better match the Linux directory structure.

    Please note that this patch also changes the "ppc" target in
    MAKEALL to "powerpc" to match this new infrastructure. But "ppc"
    is kept as an alias for now, to not break compatibility with
    scripts using this name.

    Signed-off-by: Stefan Roese
    Acked-by: Wolfgang Denk
    Acked-by: Detlev Zundel
    Acked-by: Kim Phillips
    Cc: Peter Tyser
    Cc: Anatolij Gustschin

    Stefan Roese