Date:   Mon, 20 Mar 2023 14:04:03 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     Dan Carpenter <error27@gmail.com>,
        Kloudifold <cloudifold.3125@gmail.com>
CC:     <outreachy@lists.linux.dev>, <teddy.wang@siliconmotion.com>,
        <sudipm.mukherjee@gmail.com>, <gregkh@linuxfoundation.org>,
        <linux-staging@lists.linux.dev>, <linux-kernel@vger.kernel.org>,
        <alison.schofield@intel.com>
Subject: Re: [PATCH v3] staging: sm750: Rename sm750_hw_cursor_* functions to
 snake_case
Message-ID: <6418ca43e651b_2c274e294ae@iweiny-mobl.notmuch>
References: <ZBW+NX4SLaCyEnJd@CloudiRingWorld>
 <dd1e1fcc-2a4e-438f-ad61-ef7640b8e423@kili.mountain>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <dd1e1fcc-2a4e-438f-ad61-ef7640b8e423@kili.mountain>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?+YZmdrmyii05KoaUX1WwE4R857dEI17FAXqjR/3xP4gFsTV1ssU67/8jTM1v?=
 =?us-ascii?Q?2Bo1bJl8ursHSEXr/UFO6zuU/ActjHbLIvsXl1H6ksQxq83o8CEzaEtg9FJd?=
 =?us-ascii?Q?EQ2wY4KlOZMafnFoRruSynyjKGPdtf0O3Ku9AFpj5vt2t0VyT5JvmynaOvsA?=
 =?us-ascii?Q?v/oZViv3xbYGGsKXYWS/s0S0x97hkC2gG8tn31qB4LTxAcNPHZ910Uzwaf9R?=
 =?us-ascii?Q?xrPGpbCRsZqOgt9E9iK2f3WJaYB65xvFjzSH7VcKtJGYsN3IUdAyVlwOJQYM?=
 =?us-ascii?Q?QLh3y/YR46S+4ViQKKZ3IYx723DHum5IxmReAOoANceo5WZPEaVkrc8euudm?=
 =?us-ascii?Q?84pMF4xRVUWjQoAvbagkbuwSUALeYOOnmxbIF5d28VelQ/YQOmQQC+wNrstG?=
 =?us-ascii?Q?JmjUH5zk5/MUHg7+Zs5uEoHBTdOgZfm1/bLWs+OR0UXF5m+xlkJyl0wspnDl?=
 =?us-ascii?Q?426nDyc2vTJYbugts7pAPcsdJ2h/fPAGYC6o4NxeSf74fJZ1m1iXRhmE+Iq8?=
 =?us-ascii?Q?BiVfoCVD9aqG9mSE9iGH0gk+1qKF1bHH7zjrylDoeCgu1jvODY/Ae9VWkWyg?=
 =?us-ascii?Q?zfSozZ9YPZq81n12T3juy+UoUp4ct3S87VywAL3SyDbALKIvDrUjgoEQhdq6?=
 =?us-ascii?Q?L4M9s9CEVonkEiWOVtnahg7Yu0q5pEN6twAlo5/5lHXelwNpHVQPy9LmwaDz?=
 =?us-ascii?Q?mzsXdq4eSh/SKm8Z3paGnV73XTy4e7K/p2LHIX1S/crpAzWgA+xl8cbGQsew?=
 =?us-ascii?Q?fZHg/l6y86UyUEkZmmvh7mdLe0eAbd+1nmEFJXLTG106lvTbUXQn8wrAMs65?=
 =?us-ascii?Q?RXlON5Q6K45qpcps+NXcqdGgk/LeIY3E5CIOqfVMsYLdgsUEJknIiarFnG2z?=
 =?us-ascii?Q?VKqfQmsLQCkbKArpual2SC9sA00SBi0dE1tS5xRYyMLNct0K0Pr3rjDrePLf?=
 =?us-ascii?Q?zijyQs5HTeSMHRY4LDGoALWUhqS8UwrBfGdIctIICwnSvFls9z5j0A/Q0ouM?=
 =?us-ascii?Q?i+ZUFdbUfRi2qQ3xeshEr9xVNv1cSr5K1+gPcfbArs6CUScfHtQcJd/gXW4D?=
 =?us-ascii?Q?1iMSdrQ0McTWkuig+jinakqkI6BOyzHh8fXvqg9EsF1OInWx6vIy7U1aAI5R?=
 =?us-ascii?Q?XGaxiZ/kwCzuY5XVK4PESlS/+4mo5Z3iahH8joaR4LF++DwAgEkQNsNl1FsA?=
 =?us-ascii?Q?840fu0z+Xfb1sJygwcn0fnuqh1EamMZYeF69c7ugWDZa1i4pzv+eo6h0Qnv5?=
 =?us-ascii?Q?0ehqLSxIZo0PlL7Tfl2/yNOyXMvqIOdDY6zoQ7tAQO/DOuvEuU32drnjZaeU?=
 =?us-ascii?Q?2Jfg4kmUDQXjEb7IGtBWAq91syjoCYLImMZ9SvIwXs2efFbqNFyeYVKYwgHn?=
 =?us-ascii?Q?4bMEkx0zX5YZk2W+6FOeD9WEtcPN6lsJSoTm7IJEvjQ1bO+0m9Nx9Jj4ne6J?=
 =?us-ascii?Q?D/Ys+y9bVfSkVL9H1Uyi3230MTeVw8FcPhLPPXsTeWlmRmkIIU+WKYUcWEAD?=
 =?us-ascii?Q?zAGI75f6G+muc9rL8V/1JqgVOCCXAg8mV5vRQ2fiiLO5dmppcG86NANERSVx?=
 =?us-ascii?Q?5Dqd4x1Buu/e4VgW9BNZwc3b6JoMWJlNFUHuCEWG?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 91ee47b5-0719-4f2b-bb54-08db2986a58d
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Mar 2023 21:04:08.6686
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: CLh1y+/bAJA/8jIwOTzvUNh2+5dRrgdt9mHhYjHsR4b9s3oFFcbk1st6HDR4nqgoX2soNtFKrLP5yMEgfTTQxQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB5454
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Dan Carpenter wrote:
> On Sat, Mar 18, 2023 at 09:35:49PM +0800, Kloudifold wrote:

