2011/9/14, Andreas Barth [off-list ref]:

* keguang.zhang@gmail.com (keguang.zhang@gmail.com) [110914 12:49]:

quoted

This patch adds basic support for Loongson1B
including serial, timer and interrupt handler.

I have a couple of questions. One of them is if it shouldn't be
possible to add this as part of the loongson-platform, and if we
really need a new platform. Each platform comes with some maintainence
costs which we should try to avoid. Making things more generic is
usually the right answer.

I've tried to add Loongson1 to loongson-platform (acturally loongson2
platform), but there is essential difference between them. The
loongson2 platform is something like the PC's architecture, which has
north and south bridge, while the loongson1 is SoC.
So, I think it's better that adding loongson1 as a new platform.

quoted

diff --git a/arch/mips/include/asm/mach-loongson1/irq.h

b/arch/mips/include/asm/mach-loongson1/irq.h
new file mode 100644
index 0000000..44cec4a

--- /dev/null
+++ b/arch/mips/include/asm/mach-loongson1/irq.h

@@ -0,0 +1,70 @@
+/*
+ * Copyright (c) 2011 Zhang, Keguang <keguang.zhang@gmail.com>
+ *
+ * Register mappings for Loongson1.

Can't we do the mapping via device trees, or are we not there yet?

quoted

--- /dev/null
+++ b/arch/mips/loongson1/common/clock.c

@@ -0,0 +1,164 @@
+/*
+ * Copyright (c) 2011 Zhang, Keguang <keguang.zhang@gmail.com>

Is this file not derived from any of the clock drivers we already have
in Linux?

Doesn't any of the existing clock drivers work?

Is this clock part of the CPU? Otherwise it would make sense to move
it out to the generic drivers section.

quoted

--- /dev/null
+++ b/arch/mips/loongson1/common/irq.c

@@ -0,0 +1,132 @@
+/*
+ * Copyright (c) 2011 Zhang, Keguang <keguang.zhang@gmail.com>
+ *
+ * Based on Copyright (C) 2009 Lemote Inc.

same question here. Also, do you have permission from Lemote to put
the code within GPLv2?

These code are based on the loongson platform, which is part of the
kernel code already.

quoted

diff --git a/arch/mips/loongson1/common/prom.c

b/arch/mips/loongson1/common/prom.c
new file mode 100644
index 0000000..84a25f6

--- /dev/null
+++ b/arch/mips/loongson1/common/prom.c

Can't we re-use the prom-routines from the loongson platform here? Or
even better, factor them out somewhere else in the mips or even
generic linux tree?

Same reason as question 1.

quoted

index 0000000..b34ad35

--- /dev/null
+++ b/arch/mips/loongson1/common/reset.c

quoted

+static void loongson1_halt(void)
+{
+	pr_notice("\n\n** You can safely turn off the power now **\n\n");
+	while (1) {
+		if (cpu_wait)
+			cpu_wait();
+	}
+}

This code looks familiar to me, i.e. it shouldn't be
platform-specific.




Andi

Thanks for your review!

-- 
Best Regards!
Kelvin