Re: [PATCH 1/3] Removed extra asterisks from comment beginnings, and removed unnecessary comment end

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

 



Le 07/07/2024 à 06:14, Gold Side a écrit :
 From 3574f0514207f9610a69d82d3dd0d018d6dce5dd Mon Sep 17 00:00:00 2001
From: Steven Davis <goldside000@xxxxxxxxxxx>
Date: Tue, 2 Jul 2024 22:57:24 -0400
Subject: [PATCH 1/3] Removed extra asterisk from comment beginning

It saves a byte. I'd imagine bytes are valuable in a project like this.
PS: Please forgive my previous patch email, I didn't know the guidelines well.

Hi,

no-problem, newcomers are always welcomed and they have to learn. So, IMHO, making small mistakes is part of the learning curve.


People/mailing-list should be defined according to the MAINTAINERS file. A script helps you for that: ./scripts/get_maintainer.pl

It can be run either on a patch or on a file/directory.

Here:
	./scripts/get_maintainer.pl -f kernel/module/main.c
or
	./scripts/get_maintainer.pl your_patch.patch
give:
	Luis Chamberlain <mcgrof@xxxxxxxxxx> (maintainer:MODULE SUPPORT)
	linux-modules@xxxxxxxxxxxxxxx (open list:MODULE SUPPORT)
	linux-kernel@xxxxxxxxxxxxxxx (open list:MODULE SUPPORT)

It is perfectly fine to add other people/mailing list that are relevant, as kernel-janitor.

If you are using git mail, it is possible to automate the creation of the To: and cc: fields. (see https://www.marcusfolkesson.se/blog/get_maintainers-and-git-send-email/). If using this --identify, you still can add some other --to, --cc., which is really convenient.



On your commit log, here are a few comments:
  - there should be a new-line before the Signed-off-by: line
- you can add whatever you want after the --- ending the commit description. These extra comments are informational only, and won't be part of the git history. Here, your PS should have been here.
   - imperative wording is preferred when writing commit logs

You can also give a look at https://www.kernel.org/doc/html/latest/process/submitting-patches.html, if not already done.


Welcome and happy patching :)

CJ

Signed-off-by: Steven Davis <goldside000@xxxxxxxxxxx
---
  kernel/module/main.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/module/main.c b/kernel/module/main.c
index d18a94b973e102..25e456f4381c71 100644
--- a/kernel/module/main.c
+++ b/kernel/module/main.c
@@ -450,7 +450,7 @@ bool __is_module_percpu_address(unsigned long addr, unsigned long *can_addr)
       return false;
  }
-/**
+/*
   * is_module_percpu_address() - test whether address is from module static percpu
   * @addr: address to test
   *





[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux