qmi_wwan: Add quirk for Quectel dynamic config
authorKristian Evensen <kristian.evensen@gmail.com>
Sun, 7 Apr 2019 13:39:09 +0000 (15:39 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 8 Apr 2019 23:25:56 +0000 (16:25 -0700)
commite4bf63482c309287ca84d91770ffa7dcc18e37eb
treefc8a39f41c20dc14a9ce1eb813fd677a7f8c93a0
parent22b56e827093dd1af2379d294aba7d629b3f3436
qmi_wwan: Add quirk for Quectel dynamic config

Most, if not all, Quectel devices use dynamic interface numbers, and
users are able to change the USB configuration at will. Matching on for
example interface number is therefore not possible.

Instead, the QMI device can be identified by looking at the interface
class, subclass and protocol (all 0xff), as well as the number of
endpoints. The reason we need to look at the number of endpoints, is
that the diagnostic port interface has the same class, subclass and
protocol as QMI. However, the diagnostic port only has two endpoints,
while QMI has three.

Until now, we have identified the QMI device by combining a match on
class, subclass and protocol, with a call to the function
quectel_diag_detect(). In quectel_diag_detect(), we check if the number
of endpoints matches for known Quectel vendor/product ids.

Adding new vendor/product ids to quectel_diag_detect() is not a good
long-term solution. This commit replaces the function with a quirk, and
applies the quirk to affected Quectel devices that I have been able to
test the change with (EP06, EM12 and EC25). If the quirk is set and the
number of endpoints equal two, we return from qmi_wwan_probe() with
-ENODEV.

Signed-off-by: Kristian Evensen <kristian.evensen@gmail.com>
Acked-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/qmi_wwan.c