> Johannes Berg wrote: > > On Fri, 2015-06-26 at 01:25 +0000, David Lin wrote: > > > Both the drivers are operable for the same chip part number 8897 and > > its modules (boards) manufactured out there. The PCI Device ID is a > > fixed value (by chip/manufacturer) on those modules. As said earlier, > > the two drivers are for mac80211 host mac, and firmware mac > > respectively. They serve different purposes. Users need to choose > > which driver they want to use for the same wifi hardware, and they > > cannot use both of them at the same time. The mwifiex is more widely > > used now and sta centric. We do not want to break it. > > > > I am not sure but I believe they are also multiple flavors of driver > > for some other chips. Any advice how they are handled, considering > > that changing the hardware/board is not an option? I guess we want the > > community to be better served using the same hardware. Opinion from > > others are welcomed. > > Hmm, interesting, I didn't realize it was the same device. > > Perhaps it simply shouldn't conflict at build time, and you can recommend to > blacklist one or the other? Or, in fact, return an error from probe if you can't > find the correct firmware file, so the other driver would be attempted? > Yes, current setting will only allow user to choose one of these two drivers at build time. > > > Although "-D__CHECK_ENDIAN__" does not put into Makefile, I had used > > "C=2 CF="-D__CHECK_ENDIAN__" to check warnings for endian and remove > > these warnings. > > Ok. Then I guess my issues were mostly due to 64-bit system. > I will try to build the driver under 64-bit system. > > > If you think we need to add this to Makefile and make sure zero > > warning messages, we will do that (only for sparser). > > Well it just helps not forget adding CF=-D... to the command line, it's > equivalent. I just prefer it in a sense since then if people run with > C=1 or C=2 they get that part for free, and since the driver is clean to start > with then that's a good thing to do I think. > Got it. > > I'll try to take a closer look later. > > Johannes > David, Thanks ��.n��������+%������w��{.n�����{���zW����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f