On Tue, Aug 18, 2020 at 9:03 PM Kenneth Chan <kenneth.t.chan@xxxxxxxxx> wrote: > Thank you for a patch, but it misses few crucial points: - it should have good description as commit message (body of this letter) - it should have signature of origin (Signed-off-by tag) that I can proceed (I see, it's in wrong position, use `git commit -s` to fix this) - it should be split per one logical change in a patch, e.g. 1) replacing ACPI prints by pr_*() macros, 2) split MODULE_AUTHOR() by one author per macro call, 3) etc... > Signed-off-by: Kenneth Chan <kenneth.t.chan@xxxxxxxxx> -- With Best Regards, Andy Shevchenko