[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/6] Add nRF51 DETECT signal with test
From: |
Peter Maydell |
Subject: |
Re: [PATCH 0/6] Add nRF51 DETECT signal with test |
Date: |
Mon, 24 Jul 2023 17:27:58 +0100 |
On Sat, 15 Jul 2023 at 00:27, Chris Laplante <chris@laplante.io> wrote:
>
> This patch series implements the nRF51 DETECT signal
> in the GPIO peripheral. A qtest is added exercising the signal.
>
> To implement the test, named out-GPIO IRQ interception had to be added
> to the qtest framework. I also took the opportunity to improve IRQ
> interception a bit by adding 'FAIL' responses when interception fails.
> Otherwise, it is frustrating to troubleshoot why calls to
> qtest_irq_intercept_out and friends appears to do nothing.
Thanks for this patchset and especially for the work
improving the qtest infrastructure. I've given my
comments on the different patches, and in some cases
reviewed-by tags. (Where I've given one of those, you should
add it to your commit message for the relevant patch under
your Signed-off-by: line, so that when you send the version
2 of the patchset we know that those parts are already
reviewed and don't need re-examining. If I said "make
some change; otherwise Reviewed-by" that means "make
that minor change, and then you can add the tag, etc".)
Do you have the parts of this feature that use the DETECT
signal in the POWER device, or have you not written those
yet ? If you have them, you could send those too in v2.
-- PMM
- [PATCH 2/6] qtest: implement named interception of out-GPIO, (continued)
- [PATCH 2/6] qtest: implement named interception of out-GPIO, Chris Laplante, 2023/07/14
- [PATCH 3/6] qtest: bail from irq_intercept_in if name is specified, Chris Laplante, 2023/07/14
- [PATCH 4/6] qtest: factor out qtest_install_gpio_out_intercepts, Chris Laplante, 2023/07/14
- [PATCH 6/6] qtest: microbit-test: add tests for nRF51 DETECT, Chris Laplante, 2023/07/14
- [PATCH 5/6] qtest: irq_intercept_[out/in]: return FAIL if no intercepts are installed, Chris Laplante, 2023/07/14
- Re: [PATCH 0/6] Add nRF51 DETECT signal with test,
Peter Maydell <=