#2211 closed defect (fixed)

warnings in mpc8xx BSPs

Reported by: Joel Sherrill Owned by: Sebastian Huber
Priority: high Milestone: 4.11
Component: bsps Version: 4.10
Severity: blocker Keywords:
Cc: Blocked By:
Blocking:

Description

Assigning this to Sebastian since Thomas doesn't have an account.

The mpc8xx BSPs now account for 9 of the top 10 in the "warnings per BSP" list.

Change History (5)

comment:1 Changed on Dec 12, 2014 at 2:33:06 PM by Sebastian Huber <sebastian.huber@…>

Resolution: fixed
Status: newclosed

In 3507f3f953cb9be14eea8bf3d03b6e29bba51b98/rtems:

bsp/mpc8xx: Fix warnings

close #2211

comment:2 Changed on Dec 12, 2014 at 2:38:35 PM by Joel Sherrill

Notice the heading says "mpx8xx" not mbx8xx. I think other mpc8xx BSPs got the same code as a baseline. Does your patch account for all these BSP variants?

7 powerpc-t32mppc (inBSP=4 inLibCPU=1)
20 powerpc-mbx821_001 (inBSP=18 inLibCPU=0)
20 powerpc-mbx821_002b (inBSP=18 inLibCPU=0)
20 powerpc-mbx821_002 (inBSP=18 inLibCPU=0)
20 powerpc-mbx860_001b (inBSP=18 inLibCPU=0)
20 powerpc-mbx860_002 (inBSP=18 inLibCPU=0)
20 powerpc-mbx860_1b (inBSP=18 inLibCPU=0)
20 powerpc-pghplus (inBSP=18 inLibCPU=0)
20 powerpc-tqm8xx_stk8xx (inBSP=18 inLibCPU=0)
24 powerpc-mbx860_005b (inBSP=22 inLibCPU=0)
79 powerpc-gwlcfm (inBSP=16 inLibCPU=61)

These are before your patch. I am starting a new build sweep now.

comment:3 Changed on Dec 12, 2014 at 2:41:16 PM by Sebastian Huber

Oh, I really hate this libcpu. I will fix the other BSPs.

comment:4 Changed on Dec 12, 2014 at 3:22:59 PM by Joel Sherrill

Resolution: fixed
Status: closedreopened

comment:5 Changed on Dec 12, 2014 at 3:39:48 PM by Sebastian Huber <sebastian.huber@…>

Resolution: fixed
Status: reopenedclosed

In 3cd3a260a6c623ff695d7bb9981133339e0bfff8/rtems:

bsp/mpc8xx: Fix warnings

close #2211

Note: See TracTickets for help on using tickets.