> - if ((pstrHostIfSetDrvHandler->u32Address) == (u32)NULL) { > + if (pstrHostIfSetDrvHandler->pAddress == NULL) { In the linux kernel we do not like using 'p' to stand for pointer. This new name uses CamelCase. Use scripts/checkpatch.pl to find obvious mistakes like this. The name "address" is too generic. Use a meaningful name. 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