Hi will, > On Wed, 2023-03-01 at 08:27 -0600, Will wrote: > > 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. > Ok, will drop this patch.
Regards Padmarao > On Tue, Feb 28, 2023 at 11:57 PM Padmarao Begari < > padmarao.beg...@microchip.com> wrote: > > Add __weak_symbol instead of __weak_reference for the > > cgem_set_ref_clk() function for riscv build. > > --- > > freebsd/sys/dev/cadence/if_cgem.c | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/freebsd/sys/dev/cadence/if_cgem.c > > b/freebsd/sys/dev/cadence/if_cgem.c > > index 363a9717..a12f7b43 100644 > > --- a/freebsd/sys/dev/cadence/if_cgem.c > > +++ b/freebsd/sys/dev/cadence/if_cgem.c > > @@ -1764,6 +1764,12 @@ cgem_miibus_linkchg(device_t dev) > > * Overridable weak symbol cgem_set_ref_clk(). This allows > > platforms to > > * provide a function to set the cgem's reference clock. > > */ > > +#ifdef __riscv > > +__weak_symbol int cgem_set_ref_clk(int unit, int frequency) > > +{ > > + return 0; > > +} > > +#else > > static int __used > > cgem_default_set_ref_clk(int unit, int frequency) > > { > > @@ -1771,6 +1777,7 @@ cgem_default_set_ref_clk(int unit, int > > frequency) > > return 0; > > } > > __weak_reference(cgem_default_set_ref_clk, cgem_set_ref_clk); > > +#endif /* __riscv */ > > > > /* Call to set reference clock and network config bits according > > to media. */ > > static void _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel