On Fri, Oct 30, 2015 at 12:10:42PM +0900, Glen Lee wrote: > /*create the common structure*/ > - g_linux_wlan = kzalloc(sizeof(*g_linux_wlan), GFP_KERNEL); > - if (!g_linux_wlan) > + wl = kzalloc(sizeof(struct wilc), GFP_KERNEL); This introduces a checkpatch.pl warning. The prefered style is: wl = kzalloc(sizeof(*wl), GFP_KERNEL); But the larger point is that please use checkpatch.pl. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html