Messages in this thread | | | From | Emil Velikov <> | Date | Fri, 9 Mar 2018 11:02:24 +0000 | Subject | Re: [PATCH] video: fbdev: via: remove VLA usage |
| |
On 9 March 2018 at 10:59, Eric Engestrom <eric.engestrom@imgtec.com> wrote: > On Thursday, 2018-03-08 11:39:49 -0600, Gustavo A. R. Silva wrote: >> 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 <gustavo@embeddedor.com> > > 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) > > This looks good to me: > Reviewed-by: Eric Engestrom <eric.engestrom@imgtec.com> > Same here Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
-Emil
| |