Thread (5 messages) 5 messages, 3 authors, 2014-08-26

Re: [PATCH 6/7] video: fbdev: sis: delete double assignment

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2014-08-25 11:36:45
Also in: kernel-janitors, lkml

Hi Julia,

On Sun, Aug 24, 2014 at 8:40 PM, Julia Lawall [off-list ref] wrote:
quoted
On Sat, Aug 23, 2014 at 8:33 PM, Julia Lawall [off-list ref] wrote:
quoted
---
The patches in this series do not depend on each other.

This changes the semantics of the code and is not tested.
Hence I think you should change the subject of the patch, so it's obvious
some bug is fixed, e.g. "video: fbdev: sis: Fix double assignment".
Do you want me to resend this patch with the new subject, or just keep it
in mind for the future?
That's up to the fbdev maintainer.

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