On 11/14/14 14:29, Kalle Valo wrote:
Vladimir Kondratiev<qca_vkondrat@xxxxxxxxxxxxxxxx> writes:
Introduce netdev IOCTLs, to be used by the debug tools.
Allows to read/write single dword value or
memory block, aligned to dword
Different address modes supported:
- BAR offset
- Firmware "linker" address
- target's AHB bus
Signed-off-by: Vladimir Kondratiev<qca_vkondrat@xxxxxxxxxxxxxxxx>
An ioctl interface for a wireless driver? IMHO that would have been ok
in 2004, but not in 2014. Isn't there really better way to implement
this?
If this is really driver/device specific debug tool stuff, you should
probably consider the vendor commands through nl80211 API.
Regards,
Arend
+/* Numbers SIOCDEVPRIVATE and SIOCDEVPRIVATE + 1
+ * are used by Android devices to implement PNO (preferred network offload).
+ * Albeit it is temporary solution, use different numbers to avoid conflicts
+ */
Comments like this make me even more worried that that this is just yet
another way to implement wext iwpriv interface.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html