Skip to content

Comments

minerva-ag: Modify clock status registers#2276

Closed
Victor-Jhong wants to merge 1 commit intofacebook:mainfrom
Victor-Jhong:minerva-ag/modify_clock_status_registers
Closed

minerva-ag: Modify clock status registers#2276
Victor-Jhong wants to merge 1 commit intofacebook:mainfrom
Victor-Jhong:minerva-ag/modify_clock_status_registers

Conversation

@Victor-Jhong
Copy link
Contributor

Summary:

  • remove LOSMON0(0x30) LOSMON1(0x40) LOSMON2(0x50)
  • add APLL_EVENT(0x169)

Test Plan:

  • Build code: PASS

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 19, 2025
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. (Because this pull request was imported automatically, there will not be any future comments.)

@Victor-Jhong Victor-Jhong force-pushed the minerva-ag/modify_clock_status_registers branch from a6564df to 17ddd55 Compare March 24, 2025 06:29
@facebook-github-bot
Copy link
Contributor

@Victor-Jhong has updated the pull request. You must reimport the pull request before landing.

Copy link
Contributor

@wangx6f wangx6f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you take care of the cpp-check error to make sure it passes?

@Victor-Jhong Victor-Jhong force-pushed the minerva-ag/modify_clock_status_registers branch from 17ddd55 to 89ff7b1 Compare April 1, 2025 12:24
@facebook-github-bot
Copy link
Contributor

@Victor-Jhong has updated the pull request. You must reimport the pull request before landing.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 7a91b45.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants