2011-02-22 21:23:08

by Andres Salomon

Subject: [PATCH] spi/spi_altera: fix build error

I ran into a build error while attempting to build the Feb 22nd
linux-next tree.



From: Andres Salomon <[email protected]>

Fix the following build error due to a missing semicolon:

CC [M] drivers/spi/spi_altera.o
drivers/spi/spi_altera.c:308: error: expected ‘,’ or ‘;’ before ‘extern’
make[2]: *** [drivers/spi/spi_altera.o] Error 1

Signed-off-by: Andres Salomon <[email protected]>
---
drivers/spi/spi_altera.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/spi/spi_altera.c b/drivers/spi/spi_altera.c
index 5e001f3..4813a63 100644
--- a/drivers/spi/spi_altera.c
+++ b/drivers/spi/spi_altera.c
@@ -304,7 +304,7 @@ static int __devexit altera_spi_remove(struct platform_device *dev)
static const struct of_device_id altera_spi_match[] = {
{ .compatible = "ALTR,spi-1.0", },
{},
-}
+};
MODULE_DEVICE_TABLE(of, altera_spi_match);
#else /* CONFIG_OF */
#define altera_spi_match NULL
--
1.7.2.3

2011-02-22 21:38:37

by Randy Dunlap

Subject: Re: [PATCH] spi/spi_altera: fix build error

On Tue, 22 Feb 2011 13:23:03 -0800 Andres Salomon wrote:

> I ran into a build error while attempting to build the Feb 22nd
> linux-next tree.
>
>
>
> From: Andres Salomon <[email protected]>
>
> Fix the following build error due to a missing semicolon:
>
> CC [M] drivers/spi/spi_altera.o
> drivers/spi/spi_altera.c:308: error: expected ‘,’ or ‘;’ before ‘extern’
> make[2]: *** [drivers/spi/spi_altera.o] Error 1
>
> Signed-off-by: Andres Salomon <[email protected]>

Acked-by: Randy Dunlap <[email protected]>

so ignore my combined patch. Thanks.


> ---
> drivers/spi/spi_altera.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/spi/spi_altera.c b/drivers/spi/spi_altera.c
> index 5e001f3..4813a63 100644
> --- a/drivers/spi/spi_altera.c
> +++ b/drivers/spi/spi_altera.c
> @@ -304,7 +304,7 @@ static int __devexit altera_spi_remove(struct platform_device *dev)
> static const struct of_device_id altera_spi_match[] = {
> { .compatible = "ALTR,spi-1.0", },
> {},
> -}
> +};
> MODULE_DEVICE_TABLE(of, altera_spi_match);
> #else /* CONFIG_OF */
> #define altera_spi_match NULL
> --

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***