Re: [PATCH] mmc: sdhci-tegra: Add #ifdef CONFIG_OF guard for of_find_property
On Mon, Oct 03, 2011 at 08:59:45AM -0700, Stephen Warren wrote: Igor Grinberg wrote at Monday, October 03, 2011 2:32 AM: Hi Axel, On 10/03/11 06:07, Axel Lin wrote: I got below build error with make tegra_defconfig;make Add #ifdef CONFIG_OF guard for of_find_property to fix below build error: CC drivers/mmc/host/sdhci-tegra.o drivers/mmc/host/sdhci-tegra.c: In function 'sdhci_tegra_dt_parse_pdata': drivers/mmc/host/sdhci-tegra.c:157: error: implicit declaration of function 'of_find_property' ... diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c ... @@ -154,8 +154,11 @@ static struct tegra_sdhci_platform_data * __devinit sdhci_tegra_dt_parse_pdata( plat-cd_gpio = of_get_named_gpio(np, cd-gpios, 0); plat-wp_gpio = of_get_named_gpio(np, wp-gpios, 0); plat-power_gpio = of_get_named_gpio(np, power-gpios, 0); + +#ifdef CONFIG_OF if (of_find_property(np, support-8bit, NULL)) plat-is_8bit = 1; +#endif Shouldn't we add a stub for the of_find_property() method to include/linux/of.h instead of adding more ifdefs? Or may be use of_get_property() method instead? I submitted a patch to add a stub of_find_property a little while back. Per https://lkml.org/lkml/2011/9/22/301, Grant applied it already. I'm not sure why it hasn't shown up in linux-next, since Grant's repo wasn't affected by the kernel.org downtime. Sorry for the breakage. That's because I forgot to push out the tree. :-( Sorry. I've just fixed it now. g. -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] mmc: sdhci-tegra: Add #ifdef CONFIG_OF guard for of_find_property
Hi Axel, On 10/03/11 06:07, Axel Lin wrote: I got below build error with make tegra_defconfig;make Add #ifdef CONFIG_OF guard for of_find_property to fix below build error: CC drivers/mmc/host/sdhci-tegra.o drivers/mmc/host/sdhci-tegra.c: In function 'sdhci_tegra_dt_parse_pdata': drivers/mmc/host/sdhci-tegra.c:157: error: implicit declaration of function 'of_find_property' make[3]: *** [drivers/mmc/host/sdhci-tegra.o] Error 1 make[2]: *** [drivers/mmc/host] Error 2 make[1]: *** [drivers/mmc] Error 2 make: *** [drivers] Error 2 Signed-off-by: Axel Lin axel@gmail.com --- I got the build error on linux-next (20110930). Axel drivers/mmc/host/sdhci-tegra.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c index 67176afc..3883602 100644 --- a/drivers/mmc/host/sdhci-tegra.c +++ b/drivers/mmc/host/sdhci-tegra.c @@ -154,8 +154,11 @@ static struct tegra_sdhci_platform_data * __devinit sdhci_tegra_dt_parse_pdata( plat-cd_gpio = of_get_named_gpio(np, cd-gpios, 0); plat-wp_gpio = of_get_named_gpio(np, wp-gpios, 0); plat-power_gpio = of_get_named_gpio(np, power-gpios, 0); + +#ifdef CONFIG_OF if (of_find_property(np, support-8bit, NULL)) plat-is_8bit = 1; +#endif Shouldn't we add a stub for the of_find_property() method to include/linux/of.h instead of adding more ifdefs? Or may be use of_get_property() method instead? return plat; } -- Regards, Igor. -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH] mmc: sdhci-tegra: Add #ifdef CONFIG_OF guard for of_find_property
Igor Grinberg wrote at Monday, October 03, 2011 2:32 AM: Hi Axel, On 10/03/11 06:07, Axel Lin wrote: I got below build error with make tegra_defconfig;make Add #ifdef CONFIG_OF guard for of_find_property to fix below build error: CC drivers/mmc/host/sdhci-tegra.o drivers/mmc/host/sdhci-tegra.c: In function 'sdhci_tegra_dt_parse_pdata': drivers/mmc/host/sdhci-tegra.c:157: error: implicit declaration of function 'of_find_property' ... diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c ... @@ -154,8 +154,11 @@ static struct tegra_sdhci_platform_data * __devinit sdhci_tegra_dt_parse_pdata( plat-cd_gpio = of_get_named_gpio(np, cd-gpios, 0); plat-wp_gpio = of_get_named_gpio(np, wp-gpios, 0); plat-power_gpio = of_get_named_gpio(np, power-gpios, 0); + +#ifdef CONFIG_OF if (of_find_property(np, support-8bit, NULL)) plat-is_8bit = 1; +#endif Shouldn't we add a stub for the of_find_property() method to include/linux/of.h instead of adding more ifdefs? Or may be use of_get_property() method instead? I submitted a patch to add a stub of_find_property a little while back. Per https://lkml.org/lkml/2011/9/22/301, Grant applied it already. I'm not sure why it hasn't shown up in linux-next, since Grant's repo wasn't affected by the kernel.org downtime. Sorry for the breakage. -- nvpublic
RE: [PATCH] mmc: sdhci-tegra: Add #ifdef CONFIG_OF guard for of_find_property
On Sunday, October 02, 2011 9:08 PM, Axel Lin wrote: I got below build error with make tegra_defconfig;make Add #ifdef CONFIG_OF guard for of_find_property to fix below build error: CC drivers/mmc/host/sdhci-tegra.o drivers/mmc/host/sdhci-tegra.c: In function 'sdhci_tegra_dt_parse_pdata': drivers/mmc/host/sdhci-tegra.c:157: error: implicit declaration of function 'of_find_property' make[3]: *** [drivers/mmc/host/sdhci-tegra.o] Error 1 make[2]: *** [drivers/mmc/host] Error 2 make[1]: *** [drivers/mmc] Error 2 make: *** [drivers] Error 2 Signed-off-by: Axel Lin axel@gmail.com --- I got the build error on linux-next (20110930). Axel drivers/mmc/host/sdhci-tegra.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c index 67176afc..3883602 100644 --- a/drivers/mmc/host/sdhci-tegra.c +++ b/drivers/mmc/host/sdhci-tegra.c @@ -154,8 +154,11 @@ static struct tegra_sdhci_platform_data * __devinit sdhci_tegra_dt_parse_pdata( plat-cd_gpio = of_get_named_gpio(np, cd-gpios, 0); plat-wp_gpio = of_get_named_gpio(np, wp-gpios, 0); plat-power_gpio = of_get_named_gpio(np, power-gpios, 0); + +#ifdef CONFIG_OF if (of_find_property(np, support-8bit, NULL)) plat-is_8bit = 1; +#endif I think this type of build error is going to be very common when CONFIG_OF is not defined. Wouldn't it be better to fix the issue in the linux/of.h header? It looks like some of the of_* calls are already handled properly in that header. Regards, Hartley
[PATCH] mmc: sdhci-tegra: Add #ifdef CONFIG_OF guard for of_find_property
I got below build error with make tegra_defconfig;make Add #ifdef CONFIG_OF guard for of_find_property to fix below build error: CC drivers/mmc/host/sdhci-tegra.o drivers/mmc/host/sdhci-tegra.c: In function 'sdhci_tegra_dt_parse_pdata': drivers/mmc/host/sdhci-tegra.c:157: error: implicit declaration of function 'of_find_property' make[3]: *** [drivers/mmc/host/sdhci-tegra.o] Error 1 make[2]: *** [drivers/mmc/host] Error 2 make[1]: *** [drivers/mmc] Error 2 make: *** [drivers] Error 2 Signed-off-by: Axel Lin axel@gmail.com --- I got the build error on linux-next (20110930). Axel drivers/mmc/host/sdhci-tegra.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/mmc/host/sdhci-tegra.c b/drivers/mmc/host/sdhci-tegra.c index 67176afc..3883602 100644 --- a/drivers/mmc/host/sdhci-tegra.c +++ b/drivers/mmc/host/sdhci-tegra.c @@ -154,8 +154,11 @@ static struct tegra_sdhci_platform_data * __devinit sdhci_tegra_dt_parse_pdata( plat-cd_gpio = of_get_named_gpio(np, cd-gpios, 0); plat-wp_gpio = of_get_named_gpio(np, wp-gpios, 0); plat-power_gpio = of_get_named_gpio(np, power-gpios, 0); + +#ifdef CONFIG_OF if (of_find_property(np, support-8bit, NULL)) plat-is_8bit = 1; +#endif return plat; } -- 1.7.4.1 -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html