Search Linux Wireless

RE: [PATCH v3] Add new mac80211 driver mwlwifi.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 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




[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux