Return-path: Received: from mail-yx0-f174.google.com ([209.85.213.174]:51287 "EHLO mail-yx0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750768Ab1HKQW1 convert rfc822-to-8bit (ORCPT ); Thu, 11 Aug 2011 12:22:27 -0400 Received: by yxj19 with SMTP id 19so1414996yxj.19 for ; Thu, 11 Aug 2011 09:22:27 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1313001339-11211-1-git-send-email-zajec5@gmail.com> <1313001339-11211-2-git-send-email-zajec5@gmail.com> <4E435D0E.80702@lwfinger.net> <20110811160424.GC2566@tuxdriver.com> Date: Thu, 11 Aug 2011 18:22:26 +0200 Message-ID: (sfid-20110811_182230_963751_FC48D74D) Subject: Re: [RFC][RFT][PATCH V2 2/2] b43: fix DMA on some bugged hardware From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= To: "John W. Linville" Cc: Larry Finger , linux-wireless@vger.kernel.org, b43-dev@lists.infradead.org Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: W dniu 11 sierpnia 2011 18:21 użytkownik Rafał Miłecki napisał: > W dniu 11 sierpnia 2011 18:04 użytkownik John W. Linville > napisał: >> On Thu, Aug 11, 2011 at 10:05:19AM +0200, Rafał Miłecki wrote: >>> W dniu 11 sierpnia 2011 06:39 użytkownik Larry Finger >>> napisał: >>> > On 08/10/2011 07:14 PM, Rafał Miłecki wrote: >>> >> >>> >> W dniu 10 sierpnia 2011 20:35 użytkownik Rafał Miłecki >>> >>  napisał: >>> >>> >>> >>> Some hardware with 64-bit DMA uses lower address word for setting >>> >>> routing (translation) bit. Add workaround for such boards. >>> >>> >>> >>> Signed-off-by: Rafał Miłecki >>> >>> --- >>> >>> This requires testing on at least 1 normal 64-bit DMA card and 1 32-bit >>> >>> DMA card. So far it was tested only on my 14e4:4329 where it fixed DMA. >>> >> >>> >> 14e4:4353 (BCM43224), 64-bit DMA, still works :) >>> >> >>> >> The rest will follow tomorrow. >>> > >>> > Tested with 14e4:4311 (64-bit) and 14e4:4318 (32-bit). The ssb changes were >>> > also tested with b43legacy. >>> >>> Thanks Larry. >>> >>> I've additionally tested this on 14e4:4312 (BCM4311), 32-bit DMA. Works OK. >>> >>> John: please take this patchset. >> >> Are these fixes intended for 3.1? > > The bug in DMA support is in the driver since the first moment we > enabled support for N-PHY. It's not any kind of regression. However, > it's still a bug fix. > > If it's OK to take that for 3.1, then go on. I believe it received > required testing. On the other hand, if it's going to make Dave or > Linus angry, leave it for 3.2. We have this bug since  2.6.39, we can > have it for one more release. s/Dave/David/, sorry -- Rafał