On 18 January 2015 at 19:47, Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> wrote: > Hello. > > On 1/18/2015 8:11 AM, Andy Green wrote: > >> WCN3620 can asynchronously send two new kinds of indication message, >> since we can't handle them just accept them quietly. > > >> Signed-off-by: Andy Green <andy.green@xxxxxxxxxx> >> --- >> drivers/net/wireless/ath/wcn36xx/smd.c | 6 ++++++ >> 1 file changed, 6 insertions(+) > > >> diff --git a/drivers/net/wireless/ath/wcn36xx/smd.c >> b/drivers/net/wireless/ath/wcn36xx/smd.c >> index 6398693..819741c 100644 >> --- a/drivers/net/wireless/ath/wcn36xx/smd.c >> +++ b/drivers/net/wireless/ath/wcn36xx/smd.c > > [...] >> >> @@ -2107,6 +2109,10 @@ static void wcn36xx_ind_smd_work(struct work_struct >> *work) >> msg_header = (struct wcn36xx_hal_msg_header *)hal_ind_msg->msg; >> >> switch (msg_header->msg_type) { >> + case WCN36XX_HAL_COEX_IND: >> + break; >> + case WCN36XX_HAL_AVOID_FREQ_RANGE_IND: >> + break; > > > Can't you merge these 2 cases, so that there's only one *break*? Yes will do. -Andy > [...] > > WBR, Sergei > -- 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