LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 17:58:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

Kernel: Bit mask line control options in SerialDevice::set_line_control

Browse Source 
The line control option bits (parity, stop bits, word length) were
masked and then combined incorrectly, resulting in them not being set
when requested.
This commit is contained in:
Idan Horowitz 2021-05-15 01:16:43 +03:00 committed by Andreas Kling
parent be57c424f3
commit c75ca4ea8f
Notes: sideshowbarker 2024-07-18 17:57:14 +09:00 
Author: https://github.com/IdanHo
Commit: https://github.com/SerenityOS/serenity/commit/c75ca4ea8f0
Pull-request: https://github.com/SerenityOS/serenity/pull/7116
Reviewed-by: https://github.com/awesomekling
Reviewed-by: https://github.com/supercomputer7
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Kernel/Devices/SerialDevice.cpp
View File

@ -105,7 +105,7 @@ void SerialDevice::set_line_control(ParitySelect parity_select, StopBits stop_bi
m_stop_bits = stop_bits;
m_word_length = word_length;
IO::out8(m_base_addr + 3, IO::in8(m_base_addr + 3) & (0xc0 | parity_select | stop_bits | word_length));
IO::out8(m_base_addr + 3, (IO::in8(m_base_addr + 3) & ~0x3f) | parity_select | stop_bits | word_length);
}
void SerialDevice::set_break_enable(bool break_enable)