On Mon, Nov 09, 2015 at 05:02:40AM -0500, Yin, Fengwei wrote: > > From: Andy Green <andy.green@xxxxxxxxxx> > > wcn3620 has a new message structure for the reply to some hal > commands. This patch adds the struct and helper routine that > uses it if the chip is wcn3620, or falls back to the old > helper routine. > + if (wcn->chip_version != WCN36XX_CHIP_3620 || > + len < sizeof(struct wcn36xx_hal_msg_header) + sizeof(*rsp)) > + return wcn36xx_smd_rsp_status_check(buf, len); > + Ok, disregard my question on the other patch, for some reason I read them out of order. It looks like this should work ok with non-3620 chips. -- Bob Copeland %% http://bobcopeland.com/ -- 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