search

hd-zero / hdzero-goggle

MIT License
263 stars 76 forks source link

EMUF font variant #371

Closed nerdCopter closed 11 months ago

nerdCopter commented 1 year ago
nerdCopter commented 11 months ago

i think i need to add it here, but i see there would be a mismatch of values (sort order) : https://github.com/hd-zero/hdzero-vtx/blob/554dad1d7ff123dc3c23f556aca4fa03e11d5424/src/msp_displayport.h#L54 vs https://github.com/nerdCopter/hdzero-goggle/blob/f14b97961ef382e6ad5f04f67cdd176f3a070866/src/core/osd.c#L41

i also do not know how to build the vtx firmware for testing

@ligenxxxx

propose vtx mod, but unsure.

typedef enum {
    UNKN = 0,
    ARDU,
    BTFL,
    EMUF,
    INAV,
    QUIC
} fc_variant_e;
nerdCopter commented 11 months ago

@ligenxxxx EmuFlight installed on quad: should send EMUF. but i only see fc_variant changed:ARDU. self_test.txt

nerdCopter commented 11 months ago

found the problem in EmuFlight PR's sourcecode. will repair, test, then mark ready for review. https://github.com/saidinesh5/EmuFlight/blob/e6aab0f3d5aeb97ef2c23c62522e688d1cca4bf8/src/main/io/displayport_hdzero_osd.c#L391-L392

nerdCopter commented 11 months ago

this PR confirmed working when emuflight-firmware is coded properly.

[INFO][msp_displayport.c:fcTypeDetect:258] fc_variant changed:EMUF
[INFO][osd.c:load_fc_osd_font_bmp:840] load_fc_osd_font_bmp: /mnt/extsd/resource/OSD/FC/EMUF_000.bmp...