Skip to content

Commit afbbc95

Browse files
Bugfix: Hardware version persistence.
1 parent 1ad42b2 commit afbbc95

File tree

5 files changed

+3
-27
lines changed

5 files changed

+3
-27
lines changed

examples/platform/silabs/provision/ProvisionStorageDefault.cpp

-23
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
#include <nvm3_default.h>
2626
#include <nvm3_hal_flash.h>
2727
#include <platform/CHIPDeviceConfig.h>
28-
#include <platform/silabs/MigrationManager.h>
2928
#include <platform/silabs/SilabsConfig.h>
3029
#include <platform/silabs/platformAbstraction/SilabsPlatform.h>
3130
#include <silabs_creds.h>
@@ -700,28 +699,6 @@ CHIP_ERROR Storage::GetTestEventTriggerKey(MutableByteSpan & keySpan)
700699
#endif // SL_MATTER_TEST_EVENT_TRIGGER_ENABLED
701700

702701
} // namespace Provision
703-
704-
void MigrateDacProvider(void)
705-
{
706-
constexpr uint32_t kOldKey_Creds_KeyId = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x21);
707-
constexpr uint32_t kOldKey_Creds_Base_Addr = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x22);
708-
constexpr uint32_t kOldKey_Creds_DAC_Offset = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x23);
709-
constexpr uint32_t kOldKey_Creds_DAC_Size = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x24);
710-
constexpr uint32_t kOldKey_Creds_PAI_Size = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x26);
711-
constexpr uint32_t kOldKey_Creds_PAI_Offset = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x25);
712-
constexpr uint32_t kOldKey_Creds_CD_Offset = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x27);
713-
constexpr uint32_t kOldKey_Creds_CD_Size = SilabsConfigKey(SilabsConfig::kMatterConfig_KeyBase, 0x28);
714-
715-
MigrationManager::MigrateUint32(kOldKey_Creds_KeyId, SilabsConfig::kConfigKey_Creds_KeyId);
716-
MigrationManager::MigrateUint32(kOldKey_Creds_Base_Addr, SilabsConfig::kConfigKey_Creds_Base_Addr);
717-
MigrationManager::MigrateUint32(kOldKey_Creds_DAC_Offset, SilabsConfig::kConfigKey_Creds_DAC_Offset);
718-
MigrationManager::MigrateUint32(kOldKey_Creds_DAC_Size, SilabsConfig::kConfigKey_Creds_DAC_Size);
719-
MigrationManager::MigrateUint32(kOldKey_Creds_PAI_Offset, SilabsConfig::kConfigKey_Creds_PAI_Offset);
720-
MigrationManager::MigrateUint32(kOldKey_Creds_PAI_Size, SilabsConfig::kConfigKey_Creds_PAI_Size);
721-
MigrationManager::MigrateUint32(kOldKey_Creds_CD_Offset, SilabsConfig::kConfigKey_Creds_CD_Offset);
722-
MigrationManager::MigrateUint32(kOldKey_Creds_CD_Size, SilabsConfig::kConfigKey_Creds_CD_Size);
723-
}
724-
725702
} // namespace Silabs
726703
} // namespace DeviceLayer
727704
} // namespace chip

examples/platform/silabs/provision/ProvisionStorageFlash.cpp

-3
Original file line numberDiff line numberDiff line change
@@ -734,9 +734,6 @@ CHIP_ERROR Storage::GetTestEventTriggerKey(MutableByteSpan & keySpan)
734734
#endif // SL_MATTER_TEST_EVENT_TRIGGER_ENABLED
735735

736736
} // namespace Provision
737-
738-
void MigrateDacProvider(void) {}
739-
740737
} // namespace Silabs
741738
} // namespace DeviceLayer
742739
} // namespace chip

src/platform/silabs/MigrationManager.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ static migrationData_t migrationTable[] = {
4040
{ .migrationGroup = 1, .migrationFunc = MigrateKvsMap },
4141
{ .migrationGroup = 2, .migrationFunc = MigrateDacProvider },
4242
{ .migrationGroup = 3, .migrationFunc = MigrateCounterConfigs },
43+
{ .migrationGroup = 4, .migrationFunc = MigrateHardwareVersion },
4344
// add any additional migration neccesary. migrationGroup should stay equal if done in the same commit or increment by 1 for
4445
// each new entry.
4546
};

src/platform/silabs/MigrationManager.h

+1
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ class MigrationManager
4545
void MigrateKvsMap(void);
4646
void MigrateDacProvider(void);
4747
void MigrateCounterConfigs(void);
48+
void MigrateHardwareVersion(void);
4849

4950
} // namespace Silabs
5051
} // namespace DeviceLayer

src/platform/silabs/SilabsConfig.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,7 @@ class SilabsConfig
134134
static constexpr Key kConfigKey_Provision_Request = SilabsConfigKey(kMatterFactory_KeyBase, 0x28);
135135
static constexpr Key kConfigKey_Provision_Version = SilabsConfigKey(kMatterFactory_KeyBase, 0x29);
136136
static constexpr Key kOtaTlvEncryption_KeyId = SilabsConfigKey(kMatterFactory_KeyBase, 0x30);
137+
static constexpr Key kConfigKey_HardwareVersion = SilabsConfigKey(kMatterFactory_KeyBase, 0x31);
137138

138139
// Matter Config Keys
139140
static constexpr Key kConfigKey_ServiceConfig = SilabsConfigKey(kMatterConfig_KeyBase, 0x01);
@@ -142,7 +143,6 @@ class SilabsConfig
142143
static constexpr Key kConfigKey_LastUsedEpochKeyId = SilabsConfigKey(kMatterConfig_KeyBase, 0x05);
143144
static constexpr Key kConfigKey_FailSafeArmed = SilabsConfigKey(kMatterConfig_KeyBase, 0x06);
144145
static constexpr Key kConfigKey_GroupKey = SilabsConfigKey(kMatterConfig_KeyBase, 0x07);
145-
static constexpr Key kConfigKey_HardwareVersion = SilabsConfigKey(kMatterConfig_KeyBase, 0x08);
146146
static constexpr Key kConfigKey_RegulatoryLocation = SilabsConfigKey(kMatterConfig_KeyBase, 0x09);
147147
static constexpr Key kConfigKey_CountryCode = SilabsConfigKey(kMatterConfig_KeyBase, 0x0A);
148148
static constexpr Key kConfigKey_WiFiSSID = SilabsConfigKey(kMatterConfig_KeyBase, 0x0C);

0 commit comments

Comments
 (0)