Thread (36 messages) 36 messages, 7 authors, 2007-10-30

Re: [PATCH 02/11] [POWERPC] Add 'lpb' bus type for MPC5200 LocalPlus Bus

From: Olof Johansson <hidden>
Date: 2007-10-24 00:02:29

Hi,

On Wed, Oct 24, 2007 at 01:13:15AM +0200, Marian Balakowicz wrote:
Signed-off-by: Marian Balakowicz <redacted>
---
Your patch lacks any kind of description beyond the title. (I know, it's a
real simple patch, but a real simple description would do too :)
quoted hunk ↗ jump to hunk
--- a/arch/powerpc/kernel/of_platform.c
+++ b/arch/powerpc/kernel/of_platform.c
@@ -49,6 +49,7 @@ static struct of_device_id of_default_bus_ids[] = {
 	{ .type = "plb4", },
 	{ .type = "opb", },
 	{ .type = "ebc", },
+	{ .type = "lpb", },
I thought it was consensus to keep the new bus ids only in platform
code and not add them to the default list, and just pass that list in
to of_platform_bus_probe().




-Olof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help