On 8/9/2017 02:33, Bjørn Mork wrote:


The qmi_wwan part looks fine to me. But you
will need to split it in two patches since the two
drivers are parts of different subsystems.

The option driver use interface blacklists
instead of multiple match entries. You should
probably follow the same style there. But this
is up to Johan...

Use the get_maintainer script to figure out
where the patches should be sent.


Bjørn


Whitelisting all 6 interfaces in the option driver and
then blacklist 4 of them (3 qmi + 1 Android debug) is
not efficient when you so easily can selectively whitelist
the ones (0 and 2) that the option driver should handle.
Giuseppe is doing it the right way imho.


/<Lars
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to