Burman Yan wrote:
OK.
Sorry.
I had to attach the patch, since hotmail does line wrapping, but I will
note the part regarding
the more descriptive subject.
Hotmail is probably the _worse_ email client you could use to send
patches to the kernel.
Also, some of the patches are one line per file, so I joined them
together in one single patch.
I thought that splitting that into many tiny patches will actually be
more annoying than
a single bigger patch.
The logical separation you did isn't bad, but there is no mention on the
*subject line* what section is targeted by your patch.
Also you should CC the maintainer for each section you're targeting. For
instance, for the ACPI patch you should also CC Len Brown. Check the
MAINTAINERS file for the right people to CC on each patch.
Does that mean I should send those patches again?
Afraid so.
While you're at it, you could remove casts like this one:
- pathname = (char *)kmalloc(ACPI_PATHNAME_MAX, GFP_KERNEL);
+ pathname = (char *)kzalloc(ACPI_PATHNAME_MAX, GFP_KERNEL);
if (!pathname)
return -ENOMEM;
- memset(pathname, 0, ACPI_PATHNAME_MAX);
That "(char *)" really isn't needed.
Oh, and just one more thing: don't top post.
--
Paulo Marques - www.grupopie.com
"The face of a child can say it all, especially the
mouth part of the face."
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