[...]

> 
> > ---
> > Changes in v3:
> > - Add this changelog (Philipp)
> > - Move lkp tags and link to the correct location in commit log (Alison)
> > - Update the commit msg (Philip)
> > - Update the commit log (Bagas, Julia)
> > 
> > Changes in v2:
> > - Use new function names in call sites (LKP)
> > 
> > This v2 patch was prompted by an error reported by the Linux test
> > robot, which detected the compile error.
> > Reported-by: kernel test robot <lkp@intel.com>
> > Link: https://lore.kernel.org/oe-kbuild-all/202303110849.X24WnHnM-lkp@intel.com/
> > ---
> > ---
> 
> Delete these two --- lines.
> 

Yes the 2 --- lines are not needed but...

I'd like to clarify because I think some of the tools insert these
automatically.  I've included a bunch of examples below and the patches they
generate as examples.  Please follow along if you are not sure what I mean.

To summarize you should only insert 1 '---' line in your manually created
commit message.  Do not add one to the cover letter either.  Any other lines
which are created by the tools are fine.  Just leave them.

So for this example...

_This_ is _wrong_[1]

_This_ is _correct_[2]

The correct example [2] results in a patch like [3].

But notice how B4 has added my 'cover' into the single patch, patch?  And it
has a bunch of '---' lines?


Specifically:

<quote>
---
Nothing really to be said here.  This would be a cover letter if the
patch series had more than 1 patch in it...
---
Changes in v2:
        show version info example.
---
</quote>


This is because b4 defaults to using the cover letter as part of the single
patch background information.  When using b4 to send a single patch I leave my
cover letter blank.


So if you leave the b4 cover letter blank[4] you get a correctly formatted
patch as in [5].

But _please_ _note_ that there are still 3 '---' lines in the final patch
output.  2 of which are around the changes section.  The following quote is
perfectly fine.


<quote>
---
Changes in v2:
        show version info example.
---
</quote>

I hope this helps clarify because I'm worried that folks may be trying to edit
the emails that the tools are generating and I don't think that is required.

Ira


