search

enjoy-digital / litedram

Small footprint and configurable DRAM core
Other
365 stars 115 forks source link

fix gcc warning: function used but not defined #253

Closed gsomlo closed 3 years ago

gsomlo commented 3 years ago

The 'cdelay()' function is defined in LiteX 'liblitedram', so we should not be using 'static' to declare it before calling it from other C source files.

gsomlo commented 3 years ago

CI should stop failing once https://github.com/enjoy-digital/litex/pull/923 is applied in LiteX, so maybe apply that one first.

gsomlo commented 3 years ago

CI still fails, but only because it still pulls in a pre-923 commit of LiteX:

https://github.com/enjoy-digital/litedram/runs/2677064001#step:8:825

@enjoy-digital, please have a look and let me know what you think.

enjoy-digital commented 3 years ago

Thanks @gsomlo, I merged it and will fix it if still failing.