Re: [PATCH] whitespace and comment cleanup for asus_acpi.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> This patch fixes 100 errors and 8 warnings found by checkpatch against
> the asus_acpi.c file.
> 
> It should apply cleanly to 2.6.27 and should not affect any code functionality.

Hi Mike,
Welcome!

I've applied your patch to the acpi-test tree.

A couple of things to note for your next patch...

This particular driver has a sub-maintainer, Corentin Chary,
as indicated in MAINTAINERS.  Chary will probably either Ack or Nak
this patch in response to this note, but ideally you'd have sent
the patch to Chary in the first round.

Note that this particular driver is suffering bit rot
b/c the plan is to eventually delete it in favor of
drivers/misc/asus-laptop.c.  But I agree that as long
as it is still in the tree, it should undergo basic
maintenace, even if it never gets any new features.

Also, best to avoid sending attachments if you can, because people
read comment and reply to plain text, and often don't open attachments.
>From a patch application point of view, I can handle either, but
prefer plain text because I'm also a reviwer...

ref: Documentation/SubmittingPatches

thanks!
-Len



--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux