From ef8219b087404b9b24479d5dcd60628cd76ef55e Mon Sep 17 00:00:00 2001 From: Nav Date: Sun, 11 Dec 2022 17:40:51 +0000 Subject: [PATCH] Added EEPROM size sense check to TDF validation script --- .../AVR8/Avr8TargetDescriptionFile.php | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/build/scripts/TargetDescriptionFiles/AVR8/Avr8TargetDescriptionFile.php b/build/scripts/TargetDescriptionFiles/AVR8/Avr8TargetDescriptionFile.php index 02f2f2da..9e742cf7 100644 --- a/build/scripts/TargetDescriptionFiles/AVR8/Avr8TargetDescriptionFile.php +++ b/build/scripts/TargetDescriptionFiles/AVR8/Avr8TargetDescriptionFile.php @@ -688,6 +688,17 @@ class Avr8TargetDescriptionFile extends TargetDescriptionFile $failures[] = 'Missing eeprom size.'; } + if ($this->eepromSize < 64) { + /* + * All AVR8 targets supported by Bloom have at least 64 bytes of EEPROM. This is assumed in some areas of + * Bloom's code. + * + * The purpose of this check is to remind me to address those areas of Bloom's code before adding support + * for an AVR8 target with no EEPROM. + */ + $failures[] = 'Unexpected eeprom size.'; + } + if (is_null($this->eepromPageSize)) { $failures[] = 'Missing eeprom page size.'; }