Notice: We have migrated to GitLab launching 2024-05-01 see here:

#2678 assigned enhancement

possible unnecessary includes in console drivers

Reported by: Joel Sherrill Owned by: Needs Funding
Priority: low Milestone: Indefinite
Component: bsps Version:
Severity: minor Keywords:
Cc: Blocked By:

Description (last modified by Joel Sherrill)

I am making a sweep to eliminate the include of <rtems/console.h> from <bsp.h>. This involves adding it to a few console drivers. I am seeing includes of multiple files that I am suspicious are not needed or should be corrected:

  • <malloc.h>
  • <rtems/mw_uid.h>
  • <fcntl.h>
  • use of <assert.h> - there are 100s of asserts. This is beyond my current activities.
  • <stdio.h>
  • some comments that <bsp.h> must be included before <rtems/libio.h>

I reordering in some to try a pattern of include files as follows:

  • standard C/POSIX .h files
  • standard RTEMS files without <rtems.h> as first
  • <bsp.h>
  • private .h files

This may not work but could be part of the remove rtems/console.h from bsp.h patch series.

I will not be able to investigate immediately so am filing a ticket for future follow up.

Change History (5)

comment:1 Changed on 03/29/16 at 13:22:23 by Joel Sherrill

Description: modified (diff)

comment:2 Changed on 03/30/16 at 15:21:26 by Joel Sherrill

Description: modified (diff)

comment:3 Changed on 05/11/17 at 07:31:02 by Sebastian Huber


comment:4 Changed on 06/08/17 at 07:35:15 by Sebastian Huber

Milestone: 4.12.0Indefinite
Version: 4.12

comment:5 Changed on 12/15/22 at 18:47:04 by Joel Sherrill

Owner: changed from Joel Sherrill to Needs Funding
Status: newassigned
Note: See TracTickets for help on using tickets.