lkml.org 
[lkml]   [2022]   [Jul]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 0/7] USB: cdc: add control and state defines
From
Date
> On Jul 25, 2022, at 15:58, Johan Hovold <johan@kernel.org> wrote:
>
> Several drivers use the control-line and serial-state bitmap values from
> the CDC spec, but there were no matching defines in the global CDC
> (UAPI) header.
>
> This series adds the missing defines and converts cdc-acm and f_acm to
> use them.
>
> One staging driver also had an unused set of CDC defines which are
> removed.
>
> The final patch by Yan Xinyu, which triggered this work, converts the
> usb_wwan driver to use CDC defines instead of hardcoded values in its
> line-control handling.
>
> Greg, are you ok with me taking these through USB serial (where there
> are a few more drivers that could potentially use them) or do you want
> to take the lot through your tree directly?
>
> Johan
>
>
> Johan Hovold (6):
> USB: cdc: add control-signal defines
> USB: cdc: add serial-state defines
> USB: cdc-acm: use CDC control-line defines
> USB: cdc-acm: use CDC serial-state defines
> staging: gdm724x: drop unused CDC defines
> USB: gadget: f_acm: use CDC defines
>
> Yan Xinyu (1):
> USB: serial: usb_wwan: replace DTR/RTS magic numbers with macros
>
> drivers/staging/gdm724x/gdm_tty.c | 6 -----
> drivers/usb/class/cdc-acm.c | 42 ++++++++++++++---------------
> drivers/usb/class/cdc-acm.h | 20 --------------
> drivers/usb/gadget/function/f_acm.c | 20 ++++----------
> drivers/usb/serial/usb_wwan.c | 10 ++++---
> include/uapi/linux/usb/cdc.h | 13 +++++++++
> 6 files changed, 45 insertions(+), 66 deletions(-)
>
> --
> 2.35.1
>

Thank you so much for spending your time to write this patch series!
And thank you and Greg for your reviews, they are very informative
and educational!

sdlyyxy



\
 
 \ /
  Last update: 2022-07-25 16:39    [W:0.075 / U:4.412 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site