* [PATCH 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera
@ 2025-08-06 13:47 Mael GUERIN
2025-08-06 14:52 ` Alan Stern
2025-08-06 16:44 ` [PATCH v2 " Mael GUERIN
0 siblings, 2 replies; 4+ messages in thread
From: Mael GUERIN @ 2025-08-06 13:47 UTC (permalink / raw)
To: linux-usb, usb-storage, linux-kernel
Cc: Alan Stern, Greg Kroah-Hartman, Mael GUERIN
Signed-off-by: Mael GUERIN <mael.guerin@murena.io>
---
drivers/usb/storage/unusual_devs.h | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
index 54f0b1c83..7bb5a1d75 100644
--- a/drivers/usb/storage/unusual_devs.h
+++ b/drivers/usb/storage/unusual_devs.h
@@ -262,6 +262,13 @@ UNUSUAL_DEV( 0x0421, 0x06c2, 0x0000, 0x0406,
USB_SC_DEVICE, USB_PR_DEVICE, NULL,
US_FL_MAX_SECTORS_64 ),
+/* Added by Maël GUERIN <mael.guerin@murena.io> */
+UNUSUAL_DEV( 0x0603, 0x8611, 0x0000, 0xffff,
+ "Novatek",
+ "NTK96550-based camera",
+ USB_SC_SCSI, USB_PR_BULK, NULL,
+ US_FL_BULK_IGNORE_TAG ),
+
#ifdef NO_SDDR09
UNUSUAL_DEV( 0x0436, 0x0005, 0x0100, 0x0100,
"Microtech",
--
2.50.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera
2025-08-06 13:47 [PATCH 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera Mael GUERIN
@ 2025-08-06 14:52 ` Alan Stern
2025-08-06 16:44 ` [PATCH v2 " Mael GUERIN
1 sibling, 0 replies; 4+ messages in thread
From: Alan Stern @ 2025-08-06 14:52 UTC (permalink / raw)
To: Mael GUERIN; +Cc: linux-usb, usb-storage, linux-kernel, Greg Kroah-Hartman
On Wed, Aug 06, 2025 at 03:47:21PM +0200, Mael GUERIN wrote:
> Signed-off-by: Mael GUERIN <mael.guerin@murena.io>
> ---
Patches must have a nonempty description. You can simply take the text
of your 0/1 message and put it directly in the 1/1 email as the
description. Then there won't be any need for a separate 0/1 message.
> drivers/usb/storage/unusual_devs.h | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
> index 54f0b1c83..7bb5a1d75 100644
> --- a/drivers/usb/storage/unusual_devs.h
> +++ b/drivers/usb/storage/unusual_devs.h
> @@ -262,6 +262,13 @@ UNUSUAL_DEV( 0x0421, 0x06c2, 0x0000, 0x0406,
> USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> US_FL_MAX_SECTORS_64 ),
>
> +/* Added by Maël GUERIN <mael.guerin@murena.io> */
> +UNUSUAL_DEV( 0x0603, 0x8611, 0x0000, 0xffff,
> + "Novatek",
> + "NTK96550-based camera",
> + USB_SC_SCSI, USB_PR_BULK, NULL,
> + US_FL_BULK_IGNORE_TAG ),
See the instructions at the start of this file. Entries are supposed to
be sorted by Vendor ID and then Product ID.
Alan Stern
> +
> #ifdef NO_SDDR09
> UNUSUAL_DEV( 0x0436, 0x0005, 0x0100, 0x0100,
> "Microtech",
> --
> 2.50.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera
2025-08-06 13:47 [PATCH 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera Mael GUERIN
2025-08-06 14:52 ` Alan Stern
@ 2025-08-06 16:44 ` Mael GUERIN
2025-08-06 18:18 ` Alan Stern
1 sibling, 1 reply; 4+ messages in thread
From: Mael GUERIN @ 2025-08-06 16:44 UTC (permalink / raw)
To: linux-usb, usb-storage, linux-kernel
Cc: Alan Stern, Greg Kroah-Hartman, Mael GUERIN
Add the US_FL_BULK_IGNORE_TAG quirk for Novatek NTK96550-based camera
to fix USB resets after sending SCSI vendor commands due to CBW and
CSW tags difference, leading to undesired slowness while communicating
with the device.
Please find below the copy of /sys/kernel/debug/usb/devices with my
device plugged in (listed as TechSys USB mass storage here, the
underlying chipset being the Novatek NTK96550-based camera):
T: Bus=03 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#= 3 Spd=480 MxCh= 0
D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1
P: Vendor=0603 ProdID=8611 Rev= 0.01
S: Manufacturer=TechSys
S: Product=USB Mass Storage
S: SerialNumber=966110000000100
C:* #Ifs= 1 Cfg#= 1 Atr=c0 MxPwr=100mA
I:* If#= 0 Alt= 0 #EPs= 2 Cls=08(stor.) Sub=06 Prot=50 Driver=usb-storage
E: Ad=81(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
E: Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
Signed-off-by: Mael GUERIN <mael.guerin@murena.io>
---
Thanks for your review and your advice. Here's the updated version of the
patch with a correct description as well as the unusual_devs.h file sorted
correctly.
drivers/usb/storage/unusual_devs.h | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
index 54f0b1c83..bee9f1e80 100644
--- a/drivers/usb/storage/unusual_devs.h
+++ b/drivers/usb/storage/unusual_devs.h
@@ -934,6 +934,13 @@ UNUSUAL_DEV( 0x05e3, 0x0723, 0x9451, 0x9451,
USB_SC_DEVICE, USB_PR_DEVICE, NULL,
US_FL_SANE_SENSE ),
+/* Added by Maël GUERIN <mael.guerin@murena.io> */
+UNUSUAL_DEV( 0x0603, 0x8611, 0x0000, 0xffff,
+ "Novatek",
+ "NTK96550-based camera",
+ USB_SC_SCSI, USB_PR_BULK, NULL,
+ US_FL_BULK_IGNORE_TAG ),
+
/*
* Reported by Hanno Boeck <hanno@gmx.de>
* Taken from the Lycoris Kernel
--
2.50.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera
2025-08-06 16:44 ` [PATCH v2 " Mael GUERIN
@ 2025-08-06 18:18 ` Alan Stern
0 siblings, 0 replies; 4+ messages in thread
From: Alan Stern @ 2025-08-06 18:18 UTC (permalink / raw)
To: Mael GUERIN; +Cc: linux-usb, usb-storage, linux-kernel, Greg Kroah-Hartman
The Subject line really ought to say "USB: storage: Add ..." but that's
a very minor matter; Greg can adjust it if he wants when the patch is
applied.
Apart from that...
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Alan Stern
On Wed, Aug 06, 2025 at 06:44:03PM +0200, 'Mael GUERIN' via USB Mass Storage on Linux wrote:
> Add the US_FL_BULK_IGNORE_TAG quirk for Novatek NTK96550-based camera
> to fix USB resets after sending SCSI vendor commands due to CBW and
> CSW tags difference, leading to undesired slowness while communicating
> with the device.
>
> Please find below the copy of /sys/kernel/debug/usb/devices with my
> device plugged in (listed as TechSys USB mass storage here, the
> underlying chipset being the Novatek NTK96550-based camera):
>
> T: Bus=03 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#= 3 Spd=480 MxCh= 0
> D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1
> P: Vendor=0603 ProdID=8611 Rev= 0.01
> S: Manufacturer=TechSys
> S: Product=USB Mass Storage
> S: SerialNumber=966110000000100
> C:* #Ifs= 1 Cfg#= 1 Atr=c0 MxPwr=100mA
> I:* If#= 0 Alt= 0 #EPs= 2 Cls=08(stor.) Sub=06 Prot=50 Driver=usb-storage
> E: Ad=81(I) Atr=02(Bulk) MxPS= 512 Ivl=0ms
> E: Ad=02(O) Atr=02(Bulk) MxPS= 512 Ivl=0ms
>
> Signed-off-by: Mael GUERIN <mael.guerin@murena.io>
> ---
> Thanks for your review and your advice. Here's the updated version of the
> patch with a correct description as well as the unusual_devs.h file sorted
> correctly.
>
> drivers/usb/storage/unusual_devs.h | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/usb/storage/unusual_devs.h b/drivers/usb/storage/unusual_devs.h
> index 54f0b1c83..bee9f1e80 100644
> --- a/drivers/usb/storage/unusual_devs.h
> +++ b/drivers/usb/storage/unusual_devs.h
> @@ -934,6 +934,13 @@ UNUSUAL_DEV( 0x05e3, 0x0723, 0x9451, 0x9451,
> USB_SC_DEVICE, USB_PR_DEVICE, NULL,
> US_FL_SANE_SENSE ),
>
> +/* Added by Maël GUERIN <mael.guerin@murena.io> */
> +UNUSUAL_DEV( 0x0603, 0x8611, 0x0000, 0xffff,
> + "Novatek",
> + "NTK96550-based camera",
> + USB_SC_SCSI, USB_PR_BULK, NULL,
> + US_FL_BULK_IGNORE_TAG ),
> +
> /*
> * Reported by Hanno Boeck <hanno@gmx.de>
> * Taken from the Lycoris Kernel
> --
> 2.50.1
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-08-06 18:19 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-06 13:47 [PATCH 1/1] usb-storage: Add unusual-devs entry for Novatek NTK96550-based camera Mael GUERIN
2025-08-06 14:52 ` Alan Stern
2025-08-06 16:44 ` [PATCH v2 " Mael GUERIN
2025-08-06 18:18 ` Alan Stern
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).