Thread (8 messages) 8 messages, 6 authors, 2020-03-02

Re: [PATCH] matroxfb: add Matrox MGA-G200eW board support

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2020-01-27 10:40:38
Also in: dri-devel

Hi Greg,

On Sun, Jan 26, 2020 at 8:44 AM Greg Kroah-Hartman
[off-list ref] wrote:
On Sat, Jan 25, 2020 at 02:55:06PM -0500, Rich Felker wrote:
quoted
Signed-off-by: Rich Felker <dalias@libc.org>
--
I know I don't accept patches without any changelog text, don't know
about other subsystem maintainers...
FTR, I do, iff the one-line summary says everything that needs to be said.

What's the point in writing a full paragraph like:

   Currently the foo driver does not support the bar device.
   As users may want to use the bar device, it makes perfect sense
   to add support for the bar device to the foo driver.
   Hence add support for the bar device to the foo driver.

if this doesn't add any value on top of the one-line summary?

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help