Skip to content

WIP: Minor simulation missing#274

Open
padeken wants to merge 5 commits into
masterfrom
minor_simulation_missing
Open

WIP: Minor simulation missing#274
padeken wants to merge 5 commits into
masterfrom
minor_simulation_missing

Conversation

@padeken
Copy link
Copy Markdown
Collaborator

@padeken padeken commented May 27, 2026

Hi I would like to have these sim stand-ins in basil. I think they are useful for more projects.

@cbespin
Copy link
Copy Markdown
Contributor

cbespin commented May 28, 2026

Funnily enough, part of it (at least the PLL one) is included in #272 that is about to be merged the next days. I would wait until this one is done and then check again. The RAM block will probably still be needed as I don't think it is included in #272, but I would have to check (it is quite a big one..)

@kcaisley
Copy link
Copy Markdown
Member

Actually, after chatting with @awalsemann I actually removed the PLL block. Since my version was also just a black box, we couldn't think of a good reason to keep it. But I'll chat with @padeken, since if he wants it there's obviously a use case!

@padeken
Copy link
Copy Markdown
Collaborator Author

padeken commented May 28, 2026

Hi Ned,
yes then we move it to your merge. I just need a dummy for simulation and I just used the input as output, because it worked for my simplified simulation.

@cbespin
Copy link
Copy Markdown
Contributor

cbespin commented May 28, 2026

Well, if you need some sort of PLL in a simulation with more than one clock output, maybe we can make a more flexible simulation solution with instantiating clock dividers in the PLL replacement module?

@padeken
Copy link
Copy Markdown
Collaborator Author

padeken commented May 28, 2026

Yes, and writing this I can also provide the not sim module for the RAM, which should be more efificent, but I need to test this first

@cbespin
Copy link
Copy Markdown
Contributor

cbespin commented May 28, 2026

So you need the RAM simulation module to work on a more advanced PLL one? Then we could split this and merge the RAM first.
Do you instantiate the RAM module at some point or do you use an inferred version? In this case, one would not need the simulation module, no? Just wondering, because I did not run into this issue yet with designs that infer the primitive, but that might be on my end.

@padeken
Copy link
Copy Markdown
Collaborator Author

padeken commented May 28, 2026

I think I will talk to Ned and we will make a better PLL sim and the RAM maybe I will test first and then put it here.

@padeken padeken changed the title Minor simulation missing WIP: Minor simulation missing May 28, 2026
@awalsemann
Copy link
Copy Markdown
Contributor

I belive if you need a more advanced model for a PLL, just use an encrypted model from AMD or use one of the exitsing open models available, like this one: https://github.com/nmi-leipzig/sim-x-pll
I will not stop you from developing such models on your own, but I don't see the point in us reinventing the wheel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants