Also good for me, only one minor thing mentioned below. On 20.03.2018 18:55, Ajay Singh wrote: > + if (subtype == GO_NEG_REQ || subtype == GO_NEG_RSP || > + subtype == P2P_INV_REQ || subtype == P2P_INV_RSP) { > + for (i = P2P_PUB_ACTION_SUBTYPE + 2; i < size; i++) { > + if (buff[i] == P2PELEM_ATTR_ID && > + !(memcmp(p2p_oui, &buff[i + 2], 4))) { You can remove "(" ")" around memcmp. Your choice. > + wilc_wfi_cfg_parse_rx_action(&buff[i + 6], > + size - (i + 6)); > + break; > + } > + } > + } _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel