adrv9002: fix bugs

Fix LO range
Fix PA Correction not showing
Fix enable / disable channel changes values

Signed-off-by: IonutMuthi <[email protected]>

Author: IonutMuthi

packages/adrv9002/plugins/adrv9002plugin/src/adrv9002.cpp

@@ -468,7 +468,7 @@ QWidget *Adrv9002::createRxChannelControls(const QString &title, int channel)
 
 	if(loCh) {
 		QString loAttr = QString("RX%1_LO_frequency").arg(channel + 1);
-		auto loWidget = createRangeWidget(loCh, loAttr, "[30 1 6000]", "Local Oscillator (MHz)");
+		auto loWidget = createRangeWidget(loCh, loAttr, "[30000000 1 6000000000]", "Local Oscillator (MHz)");
 
 		// Add MHz ↔ Hz conversion
 		loWidget->setDataToUIConversion(
@@ -545,7 +545,7 @@ QWidget *Adrv9002::createTxChannelControls(const QString &title, int channel)
 
 	if(loCh) {
 		QString loAttr = QString("TX%1_LO_frequency").arg(channel + 1);
-		auto loWidget = createRangeWidget(loCh, loAttr, "[30 1 6000]", "Local Oscillator (MHz)");
+		auto loWidget = createRangeWidget(loCh, loAttr, "[30000000 1 6000000000]", "Local Oscillator (MHz)");
 
 		// Add MHz ↔ Hz conversion
 		loWidget->setDataToUIConversion(
@@ -596,7 +596,7 @@ QWidget *Adrv9002::createTxChannelControls(const QString &title, int channel)
 	layout->addWidget(createCheckboxWidget(txCh, "close_loop_gain_tracking_en", "Close Loop Gain"), 8, 1);
 	layout->addWidget(createCheckboxWidget(txCh, "pa_correction_tracking_en", "PA Correction"), 9, 0);
 	layout->addWidget(createCheckboxWidget(txCh, "loopback_delay_tracking_en", "Loopback Delay"), 9, 1);
-	layout->addWidget(createCheckboxWidget(txCh, "lo_leakage_tracking_en", "LO Leakage"), 9, 0);
+	layout->addWidget(createCheckboxWidget(txCh, "lo_leakage_tracking_en", "LO Leakage"), 10, 0);
 
 	return widget;
 }

packages/adrv9002/plugins/adrv9002plugin/src/channelconfigwidget.cpp

@@ -156,36 +156,26 @@ ChannelConfigWidget::ChannelData ChannelConfigWidget::getChannelData() const
 void ChannelConfigWidget::updateControlsVisibility(bool lteMode)
 {
 	m_isLTEMode = lteMode; // Store mode state for mode-aware control logic
-	bool channelEnabled = m_enabledCb->isChecked();
 
 	// Apply mode-specific control settings
 	if(m_isLTEMode) {
 		// LTE Mode: Sample rate from predefined list (combo), bandwidth read-only
 		m_bandwidthCombo->setEnabled(false);
-
 		m_sampleRateCombo->setEditable(false);
-		m_sampleRateCombo->setEnabled(channelEnabled);
 
 		// this is a woraround caused by how Qt treats enable / disable editable logic for QCombobox
 		m_sampleRateCombo->clear();
 		m_sampleRateCombo->addItems(FrequencyTable::SAMPLE_RATES_HZ);
 
 	} else {
 		// Live Device Mode: Both sample rate and bandwidth can be edited
-		m_bandwidthCombo->setEnabled(channelEnabled);
-
 		m_sampleRateCombo->setEditable(true);
-		m_sampleRateCombo->setEnabled(channelEnabled);
+		m_bandwidthCombo->setEnabled(true);
+		m_sampleRateCombo->setEnabled(true);
 
 		// this is a woraround caused by how Qt treats enable / disable editable logic for QCombobox
 		Style::setBackgroundColor(m_sampleRateCombo, json::theme::background_primary);
 	}
-
-	// These always follow channel state regardless of mode
-	m_freqOffsetCb->setEnabled(channelEnabled);
-	if(m_rfInputCombo) {
-		m_rfInputCombo->setEnabled(channelEnabled);
-	}
 }
 
 void ChannelConfigWidget::setBandwidthReadOnly(bool readOnly) { m_bandwidthCombo->setEnabled(!readOnly); }
@@ -237,9 +227,22 @@ void ChannelConfigWidget::onEnabledChanged()
 	if(m_updatingData)
 		return;
 
-	updateControlsVisibility(m_isLTEMode);
+	bool channelEnabled = m_enabledCb->isChecked();
+
+	m_sampleRateCombo->setEnabled(channelEnabled);
+
+	if(m_isLTEMode) {
+		m_bandwidthCombo->setEnabled(false);
+	} else {
+		m_bandwidthCombo->setEnabled(channelEnabled);
+	}
+
+	m_freqOffsetCb->setEnabled(channelEnabled);
+	if(m_rfInputCombo) {
+		m_rfInputCombo->setEnabled(channelEnabled);
+	}
+
 	Q_EMIT enabledChanged(m_enabledCb->isChecked());
-	Q_EMIT channelDataChanged();
 }
 
 void ChannelConfigWidget::onSampleRateChanged()

packages/adrv9002/plugins/adrv9002plugin/src/profilegeneratorwidget.cpp

@@ -1248,6 +1248,9 @@ void ProfileGeneratorWidget::forceUpdateAllUIControls()
 
 	// Force update all 4 channel widgets with device data (enabled states completely ignored)
 	for(int i = 0; i < 4; ++i) {
+		// set channels to live device mode
+		m_channelWidgets[i]->updateControlsVisibility(false);
+
 		ChannelConfigWidget::ChannelData data;
 		if(i < 2) { // RX channels (RX1, RX2)
 			const auto &rxConfig = m_deviceConfig.rxChannels[i];