Alex Henrie (@alexhenrie) commented about dlls/ntdll/unix/serial.c:
prop->ServiceMask = SP_SERIALCOMM; prop->MaxTxQueue = 4096; prop->MaxRxQueue = 4096; - prop->MaxBaud = BAUD_115200; + prop->MaxBaud = BAUD_USER; prop->ProvSubType = PST_RS232; prop->ProvCapabilities = PCF_DTRDSR | PCF_PARITY_CHECK | PCF_RTSCTS | PCF_TOTALTIMEOUTS | PCF_INTTIMEOUTS; prop->SettableParams = SP_BAUD | SP_DATABITS | SP_HANDSHAKING | SP_PARITY | SP_PARITY_CHECK | SP_STOPBITS ; prop->SettableBaud = BAUD_075 | BAUD_110 | BAUD_134_5 | BAUD_150 | BAUD_300 | BAUD_600 | BAUD_1200 | BAUD_1800 | BAUD_2400 | BAUD_4800 | - BAUD_9600 | BAUD_19200 | BAUD_38400 | BAUD_57600 | BAUD_115200 ; + BAUD_9600 | BAUD_19200 | BAUD_38400 | BAUD_57600 | BAUD_115200 | BAUD_128K | BAUD_USER ;
It would be a good idea to add BAUD_7200, BAUD_14400, and BAUD_56K at the same time, and keep the lines wrapped to about 100 characters. I did a quick test and real Windows does report that my motherboard's PC serial port supports every baud rate up to BAUD_115200 including BAUD_USER and that its max baud is BAUD_USER. (The only baud that I am missing is BAUD_128K which I don't think my hardware supports, but it doesn't hurt to include that one too as you proposed.) Great work! My suggestion would be to squash your two commits together, add the three other missing baud rates, and retitle the commit something like "ntdll: Report all possible serial baud rates." The first word of the commit message is almost always the name of the DLL that you are changing, and the second word is almost always a verb in the imperative mood. Finally, be nice and wrap your commit message to 72 characters per line :-) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9968#note_128408