lkml.org 
[lkml]   [2021]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 1/7] staging: r8188eu: remove haldata's EEPROMVID / PID
Date
On Thursday, November 11, 2021 10:26:38 PM CET Martin Kaiser wrote:

> They are used only in a (disabled) debug print.
>
> In practice, lsusb can be used to read the actual vid and pid.

Hi Martin,

You seem to have overlooked the usual rules for writing conformant commit
messages :)

Please say "what" you did along with "why" you made the changes.

Here, and in two or three other patches of your series, you forgot to
describe "what" you did. You actually wrote it in the "Subject" lines but, as
you know for sure, commit messages must be self-contained entities.

Also, it would be nice to have a cover letter and have all seven patches in
one thread in response to the above, even if it (AFAIK) is not strictly
required.

Anyway, many thanks for your work.
Regards,

Fabio

> Signed-off-by: Martin Kaiser <martin@kaiser.cx>
> ---
> drivers/staging/r8188eu/hal/usb_halinit.c | 8 --------
> drivers/staging/r8188eu/include/rtl8188e_hal.h | 2 --
> drivers/staging/r8188eu/include/rtl8188e_spec.h | 2 --
> 3 files changed, 12 deletions(-)



\
 
 \ /
  Last update: 2021-11-12 10:03    [W:0.060 / U:1.560 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site