gpl: Allow for MBFF tray identification on flops that have scan, preset, and clear pins.#8222
Conversation
hongted
changed the title
hongted
force-pushed
the
mbff
branch
3 times, most recently
from
4844a82 to
e5c4e4c
Compare
hongted
force-pushed
the
mbff
branch
2 times, most recently
from
6205c67 to
18f2521
Compare
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
hongted
force-pushed
the
mbff
branch
2 times, most recently
from
5830a70 to
98b52cc
Compare
|
some conflicts to resolve |
hongted
force-pushed
the
mbff
branch
2 times, most recently
from
a40386d to
e1532f9
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
Resolved. Rest will be fixed once I rebase onto the parent PR. |
hongted
force-pushed
the
mbff
branch
2 times, most recently
from
2e6fc96 to
cb84969
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
|
clang-tidy review says "All clean, LGTM! 👍" |
hongted
force-pushed
the
mbff
branch
5 times, most recently
from
dd30acc to
b9e075e
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
clang-tidy review says "All clean, LGTM! 👍" |
…et, and clear ports. Signed-off-by: Ted Hong <tedhong@google.com>
|
clang-tidy review says "All clean, LGTM! 👍" |
|
Merged master so I could see the changes here without the noise of the gui changes that are already merged. |
|
clang-tidy review says "All clean, LGTM! 👍" |
| class MBFFTestFixture : public tst::Fixture | ||
| { | ||
| protected: | ||
| void SetUp() override |
There was a problem hiding this comment.
My thought is that we would create one fixture per tool that would handle all the dependency setup so that all the tests could share it. Since there is just one test for gpl now this is ok but we should plan to split it out in the future.
2 participants
No description provided.