Notice: We have migrated to GitLab launching 2024-05-01 see here: https://gitlab.rtems.org/

Opened on 01/01/18 at 00:01:11

Closed on 01/22/18 at 11:32:08

#3268 closed defect (fixed)

PowerPC BSP include naming mess.

Reported by: Chris Johns Owned by:
Priority: normal Milestone: 5.1
Component: arch/powerpc Version: 5
Severity: normal Keywords:
Cc: Blocked By:
Blocking:

Description

The PowerPC BSP family headers need some refactoring for the RTEMS 5 release. The BSP family relies on the preinstall process to get suitable headers installed to work and removing preinstall exposes this. The specific issue appears with irq.h when building the no-preinstall branch. There is a PowerPC BSP family header and a number of BSPs also have an irq.h which overrides families header. The code has #include <bsp/irq.h> and the header used depends on the include order on the GCC command line. This is fragile for any user. These headers needs to be moved to BSP specific paths, for example #include <mvme3100/irq.h>.

Change History (5)

comment:1 Changed on 01/01/18 at 00:01:31 by Chris Johns

Component: admin → arch/powerpc

comment:2 Changed on 01/01/18 at 01:16:31 by Chris Johns

Add on to the list i2c.h in the mpc5200.

comment:3 Changed on 01/04/18 at 06:19:59 by Sebastian Huber <sebastian.huber@…>

In [changeset:"e1c0d676d79da20bb5140043d388e2d9aa815421/rtems" e1c0d67/rtems]:

bsp/mpc55xxevb: Move <bsp/irq.h>

Update #3254.
Update #3268.

comment:4 Changed on 01/04/18 at 06:21:02 by Sebastian Huber <sebastian.huber@…>

In [changeset:"fb01816bd24b6398d717a69d2b95327ffb8607d5/rtems" fb01816b/rtems]:

bsps/powerpc: Move shared irq.h

This header file is only used by motorola_powerpc, so not shared.

Update #3254.
Update #3268.

comment:5 Changed on 01/22/18 at 11:32:08 by Sebastian Huber

Resolution: → fixed
Status: new → closed

This problem was fixed by [d898f6e18ec2e84977edb5024052ecca64cf39f3/rtems].

Note: See TracTickets for help on using tickets.