linux-kernel - Re: [PATCH] USB: serial: option: add Fibocom FM135-GL variants

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <160fdf7f-553a-40ed-8493-95a7d31a8b38@gmail.com>
Date: Wed, 6 Mar 2024 13:39:54 +0700
From: Lars Melin <larsm17@...il.com>
To: "Bolan Wang(Bolan)" <bolan.wang@...ocom.com>,
 "johan@...nel.org" <johan@...nel.org>,
 "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
Cc: "linux-usb@...r.kernel.org" <linux-usb@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] USB: serial: option: add Fibocom FM135-GL variants

On 2024-03-06 13:18, Bolan Wang(Bolan) wrote:
>> If the device with pid 0x01a1 only has an mbim interface as you have indicated then why do you add it to the option serial driver?
> 
> Hi Lars:
>    The pid 0x01a1 may include at, diag ports in different usb modes. Currently, only has the mbim interface.
> 
> thanks
> bolan

Hi Bolan,
so you already know that other versions of the card will have the pid 
0x01a1 and the serial interfaces will be of class ff but you don't
know what those interfaces will be used for?
You shall only add driver support for what you know today and not for 
something that might or might not be implemented in the future.

For the device with pid 0x0115 you have listed an adb interface and adb 
interfaces should not be in the option driver.

thanks
Lars