2014-09-02 11:22 GMT+08:00 Jingoo Han <[email protected]>:
> Remove unnecessary #ifdef CONFIG_OF, because this is already
> handled by the of_match_ptr macro.
>
> Signed-off-by: Jingoo Han <[email protected]>
> ---
>  drivers/spi/spi-altera.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/spi/spi-altera.c b/drivers/spi/spi-altera.c
> index 5b5709a5c957..588d428f47c8 100644
> --- a/drivers/spi/spi-altera.c
> +++ b/drivers/spi/spi-altera.c
> @@ -268,14 +268,12 @@ static int altera_spi_remove(struct platform_device 
> *dev)
>         return 0;
>  }
>
> -#ifdef CONFIG_OF
>  static const struct of_device_id altera_spi_match[] = {
>         { .compatible = "ALTR,spi-1.0", },
>         { .compatible = "altr,spi-1.0", },
>         {},
>  };
>  MODULE_DEVICE_TABLE(of, altera_spi_match);
> -#endif /* CONFIG_OF */

For !CONFIG_OF case:
        #define of_match_ptr(_ptr)      NULL
Can you explain how of_match_ptr() handle this?

Thanks,
Axel
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to