Return-path: Received: from mail-wg0-f44.google.com ([74.125.82.44]:38576 "EHLO mail-wg0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752044Ab2HJMZt convert rfc822-to-8bit (ORCPT ); Fri, 10 Aug 2012 08:25:49 -0400 Received: by wgbdr13 with SMTP id dr13so1438841wgb.1 for ; Fri, 10 Aug 2012 05:25:48 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <5024FD42.6070307@hauke-m.de> References: <1344544690-11280-1-git-send-email-zajec5@gmail.com> <5024CE29.5030500@broadcom.com> <5024FD42.6070307@hauke-m.de> Date: Fri, 10 Aug 2012 14:25:48 +0200 Message-ID: (sfid-20120810_142553_031354_4D1CE547) Subject: Re: [PATCH wireless?] bcma: keep info about parallel flash presentence From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= To: Hauke Mehrtens Cc: Arend van Spriel , linux-wireless@vger.kernel.org, "John W. Linville" Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: 2012/8/10 Hauke Mehrtens : > On 08/10/2012 11:32 AM, Rafał Miłecki wrote: >> 2012/8/10 Arend van Spriel : >>> On 08/09/2012 10:38 PM, Rafał Miłecki wrote: >>>> This allows drivers to access flash conditionally and stop crashing some >>>> devices. >>>> >>>> Signed-off-by: Rafał Miłecki >>>> Acked-by: Hauke Mehrtens >>>> --- >>>> John: I suggest taking it for wirelesss, AKA fix >>> >>> Maybe you could mentioned devices and/or drivers crashes without this patch? >> >> Yes, sure! The problematic one for me is arch/mips/bcm47xx/nvram.c. >> Right now it assumes every BCMA board has pflash. I'll submit fix for >> mips tree when this patch hits it. > > Letting this patches get merged into the wireless tree and an other > patches which depends on this one into the mips tree will cause > problems, because the wireless tree only gets merged into the mips tree > indirectly round about the rc1 release in form of the rc1 release done > by Linus. > > The patch to arch/mips/bcm47xx/nvram.c will depend on this one and you > should submit them to the same tree. I'm planning to wait until rc2 (or rc3 if needed). I know the dependency :) -- Rafał