sim: bfin: fix build warnings w/newer gcc
authorMike Frysinger <vapier@gentoo.org>
Mon, 24 May 2021 01:35:32 +0000 (21:35 -0400)
committerMike Frysinger <vapier@gentoo.org>
Mon, 24 May 2021 01:37:31 +0000 (21:37 -0400)
The bfin_otp_write_page_val func wants a pointer to an bu64[2] array,
but this code passes it a pointer to a single bu64.  It's in a struct
with a known compatible layout:
bu64 data0, data1, data2, data3;
But gcc doesn't allow these kinds of tricks anymore.  Use the more
verbose form to make the compiler happy since this is not performance
sensitive code.

sim/bfin/ChangeLog
sim/bfin/dv-bfin_otp.c

index 1ebf6818ac7735626cdcb42e9a3997c019706fd0..9c517d20bbeb7ecd0657490bcae20e7d8413e6f9 100644 (file)
@@ -1,3 +1,7 @@
+2021-05-23  Mike Frysinger  <vapier@gentoo.org>
+
+       * dv-bfin_otp.c (bfin_otp_write_page): Call bfin_otp_write_page_val2.
+
 2021-05-23  Mike Frysinger  <vapier@gentoo.org>
 
        * dv-bfin_cec.c: Include strings.h.
index 6cf1c8155c623a3b6c90de3dcc3e63089911fdb2..65afdf58b27fe62646f93a7025c7cc4357243807 100644 (file)
@@ -91,7 +91,7 @@ bfin_otp_write_page_val2 (struct bfin_otp *otp, bu16 page, bu64 lo, bu64 hi)
 static void
 bfin_otp_write_page (struct bfin_otp *otp, bu16 page)
 {
-  bfin_otp_write_page_val (otp, page, (void *)&otp->data0);
+  bfin_otp_write_page_val2 (otp, page, otp->data0, otp->data1);
 }
 
 static unsigned