Thread (7 messages) flat view 7 messages, 2 authors, 2023-07-19

Re: [v3 1/2] PCI: layerscape: Add support for Link down notification

From: Frank Li <Frank.li@nxp.com>
Date: 2023-07-19 21:15:14
Also in: imx, kernel-janitors, linux-arm-kernel, linux-pci, lkml

On Wed, Jul 19, 2023 at 10:08:16PM +0200, Markus Elfring wrote:
quoted
Cover letter just annoise people here.
How do you think about advices from another information source?

See also:
https://kernelnewbies.org/PatchSeries
"You may like to include a cover letter with your patch series."

Generally, I think cover letter will be needed only if it really
help reviewer to get main idea about patches. 

Such as my on going pathes(with cover letter):
  https://lore.kernel.org/imx/ZLglBiSz0meJm5os@lizhi-Precision-Tower-5810/T/#t (local)

Similar case without(cover leter) and accepted.
 https://lore.kernel.org/imx/20230719063425.GE151430@dragon/T/#t (local)

I don't think cover letter real help reviewer to review these two patches.

I more like to get "real problem"(such as comments about "typo").

It is just waste time to discuss if need add cover letter here.

Frank
Regards,
Markus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help