<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><div dir="ltr"><div dir="ltr">Typo in the commit message. I don't know this code very well so I'll leave content review for someone else for now.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jan 7, 2024 at 12:19 PM <<a href="mailto:berndmoessner80@gmail.com">berndmoessner80@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Bernd Moessner <<a href="mailto:berndmoessner80@gmail.com" target="_blank">berndmoessner80@gmail.com</a>><br>
<br>
When the region feature is active, flashdev assumes that<br>
a relative address is provided. It computes the abs. address and<br>
carries out the read / write. However, in this case it must not<br>
assign the abs. address to iop->offset. The relative address is<br>
required here to allow newlib to correctly read/write the next<br>
junk of data.<br></blockquote><div><br></div><div>"junk" should probably be "chunk"<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Updates #4981<br>
---<br>
 cpukit/dev/flash/flashdev.c | 10 ++++------<br>
 1 file changed, 4 insertions(+), 6 deletions(-)<br>
<br>
diff --git a/cpukit/dev/flash/flashdev.c b/cpukit/dev/flash/flashdev.c<br>
index 7bc13ed70a..b908e87023 100644<br>
--- a/cpukit/dev/flash/flashdev.c<br>
+++ b/cpukit/dev/flash/flashdev.c<br>
@@ -145,8 +145,7 @@ static int rtems_flashdev_get_abs_addr(<br>
 static int rtems_flashdev_update_and_return(<br>
   rtems_libio_t *iop,<br>
   int status,<br>
-  size_t count,<br>
-  off_t new_offset<br>
+  size_t count<br>
 );<br>
<br>
 static uint32_t rtems_flashdev_find_unallocated_region(<br>
@@ -339,7 +338,7 @@ static int rtems_flashdev_read_write(<br>
   rtems_flashdev_release( flash );<br>
<br>
   /* Update offset and return */<br>
-  return rtems_flashdev_update_and_return( iop, status, count, addr + count );<br>
+  return rtems_flashdev_update_and_return( iop, status, count );<br>
 }<br>
<br>
 static int rtems_flashdev_ioctl(<br>
@@ -599,13 +598,12 @@ static int rtems_flashdev_get_abs_addr(<br>
 static int rtems_flashdev_update_and_return(<br>
   rtems_libio_t *iop,<br>
   int status,<br>
-  size_t count,<br>
-  off_t new_offset<br>
+  size_t count<br>
 )<br>
 {<br>
   /* Update offset and return */<br>
   if ( status == 0 ) {<br>
-    iop->offset = new_offset;<br>
+    iop->offset += count;<br>
     return count;<br>
   } else {<br>
     rtems_set_errno_and_return_minus_one( status );<br>
-- <br>
2.34.1<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>