[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/6] qtest: irq_intercept_[out/in]: return FAIL if no interce
From: |
Peter Maydell |
Subject: |
Re: [PATCH 5/6] qtest: irq_intercept_[out/in]: return FAIL if no intercepts are installed |
Date: |
Mon, 24 Jul 2023 17:19:20 +0100 |
On Sat, 15 Jul 2023 at 00:27, Chris Laplante <chris@laplante.io> wrote:
>
> This is much better than just silently failing with OK.
>
> Signed-off-by: Chris Laplante <chris@laplante.io>
Makes sense. Did you do a 'make check' on an
all-targets-enabled build just to confirm we haven't
accidentally let any bogus uses of the command in while
it was returning OK for these cases?
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
thanks
-- PMM
- [PATCH 1/6] hw/gpio/nrf51: implement DETECT signal, (continued)
- [PATCH 1/6] hw/gpio/nrf51: implement DETECT signal, Chris Laplante, 2023/07/14
- [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 5/6] qtest: irq_intercept_[out/in]: return FAIL if no intercepts are installed,
Peter Maydell <=
- Re: [PATCH 0/6] Add nRF51 DETECT signal with test, Peter Maydell, 2023/07/24