Re: [PATCH] Staging: r8188eu: Fixed block comment

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

 



On venerdì 17 febbraio 2023 08:17:00 CET Khadija Kamran wrote:
> Fixed block comment by adding '*' in each line. The Check message was
> shown by using checkpatch.pl script.
> 
> Signed-off-by: Khadija Kamran <kamrankhadijadj@xxxxxxxxx>
> ---
>  drivers/staging/r8188eu/core/rtw_br_ext.c | 19 ++++++++++---------
>  1 file changed, 10 insertions(+), 9 deletions(-)
> 

Hi Khadija,

As Greg said this driver is not here any longer. This makes me think that you 
forgot at least two things...

1) Please pull and rebase the staging tree every day while working on patches 
during this period of the Outreachy project. If you had done so, you would 
have noticed that your patch cannot any more be applied.

2) I saw this patch by pure luck because you forgot to Cc the Outreachy list. 
Therefore, please don't ever forget to Cc that list. Most mentors don't 
subscribe the linux-staging list.

Thanks,

Fabio

P.S.: Please use an imperative tone, both in the subject and in the commit 
message. For example this patch could have been written the following way:

staging: axis-fifo: Add "*" to multi-line comment

Add asterisks to each line of a multi-line comment to conform to the Linux 
development guidelines. Warning reported by checkpatch.pl.

Signed-off-by: [...]









[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux