Hi Marcel, On Wed, Jan 9, 2013 at 3:08 PM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: >> -#define define_test(name, args...) \ >> +#define define_test(name, _mtu, args...) \ >> do { \ >> const struct sdp_pdu pdus[] = { \ >> args, { }, { } \ >> }; \ >> static struct test_data data; \ >> - data.mtu = 48; \ >> + data.mtu = _mtu; \ > > so using _mtu instead of mtu was the trick here. For the heck of it I > could not figure that one out. So in the end I gave up on it and > hardcoded it :( Yes, I was caught by it as well, until I realized that the preprocessor might be replacing the "mtu" part of "data.mtu" (I confirmed when I saw the cpp output). > > Patch has been applied. Thanks, -- Anderson Lizardo Instituto Nokia de Tecnologia - INdT Manaus - Brazil -- 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