Thread (5 messages) 5 messages, 4 authors, 2018-03-13

Re: [PATCH] video: fbdev: via: remove VLA usage

From: Gustavo A. R. Silva <hidden>
Date: 2018-03-13 03:29:27
Also in: dri-devel, lkml


On 03/09/2018 05:02 AM, Emil Velikov wrote:
On 9 March 2018 at 10:59, Eric Engestrom [off-list ref] wrote:
quoted
On Thursday, 2018-03-08 11:39:49 -0600, Gustavo A. R. Silva wrote:
quoted
In preparation to enabling -Wvla, remove VLA usage.

Also, fixed as part of the directive to remove all VLAs from
the kernel: https://lkml.org/lkml/2018/3/7/621

Signed-off-by: Gustavo A. R. Silva <redacted>
Sorry for my reply on v1, I missed that Emil already replied to another
post of the same patch (side note on that, making use of `-vX` and
`--in-reply-to` helps track the evolution of patches)
Yep. I know. It is just that some people prefer vX in a new thread. 
Anyway, I will take this into account for the next time.
quoted
This looks good to me:
Reviewed-by: Eric Engestrom <redacted>
Same here
Reviewed-by: Emil Velikov <redacted>

-Emil
Thanks for reviewing it, guys.
-- 
Gustavo A. R. Silva
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help