(mynewt-core) branch master updated: hw/hal: Use 32 bit for flash alignment

[jerzy]

This is an automated email from the ASF dual-hosted git repository.

jerzy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git


The following commit(s) were added to refs/heads/master by this push:
     new b822d0f49 hw/hal: Use 32 bit for flash alignment
b822d0f49 is described below

commit b822d0f49239a3cae84c6a60623d04ac2fd88420
Author: Jerzy Kasenberg <jerzy.kasenb...@codecoup.pl>
AuthorDate: Fri Nov 29 10:51:08 2024 +0100

    hw/hal: Use 32 bit for flash alignment
    
    This changes alignment type from uint8_t to uint32_t
    in several places.
    
    For flash_area_align() function 32 bits are already used
    by other platform supported by mcuboot.
    
    struct hal_flash also was using 32 bits before.
    
    Signed-off-by: Jerzy Kasenberg <jerzy.kasenb...@codecoup.pl>
---
 hw/hal/include/hal/hal_flash.h              | 2 +-
 hw/hal/src/hal_flash.c                      | 2 +-
 sys/flash_map/include/flash_map/flash_map.h | 4 ++--
 sys/flash_map/src/flash_map.c               | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/hw/hal/include/hal/hal_flash.h b/hw/hal/include/hal/hal_flash.h
index 95807ddef..74348671e 100644
--- a/hw/hal/include/hal/hal_flash.h
+++ b/hw/hal/include/hal/hal_flash.h
@@ -159,7 +159,7 @@ int hal_flash_isempty_no_buf(uint8_t id, uint32_t address, 
uint32_t num_bytes);
  *
  * @return                      The flash device's minimum write alignment.
  */
-uint8_t hal_flash_align(uint8_t flash_id);
+uint32_t hal_flash_align(uint8_t flash_id);
 
 /**
  * @brief Determines the value of an erased byte for a particular flash device.
diff --git a/hw/hal/src/hal_flash.c b/hw/hal/src/hal_flash.c
index 29cf72d49..1dd644065 100644
--- a/hw/hal/src/hal_flash.c
+++ b/hw/hal/src/hal_flash.c
@@ -53,7 +53,7 @@ hal_flash_init(void)
     return rc;
 }
 
-uint8_t
+uint32_t
 hal_flash_align(uint8_t flash_id)
 {
     const struct hal_flash *hf;
diff --git a/sys/flash_map/include/flash_map/flash_map.h 
b/sys/flash_map/include/flash_map/flash_map.h
index b0e13efed..28752cdea 100644
--- a/sys/flash_map/include/flash_map/flash_map.h
+++ b/sys/flash_map/include/flash_map/flash_map.h
@@ -60,7 +60,7 @@ struct flash_sector_range {
     uint16_t fsr_first_sector;
     uint16_t fsr_sector_count;
     uint32_t fsr_sector_size;
-    uint8_t fsr_align;
+    uint32_t fsr_align;
 };
 
 extern const struct flash_area *flash_map;
@@ -106,7 +106,7 @@ int flash_area_read_is_empty(const struct flash_area *, 
uint32_t off, void *dst,
 /*
  * Alignment restriction for flash writes.
  */
-uint8_t flash_area_align(const struct flash_area *);
+uint32_t flash_area_align(const struct flash_area *fa);
 
 /*
  * Value read from flash when it is erased.
diff --git a/sys/flash_map/src/flash_map.c b/sys/flash_map/src/flash_map.c
index c3e88dc9d..a49391b9e 100644
--- a/sys/flash_map/src/flash_map.c
+++ b/sys/flash_map/src/flash_map.c
@@ -275,7 +275,7 @@ flash_area_erase(const struct flash_area *fa, uint32_t off, 
uint32_t len)
     return hal_flash_erase(fa->fa_device_id, fa->fa_off + off, len);
 }
 
-uint8_t
+uint32_t
 flash_area_align(const struct flash_area *fa)
 {
     return hal_flash_align(fa->fa_device_id);