<div dir="ltr">This functionality is ideally implemented in a platform-specific system level control register (SLCR) driver similar to ZynqMP and Versal. You could also just leave it as-is since the default does exactly the same thing. This patch should be dropped unless you're going to define an implementation that will actually set the reference clock.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 28, 2023 at 11:57 PM Padmarao Begari <<a href="mailto:padmarao.begari@microchip.com">padmarao.begari@microchip.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">Add __weak_symbol instead of __weak_reference for the<br>
cgem_set_ref_clk() function for riscv build.<br>
---<br>
 freebsd/sys/dev/cadence/if_cgem.c | 7 +++++++<br>
 1 file changed, 7 insertions(+)<br>
<br>
diff --git a/freebsd/sys/dev/cadence/if_cgem.c b/freebsd/sys/dev/cadence/if_cgem.c<br>
index 363a9717..a12f7b43 100644<br>
--- a/freebsd/sys/dev/cadence/if_cgem.c<br>
+++ b/freebsd/sys/dev/cadence/if_cgem.c<br>
@@ -1764,6 +1764,12 @@ cgem_miibus_linkchg(device_t dev)<br>
  * Overridable weak symbol cgem_set_ref_clk().  This allows platforms to<br>
  * provide a function to set the cgem's reference clock.<br>
  */<br>
+#ifdef __riscv<br>
+__weak_symbol int cgem_set_ref_clk(int unit, int frequency)<br>
+{<br>
+       return 0;<br>
+}<br>
+#else<br>
 static int __used<br>
 cgem_default_set_ref_clk(int unit, int frequency)<br>
 {<br>
@@ -1771,6 +1777,7 @@ cgem_default_set_ref_clk(int unit, int frequency)<br>
        return 0;<br>
 }<br>
 __weak_reference(cgem_default_set_ref_clk, cgem_set_ref_clk);<br>
+#endif /* __riscv */<br>
<br>
 /* Call to set reference clock and network config bits according to media. */<br>
 static void<br>
-- <br>
2.25.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>