Hi Nagaraj, On Mon, Jun 15, 2015 at 11:55 AM, Nagaraj D R <nagaraj.dr@xxxxxxxxxxx> wrote: > Hello Luiz, > >>On Thu, Jun 11, 2015 at 1:23 PM, Nagaraj D R wrote: >>> ++correction >>> Descriptors are searched between Characteristic value declaration and characteristic definition end handle > >>No Top posting on this mailing list please. >> >>> Hello Luiz, >>> The problem that I was trying to address is for the characteristic attribute at 0Xfffe and characteristic value declaration at oxffff. >>> This is a valid scenario. Characteristic need to have a characteristic value declaration but need not have characteristic descriptors. >>> >>> Since present design search the descriptors b/w the characteristic value declaration and service end handle, >>> if service end handle and characteristic value declaration end handle are same (i.e 0xffff in a particular case) >>> then descriptor search handle will be initialized to 0X000 (i.e by incrementing characteristic value handle (0xffff) ) >>> 0x0000 handle is reserved handle + intention of searching descriptor will not be achieved. > >>Sound like a different problem than I thought, anyway it is valid so >>could you please add this explanation to the patch description and >>perhaps a comment to the code so we have it crystal clear why it is >>necessary. > > I have raised v2 with the changes you suggested, please check it. Pushed, thanks. -- Luiz Augusto von Dentz -- 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
