On Wed, Mar 20, 2013 at 04:23:16PM +0530, Mahesh Palivela wrote: > 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. > It is based on mac80211-next yes. I did it Sunday so it applies properly @ddbfe86 and does so up to 445ea4e^1, (445ea4e was Tuesday). Karl -- 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
