mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Nicolai Buchwitz <nb@tipi-net.de>
To: guido@kiener-muenchen.de
Cc: Michal Pecio <michal.pecio@gmail.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Steve Bayless <steve_bayless@keysight.com>,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
	stable@vger.kernel.org
Subject: Re: [PATCH 0/4] usb: usbtmc: fix undersized high-speed bulk packets on xHCI
Date: Wed, 21 Oct 2026 09:46:03 +0200	[thread overview]
Message-ID: <bcb7b29468175cd13f08bdd363b8c733@tipi-net.de> (raw)
In-Reply-To: <d702d2b3b66d6229855515c2eed09636@kiener-muenchen.de>

Hi Guido,

On 20.10.2026 23:12, guido@kiener-muenchen.de wrote:
> [...] Therefore every manufacturer of USBTMC devices should
> comply with the specification and cannot expect bugs in device firmware
> to be fixed by patches to the USBTMC driver on Windows, Linux, or 
> macOS.
> [...] Otherwise I cannot support adding device‑specific patches to a
> class driver.

I agree, it would be a better world if every vendor stuck to the specs.
Unfortunately reality is often different, and we are the ones stuck
with the devices on the production line.

Anyway, I get your point about keeping quirks out of the driver. It
looks like the maxpacket part can be fixed in xHCI instead, since ehci
and dwc2 already accept HS bulk endpoints below 512 bytes (see the
discussion with Michal). The ZLP issue of the Rigols can be worked
around in the application. So I'll drop this series.

Thanks,
Nicolai

  reply	other threads:[~2026-10-21  7:46 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-20 14:33 Nicolai Buchwitz
2026-10-20 14:33 ` [PATCH 1/4] usb: usbtmc: handle undersized high-speed bulk IN packets Nicolai Buchwitz
2026-10-20 14:33 ` [PATCH 2/4] usb: usbtmc: handle undersized high-speed bulk OUT packets Nicolai Buchwitz
2026-10-20 14:33 ` [PATCH 3/4] usb: usbtmc: add Rigol DP800 and DS1000Z to the packet I/O quirk table Nicolai Buchwitz
2026-10-20 14:33 ` [PATCH 4/4] usb: usbtmc: send a ZLP after a full last packet in packet I/O mode Nicolai Buchwitz
2026-10-20 16:42 ` [PATCH 0/4] usb: usbtmc: fix undersized high-speed bulk packets on xHCI Michal Pecio
2026-10-20 18:33   ` Nicolai Buchwitz
2026-10-20 21:12     ` guido
2026-10-21  7:46       ` Nicolai Buchwitz [this message]
2026-10-21 12:29         ` Guido Kiener
2026-10-20 22:56     ` Michal Pecio
2026-10-21  7:42       ` Nicolai Buchwitz
2026-10-21  9:34       ` Nicolai Buchwitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bcb7b29468175cd13f08bdd363b8c733@tipi-net.de \
    --to=nb@tipi-net.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=guido@kiener-muenchen.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=michal.pecio@gmail.com \
    --cc=stable@vger.kernel.org \
    --cc=steve_bayless@keysight.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®