Opened on 08/14/18 at 23:31:03

Closed on 08/20/18 at 06:36:58

#3502 closed defect (fixed)

PL111_LCD_CONTROL_LCD_BPP_16 Redefined

Reported by: Joel Sherrill Owned by: Sebastian Huber
Priority: normal Milestone: 5.1
Component: arch/arm Version: 5
Severity: normal Keywords:
Cc: Blocked By:
Blocking:

Description

The constant PL111_LCD_CONTROL_LCD_BPP_16 is defined twice in the file bsps/arm/include/bsp/arm-pl111-regs.h:

#define PL111_LCD_CONTROL_LCD_BPP_16 0x04U
#define PL111_LCD_CONTROL_LCD_BPP_24 0x05U
#define PL111_LCD_CONTROL_LCD_BPP_16 0x06U
#define PL111_LCD_CONTROL_LCD_BPP_12 0x07U

Given the context, I am guessing the first one should be BPP_32 but since Sebastian added the file, I am assuming he has docs and can answer this question for sure.

Change History (2)

comment:1 Changed on 08/14/18 at 23:31:18 by Joel Sherrill

Owner: set to Sebastian Huber
Status: new → assigned

comment:2 Changed on 08/20/18 at 06:36:58 by Sebastian Huber <sebastian.huber@…>

Resolution: → fixed
Status: assigned → closed

In 2cd3716/rtems:

bsps/arm: Fix PL111 register define re-definition

Close #3502.

Note: See TracTickets for help on using tickets.