* [PATCH] ipw2200: Enable LED by default @ 2010-06-16 0:55 Leann Ogasawara 2010-06-16 1:20 ` Luis R. Rodriguez 2010-06-17 8:51 ` [PATCH] " TJ 0 siblings, 2 replies; 7+ messages in thread From: Leann Ogasawara @ 2010-06-16 0:55 UTC (permalink / raw) To: reinette.chatre, ilw; +Cc: linux-wireless, TJ Hi All, As documented in 2005 in Documentation/networking/README.ipw2200, "The LED code has been reported to hang some systems when running ifconfig and is therefore disabled by default." We've however been carrying the following patch in our Ubuntu kernel for quite some time which enables the ipw2200 LED by default. This was a result of numerous user requests. We've seen no subsequent bug reports of systems hanging due to the the LED code being enabled by default. I'd therefore like to propose the following patch to enable the LED by default. This patch was originally authored by TJ. I apologize in advance that I do not have TJ's full first and last name for provenance. Thanks, Leann >From 315246037a0edab4d626de6ccb68c73d3fe61ce3 Mon Sep 17 00:00:00 2001 From: ubuntu@tjworld.net <ubuntu@tjworld.net> Date: Mon, 23 Mar 2009 20:29:28 +0000 Subject: [PATCH] ipw2200: Enable LED by default BugLink: http://bugs.launchpad.net/bugs/21367 Enable LED by default and update the MODULE_PARM_DESC. The original reason for defaulting to disabled was documented in 2005 and noted, "The LED code has been reported to hang some systems when running ifconfig and is therefore disabled by default." This no longer appears applicable and users have been requesting this be enabled for several years. Originally-by: TJ <ubuntu@tjworld.net> Signed-off-by: Tim Gardner <tim.gardner@canonical.com> Signed-off-by: Andy Whitcroft <apw@canonical.com> Acked-by: Stefan Bader <stefan.bader@canonical.com> Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com> --- Documentation/networking/README.ipw2200 | 2 +- drivers/net/wireless/ipw2x00/ipw2200.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200 index 80c7285..e4d3267 100644 --- a/Documentation/networking/README.ipw2200 +++ b/Documentation/networking/README.ipw2200 @@ -171,7 +171,7 @@ Where the supported parameter are: led Can be used to turn on experimental LED code. - 0 = Off, 1 = On. Default is 0. + 0 = Off, 1 = On. Default is 1. mode Can be used to set the default mode of the adapter. diff --git a/drivers/net/wireless/ipw2x00/ipw2200.c b/drivers/net/wireless/ipw2x00/ipw2200.c index 3aa3bb1..0805569 100644 --- a/drivers/net/wireless/ipw2x00/ipw2200.c +++ b/drivers/net/wireless/ipw2x00/ipw2200.c @@ -96,7 +96,7 @@ static int network_mode = 0; static u32 ipw_debug_level; static int associate; static int auto_create = 1; -static int led_support = 0; +static int led_support = 1; static int disable = 0; static int bt_coexist = 0; static int hwcrypto = 0; @@ -12083,7 +12083,7 @@ module_param(auto_create, int, 0444); MODULE_PARM_DESC(auto_create, "auto create adhoc network (default on)"); module_param_named(led, led_support, int, 0444); -MODULE_PARM_DESC(led, "enable led control on some systems (default 0 off)"); +MODULE_PARM_DESC(led, "enable led control on some systems (default 1 on)"); module_param(debug, int, 0444); MODULE_PARM_DESC(debug, "debug output mask"); -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ipw2200: Enable LED by default 2010-06-16 0:55 [PATCH] ipw2200: Enable LED by default Leann Ogasawara @ 2010-06-16 1:20 ` Luis R. Rodriguez 2010-06-16 2:41 ` Leann Ogasawara 2010-06-17 8:51 ` [PATCH] " TJ 1 sibling, 1 reply; 7+ messages in thread From: Luis R. Rodriguez @ 2010-06-16 1:20 UTC (permalink / raw) To: Leann Ogasawara; +Cc: reinette.chatre, ilw, linux-wireless, TJ, BrianK.Lee On Tue, Jun 15, 2010 at 5:55 PM, Leann Ogasawara <leann.ogasawara@canonical.com> wrote: > Hi All, > > As documented in 2005 in Documentation/networking/README.ipw2200, "The > LED code has been reported to hang some systems when running ifconfig > and is therefore disabled by default." We've however been carrying the > following patch in our Ubuntu kernel for quite some time which enables > the ipw2200 LED by default. This was a result of numerous user > requests. We've seen no subsequent bug reports of systems hanging due > to the the LED code being enabled by default. I'd therefore like to > propose the following patch to enable the LED by default. This patch > was originally authored by TJ. I apologize in advance that I do not > have TJ's full first and last name for provenance. > > Thanks, > Leann > > >From 315246037a0edab4d626de6ccb68c73d3fe61ce3 Mon Sep 17 00:00:00 2001 > From: ubuntu@tjworld.net <ubuntu@tjworld.net> > Date: Mon, 23 Mar 2009 20:29:28 +0000 > Subject: [PATCH] ipw2200: Enable LED by default > > BugLink: http://bugs.launchpad.net/bugs/21367 > > Enable LED by default and update the MODULE_PARM_DESC. The original > reason for defaulting to disabled was documented in 2005 and noted, "The > LED code has been reported to hang some systems when running ifconfig > and is therefore disabled by default." This no longer appears > applicable and users have been requesting this be enabled for several > years. > > Originally-by: TJ <ubuntu@tjworld.net> Does Originally-by also imply he has read the Certificate of Origin and that he agrees for his code to be used in that way? That is the purpose of the SOB anyway, but if you took his code, does that imply that he agrees to it too? This is from Documentation/SubmittingPatches Developer's Certificate of Origin 1.1 By making a contribution to this project, I certify that: (a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or (b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or (c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it. (d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved. Luis > Signed-off-by: Tim Gardner <tim.gardner@canonical.com> > Signed-off-by: Andy Whitcroft <apw@canonical.com> > Acked-by: Stefan Bader <stefan.bader@canonical.com> > Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com> > --- > Documentation/networking/README.ipw2200 | 2 +- > drivers/net/wireless/ipw2x00/ipw2200.c | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200 > index 80c7285..e4d3267 100644 > --- a/Documentation/networking/README.ipw2200 > +++ b/Documentation/networking/README.ipw2200 > @@ -171,7 +171,7 @@ Where the supported parameter are: > > led > Can be used to turn on experimental LED code. > - 0 = Off, 1 = On. Default is 0. > + 0 = Off, 1 = On. Default is 1. > > mode > Can be used to set the default mode of the adapter. > diff --git a/drivers/net/wireless/ipw2x00/ipw2200.c b/drivers/net/wireless/ipw2x00/ipw2200.c > index 3aa3bb1..0805569 100644 > --- a/drivers/net/wireless/ipw2x00/ipw2200.c > +++ b/drivers/net/wireless/ipw2x00/ipw2200.c > @@ -96,7 +96,7 @@ static int network_mode = 0; > static u32 ipw_debug_level; > static int associate; > static int auto_create = 1; > -static int led_support = 0; > +static int led_support = 1; > static int disable = 0; > static int bt_coexist = 0; > static int hwcrypto = 0; > @@ -12083,7 +12083,7 @@ module_param(auto_create, int, 0444); > MODULE_PARM_DESC(auto_create, "auto create adhoc network (default on)"); > > module_param_named(led, led_support, int, 0444); > -MODULE_PARM_DESC(led, "enable led control on some systems (default 0 off)"); > +MODULE_PARM_DESC(led, "enable led control on some systems (default 1 on)"); > > module_param(debug, int, 0444); > MODULE_PARM_DESC(debug, "debug output mask"); > -- > 1.7.0.4 > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ipw2200: Enable LED by default 2010-06-16 1:20 ` Luis R. Rodriguez @ 2010-06-16 2:41 ` Leann Ogasawara 2010-06-16 4:36 ` Luis R. Rodriguez 0 siblings, 1 reply; 7+ messages in thread From: Leann Ogasawara @ 2010-06-16 2:41 UTC (permalink / raw) To: Luis R. Rodriguez; +Cc: reinette.chatre, ilw, linux-wireless, TJ, BrianK.Lee On Tue, 2010-06-15 at 18:20 -0700, Luis R. Rodriguez wrote: > On Tue, Jun 15, 2010 at 5:55 PM, Leann Ogasawara > <leann.ogasawara@canonical.com> wrote: > > Hi All, > > > > As documented in 2005 in Documentation/networking/README.ipw2200, "The > > LED code has been reported to hang some systems when running ifconfig > > and is therefore disabled by default." We've however been carrying the > > following patch in our Ubuntu kernel for quite some time which enables > > the ipw2200 LED by default. This was a result of numerous user > > requests. We've seen no subsequent bug reports of systems hanging due > > to the the LED code being enabled by default. I'd therefore like to > > propose the following patch to enable the LED by default. This patch > > was originally authored by TJ. I apologize in advance that I do not > > have TJ's full first and last name for provenance. > > > > Thanks, > > Leann > > > > >From 315246037a0edab4d626de6ccb68c73d3fe61ce3 Mon Sep 17 00:00:00 2001 > > From: ubuntu@tjworld.net <ubuntu@tjworld.net> > > Date: Mon, 23 Mar 2009 20:29:28 +0000 > > Subject: [PATCH] ipw2200: Enable LED by default > > > > BugLink: http://bugs.launchpad.net/bugs/21367 > > > > Enable LED by default and update the MODULE_PARM_DESC. The original > > reason for defaulting to disabled was documented in 2005 and noted, "The > > LED code has been reported to hang some systems when running ifconfig > > and is therefore disabled by default." This no longer appears > > applicable and users have been requesting this be enabled for several > > years. > > > > Originally-by: TJ <ubuntu@tjworld.net> > > Does Originally-by also imply he has read the Certificate of Origin > and that he agrees for his code to be used in that way? That is the > purpose of the SOB anyway, but if you took his code, does that imply > that he agrees to it too? I'd CC'd TJ on the initial email, so I'm hoping he will respond here to your inquiry. I can say that he had submitted the original form of the patch to the Ubuntu kernel-team mailing list back in March, 2009: https://lists.ubuntu.com/archives/kernel-team/2009-March/005013.html In that original submission he did provide his official SOB. The patch was then applied with slight modification by Tim Gardner to the Ubuntu kernel. In forwarding this upstream, I wanted to give TJ the proper credit for originating the patch even though it's undergone some slight modification (ie the MODULE_PARM_DESC change). TJ had not officially signed off on this final form of the patch which is the reason I used "Originally-by:". If there is a better approach please let me know. Thanks, Leann > This is from Documentation/SubmittingPatches > > Developer's Certificate of Origin 1.1 > > By making a contribution to this project, I certify that: > > (a) The contribution was created in whole or in part by me and I > have the right to submit it under the open source license > indicated in the file; or > > (b) The contribution is based upon previous work that, to the best > of my knowledge, is covered under an appropriate open source > license and I have the right under that license to submit that > work with modifications, whether created in whole or in part > by me, under the same open source license (unless I am > permitted to submit under a different license), as indicated > in the file; or > > (c) The contribution was provided directly to me by some other > person who certified (a), (b) or (c) and I have not modified > it. > > (d) I understand and agree that this project and the contribution > are public and that a record of the contribution (including all > personal information I submit with it, including my sign-off) is > maintained indefinitely and may be redistributed consistent with > this project or the open source license(s) involved. > > > Luis > > > Signed-off-by: Tim Gardner <tim.gardner@canonical.com> > > Signed-off-by: Andy Whitcroft <apw@canonical.com> > > Acked-by: Stefan Bader <stefan.bader@canonical.com> > > Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com> > > --- > > Documentation/networking/README.ipw2200 | 2 +- > > drivers/net/wireless/ipw2x00/ipw2200.c | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200 > > index 80c7285..e4d3267 100644 > > --- a/Documentation/networking/README.ipw2200 > > +++ b/Documentation/networking/README.ipw2200 > > @@ -171,7 +171,7 @@ Where the supported parameter are: > > > > led > > Can be used to turn on experimental LED code. > > - 0 = Off, 1 = On. Default is 0. > > + 0 = Off, 1 = On. Default is 1. > > > > mode > > Can be used to set the default mode of the adapter. > > diff --git a/drivers/net/wireless/ipw2x00/ipw2200.c b/drivers/net/wireless/ipw2x00/ipw2200.c > > index 3aa3bb1..0805569 100644 > > --- a/drivers/net/wireless/ipw2x00/ipw2200.c > > +++ b/drivers/net/wireless/ipw2x00/ipw2200.c > > @@ -96,7 +96,7 @@ static int network_mode = 0; > > static u32 ipw_debug_level; > > static int associate; > > static int auto_create = 1; > > -static int led_support = 0; > > +static int led_support = 1; > > static int disable = 0; > > static int bt_coexist = 0; > > static int hwcrypto = 0; > > @@ -12083,7 +12083,7 @@ module_param(auto_create, int, 0444); > > MODULE_PARM_DESC(auto_create, "auto create adhoc network (default on)"); > > > > module_param_named(led, led_support, int, 0444); > > -MODULE_PARM_DESC(led, "enable led control on some systems (default 0 off)"); > > +MODULE_PARM_DESC(led, "enable led control on some systems (default 1 on)"); > > > > module_param(debug, int, 0444); > > MODULE_PARM_DESC(debug, "debug output mask"); > > -- > > 1.7.0.4 > > > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ipw2200: Enable LED by default 2010-06-16 2:41 ` Leann Ogasawara @ 2010-06-16 4:36 ` Luis R. Rodriguez 2010-06-16 15:57 ` [PATCH v2] " Leann Ogasawara 0 siblings, 1 reply; 7+ messages in thread From: Luis R. Rodriguez @ 2010-06-16 4:36 UTC (permalink / raw) To: Leann Ogasawara; +Cc: reinette.chatre, ilw, linux-wireless, TJ, BrianK.Lee On Tue, Jun 15, 2010 at 7:41 PM, Leann Ogasawara <leann.ogasawara@canonical.com> wrote: > On Tue, 2010-06-15 at 18:20 -0700, Luis R. Rodriguez wrote: >> On Tue, Jun 15, 2010 at 5:55 PM, Leann Ogasawara >> <leann.ogasawara@canonical.com> wrote: >> > Hi All, >> > >> > As documented in 2005 in Documentation/networking/README.ipw2200, "The >> > LED code has been reported to hang some systems when running ifconfig >> > and is therefore disabled by default." We've however been carrying the >> > following patch in our Ubuntu kernel for quite some time which enables >> > the ipw2200 LED by default. This was a result of numerous user >> > requests. We've seen no subsequent bug reports of systems hanging due >> > to the the LED code being enabled by default. I'd therefore like to >> > propose the following patch to enable the LED by default. This patch >> > was originally authored by TJ. I apologize in advance that I do not >> > have TJ's full first and last name for provenance. >> > >> > Thanks, >> > Leann >> > >> > >From 315246037a0edab4d626de6ccb68c73d3fe61ce3 Mon Sep 17 00:00:00 2001 >> > From: ubuntu@tjworld.net <ubuntu@tjworld.net> >> > Date: Mon, 23 Mar 2009 20:29:28 +0000 >> > Subject: [PATCH] ipw2200: Enable LED by default >> > >> > BugLink: http://bugs.launchpad.net/bugs/21367 >> > >> > Enable LED by default and update the MODULE_PARM_DESC. The original >> > reason for defaulting to disabled was documented in 2005 and noted, "The >> > LED code has been reported to hang some systems when running ifconfig >> > and is therefore disabled by default." This no longer appears >> > applicable and users have been requesting this be enabled for several >> > years. >> > >> > Originally-by: TJ <ubuntu@tjworld.net> >> >> Does Originally-by also imply he has read the Certificate of Origin >> and that he agrees for his code to be used in that way? That is the >> purpose of the SOB anyway, but if you took his code, does that imply >> that he agrees to it too? > > I'd CC'd TJ on the initial email, so I'm hoping he will respond here to > your inquiry. I can say that he had submitted the original form of the > patch to the Ubuntu kernel-team mailing list back in March, 2009: > > https://lists.ubuntu.com/archives/kernel-team/2009-March/005013.html > > In that original submission he did provide his official SOB. > > The patch was then applied with slight modification by Tim Gardner to > the Ubuntu kernel. In forwarding this upstream, I wanted to give TJ the > proper credit for originating the patch even though it's undergone some > slight modification (ie the MODULE_PARM_DESC change). TJ had not > officially signed off on this final form of the patch which is the > reason I used "Originally-by:". If there is a better approach please > let me know. In that case I would just recommend to keep his original SOB, then it would go through you/tim whoever it goes through until it reaches Linus. But just my $0.02. Luis ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] ipw2200: Enable LED by default 2010-06-16 4:36 ` Luis R. Rodriguez @ 2010-06-16 15:57 ` Leann Ogasawara 0 siblings, 0 replies; 7+ messages in thread From: Leann Ogasawara @ 2010-06-16 15:57 UTC (permalink / raw) To: Luis R. Rodriguez; +Cc: reinette.chatre, ilw, linux-wireless, TJ, BrianK.Lee On Tue, 2010-06-15 at 21:36 -0700, Luis R. Rodriguez wrote: > On Tue, Jun 15, 2010 at 7:41 PM, Leann Ogasawara > <leann.ogasawara@canonical.com> wrote: > > On Tue, 2010-06-15 at 18:20 -0700, Luis R. Rodriguez wrote: > >> On Tue, Jun 15, 2010 at 5:55 PM, Leann Ogasawara > >> <leann.ogasawara@canonical.com> wrote: > >> > Hi All, > >> > > >> > As documented in 2005 in Documentation/networking/README.ipw2200, "The > >> > LED code has been reported to hang some systems when running ifconfig > >> > and is therefore disabled by default." We've however been carrying the > >> > following patch in our Ubuntu kernel for quite some time which enables > >> > the ipw2200 LED by default. This was a result of numerous user > >> > requests. We've seen no subsequent bug reports of systems hanging due > >> > to the the LED code being enabled by default. I'd therefore like to > >> > propose the following patch to enable the LED by default. This patch > >> > was originally authored by TJ. I apologize in advance that I do not > >> > have TJ's full first and last name for provenance. > >> > > >> > Thanks, > >> > Leann > >> > > >> > >From 315246037a0edab4d626de6ccb68c73d3fe61ce3 Mon Sep 17 00:00:00 2001 > >> > From: ubuntu@tjworld.net <ubuntu@tjworld.net> > >> > Date: Mon, 23 Mar 2009 20:29:28 +0000 > >> > Subject: [PATCH] ipw2200: Enable LED by default > >> > > >> > BugLink: http://bugs.launchpad.net/bugs/21367 > >> > > >> > Enable LED by default and update the MODULE_PARM_DESC. The original > >> > reason for defaulting to disabled was documented in 2005 and noted, "The > >> > LED code has been reported to hang some systems when running ifconfig > >> > and is therefore disabled by default." This no longer appears > >> > applicable and users have been requesting this be enabled for several > >> > years. > >> > > >> > Originally-by: TJ <ubuntu@tjworld.net> > >> > >> Does Originally-by also imply he has read the Certificate of Origin > >> and that he agrees for his code to be used in that way? That is the > >> purpose of the SOB anyway, but if you took his code, does that imply > >> that he agrees to it too? > > > > I'd CC'd TJ on the initial email, so I'm hoping he will respond here to > > your inquiry. I can say that he had submitted the original form of the > > patch to the Ubuntu kernel-team mailing list back in March, 2009: > > > > https://lists.ubuntu.com/archives/kernel-team/2009-March/005013.html > > > > In that original submission he did provide his official SOB. > > > > The patch was then applied with slight modification by Tim Gardner to > > the Ubuntu kernel. In forwarding this upstream, I wanted to give TJ the > > proper credit for originating the patch even though it's undergone some > > slight modification (ie the MODULE_PARM_DESC change). TJ had not > > officially signed off on this final form of the patch which is the > > reason I used "Originally-by:". If there is a better approach please > > let me know. > > In that case I would just recommend to keep his original SOB, then it > would go through you/tim whoever it goes through until it reaches > Linus. But just my $0.02. Re-sending and just replacing the Originally-by: with TJ's actual SOB. Thanks, Leann >From 315246037a0edab4d626de6ccb68c73d3fe61ce3 Mon Sep 17 00:00:00 2001 From: ubuntu@tjworld.net <ubuntu@tjworld.net> Date: Mon, 23 Mar 2009 20:29:28 +0000 Subject: [PATCH] ipw2200: Enable LED by default BugLink: http://bugs.launchpad.net/bugs/21367 Enable LED by default and update the MODULE_PARM_DESC. The original reason for defaulting to disabled was documented in 2005 and noted, "The LED code has been reported to hang some systems when running ifconfig and is therefore disabled by default." This no longer appears applicable and users have been requesting this be enabled for several years. Signed-off-by: TJ <ubuntu@tjworld.net> Signed-off-by: Tim Gardner <tim.gardner@canonical.com> Signed-off-by: Andy Whitcroft <apw@canonical.com> Acked-by: Stefan Bader <stefan.bader@canonical.com> Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com> --- Documentation/networking/README.ipw2200 | 2 +- drivers/net/wireless/ipw2x00/ipw2200.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/networking/README.ipw2200 b/Documentation/networking/README.ipw2200 index 80c7285..e4d3267 100644 --- a/Documentation/networking/README.ipw2200 +++ b/Documentation/networking/README.ipw2200 @@ -171,7 +171,7 @@ Where the supported parameter are: led Can be used to turn on experimental LED code. - 0 = Off, 1 = On. Default is 0. + 0 = Off, 1 = On. Default is 1. mode Can be used to set the default mode of the adapter. diff --git a/drivers/net/wireless/ipw2x00/ipw2200.c b/drivers/net/wireless/ipw2x00/ipw2200.c index 3aa3bb1..0805569 100644 --- a/drivers/net/wireless/ipw2x00/ipw2200.c +++ b/drivers/net/wireless/ipw2x00/ipw2200.c @@ -96,7 +96,7 @@ static int network_mode = 0; static u32 ipw_debug_level; static int associate; static int auto_create = 1; -static int led_support = 0; +static int led_support = 1; static int disable = 0; static int bt_coexist = 0; static int hwcrypto = 0; @@ -12083,7 +12083,7 @@ module_param(auto_create, int, 0444); MODULE_PARM_DESC(auto_create, "auto create adhoc network (default on)"); module_param_named(led, led_support, int, 0444); -MODULE_PARM_DESC(led, "enable led control on some systems (default 0 off)"); +MODULE_PARM_DESC(led, "enable led control on some systems (default 1 on)"); module_param(debug, int, 0444); MODULE_PARM_DESC(debug, "debug output mask"); -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] ipw2200: Enable LED by default 2010-06-16 0:55 [PATCH] ipw2200: Enable LED by default Leann Ogasawara 2010-06-16 1:20 ` Luis R. Rodriguez @ 2010-06-17 8:51 ` TJ 2010-06-17 17:22 ` Luis R. Rodriguez 1 sibling, 1 reply; 7+ messages in thread From: TJ @ 2010-06-17 8:51 UTC (permalink / raw) To: Leann Ogasawara; +Cc: reinette.chatre, ilw, linux-wireless On Tue, 2010-06-15 at 17:55 -0700, Leann Ogasawara wrote: > propose the following patch to enable the LED by default. This patch > was originally authored by TJ. I apologize in advance that I do not > have TJ's full first and last name for provenance. You've already got it. TJ is my full name. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] ipw2200: Enable LED by default 2010-06-17 8:51 ` [PATCH] " TJ @ 2010-06-17 17:22 ` Luis R. Rodriguez 0 siblings, 0 replies; 7+ messages in thread From: Luis R. Rodriguez @ 2010-06-17 17:22 UTC (permalink / raw) To: TJ; +Cc: Leann Ogasawara, reinette.chatre, ilw, linux-wireless On Thu, Jun 17, 2010 at 1:51 AM, TJ <ubuntu@tjworld.net> wrote: > On Tue, 2010-06-15 at 17:55 -0700, Leann Ogasawara wrote: >> propose the following patch to enable the LED by default. This patch >> was originally authored by TJ. I apologize in advance that I do not >> have TJ's full first and last name for provenance. > > You've already got it. TJ is my full name. Also from the SubmittingPatches documentation on the kernel: then you just add a line saying Signed-off-by: Random J Developer <random@developer.example.org> using your real name (sorry, no pseudonyms or anonymous contributions.) TJ is your full name? Luis ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2010-06-17 17:22 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-06-16 0:55 [PATCH] ipw2200: Enable LED by default Leann Ogasawara 2010-06-16 1:20 ` Luis R. Rodriguez 2010-06-16 2:41 ` Leann Ogasawara 2010-06-16 4:36 ` Luis R. Rodriguez 2010-06-16 15:57 ` [PATCH v2] " Leann Ogasawara 2010-06-17 8:51 ` [PATCH] " TJ 2010-06-17 17:22 ` Luis R. Rodriguez
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).