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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html