[1]
13:39:43 > git sh
commit b513858163640433f7938ff93848a5af1e675383 (HEAD -> b4/outreachy-demo)
Author: Ira Weiny <ira.weiny@intel.com>
Date:   Mon Mar 20 13:14:51 2023 -0700

    Documentation: Example staging doc change
    
    Update the documentation with an example showing how to update the
    documentation.
    
    Signed-off-by: Ira Weiny <ira.weiny@intel.com>
    
    ---
    Changes in v2:
            show version info example.
    ---

diff --git a/Documentation/staging/index.rst b/Documentation/staging/index.rst
index ded8254bc0d7..2de034ddd059 100644
--- a/Documentation/staging/index.rst
+++ b/Documentation/staging/index.rst
@@ -1,5 +1,7 @@
 .. SPDX-License-Identifier: GPL-2.0
 
+Fixing staging documentation somehow.
+
 Unsorted Documentation
 ======================
 

[2]
13:41:03 > git sh
commit 352cbf83a2da890ba974c043ae6047a625210ea2 (HEAD -> b4/outreachy-demo)
Author: Ira Weiny <ira.weiny@intel.com>
Date:   Mon Mar 20 13:14:51 2023 -0700

    Documentation: Example staging doc change
    
    Update the documentation with an example showing how to update the
    documentation.
    
    Signed-off-by: Ira Weiny <ira.weiny@intel.com>
    ---
    Changes in v2:
            show version info example.

diff --git a/Documentation/staging/index.rst b/Documentation/staging/index.rst
index ded8254bc0d7..2de034ddd059 100644
--- a/Documentation/staging/index.rst
+++ b/Documentation/staging/index.rst
@@ -1,5 +1,7 @@
 .. SPDX-License-Identifier: GPL-2.0
 
+Fixing staging documentation somehow.
+
 Unsorted Documentation
 ======================
 

[3]
<email headers>
...
Subject: [PATCH] Documentation: Example staging doc change
...
</email headers>

Update the documentation with an example showing how to update the
documentation.

Signed-off-by: Ira Weiny <ira.weiny@intel.com>
---
Nothing really to be said here.  This would be a cover letter if the
patch series had more than 1 patch in it...
---
Changes in v2:
        show version info example.
---
 Documentation/staging/index.rst | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/staging/index.rst b/Documentation/staging/index.rst
index ded8254bc0d7..2de034ddd059 100644
--- a/Documentation/staging/index.rst
+++ b/Documentation/staging/index.rst
@@ -1,5 +1,7 @@
 .. SPDX-License-Identifier: GPL-2.0
 
+Fixing staging documentation somehow.
+
 Unsorted Documentation
 ======================
 

---
base-commit: 38e04b3e4240a6d8fb43129ebad41608db64bc6f
change-id: 20230320-outreachy-demo-bfa128532a93

Best regards,
-- 
Ira Weiny <ira.weiny@intel.com>


[4]
Cover example

# Lines starting with # will be removed from the cover letter. You can
# use them to add notes or reminders to yourself. If you want to use
# markdown headers in your cover letter, start the line with ">#".

# You can add trailers to the cover letter. Any email addresses found in
# these trailers will be added to the addresses specified/generated
# during the b4 send stage. You can also run "b4 prep --auto-to-cc" to
# auto-populate the To: and Cc: trailers based on the code being
# modified.

To: Ira Weiny <ira.weiny@intel.com>
Signed-off-by: Ira Weiny <ira.weiny@intel.com>


[5]
<email headers>
...
Subject: [PATCH] Documentation: Example staging doc change
...
</email headers>

Update the documentation with an example showing how to update the
documentation.

Signed-off-by: Ira Weiny <ira.weiny@intel.com>
---
Changes in v2:
        show version info example.
---
 Documentation/staging/index.rst | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/staging/index.rst b/Documentation/staging/index.rst
index ded8254bc0d7..2de034ddd059 100644
--- a/Documentation/staging/index.rst
+++ b/Documentation/staging/index.rst
@@ -1,5 +1,7 @@
 .. SPDX-License-Identifier: GPL-2.0
 
+Fixing staging documentation somehow.
+
 Unsorted Documentation
 ======================
 

---
base-commit: 38e04b3e4240a6d8fb43129ebad41608db64bc6f
change-id: 20230320-outreachy-demo-bfa128532a93

Best regards,
-- 
Ira Weiny <ira.weiny@intel.com>