Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> writes: > On 13/03/2024 10:58, Kalle Valo wrote: >> Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> writes: >> >>> Create the new module rtl8192d-common and move some code into it from >>> rtl8192de. Now the rtl8192de driver (PCI) and the new rtl8192du driver >>> (USB) can share some of the code. >>> >>> This is mostly the code that required little effort to make it >>> shareable. There are a few more functions which they could share, with >>> some changes. >>> >>> The only other changes in this patch should be adjusting whitespace and >>> renaming some functions. >>> >>> Tested only with RTL8192DU. >>> >>> Signed-off-by: Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> >> >> A cover letter is would be really good to have. > > I can add one. What should I write in it? Basically just a short (few sentences) introduction what the patchset adds and if there are problematic/broken/todo items, for example in this case that the firmware upload is pending. This info helps the reviewers and also having the cover letters makes the grouping in email applications better. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
