Re: [PATCH 4/4] [POWERPC] fsl_spi_init and users: stop using device_type = "spi"

On Wed, Dec 19, 2007 at 03:04:51PM -0600, Timur Tabi wrote:

Anton Vorontsov wrote:

quoted

diff --git a/arch/powerpc/sysdev/fsl_soc.c b/arch/powerpc/sysdev/fsl_soc.c
index 3cf84d0..91bac51 100644
--- a/arch/powerpc/sysdev/fsl_soc.c
+++ b/arch/powerpc/sysdev/fsl_soc.c

@@ -1242,7 +1242,7 @@ int __init fsl_spi_init(struct spi_board_info 

*board_infos,
	}

	for (np = NULL, i = 1;
-	     (np = of_find_compatible_node(np, "spi", "fsl_spi")) != NULL;
+	     (np = of_find_compatible_node(np, NULL, "fsl,spi")) != NULL;

Can you keep the original code around to look for older device trees that 
are wrong?  Backwards compatibility is important.  I don't want to have to 
update the device tree just because I update the kernel.

I though about it. Is your device tree source out of tree? Otherwise
it should be trivial to upgrade the dtb, instead of producing cruft in
the kernel. I vote for less legacy code, but lets see what others will
say. So far count is 1:1. ;-)

Thanks,

-- 
Anton Vorontsov
email: cbou@mail.ru
backup email: ya-cbou@yandex.ru
irc://irc.freenode.net/bd2