use values pointer directly

Author: microdev1

ports/esp32s2/common-hal/nvm/ByteArray.c

@@ -26,8 +26,6 @@
 
 #include "common-hal/nvm/ByteArray.h"
 
-#include <string.h>
-
 #include "py/runtime.h"
 #include "nvs_flash.h"
 
@@ -56,17 +54,14 @@ bool common_hal_nvm_bytearray_set_bytes(nvm_bytearray_obj_t *self,
         uint32_t start_index, uint8_t* values, uint32_t len) {
     char index[9];
 
-    uint8_t buffer[len];
-    memcpy(buffer, values, len);
-
     // start nvs
     nvs_handle_t handle;
     get_nvs_handle(&handle);
 
     // stage flash changes
     for (uint32_t i = 0; i < len; i++) {
         sprintf(index, "%i", start_index + i);
-        if (nvs_set_u8(handle, (const char *)index, buffer[i]) != ESP_OK) {
+        if (nvs_set_u8(handle, (const char *)index, values[i]) != ESP_OK) {
             return false;
         }
     }
@@ -84,7 +79,6 @@ bool common_hal_nvm_bytearray_set_bytes(nvm_bytearray_obj_t *self,
 void common_hal_nvm_bytearray_get_bytes(nvm_bytearray_obj_t *self,
         uint32_t start_index, uint32_t len, uint8_t* values) {
     char index[9];
-    uint8_t buffer[len];
 
     // start nvs
     nvs_handle_t handle;
@@ -93,14 +87,11 @@ void common_hal_nvm_bytearray_get_bytes(nvm_bytearray_obj_t *self,
     // get from flash
     for (uint32_t i = 0; i < len; i++) {
         sprintf(index, "%i", start_index + i);
-        if (nvs_get_u8(handle, (const char *)index, &buffer[i]) != ESP_OK) {
+        if (nvs_get_u8(handle, (const char *)index, &values[i]) != ESP_OK) {
             mp_raise_RuntimeError(translate("NVS Error"));
         }
     }
 
-    // set into values
-    memcpy(values, buffer, len);
-
     // close nvs
     nvs_close(handle);
 }