Skip to content

Commit 2cc77a4

Browse files
haslinghuishaslinghuis
committed
Fix PortHandler
1 parent d92a49a commit 2cc77a4

File tree

3 files changed

+63
-42
lines changed

3 files changed

+63
-42
lines changed

‎src/js/port_handler.js‎

Lines changed: 19 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,10 @@ PortHandler.initialize = function () {
2121
const portPickerElementSelector = "div#port-picker #port";
2222
this.portPickerElement = $(portPickerElementSelector);
2323
this.selectList = document.querySelector(portPickerElementSelector);
24-
this.initialWidth = this.selectList.offsetWidth + 12;
24+
this.initialWidth = this.selectList.offsetWidth;
25+
26+
ConfigStorage.get('showVirtualMode', res => self.showVirtualMode = res.showVirtualMode);
27+
ConfigStorage.get('showAllSerialDevices', res => self.showAllSerialDevices = res.showAllSerialDevices);
2528

2629
// fill dropdown with version numbers
2730
generateVirtualApiVersions();
@@ -33,13 +36,13 @@ PortHandler.initialize = function () {
3336
PortHandler.check = function () {
3437
const self = this;
3538

36-
ConfigStorage.get('showVirtualMode', res => self.showVirtualMode = res.showVirtualMode);
37-
ConfigStorage.get('showAllSerialDevices', res => self.showAllSerialDevices = res.showAllSerialDevices);
38-
39-
self.check_usb_devices();
40-
self.check_serial_devices();
39+
if (!self.port_available) {
40+
self.check_usb_devices();
41+
}
4142

42-
GUI.updateManualPortVisibility();
43+
if (!self.dfu_available) {
44+
self.check_serial_devices();
45+
}
4346

4447
setTimeout(function () {
4548
self.check();
@@ -109,14 +112,16 @@ PortHandler.check_usb_devices = function (callback) {
109112
}
110113
self.dfu_available = false;
111114
}
112-
if(callback) {
115+
if (callback) {
113116
callback(self.dfu_available);
114117
}
115118
if (!$('option:selected', self.portPickerElement).data().isDFU) {
116119
if (!(GUI.connected_to || GUI.connect_lock)) {
117120
FC.resetState();
118121
}
119-
self.portPickerElement.trigger('change');
122+
if (self.dfu_available) {
123+
self.portPickerElement.trigger('change');
124+
}
120125
}
121126
});
122127
};
@@ -156,6 +161,7 @@ PortHandler.removePort = function(currentPorts) {
156161
self.initialPorts.splice(self.initialPorts.indexOf(removePorts[i]), 1);
157162
}
158163
self.updatePortSelect(self.initialPorts);
164+
self.portPickerElement.trigger('change');
159165
}
160166
};
161167

@@ -186,6 +192,8 @@ PortHandler.detectPort = function(currentPorts) {
186192
TABS.firmware_flasher.boardNeedsVerification = true;
187193
}
188194

