On Wed, 2013-03-20 at 09:16 +0530, Mahesh Palivela wrote: > On Tue, 2013-03-19 at 13:28 +0100, Karl Beldan wrote: > > On Tue, Mar 19, 2013 at 05:17:06PM +0530, Mahesh Palivela wrote: > > > On Tue, 2013-03-19 at 01:23 +0100, Karl Beldan wrote: > > > > From: Karl Beldan <karl.beldan@xxxxxxxxxxxxxxxx> > > > > > + local->_oper_chandef.center_freq1 += cfreq_off; > > > > + local->_oper_chandef.center_freq2 += cfreq_off; > > > > > > can't add cfreq_off to center_freq2. Add only in case of non-zero > > > center_freq2? > > > > > At first I was like "why ?", but I hadn't spotted the places where we > > only check for "center_freq2 != 0". > > > > > > Thanks Mahesh, Have you tested vht with this ? > > Not yet Karl. I will give it a try and update you. > Karl, Is this patch based on mac80211-next kernel? I see failures/hunks when apply your patch. -- Thanks, Mahesh > -- > 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 -- 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