linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Support Castles Vega5000 PoS terminal USB
@ 2020-02-22 23:32 Ondrej Jirman
  2020-02-23  3:08 ` Lars Melin
  0 siblings, 1 reply; 3+ messages in thread
From: Ondrej Jirman @ 2020-02-22 23:32 UTC (permalink / raw)
  To: linux-usb; +Cc: Ondrej Jirman, Oliver Neukum, Greg Kroah-Hartman, open list

This terminal's USB port needs NO_UNION_NORMAL quirk to work with
cdc-acm driver.

Signed-off-by: Ondrej Jirman <megous@megous.com>
---
 drivers/usb/class/cdc-acm.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
index 62f4fb9b362f1..8577441c81a4d 100644
--- a/drivers/usb/class/cdc-acm.c
+++ b/drivers/usb/class/cdc-acm.c
@@ -1739,6 +1739,9 @@ static const struct usb_device_id acm_ids[] = {
 	{ USB_DEVICE(0x22b8, 0x2d9a),   /* modem + AT port + diagnostics + NMEA */
 	.driver_info = NO_UNION_NORMAL, /* handle only modem interface          */
 	},
+	{ USB_DEVICE(0x0ca6, 0xa050), /* Castles Technology VEGA 5000 */
+	.driver_info = NO_UNION_NORMAL, /* reports zero length descriptor */
+	},
 
 	{ USB_DEVICE(0x0572, 0x1329), /* Hummingbird huc56s (Conexant) */
 	.driver_info = NO_UNION_NORMAL, /* union descriptor misplaced on
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Support Castles Vega5000 PoS terminal USB
  2020-02-22 23:32 [PATCH] Support Castles Vega5000 PoS terminal USB Ondrej Jirman
@ 2020-02-23  3:08 ` Lars Melin
  2020-02-23 10:20   ` Ondřej Jirman
  0 siblings, 1 reply; 3+ messages in thread
From: Lars Melin @ 2020-02-23  3:08 UTC (permalink / raw)
  To: Ondrej Jirman, linux-usb; +Cc: Oliver Neukum, Greg Kroah-Hartman, open list

On 2/23/2020 06:32, Ondrej Jirman wrote:
> This terminal's USB port needs NO_UNION_NORMAL quirk to work with
> cdc-acm driver.
> 
> Signed-off-by: Ondrej Jirman <megous@megous.com>
> ---
>   drivers/usb/class/cdc-acm.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
> index 62f4fb9b362f1..8577441c81a4d 100644
> --- a/drivers/usb/class/cdc-acm.c
> +++ b/drivers/usb/class/cdc-acm.c
> @@ -1739,6 +1739,9 @@ static const struct usb_device_id acm_ids[] = {
>   	{ USB_DEVICE(0x22b8, 0x2d9a),   /* modem + AT port + diagnostics + NMEA */
>   	.driver_info = NO_UNION_NORMAL, /* handle only modem interface          */
>   	},
> +	{ USB_DEVICE(0x0ca6, 0xa050), /* Castles Technology VEGA 5000 */
> +	.driver_info = NO_UNION_NORMAL, /* reports zero length descriptor */
> +	},
>   
>   	{ USB_DEVICE(0x0572, 0x1329), /* Hummingbird huc56s (Conexant) */
>   	.driver_info = NO_UNION_NORMAL, /* union de

This quirk is in the driver since almost a year ago.

br
/Lars


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Support Castles Vega5000 PoS terminal USB
  2020-02-23  3:08 ` Lars Melin
@ 2020-02-23 10:20   ` Ondřej Jirman
  0 siblings, 0 replies; 3+ messages in thread
From: Ondřej Jirman @ 2020-02-23 10:20 UTC (permalink / raw)
  To: Lars Melin; +Cc: linux-usb, Oliver Neukum, Greg Kroah-Hartman, open list

Hello,

On Sun, Feb 23, 2020 at 10:08:04AM +0700, Lars Melin wrote:
> On 2/23/2020 06:32, Ondrej Jirman wrote:
> > This terminal's USB port needs NO_UNION_NORMAL quirk to work with
> > cdc-acm driver.
> > 
> > Signed-off-by: Ondrej Jirman <megous@megous.com>
> > ---
> >   drivers/usb/class/cdc-acm.c | 3 +++
> >   1 file changed, 3 insertions(+)
> > 
> > diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c
> > index 62f4fb9b362f1..8577441c81a4d 100644
> > --- a/drivers/usb/class/cdc-acm.c
> > +++ b/drivers/usb/class/cdc-acm.c
> > @@ -1739,6 +1739,9 @@ static const struct usb_device_id acm_ids[] = {
> >   	{ USB_DEVICE(0x22b8, 0x2d9a),   /* modem + AT port + diagnostics + NMEA */
> >   	.driver_info = NO_UNION_NORMAL, /* handle only modem interface          */
> >   	},
> > +	{ USB_DEVICE(0x0ca6, 0xa050), /* Castles Technology VEGA 5000 */
> > +	.driver_info = NO_UNION_NORMAL, /* reports zero length descriptor */
> > +	},
> >   	{ USB_DEVICE(0x0572, 0x1329), /* Hummingbird huc56s (Conexant) */
> >   	.driver_info = NO_UNION_NORMAL, /* union de
> 
> This quirk is in the driver since almost a year ago.

You're right. Thank you.

I held to the patch for way too long, and didn't notice someone else patched
it in the meantime.

regards,
	o.

> br
> /Lars
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-02-23 10:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-22 23:32 [PATCH] Support Castles Vega5000 PoS terminal USB Ondrej Jirman
2020-02-23  3:08 ` Lars Melin
2020-02-23 10:20   ` Ondřej Jirman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).