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.

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

Give don't use cells to Yosys. #188

Merged
merged 3 commits into from
Sep 20, 2023
Merged

Give don't use cells to Yosys. #188

merged 3 commits into from
Sep 20, 2023

Conversation

mithro
Copy link
Member

@mithro mithro commented Sep 18, 2023

No description provided.

@tcal-x
Copy link
Member

tcal-x commented Sep 18, 2023

Was this created using the copy tool or manually? Do we care about proper attribution?

@QuantamHD
Copy link
Collaborator

I wrote this, and I don't mind if it's not properly attributed.

@mithro
Copy link
Member Author

mithro commented Sep 18, 2023

Was this created using the copy tool or manually? Do we care about proper attribution?

This was created manually.

@mithro
Copy link
Member Author

mithro commented Sep 20, 2023

Looks like there is a real issue here;


Starting to write bazel-out/k8-fastbuild/bin/external/com_google_skywater_pdk_sky130_fd_sc_hd/timing/sky130_fd_sc_hd__ff_n40C_1v56.lib (basic from basic)
Overwriting comp_attribute,voltage_map with [['VSS', 0.0], ['KAPWR', 1.56], ['LOWLVPWR', 1.56], ['VGND', 0.0], ['VNB', 0.0], ['VPB', 1.56], ['VPWR', 1.56], ['VPWRIN', 1.56]] (existing value of {'VGND': 0.0, 'VNB': 0.0} )
   Finish writing bazel-out/k8-fastbuild/bin/external/com_google_skywater_pdk_sky130_fd_sc_hd/timing/sky130_fd_sc_hd__ff_n40C_1v56.lib

synthesis/tests/counter.v {}
{} {}
ERROR: /workspace/synthesis/tests/BUILD:84:15: SynthesizingRTL synthesis/tests/verilog_counter_asap7_synth_synth_output.v failed: (Exit 1): yosys failed: error executing command bazel-out/k8-opt-exec-2B5CBBC6/bin/external/at_clifford_yosys/yosys -q -q -Q -l bazel-out/k8-fastbuild/bin/synthesis/tests/verilog_counter_asap7_synth_yosys_output.log -c synthesis/synth.tcl

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
ERROR: TCL interpreter returned an error: Yosys command produced an error

@mithro
Copy link
Member Author

mithro commented Sep 20, 2023

Looks like we need to upgrade the version of abc?

ERROR: Command syntax error: Unknown option or option in arguments.
> abc -liberty bazel-out/k8-fastbuild/bin/external/com_google_skywater_pdk_sky130_fd_sc_hd/timing/sky130_fd_sc_hd__ff_100C_1v95.lib -dff -g aig -dont_use sky130_fd_sc_hd__probe_p_* -dont_use sky130_fd_sc_hd__probec_p_*
>                                                                                                                                               ^
ERROR: TCL interpreter returned an error: Yosys command produced an error

@mithro
Copy link
Member Author

mithro commented Sep 20, 2023

I've upgraded both Yosys and ABC and the build runs fine locally now, will see what cloud build thinks.

@QuantamHD QuantamHD merged commit af98f9a into hdl:main Sep 20, 2023
4 checks passed
@mithro mithro deleted the dont-use-args branch September 20, 2023 17:06
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.

3 participants