Re: [PATCH v4] USB: serial: option: Add missing RSVD(5) flag for Rolling RW135R-GL

[Johan Hovold]

On Fri, May 15, 2026 at 11:31:05AM +0800, zwq2226404116@xxxxxxx wrote:
> From: Wanquan Zhong <wanquan.zhong@xxxxxxxxxxx>
> 
> The RW135R-GL MBIM interface entry was missing the .driver_info = RSVD(5) flag.
> 
> Without this flag, the option driver binds to the MBIM interface
> and conflicts with the cdc_mbim driver, causing AT/MBIM communication
> failures. This matches the handling of other Rolling Wireless MBIM devices.

Please reply to this mail and explain why you think reserving interface
number 5 (i.e. the ADB interface) prevents the driver to binding to the
non-vendor MBIM interfaces number 0 and 1.

Don't send any new versions until you've explained this.
 
> This patch adds the missing RSVD(5) quirk for Rolling RW135R-GL (33f8:1003)
> to prevent the option driver from probing the MBIM control interface,
> which avoids conflicts with cdc_mbim and fixes communication failures.
> 
> This aligns with the configuration used by all other Rolling Wireless MBIM modems.
> 
> v4:
> - Fixes tag corrected

No, it's still using the wrong summary (i.e. the text inside
parenthesis). I already sent you the right one in an earlier reply, but
you can also look it up using git log.

> - Device table entry formatting aligned with existing pattern

Move the changelog after the --- line.

You also removed the usb-devices output from the commit message. Please
add it back.
 
> Tested successfully on multiple interface configurations:
> - mbim + diag + AT + pipe
> - mbim + diag + AT + ADB + pipe
> - mbim + pipe
> 
> Fixes: 01e8d0f74222 ("usb: serial: add support for Rolling Wireless RW135R-GL (33f8:1003)")
> Signed-off-by: Wanquan Zhong <wanquan.zhong@xxxxxxxxxxx>
> ---
>  drivers/usb/serial/option.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Johan