ldube opened a new pull request, #19986:
URL: https://github.com/apache/nuttx/pull/19986
## Summary
Fixes several bugs in EDID parsing and consolidates syslog output in
videomode_dump to prevent broken lines.
Specific changes include:
- Corrected bitwise masking for _HACT_HI (0xf0) and _HBLK_HI (0x0f) to
properly extract the upper bits of horizontal active and blanking timings.
- Multiplied raw EDID pixel clock by 10 to convert it into kHz, matching
the expectation of the videomode struct dotclock field.
- Combined fragmented syslog calls in videomode_dump into a single line to
prevent unwanted newlines from splitting the output across multiple logs.
## Testing
Used MIPS CI20 display driver to call edid_dump.
**Before**
```
1024x768 @ 60Hz
(65000 1048 1184 1344 771 777 806
-H -V)
1024x768 @ 70Hz
(75000 1048 1184 1328 771 777 806
-H -V)
1024x768 @ 75Hz
(78750 1040 1136 1312 769 772 800
+H +V)
80x768 @ 42Hz
(8550 144 256 256 771 777 795
+H +V)
0x768 @ 79Hz
(10225 80 208 160 771 778 805
-H +V)
Preferred Mode:
80x768 @ 42Hz
```
**After**
```
1024x768 @ 60Hz (65000 1048 1184 1344 771 777 806 -H -V)
1024x768 @ 70Hz (75000 1048 1184 1328 771 777 806 -H -V)
1024x768 @ 75Hz (78750 1040 1136 1312 769 772 800 +H +V)
1360x768 @ 60Hz (85500 1424 1536 1792 771 777 795 +H +V)
1280x768 @ 75Hz (102250 1360 1488 1696 771 778 805 -H +V)
Preferred Mode: 1360x768 @ 60Hz
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
