Merge pull request #18 from romain145/multi_page_write_fix

nseidle · web-flow · commit 0bd21b169656 · 2022-11-09T08:06:20.000-07:00
Fix multi-page write
diff --git a/src/SparkFun_External_EEPROM.cpp b/src/SparkFun_External_EEPROM.cpp
@@ -209,7 +209,7 @@ void ExternalEEPROM::write(uint32_t eepromLocation, const uint8_t *dataToWrite,
       uint16_t pageNumber1 = (eepromLocation + recorded) / settings.pageSize_bytes;
       uint16_t pageNumber2 = (eepromLocation + recorded + amtToWrite - 1) / settings.pageSize_bytes;
       if (pageNumber2 > pageNumber1)
-        amtToWrite = (pageNumber2 * settings.pageSize_bytes) - (eepromLocation + recorded); //Limit the read amt to go right up to edge of page barrier
+        amtToWrite = ((pageNumber1+1) * settings.pageSize_bytes) - (eepromLocation + recorded); //Limit the write amt to go right up to edge of page barrier
     }
 
     uint8_t i2cAddress = settings.deviceAddress;

Original file line number	Diff line number	Diff line change
`@@ -209,7 +209,7 @@ void ExternalEEPROM::write(uint32_t eepromLocation, const uint8_t *dataToWrite,`
`209`	`209`	`uint16_t pageNumber1 = (eepromLocation + recorded) / settings.pageSize_bytes;`
`210`	`210`	`uint16_t pageNumber2 = (eepromLocation + recorded + amtToWrite - 1) / settings.pageSize_bytes;`
`211`	`211`	`if (pageNumber2 > pageNumber1)`
`212`		`- amtToWrite = (pageNumber2 * settings.pageSize_bytes) - (eepromLocation + recorded); //Limit the read amt to go right up to edge of page barrier`
	`212`	`+ amtToWrite = ((pageNumber1+1) * settings.pageSize_bytes) - (eepromLocation + recorded); //Limit the write amt to go right up to edge of page barrier`
`213`	`213`	`}`
`214`	`214`
`215`	`215`	`uint8_t i2cAddress = settings.deviceAddress;`