From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754887AbbCELkc (ORCPT ); Thu, 5 Mar 2015 06:40:32 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:37049 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750965AbbCELka convert rfc822-to-8bit (ORCPT ); Thu, 5 Mar 2015 06:40:30 -0500 From: Kalle Valo To: =?utf-8?Q?Rafa=C5=82_Mi=C5=82ecki?= Cc: Chen Gang , "linux-wireless\@vger.kernel.org" , "linux-kernel\@vger.kernel.org" , Hauke Mehrtens , Arnd Bergmann , Jim Davis , Michael =?utf-8?Q?B=C3=BCsch?= Subject: Re: [PATCH] bcma: Kconfig: Let it depend on PCI References: Date: Thu, 05 Mar 2015 13:40:23 +0200 In-Reply-To: (=?utf-8?Q?=22Rafa=C5=82_Mi=C5=82ecki=22's?= message of "Wed, 4 Mar 2015 23:23:32 +0100") Message-ID: <87r3t365aw.fsf@kamboji.qca.qualcomm.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Rafał Miłecki writes: > Kalle, I guess the recent discussion and work on this problem could be > confusing, so let me try to sum it up it a bit. > > First of all, please note there are 3 awaiting bcma patches that > should be applied independently of what we decide to do with this > patch. They are of course inspired by the recent building problems. > List of these patches: > [PATCH next] bcma: make bcma_host_pci_(up|down) calls safe for every config > [PATCH next] bcma: move internal function declarations to private header > [PATCH next] bcma: prepare Kconfig symbol for PCI driver > > Now, the building problem is obviously caused by me, my work on > driver_pcie2.c and using pcie_set_readrq there without making sure > that is PCI available. I'm sorry for that. > > All 3 above patches are moving us toward the the most optimal solution > of this problem. Depending on PCI only when it's really required. > There is still one more change missing that I'm working on. It'll take > me about 2 more days to get the last patch. > On the other way, patch proposed by Chen fixes building problem right > now. It's much simpler but bumps bcma requirements a bit too high. > bcma doesn't really have to depend on PCI. Thanks for the good summary, it clarified things for me. > So you have 2 options there and I'll be happy with whatever you choose to do: > 1) Pick Chen patch now and in ~2 days apply my final fix + revert Chen > patch. I'll prefer this option so that we can quickly solve the build problem. Also once your final fix is ready, please remember that you need to revert Chen's patch within the same patch. -- Kalle Valo