From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Chen Subject: Re: [PATCH V2] leds: trigger: Introduce an USB port trigger Date: Mon, 18 Jul 2016 13:40:10 +0800 Message-ID: <20160718054010.GE31293@shlinux2> References: <1468239883-22695-1-git-send-email-zajec5@gmail.com> <1468617060-29830-1-git-send-email-zajec5@gmail.com> <20160718023126.GB31293@shlinux2> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= Cc: Richard Purdie , Jacek Anaszewski , Felipe Balbi , "linux-usb@vger.kernel.org" , Rob Herring , Mark Rutland , Jonathan Corbet , Sakari Ailus , Ezequiel Garcia , Boris Brezillon , Pavel Machek , Geert Uytterhoeven , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , "open list:DOCUMENTATION" , "open list:LED SUBSYSTEM" List-Id: devicetree@vger.kernel.org On Mon, Jul 18, 2016 at 06:44:49AM +0200, Rafa=C5=82 Mi=C5=82ecki wrote= : > On 18 July 2016 at 04:31, Peter Chen wrote: > > On Fri, Jul 15, 2016 at 11:10:45PM +0200, Rafa=C5=82 Mi=C5=82ecki w= rote: > >> + > >> +usbport trigger: > >> +- usb-ports : List of USB ports that usbport should observed for = turning on a > >> + given LED. > >> + > > > > %s/should/should be >=20 > Thanks. >=20 >=20 > >> diff --git a/drivers/leds/trigger/ledtrig-usbport.c b/drivers/leds= /trigger/ledtrig-usbport.c > >> new file mode 100644 > >> index 0000000..97b064c > >> --- /dev/null > >> +++ b/drivers/leds/trigger/ledtrig-usbport.c > >> @@ -0,0 +1,206 @@ > >> +/* > >> + * USB port LED trigger > >> + * > >> + * Copyright (C) 2016 Rafa=C5=82 Mi=C5=82ecki > >> + * > >> + * This program is free software; you can redistribute it and/or = modify > >> + * it under the terms of the GNU General Public License as publis= hed by > >> + * the Free Software Foundation; either version 2 of the License,= or (at > >> + * your option) any later version. > >> + */ > > > > GPL v2 only. > > > >> +MODULE_AUTHOR("Rafa=C5=82 Mi=C5=82ecki "= ); > >> +MODULE_DESCRIPTION("USB port trigger"); > >> +MODULE_LICENSE("GPL"); > > > > GPL v2 >=20 > What's the reason for this? I don't have any real preference, but I > never heard heard about kernel/Linux preference neither. >=20 https://en.wikipedia.org/wiki/Linux_kernel --=20 Best Regards, Peter Chen