2016-05-11 22:55 GMT+02:00 Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx>: > However, this patch changes more. I did not get back on your question > regarding beacon period and dtim. Sorry for that as it means this patch > still need rework. Those values are obtained differently and filled in > by the function brcmf_fill_bss_param() (called in line 2530). When you > do that make sure the subject prefix is 'PATCH v2' iso 'PATCH' (using > git format-patch option --subject-prefix=..) and add a changelog after > your signed-off line as shown below: > > Signed-off-by: Jaap Jan Meijer <jjmeijer88@xxxxxxxxx> > --- > change log: > v2: > - bla... > --- > .../broadcom/brcm80211/brcmfmac/cfg80211.c | 42 > ++++++++++++++++++++++ > > Don't get discouraged :-D Nederlander? I won't but I hope you won't either and yes, what gave it away? ;) 2016-05-12 8:32 GMT+02:00 Kalle Valo <kvalo@xxxxxxxxxxxxxx>: > > This fixed a regression, right? So then a Fixes line would be nice: > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches#n171 > > -- > Kalle Valo Thank you for the feedback guys, I will send a new patch shortly. -- 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