Hi Inga,
On Thu, Jun 3, 2021 at 12:38 PM Tedd Ho-Jeong An
<tedd.an@xxxxxxxxxxxxxxx> wrote:
>
> Hi Inga,
>
> On Wed, 2021-06-02 at 17:04 -0700, Inga Stotland wrote:
> > This changes check for setup complete in client_cmd_complete() callback
> > from tester_setup_complete() to test_setup_condition_complete(). This
> > allows for combining setup conditions when setup_bthost() is called.
>
> Reviewed-by: Tedd Ho-Jeong An <tedd.an@xxxxxxxxx>
>
> > ---
> > tools/mgmt-tester.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/tools/mgmt-tester.c b/tools/mgmt-tester.c
> > index 4dd3490c2..657b7aaa0 100644
> > --- a/tools/mgmt-tester.c
> > +++ b/tools/mgmt-tester.c
> > @@ -5831,7 +5831,7 @@ static void client_cmd_complete(uint16_t opcode, uint8_t status,
> > if (status)
> > tester_setup_failed();
> > else
> > - tester_setup_complete();
> > + test_setup_condition_complete(data);
> > }
> >
> > static void setup_bthost(void)
> > @@ -5842,6 +5842,7 @@ static void setup_bthost(void)
> >
> > bthost = hciemu_client_get_host(data->hciemu);
> > bthost_set_cmd_complete_cb(bthost, client_cmd_complete, data);
> > + test_add_setup_condition(data);
> >
> > if (data->hciemu_type == HCIEMU_TYPE_LE ||
> > test->client_enable_adv) {
>
> Regards,
> Tedd
>
Applied, thanks.
--
Luiz Augusto von Dentz
