* Re: Fw: USB: core: WARN if pipe direction != setup packet direction [not found] <HE1PR0602MB32762856D2728450104CE1C6A1219@HE1PR0602MB3276.eurprd06.prod.outlook.com> @ 2021-05-29 19:21 ` Alan Stern [not found] ` <HE1PR0602MB3276072A2816DCBD1EB7DEB4A1219@HE1PR0602MB3276.eurprd06.prod.outlook.com> 0 siblings, 1 reply; 5+ messages in thread From: Alan Stern @ 2021-05-29 19:21 UTC (permalink / raw) To: Eero; +Cc: Mauro Carvalho Chehab, linux-media, USB mailing list Once again, please always send these emails to the mailing list as well as to me. If this means sending from your gmail address instead of the hotmail address, so be it. On Sat, May 29, 2021 at 09:56:05PM +0300, Eero wrote: > Hi, > > The patch removed the warning, but the usb dvb-t stick does not work > because of looping messages in dmesg: > [ 418.485387] mxl5005s I2C write failed > [ 418.658410] mxl5005s I2C write failed > > Those messages come from: > > linux-next-next-20210528/drivers/media/tuners/mxl5005s.c > > static int mxl5005s_writereg(struct dvb_frontend *fe, u8 reg, u8 val, > int latch) { > struct mxl5005s_state *state = fe->tuner_priv; > u8 buf[3] = { reg, val, MXL5005S_LATCH_BYTE }; > struct i2c_msg msg = { .addr = state->config->i2c_address, > .flags = 0, .buf = buf, .len = 3 }; > > if (latch == 0) > msg.len = 2; > > dprintk(2, "%s(0x%x, 0x%x, 0x%x)\n", __func__, reg, val, > msg.addr); > > if (i2c_transfer(state->i2c, &msg, 1) != 1) { > printk(KERN_WARNING "mxl5005s I2C write failed\n"); > return -EREMOTEIO; > } > return 0; > } This is a completely different error. It should be reported to the maintainers of the media drivers (CC'ed). Does the same problem occur if you don't apply Johan's rtl28xxu patch and you revert commit 5cc59c418fde from 5.13.rc3-next? Alan Stern ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <HE1PR0602MB3276072A2816DCBD1EB7DEB4A1219@HE1PR0602MB3276.eurprd06.prod.outlook.com>]
* Re: USB: core: WARN if pipe direction != setup packet direction [not found] ` <HE1PR0602MB3276072A2816DCBD1EB7DEB4A1219@HE1PR0602MB3276.eurprd06.prod.outlook.com> @ 2021-05-29 19:58 ` Alan Stern 2021-05-30 10:18 ` Johan Hovold 0 siblings, 1 reply; 5+ messages in thread From: Alan Stern @ 2021-05-29 19:58 UTC (permalink / raw) To: Eero; +Cc: Johan Hovold, USB mailing list Please don't top-post. And for the third time, please CC your emails to the mailing list. If you don't, I will stop replying. On Sat, May 29, 2021 at 10:41:51PM +0300, Eero wrote: > Hi, > > The dvb-t stick works when I did that. Thank you. > > BR, Eero > > On Sat, 29 May 2021 15:21:36 -0400 > > > > Does the same problem occur if you don't apply Johan's rtl28xxu patch > > and you revert commit 5cc59c418fde from 5.13.rc3-next? > > > > Alan Stern Johan, it appears that your patch making an empty I2C read actually transfer one byte somehow causes a problem for this dvb-t stick. Below is a copy of the original problem report. Any ideas? Alan Stern The patch removed the warning, but the usb dvb-t stick does not work because of looping messages in dmesg: [ 418.485387] mxl5005s I2C write failed [ 418.658410] mxl5005s I2C write failed Those messages come from: linux-next-next-20210528/drivers/media/tuners/mxl5005s.c static int mxl5005s_writereg(struct dvb_frontend *fe, u8 reg, u8 val, int latch) { struct mxl5005s_state *state = fe->tuner_priv; u8 buf[3] = { reg, val, MXL5005S_LATCH_BYTE }; struct i2c_msg msg = { .addr = state->config->i2c_address, .flags = 0, .buf = buf, .len = 3 }; if (latch == 0) msg.len = 2; dprintk(2, "%s(0x%x, 0x%x, 0x%x)\n", __func__, reg, val, msg.addr); if (i2c_transfer(state->i2c, &msg, 1) != 1) { printk(KERN_WARNING "mxl5005s I2C write failed\n"); return -EREMOTEIO; } return 0; } BR, Eero ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: USB: core: WARN if pipe direction != setup packet direction 2021-05-29 19:58 ` Alan Stern @ 2021-05-30 10:18 ` Johan Hovold 0 siblings, 0 replies; 5+ messages in thread From: Johan Hovold @ 2021-05-30 10:18 UTC (permalink / raw) To: Alan Stern; +Cc: Eero, USB mailing list On Sat, May 29, 2021 at 03:58:31PM -0400, Alan Stern wrote: > Please don't top-post. And for the third time, please CC your emails > to the mailing list. If you don't, I will stop replying. > > On Sat, May 29, 2021 at 10:41:51PM +0300, Eero wrote: > > Hi, > > > > The dvb-t stick works when I did that. Thank you. > > > > BR, Eero > > > > On Sat, 29 May 2021 15:21:36 -0400 > > > > > > Does the same problem occur if you don't apply Johan's rtl28xxu patch > > > and you revert commit 5cc59c418fde from 5.13.rc3-next? > > > > > > Alan Stern > > Johan, it appears that your patch making an empty I2C read actually > transfer one byte somehow causes a problem for this dvb-t stick. > > Below is a copy of the original problem report. Any ideas? I'd start with verifying that the type is still detected correctly so that the driver uses the right implementation (i.e. for rtl2831u instead of rtl2832u or vice versa). If that isn't the problem, it could be the 1-byte read request itself that throws the firmware off and causes later accesses to fail even if that seems a bit far-fetched. > The patch removed the warning, but the usb dvb-t stick does not work > because of looping messages in dmesg: > [ 418.485387] mxl5005s I2C write failed > [ 418.658410] mxl5005s I2C write failed Eero, could you try the below patch on top of linux-next (i.e. without the zero-length request patch) and send us the logs? Johan From eda5deca4cbdebe21718bb13f76b8eed0673f9be Mon Sep 17 00:00:00 2001 From: Johan Hovold <johan@kernel.org> Date: Mon, 24 May 2021 10:55:19 +0200 Subject: [PATCH] media: rtl28xxu: add type-detection instrumentation --- drivers/media/usb/dvb-usb-v2/rtl28xxu.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/media/usb/dvb-usb-v2/rtl28xxu.c b/drivers/media/usb/dvb-usb-v2/rtl28xxu.c index 97ed17a141bb..21e565603108 100644 --- a/drivers/media/usb/dvb-usb-v2/rtl28xxu.c +++ b/drivers/media/usb/dvb-usb-v2/rtl28xxu.c @@ -612,8 +612,10 @@ static int rtl28xxu_read_config(struct dvb_usb_device *d) static int rtl28xxu_identify_state(struct dvb_usb_device *d, const char **name) { struct rtl28xxu_dev *dev = d_to_priv(d); + u8 buf[1]; int ret; struct rtl28xxu_req req_demod_i2c = {0x0020, CMD_I2C_DA_RD, 0, NULL}; + struct rtl28xxu_req req_demod_i2c2 = {0x0020, CMD_I2C_DA_RD, 1, buf}; dev_dbg(&d->intf->dev, "\n"); @@ -622,6 +624,11 @@ static int rtl28xxu_identify_state(struct dvb_usb_device *d, const char **name) * by old RTL2831U. */ ret = rtl28xxu_ctrl_msg(d, &req_demod_i2c); + dev_info(&d->intf->dev, "%s - ret1 = %d\n", __func__, ret); + + ret = rtl28xxu_ctrl_msg(d, &req_demod_i2c2); + dev_info(&d->intf->dev, "%s - ret2 = %d\n", __func__, ret); + if (ret == -EPIPE) { dev->chip_id = CHIP_ID_RTL2831U; } else if (ret == 0) { -- 2.31.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: USB: core: WARN if pipe direction != setup packet direction @ 2021-05-29 18:22 Eero Lehtinen 0 siblings, 0 replies; 5+ messages in thread From: Eero Lehtinen @ 2021-05-29 18:22 UTC (permalink / raw) To: linux-usb Hi, I tested the 5.13.0-rc3-next-20210528 kernel from kernel.org and there is this warning at boot. My usb devices work fine. [ 3.142879] ------------[ cut here ]------------ [ 3.142881] usb 1-1: BOGUS control dir, pipe 80000280 doesn't match bRequestType c0 [ 3.142896] WARNING: CPU: 11 PID: 1444 at drivers/usb/core/urb.c:410 usb_submit_urb+0x60e/0x650 [ 3.142904] Modules linked in: dvb_usb_rtl28xxu(+) videobuf2_vmalloc videobuf2_memops dvb_usb_v2 videobuf2_v4l2 videobuf2_common dvb_core videodev input_leds snd_seq_midi snd_seq_midi_event snd_usb_audio snd_hwdep snd_usbmidi_lib snd_rawmidi mc iptable_nat nf_nat nf_conntrack nf_defrag_ipv6 nf_defrag_ipv4 iptable_mangle iptable_filter bluetooth ecdh_generic ecc amdgpu snd_hda_codec_realtek kvm_amd snd_hda_codec_generic snd_hda_codec_hdmi kvm snd_hda_intel snd_intel_dspcfg snd_hda_codec irqbypass mfd_core iommu_v2 gpu_sched snd_hda_core snd_pcm k10temp acpi_cpufreq processor efivarfs ip_tables x_tables ipv6 efivars i2c_piix4 r8169 realtek xhci_pci nvme xhci_hcd nvme_core [ 3.142924] CPU: 11 PID: 1444 Comm: systemd-udevd Tainted: G W 5.13.0-rc3-next-20210528 #1 [ 3.142926] Hardware name: System manufacturer System Product Name/PRIME B450M-K, BIOS 3002 03/11/2021 [ 3.142927] RIP: 0010:usb_submit_urb+0x60e/0x650 [ 3.142930] Code: 89 44 24 10 89 4c 24 08 e8 2f a5 f4 ff 44 8b 44 24 10 8b 4c 24 08 48 c7 c7 a0 36 07 82 48 8b 54 24 18 48 89 c6 e8 d4 15 1d 00 <0f> 0b 8b 45 5c 25 ff fd 80 ff 45 85 f6 0f 85 36 ff ff ff 80 cc 02 [ 3.142932] RSP: 0018:ffff888104807ac0 EFLAGS: 00010282 [ 3.142935] RAX: 0000000000000047 RBX: 0000000000000000 RCX: 0000000000000000 [ 3.142936] RDX: ffff88841eade380 RSI: ffff88841ead7510 RDI: ffff88841ead7510 [ 3.142937] RBP: ffff888117043d80 R08: ffffffff82305068 R09: 0000000000009ffb [ 3.142938] R10: 00000000ffffe000 R11: 3fffffffffffffff R12: ffff888102265000 [ 3.142938] R13: 0000000000000000 R14: 0000000000000001 R15: ffff888102265058 [ 3.142939] FS: 00007f24b42f58c0(0000) GS:ffff88841eac0000(0000) knlGS:0000000000000000 [ 3.142941] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 3.142942] CR2: 000055b6d7d7c040 CR3: 0000000105fe3000 CR4: 00000000003506e0 [ 3.142943] Call Trace: [ 3.142945] usb_start_wait_urb+0x50/0xd0 [ 3.142947] usb_control_msg+0xbd/0x110 [ 3.142951] rtl28xxu_ctrl_msg+0x62/0x160 [dvb_usb_rtl28xxu] [ 3.142956] rtl28xxu_identify_state+0x32/0x70 [dvb_usb_rtl28xxu] [ 3.142960] dvb_usbv2_probe+0x10f/0x180 [dvb_usb_v2] [ 3.142963] ? pm_runtime_enable+0x16/0xb0 [ 3.142966] ? __pm_runtime_set_status+0x145/0x2a0 [ 3.142968] usb_probe_interface+0xd0/0x250 [ 3.142970] really_probe+0xe5/0x450 [ 3.142973] driver_probe_device+0x51/0xb0 [ 3.142975] device_driver_attach+0x9c/0xb0 [ 3.142976] __driver_attach+0x7c/0x110 [ 3.142978] ? device_driver_attach+0xb0/0xb0 [ 3.142979] bus_for_each_dev+0x5c/0x90 [ 3.142982] bus_add_driver+0x101/0x1b0 [ 3.142984] driver_register+0x86/0xd0 [ 3.142986] usb_register_driver+0x7f/0x120 [ 3.142987] ? 0xffffffffa0436000 [ 3.142989] do_one_initcall+0x34/0x140 [ 3.142992] ? __cond_resched+0x11/0x40 [ 3.142995] ? kmem_cache_alloc+0x35/0x150 [ 3.142999] do_init_module+0x51/0x230 [ 3.143001] __do_sys_finit_module+0x8f/0xc0 [ 3.143003] do_syscall_64+0x34/0x90 [ 3.143005] entry_SYSCALL_64_after_hwframe+0x44/0xae [ 3.143007] RIP: 0033:0x7f24b47ae9b9 [ 3.143008] Code: 00 c3 66 2e 0f 1f 84 00 00 00 00 00 0f 1f 44 00 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a7 54 0c 00 f7 d8 64 89 01 48 [ 3.143009] RSP: 002b:00007ffed1ac02e8 EFLAGS: 00000246 ORIG_RAX: 0000000000000139 [ 3.143011] RAX: ffffffffffffffda RBX: 000055b6d7e8b070 RCX: 00007f24b47ae9b9 [ 3.143012] RDX: 0000000000000000 RSI: 00007f24b4939e2d RDI: 0000000000000010 [ 3.143013] RBP: 0000000000020000 R08: 0000000000000000 R09: 000055b6d7e770a0 [ 3.143013] R10: 0000000000000010 R11: 0000000000000246 R12: 00007f24b4939e2d [ 3.143014] R13: 0000000000000000 R14: 000055b6d7e8c070 R15: 000055b6d7e8b070 [ 3.143015] ---[ end trace 74ad8cc11087d2a8 ]--- ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <HE1PR0602MB327635A49DB939C8255FB1CCA1219@HE1PR0602MB3276.eurprd06.prod.outlook.com>]
* Re: USB: core: WARN if pipe direction != setup packet direction [not found] <HE1PR0602MB327635A49DB939C8255FB1CCA1219@HE1PR0602MB3276.eurprd06.prod.outlook.com> @ 2021-05-29 14:30 ` Alan Stern 0 siblings, 0 replies; 5+ messages in thread From: Alan Stern @ 2021-05-29 14:30 UTC (permalink / raw) To: Eero; +Cc: USB mailing list Messages like this should always be sent to the linux-usb mailing list so that all the developers can see them. On Sat, May 29, 2021 at 01:15:25PM +0300, Eero wrote: > Hi, > > I tested the 5.13.0-rc3-next-20210528 kernel from kernel.org and there > is this warning at boot. My usb devices work fine. > > [ 3.142879] ------------[ cut here ]------------ > [ 3.142881] usb 1-1: BOGUS control dir, pipe 80000280 doesn't match > bRequestType c0 [ 3.142896] WARNING: CPU: 11 PID: 1444 at > drivers/usb/core/urb.c:410 usb_submit_urb+0x60e/0x650 [ 3.142904] > Modules linked in: dvb_usb_rtl28xxu(+) videobuf2_vmalloc > videobuf2_memops dvb_usb_v2 videobuf2_v4l2 videobuf2_common dvb_core > videodev input_leds snd_seq_midi snd_seq_midi_event snd_usb_audio > snd_hwdep snd_usbmidi_lib snd_rawmidi mc iptable_nat nf_nat > nf_conntrack nf_defrag_ipv6 nf_defrag_ipv4 iptable_mangle > iptable_filter bluetooth ecdh_generic ecc amdgpu snd_hda_codec_realtek > kvm_amd snd_hda_codec_generic snd_hda_codec_hdmi kvm snd_hda_intel > snd_intel_dspcfg snd_hda_codec irqbypass mfd_core iommu_v2 gpu_sched > snd_hda_core snd_pcm k10temp acpi_cpufreq processor efivarfs ip_tables > x_tables ipv6 efivars i2c_piix4 r8169 realtek xhci_pci nvme xhci_hcd > nvme_core [ 3.142924] CPU: 11 PID: 1444 Comm: systemd-udevd Tainted: > G W 5.13.0-rc3-next-20210528 #1 [ 3.142926] Hardware > name: System manufacturer System Product Name/PRIME B450M-K, BIOS 3002 > 03/11/2021 [ 3.142927] RIP: 0010:usb_submit_urb+0x60e/0x650 [ > 3.142930] Code: 89 44 24 10 89 4c 24 08 e8 2f a5 f4 ff 44 8b 44 24 10 > 8b 4c 24 08 48 c7 c7 a0 36 07 82 48 8b 54 24 18 48 89 c6 e8 d4 15 1d 00 > <0f> 0b 8b 45 5c 25 ff fd 80 ff 45 85 f6 0f 85 36 ff ff ff 80 cc 02 [ > 3.142932] RSP: 0018:ffff888104807ac0 EFLAGS: 00010282 [ 3.142935] > RAX: 0000000000000047 RBX: 0000000000000000 RCX: 0000000000000000 [ > 3.142936] RDX: ffff88841eade380 RSI: ffff88841ead7510 RDI: > ffff88841ead7510 [ 3.142937] RBP: ffff888117043d80 R08: > ffffffff82305068 R09: 0000000000009ffb [ 3.142938] R10: > 00000000ffffe000 R11: 3fffffffffffffff R12: ffff888102265000 [ > 3.142938] R13: 0000000000000000 R14: 0000000000000001 R15: > ffff888102265058 [ 3.142939] FS: 00007f24b42f58c0(0000) > GS:ffff88841eac0000(0000) knlGS:0000000000000000 [ 3.142941] CS: > 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 3.142942] CR2: > 000055b6d7d7c040 CR3: 0000000105fe3000 CR4: 00000000003506e0 [ > 3.142943] Call Trace: [ 3.142945] usb_start_wait_urb+0x50/0xd0 [ > 3.142947] usb_control_msg+0xbd/0x110 [ 3.142951] > rtl28xxu_ctrl_msg+0x62/0x160 [dvb_usb_rtl28xxu] [ 3.142956] > rtl28xxu_identify_state+0x32/0x70 [dvb_usb_rtl28xxu] [ 3.142960] > dvb_usbv2_probe+0x10f/0x180 [dvb_usb_v2] [ 3.142963] ? > pm_runtime_enable+0x16/0xb0 [ 3.142966] ? > __pm_runtime_set_status+0x145/0x2a0 [ 3.142968] > usb_probe_interface+0xd0/0x250 [ 3.142970] really_probe+0xe5/0x450 > [ 3.142973] driver_probe_device+0x51/0xb0 [ 3.142975] > device_driver_attach+0x9c/0xb0 [ 3.142976] > __driver_attach+0x7c/0x110 [ 3.142978] ? > device_driver_attach+0xb0/0xb0 [ 3.142979] > bus_for_each_dev+0x5c/0x90 [ 3.142982] bus_add_driver+0x101/0x1b0 [ > 3.142984] driver_register+0x86/0xd0 [ 3.142986] > usb_register_driver+0x7f/0x120 [ 3.142987] ? 0xffffffffa0436000 [ > 3.142989] do_one_initcall+0x34/0x140 [ 3.142992] ? > __cond_resched+0x11/0x40 [ 3.142995] ? kmem_cache_alloc+0x35/0x150 > [ 3.142999] do_init_module+0x51/0x230 [ 3.143001] > __do_sys_finit_module+0x8f/0xc0 [ 3.143003] do_syscall_64+0x34/0x90 > [ 3.143005] entry_SYSCALL_64_after_hwframe+0x44/0xae [ 3.143007] > RIP: 0033:0x7f24b47ae9b9 [ 3.143008] Code: 00 c3 66 2e 0f 1f 84 00 > 00 00 00 00 0f 1f 44 00 00 48 89 f8 48 89 f7 48 89 d6 48 89 ca 4d 89 c2 > 4d 89 c8 4c 8b 4c 24 08 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a7 > 54 0c 00 f7 d8 64 89 01 48 [ 3.143009] RSP: 002b:00007ffed1ac02e8 > EFLAGS: 00000246 ORIG_RAX: 0000000000000139 [ 3.143011] RAX: > ffffffffffffffda RBX: 000055b6d7e8b070 RCX: 00007f24b47ae9b9 [ > 3.143012] RDX: 0000000000000000 RSI: 00007f24b4939e2d RDI: > 0000000000000010 [ 3.143013] RBP: 0000000000020000 R08: > 0000000000000000 R09: 000055b6d7e770a0 [ 3.143013] R10: > 0000000000000010 R11: 0000000000000246 R12: 00007f24b4939e2d [ > 3.143014] R13: 0000000000000000 R14: 000055b6d7e8c070 R15: > 000055b6d7e8b070 [ 3.143015] ---[ end trace 74ad8cc11087d2a8 ]--- This should be fixed by: https://lore.kernel.org/r/20210524110920.24599-4-johan@kernel.org Alan Stern ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-05-30 10:18 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <HE1PR0602MB32762856D2728450104CE1C6A1219@HE1PR0602MB3276.eurprd06.prod.outlook.com>
2021-05-29 19:21 ` Fw: USB: core: WARN if pipe direction != setup packet direction Alan Stern
[not found] ` <HE1PR0602MB3276072A2816DCBD1EB7DEB4A1219@HE1PR0602MB3276.eurprd06.prod.outlook.com>
2021-05-29 19:58 ` Alan Stern
2021-05-30 10:18 ` Johan Hovold
2021-05-29 18:22 Eero Lehtinen
[not found] <HE1PR0602MB327635A49DB939C8255FB1CCA1219@HE1PR0602MB3276.eurprd06.prod.outlook.com>
2021-05-29 14:30 ` 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).