On Mon, Sep 28, 2015 at 11:43:55PM +0530, Chandra S Gorentla wrote: > - The message queue is replaced with standard Linux linked list > - kmem_cache is used for list members > - A check for return value of receive method is added > - GFP_ATOMIC is changed to GFP_KERNEL > - A few other related minor changes These should be listed and explained. > > while (1) { > - wilc_mq_recv(&gMsgQHostIF, &msg, sizeof(struct host_if_msg), &u32Ret); > + ret = wilc_mq_recv(&gMsgQHostIF, &msg, > + sizeof(struct host_if_msg), &u32Ret); > + if (ret) > + continue; > + I asked before if this was a forever loop and never got a response. Also what does this have to do with list macros? regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel