* Re: Re: [PATCH][RESEND] socket interface for IPMI against 2.5.66-bk [not found] ` <3E8D9CD2.8060506@acm.org> @ 2003-04-07 1:26 ` Louis Zhuang 2003-04-07 15:18 ` Corey Minyard 0 siblings, 1 reply; 3+ messages in thread From: Louis Zhuang @ 2003-04-07 1:26 UTC (permalink / raw) To: Corey Minyard; +Cc: OPENIPMIML, LKML On Fri, 2003-04-04 at 22:55, Corey Minyard wrote: > I've merged this in, but I made some adjustments: > > * I added a copyright to include/net/ipmi.h (just copied from > net/ipmi/af_ipmi.c) Just curious, should we add copyright even when I have announced it by 'MODULE_LICENSE("GPL")'? I dislike too many leagal text in code... > * I added usage comments to include/net/ipmi.h > * I modified the ioctls to return EFAULT on a bad user address. > * I modified SIOCIPMIGETEVENT to take a pointer to an integer, not > an integer. That's the way ioctls work, they always take a pointer, > even though it sucks, IMHO. > > I also adapted it to the 2.4 kernel. > > I'll put out an experimental patch for this sometime soon (with a bunch > of other stuff). I also have SMIC and full ACPI support as part of it, > and I'm waiting for them. > > - -Corey -- Yours truly, Louis Zhuang --------------- My words are my own... Fault Injection Test Harness Project BK tree: http://fault-injection.bkbits.net/linux-2.5 Home Page: http://sf.net/projects/fault-injection Open HPI Project Home Page: http://sf.net/projects/openhpi ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][RESEND] socket interface for IPMI against 2.5.66-bk 2003-04-07 1:26 ` Re: [PATCH][RESEND] socket interface for IPMI against 2.5.66-bk Louis Zhuang @ 2003-04-07 15:18 ` Corey Minyard 2003-04-07 14:51 ` Alan Cox 0 siblings, 1 reply; 3+ messages in thread From: Corey Minyard @ 2003-04-07 15:18 UTC (permalink / raw) To: Louis Zhuang; +Cc: OPENIPMIML, LKML -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Louis Zhuang wrote: >On Fri, 2003-04-04 at 22:55, Corey Minyard wrote: > >>I've merged this in, but I made some adjustments: >> >> * I added a copyright to include/net/ipmi.h (just copied from >>net/ipmi/af_ipmi.c) > >Just curious, should we add copyright even when I have announced it by >'MODULE_LICENSE("GPL")'? I dislike too many leagal text in code... I guess it's up to you, but you had added the header in all files but that one. Corey -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQE+kZa0IXnXXONXERcRAtd0AKCjBePfZPzdE+c9n/aBH0yqYI3SGwCdGZZc GbHp0DsHLsa1yF/W/T7kYRc= =pg9e -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][RESEND] socket interface for IPMI against 2.5.66-bk 2003-04-07 15:18 ` Corey Minyard @ 2003-04-07 14:51 ` Alan Cox 0 siblings, 0 replies; 3+ messages in thread From: Alan Cox @ 2003-04-07 14:51 UTC (permalink / raw) To: Corey Minyard; +Cc: Louis Zhuang, OPENIPMIML, LKML > >Just curious, should we add copyright even when I have announced it by > >'MODULE_LICENSE("GPL")'? I dislike too many leagal text in code... > > I guess it's up to you, but you had added the header in all files but > that one. It is a good idea to do so (although its a matter of taste too). It doesn't need a foot long pile of blurb. Its quite sufficient to say Copyright (C) <year> <author) and state where the GPL is found. Some people advise that the no warranty piece by present in each file and you'll see IBM I think do that. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2003-04-07 15:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1049363835.1168.6.camel@hawk.sh.intel.com>
[not found] ` <3E8C63D4.8040807@mvista.com>
[not found] ` <1049433965.1165.2.camel@hawk.sh.intel.com>
[not found] ` <3E8D9CD2.8060506@acm.org>
2003-04-07 1:26 ` Re: [PATCH][RESEND] socket interface for IPMI against 2.5.66-bk Louis Zhuang
2003-04-07 15:18 ` Corey Minyard
2003-04-07 14:51 ` Alan Cox
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox