Skip to content

Controller with bus#110

Draft
codesavyani wants to merge 14 commits intomasterfrom
controller_with_bus
Draft

Controller with bus#110
codesavyani wants to merge 14 commits intomasterfrom
controller_with_bus

Conversation

@codesavyani
Copy link
Copy Markdown
Collaborator

@codesavyani codesavyani commented Oct 28, 2025

feat(Physical Design): Implementing a bus from the controller to solve utilization problem in the controller

Description

There is a utilization problem if we place controller on the left side of the resources because then the controller needs to be of height equal to all the resources.

These changes instead support controller to be below the resources in the physical design to decrease the overall area allotted to the controller.

To support this, a bus structure is implemented to pass instr, valid and activate signal to all the resources from the controller

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

This has been tested by running tests mul_32_1_1, mul_32_3_1 and mul_512_3_1

Checklist

  • [* ] My code follows the style guidelines of this project
  • [*] I have performed a self-review of my code
  • [*] I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • [*] My changes generate no new warnings
  • [*] I have added tests that prove my fix is effective or that my feature works
  • [*] New and existing unit tests pass locally with my changes
  • [*] Any dependent changes have been merged and published in downstream modules

@codesavyani codesavyani self-assigned this Oct 30, 2025
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.

2 participants