Merge pull request #13699 from boraozgen/bugfix/sfdp-find-addr-region

Fix sfdp_find_addr_region algorithm

Author: 0xc0170

drivers/include/drivers/internal/SFDP.h

@@ -129,7 +129,7 @@ int sfdp_detect_erase_types_inst_and_size(uint8_t *bptbl_ptr, sfdp_hdr_info &sfd
  *
  * @return Region number
  */
-int sfdp_find_addr_region(bd_size_t offset, const sfdp_hdr_info &sfdp_info);
+int sfdp_find_addr_region(bd_addr_t offset, const sfdp_hdr_info &sfdp_info);
 
 /** Finds the largest Erase Type of the Region to which the offset belongs to
  *

drivers/source/SFDP.cpp

@@ -373,20 +373,20 @@ int sfdp_detect_erase_types_inst_and_size(uint8_t *bptbl_ptr, sfdp_hdr_info &sfd
     return 0;
 }
 
-int sfdp_find_addr_region(bd_size_t offset, const sfdp_hdr_info &sfdp_info)
+int sfdp_find_addr_region(bd_addr_t offset, const sfdp_hdr_info &sfdp_info)
 {
-    if ((offset > sfdp_info.bptbl.device_size_bytes) || (sfdp_info.smptbl.region_cnt == 0)) {
+    if ((offset >= sfdp_info.bptbl.device_size_bytes) || (sfdp_info.smptbl.region_cnt == 0)) {
         return -1;
     }
 
     if (sfdp_info.smptbl.region_cnt == 1) {
         return 0;
     }
 
-    for (int idx = sfdp_info.smptbl.region_cnt - 2; idx >= 0; idx--) {
+    for (int idx = 0; idx < sfdp_info.smptbl.region_cnt; idx++) {
 
-        if (offset > sfdp_info.smptbl.region_high_boundary[idx]) {
-            return (idx + 1);
+        if (offset <= sfdp_info.smptbl.region_high_boundary[idx]) {
+            return idx;
         }
     }
     return -1;