Re: [PATCH v3 2/2] usb: Add LED triggers for USB activity

From: Felipe Balbi
Date: Wed Aug 27 2014 - 15:30:50 EST


On Wed, Aug 27, 2014 at 12:27:51PM -0700, Greg Kroah-Hartman wrote:
> On Wed, Aug 27, 2014 at 03:03:45PM +0200, Michal Sojka wrote:
> > With this patch, USB activity can be signaled by blinking a LED. There
> > are two triggers, one for activity on USB host and one for USB gadget.
> >
> > Both trigger should work with all host/device controllers. Tested only
> > with musb.
> >
> > Signed-off-by: Michal Sojka <sojka@xxxxxxxxx>
> > ---
> > drivers/usb/Kconfig | 11 ++++++
> > drivers/usb/common/Makefile | 5 ++-
> > drivers/usb/common/{usb-common.c => common.c} | 0
>
> I don't mind renaming files, but don't "hide" it in this patch, do it in
> a separate patch please, and then base your next patch on this one.

+1

yeah, this prevents us from reviewing the actual changes

--
balbi

Attachment: signature.asc
Description: Digital signature