Change in coreboot[master]: drivers/spi: ignore -Wvla on clang too

[Name of user not set (Code Review)]

[e-mail] has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/35865 )

Change subject: drivers/spi: ignore -Wvla on clang too
......................................................................

drivers/spi: ignore -Wvla on clang too

Change-Id: I99bc6877680b32f2bae78437ab0482baa65496d8
Signed-off-by: Greg V 
---
M src/drivers/spi/spi_flash.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/35865/1

diff --git a/src/drivers/spi/spi_flash.c b/src/drivers/spi/spi_flash.c
index 5dbe1f4..9dfecea 100644
--- a/src/drivers/spi/spi_flash.c
+++ b/src/drivers/spi/spi_flash.c
@@ -100,8 +100,8 @@
 #pragma GCC diagnostic push
 #if defined(__GNUC__) && !defined(__clang__)
 #pragma GCC diagnostic ignored "-Wstack-usage="
-#pragma GCC diagnostic ignored "-Wvla"
 #endif
+#pragma GCC diagnostic ignored "-Wvla"
 int spi_flash_cmd_write(const struct spi_slave *spi, const u8 *cmd,
    		size_t cmd_len, const void *data, size_t data_len)
 {

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35865 )

Change subject: drivers/spi: ignore -Wvla on clang too
......................................................................

Patch Set 1: Code-Review+1

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35865 )

Change subject: drivers/spi: ignore -Wvla on clang too
......................................................................

Patch Set 1: Code-Review+2

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/35865 )

Change subject: drivers/spi: ignore -Wvla on clang too
......................................................................

drivers/spi: ignore -Wvla on clang too

Change-Id: I99bc6877680b32f2bae78437ab0482baa65496d8
Signed-off-by: Greg V 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35865
Tested-by: build bot (Jenkins) 
Reviewed-by: Paul Menzel 
Reviewed-by: Julius Werner 
---
M src/drivers/spi/spi_flash.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Julius Werner: Looks good to me, approved

diff --git a/src/drivers/spi/spi_flash.c b/src/drivers/spi/spi_flash.c
index 5dbe1f4..9dfecea 100644
--- a/src/drivers/spi/spi_flash.c
+++ b/src/drivers/spi/spi_flash.c
@@ -100,8 +100,8 @@
 #pragma GCC diagnostic push
 #if defined(__GNUC__) && !defined(__clang__)
 #pragma GCC diagnostic ignored "-Wstack-usage="
-#pragma GCC diagnostic ignored "-Wvla"
 #endif
+#pragma GCC diagnostic ignored "-Wvla"
 int spi_flash_cmd_write(const struct spi_slave *spi, const u8 *cmd,
    		size_t cmd_len, const void *data, size_t data_len)
 {