Amitkumar Karwar <akarwar@xxxxxxxxxxx> writes: >> From: Kalle Valo [mailto:kvalo@xxxxxxxxxxxxxx] >> Sent: Tuesday, May 24, 2016 1:53 AM >> To: Amitkumar Karwar >> Cc: linux-wireless@xxxxxxxxxxxxxxx >> Subject: Re: [PATCH] mwifiex: handle edmac vendor command >> >> Amitkumar Karwar <akarwar@xxxxxxxxxxx> writes: >> >> > Hi Kalle, >> > >> >> From: Amitkumar Karwar [mailto:akarwar@xxxxxxxxxxx] >> >> Sent: Friday, April 29, 2016 9:28 PM >> >> To: linux-wireless@xxxxxxxxxxxxxxx >> >> Cc: Jeff CF Chen; Amitkumar Karwar >> >> Subject: [PATCH] mwifiex: handle edmac vendor command >> >> >> >> From: chunfan chen <jeffc@xxxxxxxxxxx> >> >> >> >> Userspace can configure edmac values through a custom vendor command. >> >> They will be used by hardware for adaptivity. >> >> >> >> Signed-off-by: chunfan chen <jeffc@xxxxxxxxxxx> >> >> Signed-off-by: Amitkumar Karwar <akarwar@xxxxxxxxxxx> >> >> [deleted over 200 lines of unnecessary quotes] >> >> > This patch seems to have deferred. We basically want a way to download >> > a vendor specific configuration to our firmware. Do you have any >> > suggestions on how can achieve this in better way? >> > >> > I can see below iw command suits our requirement. >> > >> > iw dev <devname> vendor send <oui> <subcmd> <filename|-|hex data> >> > >> > Please guide. >> >> It was deferred because use of the nl80211 vendor interface (which I >> don't think belong to upstream drivers). I'll take a look at this patch >> in detail after the merge window. >> > > Did you get a chance to check this patch? > Please let me know if you have any suggestions. Sorry for the delay. I have been thinking about vendor commands quite a lot and I don't think they belong to upstream drivers. For regular use cases (used by normal users) we have nl80211, for developer and testing purposes we have debugfs and for manufacturing type of tests we have nl80211 testmode interface. The focus of development should be adding new functionality to nl80211 (and other generic interfaces), not to driver specific vendor commands. I know brcm80211 and ti's wlcore have few vendor commands but I'm hoping to remove them sometime soon. Thoughts? -- Kalle Valo
