* [PATCH] [ARM] dove: add support for CM-A510 machine.
@ 2010-10-07 7:36 Konstantin Sinyuk
2010-10-07 17:44 ` saeed bishara
2010-10-13 7:41 ` Mike Rapoport
0 siblings, 2 replies; 13+ messages in thread
From: Konstantin Sinyuk @ 2010-10-07 7:36 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Konstantin Sinyuk <kostyas@compulab.co.il>
Signed-off-by: Mike Rapoport <mike@compulab.co.il>
---
arch/arm/mach-dove/Kconfig | 6 +++
arch/arm/mach-dove/Makefile | 1 +
arch/arm/mach-dove/cm-a510.c | 96 ++++++++++++++++++++++++++++++++++++++++++
3 files changed, 103 insertions(+), 0 deletions(-)
create mode 100644 arch/arm/mach-dove/cm-a510.c
diff --git a/arch/arm/mach-dove/Kconfig b/arch/arm/mach-dove/Kconfig
index 3b9a32a..a4ed390 100644
--- a/arch/arm/mach-dove/Kconfig
+++ b/arch/arm/mach-dove/Kconfig
@@ -9,6 +9,12 @@ config MACH_DOVE_DB
Say 'Y' here if you want your kernel to support the
Marvell DB-MV88AP510 Development Board.
+ config MACH_CM_A510
+ bool "CompuLab CM-A510 Board"
+ help
+ Say 'Y' here if you want your kernel to support the
+ CompuLab CM-A510 Board.
+
endmenu
endif
diff --git a/arch/arm/mach-dove/Makefile b/arch/arm/mach-dove/Makefile
index 7ab3be5..f74f549 100644
--- a/arch/arm/mach-dove/Makefile
+++ b/arch/arm/mach-dove/Makefile
@@ -1,3 +1,4 @@
obj-y += common.o addr-map.o irq.o pcie.o
obj-$(CONFIG_MACH_DOVE_DB) += dove-db-setup.o
+obj-$(CONFIG_MACH_CM_A510) += cm-a510.o
diff --git a/arch/arm/mach-dove/cm-a510.c b/arch/arm/mach-dove/cm-a510.c
new file mode 100644
index 0000000..321c961
--- /dev/null
+++ b/arch/arm/mach-dove/cm-a510.c
@@ -0,0 +1,96 @@
+/*
+ * arch/arm/mach-dove/cm-a510.c
+ *
+ * Copyright (C) 2010 CompuLab, Ltd.
+ * Konstantin Sinyuk <kostyas@compulab.co.il>
+ *
+ * Based on Marvell DB-MV88AP510-BP Development Board Setup
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+#include <linux/kernel.h>
+#include <linux/init.h>
+#include <linux/ata_platform.h>
+#include <linux/mv643xx_eth.h>
+#include <linux/spi/spi.h>
+#include <linux/spi/flash.h>
+
+#include <asm/mach-types.h>
+#include <asm/mach/arch.h>
+
+#include <mach/dove.h>
+
+#include "common.h"
+
+static struct mv643xx_eth_platform_data cm_a510_ge00_data = {
+ .phy_addr = MV643XX_ETH_PHY_ADDR_DEFAULT,
+};
+
+static struct mv_sata_platform_data cm_a510_sata_data = {
+ .n_ports = 1,
+};
+
+/*
+ * SPI Devices:
+ * SPI0: 1M Flash Winbond w25q32bv
+ */
+static const struct flash_platform_data cm_a510_spi_flash_data = {
+ .type = "w25q32bv",
+};
+
+static struct spi_board_info __initdata cm_a510_spi_flash_info[] = {
+ {
+ .modalias = "m25p80",
+ .platform_data = &cm_a510_spi_flash_data,
+ .irq = -1,
+ .max_speed_hz = 20000000,
+ .bus_num = 0,
+ .chip_select = 0,
+ },
+};
+
+static int __init cm_a510_pci_init(void)
+{
+ if (machine_is_cm_a510())
+ dove_pcie_init(1, 1);
+
+ return 0;
+}
+
+subsys_initcall(cm_a510_pci_init);
+
+/* Board Init */
+static void __init cm_a510_init(void)
+{
+ /*
+ * Basic Dove setup. Needs to be called early.
+ */
+ dove_init();
+
+ dove_ge00_init(&cm_a510_ge00_data);
+ dove_ehci0_init();
+ dove_ehci1_init();
+ dove_sata_init(&cm_a510_sata_data);
+ dove_sdio0_init();
+ dove_sdio1_init();
+ dove_spi0_init();
+ dove_spi1_init();
+ dove_uart0_init();
+ dove_uart1_init();
+ dove_i2c_init();
+ spi_register_board_info(cm_a510_spi_flash_info,
+ ARRAY_SIZE(cm_a510_spi_flash_info));
+}
+
+MACHINE_START(CM_A510, "Compulab CM-A510 Board")
+ .phys_io = DOVE_SB_REGS_PHYS_BASE,
+ .io_pg_offst = ((DOVE_SB_REGS_VIRT_BASE) >> 18) & 0xfffc,
+ .boot_params = 0x00000100,
+ .init_machine = cm_a510_init,
+ .map_io = dove_map_io,
+ .init_irq = dove_init_irq,
+ .timer = &dove_timer,
+MACHINE_END
--
1.7.0.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-10-07 7:36 [PATCH] [ARM] dove: add support for CM-A510 machine Konstantin Sinyuk
@ 2010-10-07 17:44 ` saeed bishara
2010-10-10 6:46 ` Mike Rapoport
2010-10-13 7:41 ` Mike Rapoport
1 sibling, 1 reply; 13+ messages in thread
From: saeed bishara @ 2010-10-07 17:44 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Oct 7, 2010 at 9:36 AM, Konstantin Sinyuk
<kostyas@compulab.co.il> wrote:
> Signed-off-by: Konstantin Sinyuk <kostyas@compulab.co.il>
> Signed-off-by: Mike Rapoport <mike@compulab.co.il>
> ---
> ?arch/arm/mach-dove/Kconfig ? | ? ?6 +++
> ?arch/arm/mach-dove/Makefile ?| ? ?1 +
> ?arch/arm/mach-dove/cm-a510.c | ? 96 ++++++++++++++++++++++++++++++++++++++++++
> ?3 files changed, 103 insertions(+), 0 deletions(-)
> ?create mode 100644 arch/arm/mach-dove/cm-a510.c
>
> diff --git a/arch/arm/mach-dove/Kconfig b/arch/arm/mach-dove/Kconfig
> index 3b9a32a..a4ed390 100644
> --- a/arch/arm/mach-dove/Kconfig
> +++ b/arch/arm/mach-dove/Kconfig
> @@ -9,6 +9,12 @@ config MACH_DOVE_DB
> ? ? ? ? ?Say 'Y' here if you want your kernel to support the
> ? ? ? ? ?Marvell DB-MV88AP510 Development Board.
>
> + config MACH_CM_A510
> + ? ? ? bool "CompuLab CM-A510 Board"
> + ? ? ? help
> + ? ? ? ? Say 'Y' here if you want your kernel to support the
> + ? ? ? ? CompuLab CM-A510 Board.
> +
> ?endmenu
>
> ?endif
> diff --git a/arch/arm/mach-dove/Makefile b/arch/arm/mach-dove/Makefile
> index 7ab3be5..f74f549 100644
> --- a/arch/arm/mach-dove/Makefile
> +++ b/arch/arm/mach-dove/Makefile
> @@ -1,3 +1,4 @@
> ?obj-y ? ? ? ? ? ? ? ? ? ? ? ? ?+= common.o addr-map.o irq.o pcie.o
>
> ?obj-$(CONFIG_MACH_DOVE_DB) ? ? += dove-db-setup.o
> +obj-$(CONFIG_MACH_CM_A510) ? ? += cm-a510.o
> diff --git a/arch/arm/mach-dove/cm-a510.c b/arch/arm/mach-dove/cm-a510.c
> new file mode 100644
> index 0000000..321c961
> --- /dev/null
> +++ b/arch/arm/mach-dove/cm-a510.c
> @@ -0,0 +1,96 @@
> +/*
> + * arch/arm/mach-dove/cm-a510.c
> + *
> + * Copyright (C) 2010 CompuLab, Ltd.
> + * Konstantin Sinyuk <kostyas@compulab.co.il>
> + *
> + * Based on Marvell DB-MV88AP510-BP Development Board Setup
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. ?This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/ata_platform.h>
> +#include <linux/mv643xx_eth.h>
> +#include <linux/spi/spi.h>
> +#include <linux/spi/flash.h>
> +
> +#include <asm/mach-types.h>
> +#include <asm/mach/arch.h>
> +
> +#include <mach/dove.h>
> +
> +#include "common.h"
> +
> +static struct mv643xx_eth_platform_data cm_a510_ge00_data = {
> + ? ? ? .phy_addr ? ? ? = MV643XX_ETH_PHY_ADDR_DEFAULT,
> +};
> +
> +static struct mv_sata_platform_data cm_a510_sata_data = {
> + ? ? ? .n_ports ? ? ? ?= 1,
> +};
> +
> +/*
> + * SPI Devices:
> + * SPI0: 1M Flash Winbond w25q32bv
> + */
> +static const struct flash_platform_data cm_a510_spi_flash_data = {
> + ? ? ? .type ? ? ? ? ? = "w25q32bv",
> +};
> +
> +static struct spi_board_info __initdata cm_a510_spi_flash_info[] = {
> + ? ? ? {
> + ? ? ? ? ? ? ? .modalias ? ? ? = "m25p80",
> + ? ? ? ? ? ? ? .platform_data ?= &cm_a510_spi_flash_data,
> + ? ? ? ? ? ? ? .irq ? ? ? ? ? ?= -1,
> + ? ? ? ? ? ? ? .max_speed_hz ? = 20000000,
> + ? ? ? ? ? ? ? .bus_num ? ? ? ?= 0,
> + ? ? ? ? ? ? ? .chip_select ? ?= 0,
> + ? ? ? },
> +};
> +
> +static int __init cm_a510_pci_init(void)
> +{
> + ? ? ? if (machine_is_cm_a510())
> + ? ? ? ? ? ? ? dove_pcie_init(1, 1);
> +
> + ? ? ? return 0;
> +}
> +
> +subsys_initcall(cm_a510_pci_init);
> +
> +/* Board Init */
> +static void __init cm_a510_init(void)
> +{
> + ? ? ? /*
> + ? ? ? ?* Basic Dove setup. Needs to be called early.
> + ? ? ? ?*/
> + ? ? ? dove_init();
> +
> + ? ? ? dove_ge00_init(&cm_a510_ge00_data);
> + ? ? ? dove_ehci0_init();
> + ? ? ? dove_ehci1_init();
> + ? ? ? dove_sata_init(&cm_a510_sata_data);
> + ? ? ? dove_sdio0_init();
> + ? ? ? dove_sdio1_init();
> + ? ? ? dove_spi0_init();
> + ? ? ? dove_spi1_init();
> + ? ? ? dove_uart0_init();
> + ? ? ? dove_uart1_init();
> + ? ? ? dove_i2c_init();
> + ? ? ? spi_register_board_info(cm_a510_spi_flash_info,
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ARRAY_SIZE(cm_a510_spi_flash_info));
Konstantin, are you sure your board has all the above interfaces?
other than that the patch looks fine to me.
> +}
> +
> +MACHINE_START(CM_A510, "Compulab CM-A510 Board")
> + ? ? ? .phys_io ? ? ? ?= DOVE_SB_REGS_PHYS_BASE,
> + ? ? ? .io_pg_offst ? ?= ((DOVE_SB_REGS_VIRT_BASE) >> 18) & 0xfffc,
> + ? ? ? .boot_params ? ?= 0x00000100,
> + ? ? ? .init_machine ? = cm_a510_init,
> + ? ? ? .map_io ? ? ? ? = dove_map_io,
> + ? ? ? .init_irq ? ? ? = dove_init_irq,
> + ? ? ? .timer ? ? ? ? ?= &dove_timer,
> +MACHINE_END
> --
> 1.7.0.4
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-10-07 17:44 ` saeed bishara
@ 2010-10-10 6:46 ` Mike Rapoport
0 siblings, 0 replies; 13+ messages in thread
From: Mike Rapoport @ 2010-10-10 6:46 UTC (permalink / raw)
To: linux-arm-kernel
saeed bishara wrote:
> On Thu, Oct 7, 2010 at 9:36 AM, Konstantin Sinyuk
> <kostyas@compulab.co.il> wrote:
>> Signed-off-by: Konstantin Sinyuk <kostyas@compulab.co.il>
>> Signed-off-by: Mike Rapoport <mike@compulab.co.il>
>> ---
[ snip ]
>> +/* Board Init */
>> +static void __init cm_a510_init(void)
>> +{
>> + /*
>> + * Basic Dove setup. Needs to be called early.
>> + */
>> + dove_init();
>> +
>> + dove_ge00_init(&cm_a510_ge00_data);
>> + dove_ehci0_init();
>> + dove_ehci1_init();
>> + dove_sata_init(&cm_a510_sata_data);
>> + dove_sdio0_init();
>> + dove_sdio1_init();
>> + dove_spi0_init();
>> + dove_spi1_init();
>> + dove_uart0_init();
>> + dove_uart1_init();
>> + dove_i2c_init();
>> + spi_register_board_info(cm_a510_spi_flash_info,
>> + ARRAY_SIZE(cm_a510_spi_flash_info));
> Konstantin, are you sure your board has all the above interfaces?
> other than that the patch looks fine to me.
Our board indeed has all the above interfaces. Moreover, it uses other Dove
interfaces that haven't yet hit the mainline.
>> +}
>> +
>> +MACHINE_START(CM_A510, "Compulab CM-A510 Board")
>> + .phys_io = DOVE_SB_REGS_PHYS_BASE,
>> + .io_pg_offst = ((DOVE_SB_REGS_VIRT_BASE) >> 18) & 0xfffc,
>> + .boot_params = 0x00000100,
>> + .init_machine = cm_a510_init,
>> + .map_io = dove_map_io,
>> + .init_irq = dove_init_irq,
>> + .timer = &dove_timer,
>> +MACHINE_END
>> --
>> 1.7.0.4
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-10-07 7:36 [PATCH] [ARM] dove: add support for CM-A510 machine Konstantin Sinyuk
2010-10-07 17:44 ` saeed bishara
@ 2010-10-13 7:41 ` Mike Rapoport
2010-11-10 13:18 ` Russell King - ARM Linux
1 sibling, 1 reply; 13+ messages in thread
From: Mike Rapoport @ 2010-10-13 7:41 UTC (permalink / raw)
To: linux-arm-kernel
Hi Russell,
Shell we send it to your patch system?
Konstantin Sinyuk wrote:
> Signed-off-by: Konstantin Sinyuk <kostyas@compulab.co.il>
> Signed-off-by: Mike Rapoport <mike@compulab.co.il>
> ---
> arch/arm/mach-dove/Kconfig | 6 +++
> arch/arm/mach-dove/Makefile | 1 +
> arch/arm/mach-dove/cm-a510.c | 96 ++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 103 insertions(+), 0 deletions(-)
> create mode 100644 arch/arm/mach-dove/cm-a510.c
>
> diff --git a/arch/arm/mach-dove/Kconfig b/arch/arm/mach-dove/Kconfig
> index 3b9a32a..a4ed390 100644
> --- a/arch/arm/mach-dove/Kconfig
> +++ b/arch/arm/mach-dove/Kconfig
> @@ -9,6 +9,12 @@ config MACH_DOVE_DB
> Say 'Y' here if you want your kernel to support the
> Marvell DB-MV88AP510 Development Board.
>
> + config MACH_CM_A510
> + bool "CompuLab CM-A510 Board"
> + help
> + Say 'Y' here if you want your kernel to support the
> + CompuLab CM-A510 Board.
> +
> endmenu
>
> endif
> diff --git a/arch/arm/mach-dove/Makefile b/arch/arm/mach-dove/Makefile
> index 7ab3be5..f74f549 100644
> --- a/arch/arm/mach-dove/Makefile
> +++ b/arch/arm/mach-dove/Makefile
> @@ -1,3 +1,4 @@
> obj-y += common.o addr-map.o irq.o pcie.o
>
> obj-$(CONFIG_MACH_DOVE_DB) += dove-db-setup.o
> +obj-$(CONFIG_MACH_CM_A510) += cm-a510.o
> diff --git a/arch/arm/mach-dove/cm-a510.c b/arch/arm/mach-dove/cm-a510.c
> new file mode 100644
> index 0000000..321c961
> --- /dev/null
> +++ b/arch/arm/mach-dove/cm-a510.c
> @@ -0,0 +1,96 @@
> +/*
> + * arch/arm/mach-dove/cm-a510.c
> + *
> + * Copyright (C) 2010 CompuLab, Ltd.
> + * Konstantin Sinyuk <kostyas@compulab.co.il>
> + *
> + * Based on Marvell DB-MV88AP510-BP Development Board Setup
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +#include <linux/kernel.h>
> +#include <linux/init.h>
> +#include <linux/ata_platform.h>
> +#include <linux/mv643xx_eth.h>
> +#include <linux/spi/spi.h>
> +#include <linux/spi/flash.h>
> +
> +#include <asm/mach-types.h>
> +#include <asm/mach/arch.h>
> +
> +#include <mach/dove.h>
> +
> +#include "common.h"
> +
> +static struct mv643xx_eth_platform_data cm_a510_ge00_data = {
> + .phy_addr = MV643XX_ETH_PHY_ADDR_DEFAULT,
> +};
> +
> +static struct mv_sata_platform_data cm_a510_sata_data = {
> + .n_ports = 1,
> +};
> +
> +/*
> + * SPI Devices:
> + * SPI0: 1M Flash Winbond w25q32bv
> + */
> +static const struct flash_platform_data cm_a510_spi_flash_data = {
> + .type = "w25q32bv",
> +};
> +
> +static struct spi_board_info __initdata cm_a510_spi_flash_info[] = {
> + {
> + .modalias = "m25p80",
> + .platform_data = &cm_a510_spi_flash_data,
> + .irq = -1,
> + .max_speed_hz = 20000000,
> + .bus_num = 0,
> + .chip_select = 0,
> + },
> +};
> +
> +static int __init cm_a510_pci_init(void)
> +{
> + if (machine_is_cm_a510())
> + dove_pcie_init(1, 1);
> +
> + return 0;
> +}
> +
> +subsys_initcall(cm_a510_pci_init);
> +
> +/* Board Init */
> +static void __init cm_a510_init(void)
> +{
> + /*
> + * Basic Dove setup. Needs to be called early.
> + */
> + dove_init();
> +
> + dove_ge00_init(&cm_a510_ge00_data);
> + dove_ehci0_init();
> + dove_ehci1_init();
> + dove_sata_init(&cm_a510_sata_data);
> + dove_sdio0_init();
> + dove_sdio1_init();
> + dove_spi0_init();
> + dove_spi1_init();
> + dove_uart0_init();
> + dove_uart1_init();
> + dove_i2c_init();
> + spi_register_board_info(cm_a510_spi_flash_info,
> + ARRAY_SIZE(cm_a510_spi_flash_info));
> +}
> +
> +MACHINE_START(CM_A510, "Compulab CM-A510 Board")
> + .phys_io = DOVE_SB_REGS_PHYS_BASE,
> + .io_pg_offst = ((DOVE_SB_REGS_VIRT_BASE) >> 18) & 0xfffc,
> + .boot_params = 0x00000100,
> + .init_machine = cm_a510_init,
> + .map_io = dove_map_io,
> + .init_irq = dove_init_irq,
> + .timer = &dove_timer,
> +MACHINE_END
> --
> 1.7.0.4
>
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-10-13 7:41 ` Mike Rapoport
@ 2010-11-10 13:18 ` Russell King - ARM Linux
2010-11-10 13:32 ` Eric Miao
` (2 more replies)
0 siblings, 3 replies; 13+ messages in thread
From: Russell King - ARM Linux @ 2010-11-10 13:18 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
> Hi Russell,
> Shell we send it to your patch system?
I've no idea what to do about this. I can find no entry in MAINTAINERS
for anyone who looks after Marvell Dove, so I assume it's another bit
of orphaned code.
It seems that it started off being looked after by Nico, and now that
Nico has moved on to pastures new, it's (presumably) maintainer-less.
Is there someone out there who'd be willing to look after this class of
platforms and put their name in MAINTAINERS for it?
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 13:18 ` Russell King - ARM Linux
@ 2010-11-10 13:32 ` Eric Miao
2010-11-10 13:42 ` saeed bishara
2010-11-10 21:12 ` Nicolas Pitre
2 siblings, 0 replies; 13+ messages in thread
From: Eric Miao @ 2010-11-10 13:32 UTC (permalink / raw)
To: linux-arm-kernel
Saeed?
Sent from my iPhone
On Nov 10, 2010, at 8:18 AM, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote:
> On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
>> Hi Russell,
>> Shell we send it to your patch system?
>
> I've no idea what to do about this. I can find no entry in MAINTAINERS
> for anyone who looks after Marvell Dove, so I assume it's another bit
> of orphaned code.
>
> It seems that it started off being looked after by Nico, and now that
> Nico has moved on to pastures new, it's (presumably) maintainer-less.
>
> Is there someone out there who'd be willing to look after this class of
> platforms and put their name in MAINTAINERS for it?
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 13:18 ` Russell King - ARM Linux
2010-11-10 13:32 ` Eric Miao
@ 2010-11-10 13:42 ` saeed bishara
2010-11-10 13:45 ` Russell King - ARM Linux
2010-11-10 14:28 ` Mike Rapoport
2010-11-10 21:12 ` Nicolas Pitre
2 siblings, 2 replies; 13+ messages in thread
From: saeed bishara @ 2010-11-10 13:42 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Nov 10, 2010 at 3:18 PM, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
>> Hi Russell,
>> Shell we send it to your patch system?
>
> I've no idea what to do about this. ?I can find no entry in MAINTAINERS
> for anyone who looks after Marvell Dove, so I assume it's another bit
> of orphaned code.
>
> It seems that it started off being looked after by Nico, and now that
> Nico has moved on to pastures new, it's (presumably) maintainer-less.
>
> Is there someone out there who'd be willing to look after this class of
> platforms and put their name in MAINTAINERS for it?
yes, I'ld. should I send a patch to update the MAINTAINERS file?
saeed
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 13:42 ` saeed bishara
@ 2010-11-10 13:45 ` Russell King - ARM Linux
2010-11-10 13:51 ` saeed bishara
2010-11-10 14:28 ` Mike Rapoport
1 sibling, 1 reply; 13+ messages in thread
From: Russell King - ARM Linux @ 2010-11-10 13:45 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Nov 10, 2010 at 03:42:57PM +0200, saeed bishara wrote:
> On Wed, Nov 10, 2010 at 3:18 PM, Russell King - ARM Linux
> <linux@arm.linux.org.uk> wrote:
> > On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
> >> Hi Russell,
> >> Shell we send it to your patch system?
> >
> > I've no idea what to do about this. ?I can find no entry in MAINTAINERS
> > for anyone who looks after Marvell Dove, so I assume it's another bit
> > of orphaned code.
> >
> > It seems that it started off being looked after by Nico, and now that
> > Nico has moved on to pastures new, it's (presumably) maintainer-less.
> >
> > Is there someone out there who'd be willing to look after this class of
> > platforms and put their name in MAINTAINERS for it?
> yes, I'ld. should I send a patch to update the MAINTAINERS file?
Yes, and does the CM-A510 patch get your ack, and/or are you going to
deal with it?
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 13:45 ` Russell King - ARM Linux
@ 2010-11-10 13:51 ` saeed bishara
0 siblings, 0 replies; 13+ messages in thread
From: saeed bishara @ 2010-11-10 13:51 UTC (permalink / raw)
To: linux-arm-kernel
>> > Is there someone out there who'd be willing to look after this class of
>> > platforms and put their name in MAINTAINERS for it?
>> yes, I'ld. should I send a ?patch to update the MAINTAINERS file?
>
> Yes, and does the CM-A510 patch get your ack, and/or are you going to
> deal with it?
I reviewed it, it looks fine to me.
saeed
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 13:42 ` saeed bishara
2010-11-10 13:45 ` Russell King - ARM Linux
@ 2010-11-10 14:28 ` Mike Rapoport
1 sibling, 0 replies; 13+ messages in thread
From: Mike Rapoport @ 2010-11-10 14:28 UTC (permalink / raw)
To: linux-arm-kernel
On 11/10/10 15:42, saeed bishara wrote:
> On Wed, Nov 10, 2010 at 3:18 PM, Russell King - ARM Linux
> <linux@arm.linux.org.uk> wrote:
>> On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
>>> Hi Russell,
>>> Shell we send it to your patch system?
>>
>> I've no idea what to do about this. I can find no entry in MAINTAINERS
>> for anyone who looks after Marvell Dove, so I assume it's another bit
>> of orphaned code.
>>
>> It seems that it started off being looked after by Nico, and now that
>> Nico has moved on to pastures new, it's (presumably) maintainer-less.
>>
>> Is there someone out there who'd be willing to look after this class of
>> platforms and put their name in MAINTAINERS for it?
> yes, I'ld. should I send a patch to update the MAINTAINERS file?
Saeed,
Please add me as co-maintener.
> saeed
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 13:18 ` Russell King - ARM Linux
2010-11-10 13:32 ` Eric Miao
2010-11-10 13:42 ` saeed bishara
@ 2010-11-10 21:12 ` Nicolas Pitre
2010-11-11 10:59 ` Mike Rapoport
2 siblings, 1 reply; 13+ messages in thread
From: Nicolas Pitre @ 2010-11-10 21:12 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, 10 Nov 2010, Russell King - ARM Linux wrote:
> On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
> > Hi Russell,
> > Shell we send it to your patch system?
>
> I've no idea what to do about this. I can find no entry in MAINTAINERS
> for anyone who looks after Marvell Dove, so I assume it's another bit
> of orphaned code.
>
> It seems that it started off being looked after by Nico, and now that
> Nico has moved on to pastures new, it's (presumably) maintainer-less.
I still gather Orion/Kirkwood/Dove patches sent my way and give them a
quick sanity check.
Nicolas
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-10 21:12 ` Nicolas Pitre
@ 2010-11-11 10:59 ` Mike Rapoport
2010-11-11 16:18 ` Russell King - ARM Linux
0 siblings, 1 reply; 13+ messages in thread
From: Mike Rapoport @ 2010-11-11 10:59 UTC (permalink / raw)
To: linux-arm-kernel
On 11/10/10 23:12, Nicolas Pitre wrote:
> On Wed, 10 Nov 2010, Russell King - ARM Linux wrote:
>
>> On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
>>> Hi Russell,
>>> Shell we send it to your patch system?
>>
>> I've no idea what to do about this. I can find no entry in MAINTAINERS
>> for anyone who looks after Marvell Dove, so I assume it's another bit
>> of orphaned code.
>>
>> It seems that it started off being looked after by Nico, and now that
>> Nico has moved on to pastures new, it's (presumably) maintainer-less.
>
> I still gather Orion/Kirkwood/Dove patches sent my way and give them a
> quick sanity check.
Russell, Nicolas,
I'm little bit confused now. Should the Dove patches go to the patch tracking
system or through Nicolas orion tree?
>
> Nicolas
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] [ARM] dove: add support for CM-A510 machine.
2010-11-11 10:59 ` Mike Rapoport
@ 2010-11-11 16:18 ` Russell King - ARM Linux
0 siblings, 0 replies; 13+ messages in thread
From: Russell King - ARM Linux @ 2010-11-11 16:18 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Nov 11, 2010 at 12:59:04PM +0200, Mike Rapoport wrote:
> On 11/10/10 23:12, Nicolas Pitre wrote:
> > On Wed, 10 Nov 2010, Russell King - ARM Linux wrote:
> >
> >> On Wed, Oct 13, 2010 at 09:41:14AM +0200, Mike Rapoport wrote:
> >>> Hi Russell,
> >>> Shell we send it to your patch system?
> >>
> >> I've no idea what to do about this. I can find no entry in MAINTAINERS
> >> for anyone who looks after Marvell Dove, so I assume it's another bit
> >> of orphaned code.
> >>
> >> It seems that it started off being looked after by Nico, and now that
> >> Nico has moved on to pastures new, it's (presumably) maintainer-less.
> >
> > I still gather Orion/Kirkwood/Dove patches sent my way and give them a
> > quick sanity check.
>
> Russell, Nicolas,
> I'm little bit confused now. Should the Dove patches go to the patch tracking
> system or through Nicolas orion tree?
I have no idea - I was surprised that you were asking me about Dove
stuff as I've never had anything to do with Dove.
Looking at Dove's history, it has been looked after by Nicolas, so it's
up to Nicolas to declare what his intention is, and sort out replacement
maintainers if he doesn't want to look after it anymore. In any case,
the missing MAINTAINERS entry needs fixing to indicate who should accept
patches for it, or whether mach-dove is orphaned.
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2010-11-11 16:18 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-10-07 7:36 [PATCH] [ARM] dove: add support for CM-A510 machine Konstantin Sinyuk
2010-10-07 17:44 ` saeed bishara
2010-10-10 6:46 ` Mike Rapoport
2010-10-13 7:41 ` Mike Rapoport
2010-11-10 13:18 ` Russell King - ARM Linux
2010-11-10 13:32 ` Eric Miao
2010-11-10 13:42 ` saeed bishara
2010-11-10 13:45 ` Russell King - ARM Linux
2010-11-10 13:51 ` saeed bishara
2010-11-10 14:28 ` Mike Rapoport
2010-11-10 21:12 ` Nicolas Pitre
2010-11-11 10:59 ` Mike Rapoport
2010-11-11 16:18 ` Russell King - ARM Linux
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).