Return-path: Received: from nbd.name ([46.4.11.11]:39914 "EHLO nbd.name" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751406Ab0KLT4K (ORCPT ); Fri, 12 Nov 2010 14:56:10 -0500 Message-ID: <4CDD9BB8.1040305@openwrt.org> Date: Fri, 12 Nov 2010 20:55:36 +0100 From: Felix Fietkau MIME-Version: 1.0 To: Wojciech Dubowik CC: Bob Copeland , linville@tuxdriver.com, linux-wireless@vger.kernel.org Subject: Re: [PATCH 8/9] ath5k: AHB port. Add AHB bus support. References: <1298013311.4723.1289591505420.JavaMail.root@idefix> In-Reply-To: <1298013311.4723.1289591505420.JavaMail.root@idefix> Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On 2010-11-12 8:51 PM, Wojciech Dubowik wrote: >> From: "Felix Fietkau" >> To: "Bob Copeland" >> Cc: "Wojciech Dubowik" , linville@tuxdriver.com, linux-wireless@vger.kernel.org >> Sent: Freitag, 12. November 2010 20:36:51 >> Subject: Re: [PATCH 8/9] ath5k: AHB port. Add AHB bus support. >> On 2010-11-12 8:26 PM, Bob Copeland wrote: >> > On Thu, Nov 11, 2010 at 9:58 AM, Wojciech Dubowik >> > wrote: >> >> >> >> AHB specific functions are now in ahb.c file. There is no common >> >> init functions which tries first pci and then ahb becasue on some >> > >> > 'because' >> > >> >> --- a/drivers/net/wireless/ath/ath5k/Makefile >> >> +++ b/drivers/net/wireless/ath/ath5k/Makefile >> >> @@ -15,5 +15,5 @@ ath5k-y += rfkill.o >> >> ath5k-y += ani.o >> >> ath5k-y += sysfs.o >> >> ath5k-$(CONFIG_ATH5K_DEBUG) += debug.o >> >> -ath5k-y += pci.o >> >> +ath5k-y += $(if $(CONFIG_ATHEROS_AR231X),ahb,pci).o >> >> obj-$(CONFIG_ATH5K) += ath5k.o >> > >> > This is not the right way.. what if someone wants both in one >> > kernel? >> Having both in the kernel is completely pointless. There are no >> devices >> that can support both, nor will there ever be. >> >> - Felix > > I agree. The only advantage would be that some of the typos and missing > functions in either of the files would be detected during compile time. > My AR5312 board crashes when I enable PCI. I guess it's fixable but it's > out of the scope of the ath5k driver... Well, AR5312 does not have PCI. Only AR2315 has it, and there is only one device that even uses it (Atheros never officially supported it). - Felix