fixed review comments

src/VehicleSetup/FirmwareUpgrade.qml

@@ -139,7 +139,7 @@ QGCView {
         QGCViewDialog {
             anchors.fill: parent
  
-            property bool showFirmwareTypeSelection: apmFlightStack.checked || advancedMode.checked
+            property bool showFirmwareTypeSelection: advancedMode.checked
             property bool px4Flow:              controller.boardType == "PX4 Flow"
 
             function accept() {

src/VehicleSetup/FirmwareUpgradeController.h

@@ -174,6 +174,7 @@ private slots:
 
 private:
     void _getFirmwareFile(FirmwareIdentifier firmwareIDTrinity);
+    void _initFirmwareHash();
     void _downloadFirmware(void);
     void _appendStatusLog(const QString& text, bool critical = false);
     void _errorCancel(const QString& msg);
@@ -181,6 +182,13 @@ private:
     QString _portName;
     QString _portDescription;
 
+    // firmware hashes
+    QHash<FirmwareIdentifier, QString> _rgPX4FMUV2Firmware;
+    QHash<FirmwareIdentifier, QString> _rgAeroCoreFirmware;
+    QHash<FirmwareIdentifier, QString> _rgPX4FMUV1Firmware;
+    QHash<FirmwareIdentifier, QString> _rgPX4FLowFirmware;
+    QHash<FirmwareIdentifier, QString> _rg3DRRadioFirmware;
+
     /// Information which comes back from the bootloader
     bool        _bootloaderFound;           ///< true: we have received the foundBootloader signals
     uint32_t    _bootloaderVersion;         ///< Bootloader version