On 04/08/2016 02:01 PM, Jouni Malinen wrote:
On Fri, Apr 08, 2016 at 10:28:41AM -0700, Ben Greear wrote:
Could you just accept my patch as well as your patch? They do similar
things, but cater to different API, and I am happy with using the config
file.
I don't see any point in providing two mechanisms for doing the same
thing. The only thing I'd consider is adding configuration parameters
that provide default values for the VENDOR_ELEM values. That said, I'm
not sure I really buy that use case as something that would justify the
extra complexity.
In the long term, it's easier for me if I can get patches upstream.
So, I will try to code up a patch to set the VENDOR_ELEM values using
the .conf file if you will accept the patch.
But, if you will not, then I'll just carry my v2 patch in my own
tree and deal with merge conficts as required.
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap
