Hi Jose, * Gustavo F. Padovan <gustavo@xxxxxxxxxxx> [2010-07-22 20:37:22 -0300]: > Hi Jose, > > * Jose Antonio Santos Cadenas <santoscadenas@xxxxxxxxx> [2010-07-22 10:56:22 +0200]: > > > From: José Antonio Santos-Cadenas <santoscadenas@xxxxxxxxx> > > > > MCL should transite to properly state when a create_mdl_req is > > not SUCCESS > > --- > > mcap/mcap.c | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/mcap/mcap.c b/mcap/mcap.c > > index cf92368..1dfe083 100644 > > --- a/mcap/mcap.c > > +++ b/mcap/mcap.c > > @@ -1260,6 +1260,7 @@ fail: > > mcl->mdls = g_slist_remove(mcl->mdls, mdl); > > g_free(mdl); > > g_error_free(gerr); > > + update_mcl_state(mcl); > > return close; > > } > > The same should apply here. Can you merge this with the original commit > that implemented create_mdl_req(), can't you? There should be another patches here in the same situation, I'm not checking all of them for this. Please check them and fix into the original commits when possible. That can make the review easier. ;) -- Gustavo F. Padovan http://padovan.org -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html