opentitan icon indicating copy to clipboard operation
opentitan copied to clipboard

Published 20 hours ago • verified publisher icon lowRISC

[dv/pwm] Post V2S TODO list

Open cindychip opened this issue 3 years ago • 7 comments

Because all V2S requirements are met, I am going to move the remaining items to post V2:

  • [ ] pwm_env_pkg functional coverage is low.

image

  • [ ] Check why pwm_en_cio_o is not excluded from UNR.

  • [x] Martin: help check issue: https://github.com/lowRISC/opentitan/issues/9999

estimate 8

cindychip avatar Jul 11 '22 22:07 cindychip

@sriyerg it seems like you are going to help with pwm V3. These are the TODO items discussed in the PWM V2 review :)

cindychip avatar Dec 06 '22 19:12 cindychip

Triaged for pwm. Assigning to M2.5 with https://github.com/lowRISC/opentitan/labels/Priority%3AP1 because low functional coverage may hide functional bugs.

andreaskurth avatar Feb 27 '23 10:02 andreaskurth

Estimate range: 4 - 8

GregAC avatar Mar 23 '23 17:03 GregAC

Assigning to @andreaskurth since @sriyerg is no longer on this project. Please reassign to the appropriate person if necessary.

jonmichelson avatar May 02 '23 22:05 jonmichelson

Assigning @marnovandermaas as per SoT assignment

andreaskurth avatar May 03 '23 12:05 andreaskurth

This seems like a V3 task and probably doesn't need to be in M5. Marking as a triage priority accordingly.

rswarbrick avatar Jun 18 '24 13:06 rswarbrick

Moving to M7, where we target V3 completion.

rswarbrick avatar Jun 19 '24 09:06 rswarbrick

With a run from yesterday (and a few local commits that are in PRs), I see: Image

Still not great, but marginally better than when this was originally opened. Remaining effort estimate: 2 days.

rswarbrick avatar Nov 13 '24 11:11 rswarbrick

The question about toggle coverage of pwm_en_cio_o should be made moot by #25140. (Maybe it wasn't caught because we haven't got the UNR flow configured for toggle coverage, but it doesn't really matter)

rswarbrick avatar Nov 14 '24 13:11 rswarbrick