[PATCH 4/6] staging: wilc1000: rework address value.
Dan Carpenter
dan.carpenter at oracle.com
Tue Jun 30 09:31:47 UTC 2015
> - 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
More information about the devel
mailing list