195+
self.portPickerElement.trigger('change');
196+
189197
// auto-connect if enabled
190198
if (GUI.auto_connect && !GUI.connecting_to && !GUI.connected_to) {
191199
// start connect procedure. We need firmware flasher protection over here
@@ -267,7 +275,7 @@ PortHandler.selectPort = function(ports) {
267275
const pathSelect = ports[i].path;
268276
const isWindows = (OS === 'Windows');
269277
const isTty = pathSelect.includes('tty');
270-
const deviceRecognized = portName.includes('STM') || portName.includes('CP210');
278+
const deviceRecognized = portName.includes('STM') || portName.includes('CP210') || portName.startsWith('SPR');
271279
const legacyDeviceRecognized = portName.includes('usb');
272280
if (isWindows && deviceRecognized || isTty && (deviceRecognized || legacyDeviceRecognized)) {
273281
this.portPickerElement.val(pathSelect);
@@ -293,7 +301,7 @@ PortHandler.setPortsInputWidth = function() {
293301
return max;
294302
}
295303

296-
const correction = 24; // account for up/down button and spacing
304+
const correction = 32; // account for up/down button and spacing
297305
let width = findMaxLengthOption(this.selectList) + correction;
298306

299307
width = (width > this.initialWidth) ? width : this.initialWidth;

‎src/js/serial_backend.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ let connectionTimestamp;
55
let clicks = false;
66

77
function initializeSerialBackend() {
8-
GUI.updateManualPortVisibility = function(){
8+
GUI.updateManualPortVisibility = function() {
99
const selected_port = $('div#port-picker #port option:selected');
1010
if (selected_port.data().isManual) {
1111
$('#port-override-option').show();

‎src/js/tabs/firmware_flasher.js‎

Lines changed: 43 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -851,15 +851,24 @@ firmware_flasher.initialize = function (callback) {
851851
if (String(portPickerElement.val()) !== '0') {
852852
const port = String(portPickerElement.val());
853853
let baud = 115200;
854+
854855
if ($('input.flash_manual_baud').is(':checked')) {
855856
baud = parseInt($('#flash_manual_baud_rate').val());
856857
}
858+
857859
GUI.log(i18n.getMessage('firmwareFlasherDetectBoardQuery'));
858-
if (!(serial.connected || serial.connectionId)) {
859-
serial.connect(port, {bitrate: baud}, onConnect);
860+
861+
const isLoaded = self.releases ? Object.keys(self.releases).length > 0 : false;
862+
863+
if (isLoaded) {
864+
if (!(serial.connected || serial.connectionId)) {
865+
serial.connect(port, {bitrate: baud}, onConnect);
866+
} else {
867+
console.warn('Attempting to connect while there still is a connection', serial.connected, serial.connectionId);
868+
serial.disconnect();
869+
}
860870
} else {
861-
console.warn('Attempting to connect while there still is a connection', serial.connected, serial.connectionId);
862-
serial.disconnect();
871+
console.log('Releases not loaded yet');
863872
}
864873
} else {
865874
GUI.log(i18n.getMessage('firmwareFlasherNoValidPort'));
@@ -876,24 +885,22 @@ firmware_flasher.initialize = function (callback) {
876885
if (isClickable) {
877886
isClickable = false;
878887
verifyBoard();
879-
setTimeout(() => isClickable = true, 1000);
888+
setTimeout(() => {
889+
detectBoardElement.removeClass('disabled');
890+
isClickable = true;
891+
}, 1000);
880892
}
881893
});
882894

883895
function updateDetectBoardButton() {
884-
const isDfu = portPickerElement.val().includes('DFU');
896+
const isDfu = PortHandler.dfu_available;
885897
const isBusy = GUI.connect_lock;
886-
const isLoaded = self.releases ? Object.keys(self.releases).length > 0 : false;
887-
const isAvailable = PortHandler.port_available || false;
888-
const isButtonDisabled = isDfu || isBusy || !isLoaded || !isAvailable;
898+
const isAvailable = PortHandler.port_available;
899+
const isButtonDisabled = isDfu || isBusy || !isAvailable;
889900

890901
detectBoardElement.toggleClass('disabled', isButtonDisabled);
891902
}
892903

893-
document.querySelector('select[name="build_type"]').addEventListener('change', updateDetectBoardButton);
894-
document.querySelector('select[name="board"]').addEventListener('change', updateDetectBoardButton);
895-
document.querySelector('select[name="firmware_version"]').addEventListener('change', updateDetectBoardButton);
896-
897904
ConfigStorage.get('erase_chip', function (result) {
898905
if (result.erase_chip) {
899906
$('input.erase_chip').prop('checked', true);
@@ -1125,11 +1132,14 @@ firmware_flasher.initialize = function (callback) {
11251132
});
11261133

11271134
const exitDfuElement = $('a.exit_dfu');
1135+
11281136
exitDfuElement.click(function () {
1129-
if (!$(this).hasClass('disabled')) {
1137+
if (!exitDfuElement.hasClass('disabled')) {
1138+
exitDfuElement.addClass("disabled");
11301139
if (!GUI.connect_lock) { // button disabled while flashing is in progress
11311140
analytics.sendEvent(analytics.EVENT_CATEGORIES.FLASHING, 'ExitDfu', null);
11321141
try {
1142+
console.log('Closing DFU');
11331143
STM32DFU.connect(usbDevices, self.parsed_hex, { exitDfu: true });
11341144
} catch (e) {
11351145
console.log(`Exiting DFU failed: ${e.message}`);
@@ -1138,25 +1148,27 @@ firmware_flasher.initialize = function (callback) {
11381148
}
11391149
});
11401150

1141-
portPickerElement.change(function () {
1142-
if (!GUI.connect_lock) {
1143-
if ($('option:selected', this).data().isDFU) {
1144-
exitDfuElement.removeClass('disabled');
1145-
} else {
1146-
// Porthandler resets board on port detect
1147-
if (self.boardNeedsVerification) {
1148-
// reset to prevent multiple calls
1149-
self.boardNeedsVerification = false;
1150-
verifyBoard();
1151-
}
1151+
portPickerElement.on('change', function () {
1152+
if (GUI.active_tab === 'firmware_flasher') {
1153+
if (!GUI.connect_lock) {
1154+
if ($('option:selected', this).data().isDFU) {
1155+
exitDfuElement.removeClass('disabled');
1156+
} else {
1157+
// Porthandler resets board on port detect
1158+
if (self.boardNeedsVerification) {
1159+
// reset to prevent multiple calls
1160+
self.boardNeedsVerification = false;
1161+
verifyBoard();
1162+
}
11521163

1153-
$("a.load_remote_file").removeClass('disabled');
1154-
$("a.load_file").removeClass('disabled');
1155-
exitDfuElement.addClass('disabled');
1164+
$("a.load_remote_file").removeClass('disabled');
1165+
$("a.load_file").removeClass('disabled');
1166+
exitDfuElement.addClass('disabled');
1167+
}
11561168
}
1169+
updateDetectBoardButton();
11571170
}
1158-
updateDetectBoardButton();
1159-
}).change();
1171+
}).trigger('change');
11601172

11611173
$('a.flash_firmware').click(function () {
11621174
if (!$(this).hasClass('disabled')) {
@@ -1226,6 +1238,7 @@ firmware_flasher.initialize = function (callback) {
12261238

12271239
function startFlashing() {
12281240
exitDfuElement.addClass('disabled');
1241+
$('a.flash_firmware').addClass('disabled');
12291242
$("a.load_remote_file").addClass('disabled');
12301243
$("a.load_file").addClass('disabled');
12311244
if (!GUI.connect_lock) { // button disabled while flashing is in progress

0 commit comments

Comments
 (0)