Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

LDO Vin range is not useful for powering sky130A standard cells #2

Closed
shalan opened this issue May 12, 2021 · 8 comments
Closed

LDO Vin range is not useful for powering sky130A standard cells #2

shalan opened this issue May 12, 2021 · 8 comments
Assignees
@lulu9312

Comments

@shalan
Copy link

shalan commented May 12, 2021 •
edited
Loading

According to this, the LDO generator cannot generate 1.8V LDO needed to power digital macros implemented using sky130 SCLs (e.g., hd, hell, ...)
@msaligane
Copy link
Member

That needs to be updated. FYI, the LDO has not been released yet using the open source tools.

@msaligane
Copy link
Member

@wenbodd can you update these:

      "vin": {
        "max": 1.3,
        "min": 0.6}

@shalan
Copy link
Author

shalan commented May 12, 2021

Is it characterize for Vin beyond 1.3V? The FASoC website lists the same range (0.6 to 1.3V) on the LDO specs table.

@msaligane
Copy link
Member

msaligane commented May 12, 2021 •
edited
Loading

Yes, the sky130 version of the LDO takes in 3.3v/5v and generated 1.8v. These values are from other technologies we are supporting and it seems this file was not updated.

I would advise not using this code for now. I will update this when it is ready.

@mithro
Copy link

mithro commented May 12, 2021

Can you please add an example which generates something useful for SKY130?

@msaligane
Copy link
Member

Sure.
The temp sensor is the only thing released for now and the example is shown in the readme. LDO and SAR ADC are wip.
We should have updates around early June.

luccareinehr added a commit to luccareinehr/OpenFASOC that referenced this issue Jun 30, 2022
@luccareinehr
Add __init__ files for subpackage structure in the Pythonic API
d296066 
After this commit, the package currently looks like this:

openfasoc (package)
  config.py
  generators (subpackage idea-fasoc#1)
    temp_sense_gen (subpackage idea-fasoc#2)
        tools (subpackage idea-fasoc#3)
        function.py
        TEMP_netlist.py
@msaligane msaligane assigned msaligane and unassigned msaligane Aug 18, 2022
msaligane pushed a commit that referenced this issue Sep 8, 2022
@lulu9312
Merge pull request #2 from lulu9312/lulu9312-patch-2
be127c7 
Config file updated
@msaligane
Copy link
Member

This should be fixed now. @lulu9312 ?

msaligane pushed a commit that referenced this issue Dec 2, 2022
@alibillalhammoud
Merge pull request #2 from lulu9312/main
7998122 
DRC violations fixed and latch placement fixed
@lulu9312
Copy link
Contributor

Yes @msaligane This is fixed. We can close this issue.

msaligane pushed a commit that referenced this issue May 17, 2023
@Jerry-lqx
Merge pull request #2 from saicharan0112/patch-3
ae46f64 
Update cryo_sky130hd.yml
gkielian pushed a commit to gkielian/OpenFASOC that referenced this issue Nov 15, 2023
@Wen-Tian-Pineapple
Merge pull request idea-fasoc#2 from idea-fasoc/main
6fd6427 
update
msaligane pushed a commit that referenced this issue Jun 5, 2024
@chetanyagoyal
Merge pull request #2 from chetanyagoyal/patch-1
ce42c42 
comment ldo from build and test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lulu9312 lulu9312

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mithro @shalan @msaligane @lulu9312 @saicharan0112