Use 2 bits for msp displayport font page

[mmosca]

May allow for future 4 page fonts

[mmosca]

I may change the fontPage uint8_t array to attr uint8_t array and combine with the blink flag.

[RomanLut]

Blinking attribute handling can be removed from displayport_msp_osd.c completely.

Displayport_msp_osd does not support blinking attibute:

inav/src/main/io/displayport_msp_osd.c

Line 376 in 1dc9105

static textAttributes_t supportedTextAttributes(const displayPort_t *displayPort)

Software blinking is used, blinking attibute is not applied to characters in display.c:

inav/src/main/drivers/display.c

Line 49 in 1dc9105

static bool displayAttributesRequireEmulation(displayPort_t *instance, textAttributes_t attr)

inav/src/main/drivers/display.c

Line 206 in 1dc9105

if (displayAttributesRequireEmulation(instance, attr)) {

inav/src/main/drivers/display.c

Line 66 in 1dc9105

TEXT_ATTRIBUTES_REMOVE_BLINK(*attr);

So this code never executes:

inav/src/main/io/displayport_msp_osd.c

Line 322 in 1dc9105

if (blink) {

and can be removed unless you plan to implement useDeviceBlink option similar to BF.

[MrD-RC]

There's no need to remove blink. It is in the API. It should be handled as specified by the API.

[RomanLut]

Should be optional then, displayport_msp_use_device_blink is OFF by default in betaflight, I guess for a reason.