2015-05-09 21:02 GMT+09:00 Kalle Valo <kvalo@xxxxxxxxxxxxxx>: > Taehee Yoo <ap420073@xxxxxxxxx> writes: > >> 2015-05-08 22:01 GMT+09:00 Kalle Valo <kvalo@xxxxxxxxxxxxxx>: >>> Taehee Yoo <ap420073@xxxxxxxxx> writes: >>> >>>> 1. Remove setting REG_BCN_MAX_ERR code in _rtl92cu_set_media_status(). >>>> Initialization routine set REG_BCN_MAX_ERR to 0xFF. >>>> also _rtl92cu_set_media_status set REG_BCN_MAX_ERR to same value. >>>> so i remove this code. >>>> >>>> 2. Remove rtl92c_init_beacon_max_error's parameter. >>>> because parameter "infra_mode" is not used. >>>> >>>> Signed-off-by: Taehee Yoo <ap420073@xxxxxxxxx> >>> >>> Please split the patches into two. Having a numbered list of changes in >>> the commit log is a strong indication that the patch should be split >>> into smaller patches. >>> >>> -- >>> Kalle Valo >> >> Thank you for your review! >> I'll resend two patchs. > > Thanks. But when you send new versions please mark them as v2, v3 and so > on as that makes it easy for me to see what is the latest version. > > -- > Kalle Valo I already sent two patchs as "rtlwifi: rtl8192cu: Remove setting REG_BCN_MAX_ERR code in _rtl92cu_set_media_status()." and "rtlwifi: rtl8192cu: Remove rtl92c_init_beacon_max_error's parameter" is it right? or would I resend as V2? -- 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