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

QL-MemoryBank-Flatten "num_wl" is ignored when "num_regions" is defined #1901

Open
mustafaarslan0 opened this issue Nov 18, 2024 · 1 comment

Comments

@mustafaarslan0
Copy link
Contributor

Describe the bug
"num_wl" of "ql_memory_bank_config_setting" is ignored when "num_regions" of "configuration_protocol" is defined (even if num_regions=1). It works if "num_regions" is not defined.

  <configuration_protocol>
    <organization type="ql_memory_bank" circuit_model_name="SRAM" num_regions="4">
      <bl protocol="flatten"/>
      <wl protocol="flatten"/>
    </organization>
    <ql_memory_bank_config_setting>
      <pb_type name="clb" num_wl="32"/>
    </ql_memory_bank_config_setting>

To Reproduce

  1. Clone latest main OpenFPGA repository
  2. Define "num_regions" in OpenFPGA arch file: k4_N4_40nm_qlbankflatten_defined_wl_openfpga.xml
  3. Execute OpenFPGA task: ql_memory_bank_flatten_defined_wl
  4. See that "num_wl" is ignored and OpenFPGA try to equalize number of BL and WLs.
@tangxifan
Copy link
Collaborator

@mustafaarslan0 Perhpas, the contributor @chungshien-chai is the best person to answer.
Here are my opinions:

  • If it is not specified in the current documentation, we should document the behavior. The additional syntax should be well documented under the section
  • I believe it is a bug. We should fix that in a near future.

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

No branches or pull requests

2 participants