On Sat, Mar 17, 2018 at 11:07:27AM +1100, Benjamin Herrenschmidt wrote:
On Sat, 2018-03-17 at 04:29 +0800, kbuild test robot wrote:
Hi Benjamin,
I love your patch! Perhaps something to improve:
[auto build test WARNING on v4.16-rc4]
[if your patch is applied to the wrong git tree, please drop
On Fri, Jan 26, 2018 at 10:58:56AM +1100, Benjamin Herrenschmidt wrote:
On Wed, 2018-01-24 at 11:30 +0800, kbuild test robot wrote:
Hi Benjamin,
I love your patch! Perhaps something to improve:
[auto build test WARNING on balbi-usb/next]
[also build test WARNING on v4.15-rc9 next-20180119]
[if
kbuild just spit out the following warning for this patch:
drivers/usb/host/max3421-hcd.c:1707:2-8: preceding lock on line 1702
Error ids grouped by kconfigs:
recent_errors
└── i386-allmodconfig
└── drivers-usb-host-max3421-hcd.c:preceding-lock-on-line
I don't really know what it means, a
Hi Josh,
On Mon, Oct 02, 2017 at 04:31:09PM -0500, Josh Poimboeuf wrote:
On Mon, Oct 02, 2017 at 04:26:54PM -0500, Josh Poimboeuf wrote:
Fengguang, assuming it's reliably recreatable, any chance you could
recreate with the following patch?
Sorry, here's a version which actually compiles.
dif
Hi Josh,
On Mon, Oct 02, 2017 at 04:31:09PM -0500, Josh Poimboeuf wrote:
On Mon, Oct 02, 2017 at 04:26:54PM -0500, Josh Poimboeuf wrote:
Fengguang, assuming it's reliably recreatable, any chance you could
recreate with the following patch?
Sure, I'll try your patch on v4.14-rc3 since it looks
The dummy_udc WARN is gone with this patch for 100+ times of boot tests.
Tested-by: Xiaolong Ye
Thanks,
Xiaolong
Thanks, Xiaolong. I will submit a patch for it.
Thank you Peter! We've been noised by this warning for quite some time.
Fengguang
--
To unsubscribe from this list: send the lin
[Sorry, resend to correct Felipe's email address]
Greetings,
This debug patch possibly discloses some USB/I2C bugs. Since the USB
warning shows up earlier in dmesg, it might also be the root cause of
the I2C bug. The attached reproduce-* script may help debug them.
https://git.kernel.org/pub/scm
Greetings,
This debug patch possibly discloses some USB/I2C bugs. Since the USB
warning shows up earlier in dmesg, it might also be the root cause of
the I2C bug. The attached reproduce-* script may help debug them.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
commit
On Mon, Sep 12, 2016 at 11:36:20AM +0800, Peter Chen wrote:
On Mon, Sep 12, 2016 at 07:16:07AM +0800, kbuild test robot wrote:
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-next
head: ab57f86198d6ff20371613d4a02fd4841972a5c0
commit: ad764c49f65ac171e493e6baf39bc8ba
Hi Peter,
On Mon, Sep 12, 2016 at 11:36:20AM +0800, Peter Chen wrote:
On Mon, Sep 12, 2016 at 07:16:07AM +0800, kbuild test robot wrote:
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-next
head: ab57f86198d6ff20371613d4a02fd4841972a5c0
commit: ad764c49f65ac171e493e
>
> sizeof when applied to a pointer typed expression gives the size of the
> pointer
>
> Generated by: scripts/coccinelle/misc/noderef.cocci
>
> CC: Masaki Ota
> Signed-off-by: Fengguang Wu
> ---
>
> hid-alps.c |5 +++--
> 1 file changed, 3 inse
drivers/usb/gadget/function/f_hid.c:852:21: sparse: symbol 'hidg_alloc' was not
declared. Should it be static?
Signed-off-by: Fengguang Wu
---
f_hid.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/gadget/function/f_hid.c
b/drivers/usb/gadge
drivers/usb/gadget/udc/r8a66597-udc.c:1849:1-3: WARNING: PTR_ERR_OR_ZERO can be
used
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
Generated by: scripts/coccinelle/api/ptr_ret.cocci
Signed-off-by: Fengguang Wu
---
r8a66597-udc.c |5 +
1 file changed, 1 insertion(+), 4
Hi Dan,
Just in case it helps, here we find a different bug message in the
same commit.
git://git.kernel.org/pub/scm/linux/kernel/git/djbw/usb.git td-fragments-v1
commit e65e21a542cab81d794db4e5fe919c4e1d624ea7
Author: Dan Williams
AuthorDate: Tue Jul 22 00:08:51 2014 -0700
Commit: Dan
Greetings,
These *huge number* of *long standing* errors make me wonder whether
the below USB driver code should be disabled on s390?
tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux master
head: 9064171268d838b8f283fe111ef086b9479d059a
commit: dced35aeb0367dda2636ee9ee914bd
On Tue, Mar 05, 2013 at 05:57:34PM +0800, Fengguang Wu wrote:
> On Mon, Mar 04, 2013 at 10:28:09PM +0100, Sebastian Andrzej Siewior wrote:
> > Fengguang Wu run into a kernel ops after he complied dummy_hcd and g_cdc
> > into the kernel. The problem was that u_serial was used
On Mon, Mar 04, 2013 at 10:28:09PM +0100, Sebastian Andrzej Siewior wrote:
> Fengguang Wu run into a kernel ops after he complied dummy_hcd and g_cdc
> into the kernel. The problem was that u_serial was used by g_cdc before
> u_serial was initialized. In the module case eveything is initi
On Sat, Mar 02, 2013 at 02:41:07PM +0100, Sebastian Andrzej Siewior wrote:
> On 03/02/2013 02:37 PM, Fengguang Wu wrote:
> > On Wed, Feb 27, 2013 at 12:49:35PM +0200, Felipe Balbi wrote:
> >> Hi,
> >>
> >> On Wed, Feb 06, 2013 at 10:04:24AM +0800, Fengguang Wu w
On Wed, Feb 27, 2013 at 12:49:35PM +0200, Felipe Balbi wrote:
> Hi,
>
> On Wed, Feb 06, 2013 at 10:04:24AM +0800, Fengguang Wu wrote:
> > Greetings,
> >
> > I got the below warning and the first bad commit is
>
> can you send a fixup patch ?
Sorry I don'
Signed-off-by: Fengguang Wu
---
drivers/usb/gadget/zero.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-next.orig/drivers/usb/gadget/zero.c 2013-02-03 09:50:18.426689812
+0800
+++ linux-next/drivers/usb/gadget/zero.c2013-02-03 09:53:35.162694484
+0800
@@ -63,7
Signed-off-by: Fengguang Wu
---
drivers/usb/gadget/u_serial.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-next.orig/drivers/usb/gadget/u_serial.c 2013-02-03
09:50:18.422689811 +0800
+++ linux-next/drivers/usb/gadget/u_serial.c2013-02-03 09:50:27.570690026
On Mon, Jan 21, 2013 at 10:13:43AM +0200, Felipe Balbi wrote:
> On Sun, Jan 20, 2013 at 05:24:05PM -0800, Chao Xie wrote:
[snip]
> > I am formatting and re-test the patches. I will send out today.
> > Thanks.
>
>
> great, please make sure to compile test on x86 and ARM with
> allyesconfig, allnoc
Signed-off-by: Fengguang Wu
---
drivers/usb/misc/usb3503.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- linux-next.orig/drivers/usb/misc/usb3503.c 2013-01-12 13:12:34.367933402
+0800
+++ linux-next/drivers/usb/misc/usb3503.c 2013-01-12 13:12:37.251933471
+0800
@@ -178,7
Dongjin,
> Do I need to send another patch on top of it to fix the warning or
Yes, you'll need to send another incremental fix to Greg.
Thanks,
Fengguang
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo i
Dongjin Kim 2012-12-08 239
6a099c63 Dongjin Kim 2012-12-08 240dev_info(&i2c->dev, "%s: probed on %s
mode\n", __func__,
6a099c63 Dongjin Kim 2012-12-08 241(hub->mode ==
USB3503_MODE_HUB) ? "hub" : "standby");
---
0-DAY kernel build
--*/
4cf2503c Yoshihiro Shimoda 2007-05-10 @1748 static struct
platform_driver m66592_driver = {
598f22e1 Yoshihiro Shimoda 2007-07-17 1749 .remove =
__exit_p(m66592_remove),
4cf2503c Yoshihiro Shimoda 2007-05-10 1750
ree(filter_mask);
> >bbd9f9ee Steve Glendinning 2012-10-26 1137
> >check_warn_return(ret, "Error writing WUFF");
> >bbd9f9ee Steve Glendinning 2012-10-26 1138 }
> >06a221be Ming Lei 2012-11-06 1139
> >k
On Fri, Nov 02, 2012 at 03:13:54PM -0400, Alan Stern wrote:
> On Sat, 3 Nov 2012, Fengguang Wu wrote:
>
> > tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git
> > usb-next
> > head: bc8d51ea7e8ae0abb90fa89407b55a7e0bcb0a2a
> > commit: 09f6ffde2e
7 Benjamin Herrenschmidt 2006-12-15 679 }
8cd42e97 Kumar Gala 2006-01-20 680
---
0-DAY kernel build testing backend Open Source Technology Center
Fengguang Wu, Yuanhan Liu Intel Corporation
--
To unsubscribe from this list: send the line
On Fri, Nov 02, 2012 at 11:13:56AM +0200, Felipe Balbi wrote:
> Hi,
>
> On Fri, Nov 02, 2012 at 02:05:52AM +0800, kbuild test robot wrote:
> > tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git
> > usb-next
> > head: 99f91934a907df31ba878dfdd090002049dc476a
> > commit: 3e02320
Hi Alan,
> It's hard to know which headers are needed for foreign architectures.
> Still, I think this ought to fix the problems you found. Can you test
> this patch?
Yes, it works. Thanks for the quick fix!
Thanks,
Fengguang
> Index: usb-3.7/drivers/usb/host/ehci-platform.c
> ==
njamin Herrenschmidt 2006-12-28 676 #else
68f50e52 Al Viro2007-02-09 677 writel(val, regs);
d728e327 Benjamin Herrenschmidt 2006-12-28 678 #endif
---
0-DAY kernel build testing backend Open Source Technology Center
Fengguang Wu, Yuanhan Liu
On Thu, Nov 01, 2012 at 04:47:14PM +0300, Max Filippov wrote:
> On Thu, Nov 1, 2012 at 5:23 PM, Felipe Balbi wrote:
> > Hi,
> >
> > On Thu, Nov 01, 2012 at 06:04:20AM -0700, Greg KH wrote:
> >> On Thu, Nov 01, 2012 at 12:50:34PM +0200, Felipe Balbi wrote:
> >> > On Thu, Nov 01, 2012 at 12:33:39PM
On Wed, Oct 24, 2012 at 04:37:20PM +0300, Felipe Balbi wrote:
> Hi,
>
> On Wed, Oct 24, 2012 at 09:23:23PM +0800, Fengguang Wu wrote:
> > Sorry, this should really be CCed to the media list.
> > I'll use the list recommended by get_maintainer.pl in future.
>
>
On Thu, Sep 27, 2012 at 10:10:26AM -0700, Greg KH wrote:
> On Thu, Sep 27, 2012 at 02:24:13PM +0800, Fengguang Wu wrote:
> > Signed-off-by: Fengguang Wu
> > ---
> > drivers/usb/misc/ezusb.c | 16
> > 1 file changed, 8 insertions(+), 8 deleti
Signed-off-by: Fengguang Wu
---
drivers/usb/misc/ezusb.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-next
head: 70c048a238c780c226eb4b115ebaa908cb3b34ec
commit
if (be16_to_cpu(devinfo->response.tag) != tag) {
668 shost_printk(KERN_INFO, shost,
669 "%s: %s failed (wrong tag %d/%d)\n", __func__,
670 fname, be16_to_cpu(devinfo->response.tag),
tag);
671 return FAILED
= serial_drivers; *sd; ++sd) {
1414 if ((*sd)->reset_resume)
> 1415 udriver->reset_resume = usb_serial_reset_resume;
> 1416 break;
1417 }
---
0-DAY kernel build testing backend Open Source Technology Centre
ty = tty_port_tty_get(&port->port);
175 if (tty) {
---
0-DAY kernel build testing backend Open Source Technology Centre
Fengguang Wu, Yuanhan Liu Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-us
Open Source Technology Centre
Fengguang Wu Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
spin_unlock(&ci->lock);
1687
1688 return retval;
1689 }
1690
1691 /**
1692 * udc_release: driver release function
1693 * @dev: device
---
0-DAY kernel build testing backend Open Source Technology Centre
Fengguang Wu I
On Wed, Sep 12, 2012 at 03:02:13PM +0300, Felipe Balbi wrote:
> Hi,
>
> On Wed, Sep 12, 2012 at 11:50:57AM +0300, Dan Carpenter wrote:
> > On Wed, Sep 12, 2012 at 04:26:22PM +0800, Fengguang Wu wrote:
> > > Hi Felipe,
> > >
> > > FYI, there are new sm
3 return port_is_power_on(hub, portstatus);
2664 }
---
0-DAY kernel build testing backend Open Source Technology Centre
Fengguang Wu Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body o
On Mon, Sep 10, 2012 at 07:05:15PM +0200, Sebastian Andrzej Siewior wrote:
> On 09/10/2012 06:40 PM, Fengguang Wu wrote:
> >tree: git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git master
> >head: d9c88901337158c9f253a7de58a10b5125d
escriptor otg_descriptor = {
96 .bLength = sizeof otg_descriptor,
97 .bDescriptorType = USB_DT_OTG,
98
99 /* REVISIT SRP-only hardware is possible, although
---
0-DAY kernel build testing backend Open Source Technology Centre
Fen
h to balbi's tree
> for fixing this.
> Also added you as a reviewer in the to list of the patch.
> Plz verify and let me know if you still see any issues.
It should be fine if you already verified the patch with allmodconfig
build test.
When a commit tagged with
Reported-by: Feng
On Fri, Aug 17, 2012 at 05:27:08PM +0800, Fengguang Wu wrote:
> Hi Bjørn,
>
> > -static const struct usb_device_id id_table[] __devinitconst = {
> > +static const struct usb_device_id id_table[] = {
>
> Good catch! It magically fixed the oops. So that id_table w
Hi Bjørn,
> -static const struct usb_device_id id_table[] __devinitconst = {
> +static const struct usb_device_id id_table[] = {
Good catch! It magically fixed the oops. So that id_table was
freed sometime with __devinitconst?
There are some more "usb_device_id .* __devinitconst" users in the sy
On Sun, Aug 05, 2012 at 09:58:26AM -0700, Greg KH wrote:
> On Sun, Aug 05, 2012 at 10:59:38AM +0800, Fengguang Wu wrote:
> > Hi all,
> >
> > This line triggers an oops in kvm boot test:
> >
> > usb_match_id():
> > ==> 748 for (; id->idVen
49 matches
Mail list logo