* [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
@ 2012-09-14 9:20 Santosh Shilimkar
2012-09-17 21:51 ` Tony Lindgren
0 siblings, 1 reply; 10+ messages in thread
From: Santosh Shilimkar @ 2012-09-14 9:20 UTC (permalink / raw)
To: linux-arm-kernel
Cc: linux-omap, linux-kernel, Santosh Shilimkar, Arnd Bergmann,
Tony Lindgren
OMAP interconnect drivers are used for the interconnect error handling.
Since they are bus driver, lets move it to newly created drivers/bus.
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Tony Lindgren <tony@atomide.com>
Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
---
Patch just moves OMAP interconnect drivers as is to the newly created
driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
tree and test on all OMAP boards.
arch/arm/mach-omap2/Kconfig | 2 ++
arch/arm/mach-omap2/Makefile | 5 -----
drivers/bus/Kconfig | 6 ++++++
drivers/bus/Makefile | 3 +++
{arch/arm/mach-omap2 => drivers/bus}/omap_l3_noc.c | 0
{arch/arm/mach-omap2 => drivers/bus}/omap_l3_noc.h | 0
{arch/arm/mach-omap2 => drivers/bus}/omap_l3_smx.c | 0
{arch/arm/mach-omap2 => drivers/bus}/omap_l3_smx.h | 0
8 files changed, 11 insertions(+), 5 deletions(-)
rename {arch/arm/mach-omap2 => drivers/bus}/omap_l3_noc.c (100%)
rename {arch/arm/mach-omap2 => drivers/bus}/omap_l3_noc.h (100%)
rename {arch/arm/mach-omap2 => drivers/bus}/omap_l3_smx.c (100%)
rename {arch/arm/mach-omap2 => drivers/bus}/omap_l3_smx.h (100%)
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index dd2db02..7d3c8ab 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -44,6 +44,7 @@ config ARCH_OMAP3
select ARM_CPU_SUSPEND if PM
select MULTI_IRQ_HANDLER
select SOC_HAS_OMAP2_SDRC
+ select OMAP_INTERCONNECT
config ARCH_OMAP4
bool "TI OMAP4"
@@ -63,6 +64,7 @@ config ARCH_OMAP4
select USB_ARCH_HAS_EHCI if USB_SUPPORT
select ARM_CPU_SUSPEND if PM
select ARCH_NEEDS_CPU_IDLE_COUPLED
+ select OMAP_INTERCONNECT
config SOC_OMAP5
bool "TI OMAP5"
diff --git a/arch/arm/mach-omap2/Makefile b/arch/arm/mach-omap2/Makefile
index f6a24b3..7fed980 100644
--- a/arch/arm/mach-omap2/Makefile
+++ b/arch/arm/mach-omap2/Makefile
@@ -199,11 +199,6 @@ obj-$(CONFIG_ARCH_OMAP4) += omap_hwmod_44xx_data.o
# EMU peripherals
obj-$(CONFIG_OMAP3_EMU) += emu.o
-# L3 interconnect
-obj-$(CONFIG_ARCH_OMAP3) += omap_l3_smx.o
-obj-$(CONFIG_ARCH_OMAP4) += omap_l3_noc.o
-obj-$(CONFIG_SOC_OMAP5) += omap_l3_noc.o
-
obj-$(CONFIG_OMAP_MBOX_FWK) += mailbox_mach.o
mailbox_mach-objs := mailbox.o
diff --git a/drivers/bus/Kconfig b/drivers/bus/Kconfig
index 6270415..bbec35d 100644
--- a/drivers/bus/Kconfig
+++ b/drivers/bus/Kconfig
@@ -12,4 +12,10 @@ config OMAP_OCP2SCP
OCP2SCP and in OMAP5, both USB PHY and SATA PHY is connected via
OCP2SCP.
+config OMAP_INTERCONNECT
+ tristate "OMAP INTERCONNECT DRIVER"
+ depends on ARCH_OMAP2PLUS
+
+ help
+ Driver to enable OMAP interconnect error handling driver.
endmenu
diff --git a/drivers/bus/Makefile b/drivers/bus/Makefile
index 0ec50bc..45d997c 100644
--- a/drivers/bus/Makefile
+++ b/drivers/bus/Makefile
@@ -3,3 +3,6 @@
#
obj-$(CONFIG_OMAP_OCP2SCP) += omap-ocp2scp.o
+
+# Interconnect bus driver for OMAP SoCs.
+obj-$(CONFIG_OMAP_INTERCONNECT) += omap_l3_smx.o omap_l3_noc.o
diff --git a/arch/arm/mach-omap2/omap_l3_noc.c b/drivers/bus/omap_l3_noc.c
similarity index 100%
rename from arch/arm/mach-omap2/omap_l3_noc.c
rename to drivers/bus/omap_l3_noc.c
diff --git a/arch/arm/mach-omap2/omap_l3_noc.h b/drivers/bus/omap_l3_noc.h
similarity index 100%
rename from arch/arm/mach-omap2/omap_l3_noc.h
rename to drivers/bus/omap_l3_noc.h
diff --git a/arch/arm/mach-omap2/omap_l3_smx.c b/drivers/bus/omap_l3_smx.c
similarity index 100%
rename from arch/arm/mach-omap2/omap_l3_smx.c
rename to drivers/bus/omap_l3_smx.c
diff --git a/arch/arm/mach-omap2/omap_l3_smx.h b/drivers/bus/omap_l3_smx.h
similarity index 100%
rename from arch/arm/mach-omap2/omap_l3_smx.h
rename to drivers/bus/omap_l3_smx.h
--
1.7.9.5
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-14 9:20 [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/ Santosh Shilimkar
@ 2012-09-17 21:51 ` Tony Lindgren
2012-09-19 14:58 ` Arnd Bergmann
2012-09-19 18:57 ` Arnd Bergmann
0 siblings, 2 replies; 10+ messages in thread
From: Tony Lindgren @ 2012-09-17 21:51 UTC (permalink / raw)
To: Santosh Shilimkar
Cc: linux-arm-kernel, linux-omap, linux-kernel, Arnd Bergmann
* Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
> OMAP interconnect drivers are used for the interconnect error handling.
> Since they are bus driver, lets move it to newly created drivers/bus.
>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Tony Lindgren <tony@atomide.com>
> Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> ---
> Patch just moves OMAP interconnect drivers as is to the newly created
> driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
> tree and test on all OMAP boards.
Great, looks like this should not conflict with other
omap patches queued, so Arnd should probably take this into
the bus branch:
Acked-by: Tony Lindgren <tony@atomide.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-17 21:51 ` Tony Lindgren
@ 2012-09-19 14:58 ` Arnd Bergmann
2012-09-19 15:10 ` Olof Johansson
2012-09-19 16:21 ` Shilimkar, Santosh
2012-09-19 18:57 ` Arnd Bergmann
1 sibling, 2 replies; 10+ messages in thread
From: Arnd Bergmann @ 2012-09-19 14:58 UTC (permalink / raw)
To: Tony Lindgren
Cc: Santosh Shilimkar, linux-arm-kernel, linux-omap, linux-kernel,
Olof Johansson
On Monday 17 September 2012, Tony Lindgren wrote:
> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
> > OMAP interconnect drivers are used for the interconnect error handling.
> > Since they are bus driver, lets move it to newly created drivers/bus.
> >
> > Cc: Arnd Bergmann <arnd@arndb.de>
> > Cc: Tony Lindgren <tony@atomide.com>
> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> > ---
> > Patch just moves OMAP interconnect drivers as is to the newly created
> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
> > tree and test on all OMAP boards.
>
> Great, looks like this should not conflict with other
> omap patches queued, so Arnd should probably take this into
> the bus branch:
>
> Acked-by: Tony Lindgren <tony@atomide.com>
Ok, added to the branch. In the future please Cc both Olof and me when submitting
patches for inclusion.
Olof: I figured it would be easier if I just add this one than relaying it to
you.
Arnd
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-19 14:58 ` Arnd Bergmann
@ 2012-09-19 15:10 ` Olof Johansson
2012-09-19 16:21 ` Shilimkar, Santosh
1 sibling, 0 replies; 10+ messages in thread
From: Olof Johansson @ 2012-09-19 15:10 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Tony Lindgren, Santosh Shilimkar, linux-arm-kernel, linux-omap,
linux-kernel
On Wed, Sep 19, 2012 at 7:58 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Monday 17 September 2012, Tony Lindgren wrote:
>> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
>> > OMAP interconnect drivers are used for the interconnect error handling.
>> > Since they are bus driver, lets move it to newly created drivers/bus.
>> >
>> > Cc: Arnd Bergmann <arnd@arndb.de>
>> > Cc: Tony Lindgren <tony@atomide.com>
>> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
>> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>> > ---
>> > Patch just moves OMAP interconnect drivers as is to the newly created
>> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
>> > tree and test on all OMAP boards.
>>
>> Great, looks like this should not conflict with other
>> omap patches queued, so Arnd should probably take this into
>> the bus branch:
>>
>> Acked-by: Tony Lindgren <tony@atomide.com>
>
> Ok, added to the branch. In the future please Cc both Olof and me when submitting
> patches for inclusion.
>
> Olof: I figured it would be easier if I just add this one than relaying it to
> you.
Sounds good, thanks for the heads up.
-Olof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-19 14:58 ` Arnd Bergmann
2012-09-19 15:10 ` Olof Johansson
@ 2012-09-19 16:21 ` Shilimkar, Santosh
1 sibling, 0 replies; 10+ messages in thread
From: Shilimkar, Santosh @ 2012-09-19 16:21 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Tony Lindgren, linux-arm-kernel, linux-omap, linux-kernel,
Olof Johansson
On Wed, Sep 19, 2012 at 8:28 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Monday 17 September 2012, Tony Lindgren wrote:
>> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
>> > OMAP interconnect drivers are used for the interconnect error handling.
>> > Since they are bus driver, lets move it to newly created drivers/bus.
>> >
>> > Cc: Arnd Bergmann <arnd@arndb.de>
>> > Cc: Tony Lindgren <tony@atomide.com>
>> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
>> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>> > ---
>> > Patch just moves OMAP interconnect drivers as is to the newly created
>> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
>> > tree and test on all OMAP boards.
>>
>> Great, looks like this should not conflict with other
>> omap patches queued, so Arnd should probably take this into
>> the bus branch:
>>
>> Acked-by: Tony Lindgren <tony@atomide.com>
>
> Ok, added to the branch. In the future please Cc both Olof and me when submitting
> patches for inclusion.
>
Thanks Arnd. Sorry I missed to Cc Olof on the patch. Will take care of
this in future.
Regards
santosh
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-17 21:51 ` Tony Lindgren
2012-09-19 14:58 ` Arnd Bergmann
@ 2012-09-19 18:57 ` Arnd Bergmann
2012-09-19 19:11 ` Tony Lindgren
2012-09-20 5:56 ` Shilimkar, Santosh
1 sibling, 2 replies; 10+ messages in thread
From: Arnd Bergmann @ 2012-09-19 18:57 UTC (permalink / raw)
To: Tony Lindgren
Cc: Santosh Shilimkar, linux-arm-kernel, linux-omap, linux-kernel
On Monday 17 September 2012, Tony Lindgren wrote:
> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
> > OMAP interconnect drivers are used for the interconnect error handling.
> > Since they are bus driver, lets move it to newly created drivers/bus.
> >
> > Cc: Arnd Bergmann <arnd@arndb.de>
> > Cc: Tony Lindgren <tony@atomide.com>
> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> > ---
> > Patch just moves OMAP interconnect drivers as is to the newly created
> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
> > tree and test on all OMAP boards.
>
> Great, looks like this should not conflict with other
> omap patches queued, so Arnd should probably take this into
> the bus branch:
>
> Acked-by: Tony Lindgren <tony@atomide.com>
It turns out that the patch actually did conflict and we now have a broken
omap2plus_defconfig. The patch below seems to fix it, but please verify
that this makes sense.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
diff --git a/drivers/bus/omap_l3_noc.c b/drivers/bus/omap_l3_noc.c
index f447e02..ab911a3 100644
--- a/drivers/bus/omap_l3_noc.c
+++ b/drivers/bus/omap_l3_noc.c
@@ -28,7 +28,6 @@
#include <linux/kernel.h>
#include <linux/slab.h>
-#include "soc.h"
#include "omap_l3_noc.h"
/*
@@ -191,7 +190,7 @@ static int __devinit omap4_l3_probe(struct platform_device *pdev)
IRQF_DISABLED, "l3-dbg-irq", l3);
if (ret) {
pr_crit("L3: request_irq failed to register for 0x%x\n",
- 9 + OMAP44XX_IRQ_GIC_START);
+ l3->debug_irq);
goto err3;
}
@@ -201,7 +200,7 @@ static int __devinit omap4_l3_probe(struct platform_device *pdev)
IRQF_DISABLED, "l3-app-irq", l3);
if (ret) {
pr_crit("L3: request_irq failed to register for 0x%x\n",
- 10 + OMAP44XX_IRQ_GIC_START);
+ l3->app_irq);
goto err4;
}
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-19 18:57 ` Arnd Bergmann
@ 2012-09-19 19:11 ` Tony Lindgren
2012-09-20 5:56 ` Shilimkar, Santosh
1 sibling, 0 replies; 10+ messages in thread
From: Tony Lindgren @ 2012-09-19 19:11 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Santosh Shilimkar, linux-arm-kernel, linux-omap, linux-kernel
* Arnd Bergmann <arnd@arndb.de> [120919 11:59]:
> On Monday 17 September 2012, Tony Lindgren wrote:
> > * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
> > > OMAP interconnect drivers are used for the interconnect error handling.
> > > Since they are bus driver, lets move it to newly created drivers/bus.
> > >
> > > Cc: Arnd Bergmann <arnd@arndb.de>
> > > Cc: Tony Lindgren <tony@atomide.com>
> > > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> > > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> > > ---
> > > Patch just moves OMAP interconnect drivers as is to the newly created
> > > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
> > > tree and test on all OMAP boards.
> >
> > Great, looks like this should not conflict with other
> > omap patches queued, so Arnd should probably take this into
> > the bus branch:
> >
> > Acked-by: Tony Lindgren <tony@atomide.com>
>
> It turns out that the patch actually did conflict and we now have a broken
> omap2plus_defconfig. The patch below seems to fix it, but please verify
> that this makes sense.
Heh a conflict caused by hard coded irqs for the error message :)
Looks like a good fix that removes some unncecessary dependencies:
Acked-by: Tony Lindgren <tony@atomide.com>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>
> diff --git a/drivers/bus/omap_l3_noc.c b/drivers/bus/omap_l3_noc.c
> index f447e02..ab911a3 100644
> --- a/drivers/bus/omap_l3_noc.c
> +++ b/drivers/bus/omap_l3_noc.c
> @@ -28,7 +28,6 @@
> #include <linux/kernel.h>
> #include <linux/slab.h>
>
> -#include "soc.h"
> #include "omap_l3_noc.h"
>
> /*
> @@ -191,7 +190,7 @@ static int __devinit omap4_l3_probe(struct platform_device *pdev)
> IRQF_DISABLED, "l3-dbg-irq", l3);
> if (ret) {
> pr_crit("L3: request_irq failed to register for 0x%x\n",
> - 9 + OMAP44XX_IRQ_GIC_START);
> + l3->debug_irq);
> goto err3;
> }
>
> @@ -201,7 +200,7 @@ static int __devinit omap4_l3_probe(struct platform_device *pdev)
> IRQF_DISABLED, "l3-app-irq", l3);
> if (ret) {
> pr_crit("L3: request_irq failed to register for 0x%x\n",
> - 10 + OMAP44XX_IRQ_GIC_START);
> + l3->app_irq);
> goto err4;
> }
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-19 18:57 ` Arnd Bergmann
2012-09-19 19:11 ` Tony Lindgren
@ 2012-09-20 5:56 ` Shilimkar, Santosh
2012-09-20 23:22 ` Olof Johansson
1 sibling, 1 reply; 10+ messages in thread
From: Shilimkar, Santosh @ 2012-09-20 5:56 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: Tony Lindgren, linux-arm-kernel, linux-omap, linux-kernel
On Thu, Sep 20, 2012 at 12:27 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Monday 17 September 2012, Tony Lindgren wrote:
>> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
>> > OMAP interconnect drivers are used for the interconnect error handling.
>> > Since they are bus driver, lets move it to newly created drivers/bus.
>> >
>> > Cc: Arnd Bergmann <arnd@arndb.de>
>> > Cc: Tony Lindgren <tony@atomide.com>
>> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
>> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>> > ---
>> > Patch just moves OMAP interconnect drivers as is to the newly created
>> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
>> > tree and test on all OMAP boards.
>>
>> Great, looks like this should not conflict with other
>> omap patches queued, so Arnd should probably take this into
>> the bus branch:
>>
>> Acked-by: Tony Lindgren <tony@atomide.com>
>
> It turns out that the patch actually did conflict and we now have a broken
> omap2plus_defconfig. The patch below seems to fix it, but please verify
> that this makes sense.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>
Looks correct.
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-20 5:56 ` Shilimkar, Santosh
@ 2012-09-20 23:22 ` Olof Johansson
2012-09-20 23:54 ` Tony Lindgren
0 siblings, 1 reply; 10+ messages in thread
From: Olof Johansson @ 2012-09-20 23:22 UTC (permalink / raw)
To: Shilimkar, Santosh
Cc: Arnd Bergmann, Tony Lindgren, linux-arm-kernel, linux-omap, linux-kernel
On Wed, Sep 19, 2012 at 10:56 PM, Shilimkar, Santosh
<santosh.shilimkar@ti.com> wrote:
> On Thu, Sep 20, 2012 at 12:27 AM, Arnd Bergmann <arnd@arndb.de> wrote:
>> On Monday 17 September 2012, Tony Lindgren wrote:
>>> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
>>> > OMAP interconnect drivers are used for the interconnect error handling.
>>> > Since they are bus driver, lets move it to newly created drivers/bus.
>>> >
>>> > Cc: Arnd Bergmann <arnd@arndb.de>
>>> > Cc: Tony Lindgren <tony@atomide.com>
>>> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
>>> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>>> > ---
>>> > Patch just moves OMAP interconnect drivers as is to the newly created
>>> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
>>> > tree and test on all OMAP boards.
>>>
>>> Great, looks like this should not conflict with other
>>> omap patches queued, so Arnd should probably take this into
>>> the bus branch:
>>>
>>> Acked-by: Tony Lindgren <tony@atomide.com>
>>
>> It turns out that the patch actually did conflict and we now have a broken
>> omap2plus_defconfig. The patch below seems to fix it, but please verify
>> that this makes sense.
>>
>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>>
> Looks correct.
>
> Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
I ended up applying something very similar to this and carried the
ACKs on that, but I broke it up in two pieces; one for the
drivers/ocp2scp branch to fix the irq numbers, and another to just
for-next to deal with the soc.h include file (since that should
probably have been fixed at branch merge time).
Pushed out to for-next as well.
-Olof
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/
2012-09-20 23:22 ` Olof Johansson
@ 2012-09-20 23:54 ` Tony Lindgren
0 siblings, 0 replies; 10+ messages in thread
From: Tony Lindgren @ 2012-09-20 23:54 UTC (permalink / raw)
To: Olof Johansson
Cc: Shilimkar, Santosh, Arnd Bergmann, linux-arm-kernel, linux-omap,
linux-kernel
* Olof Johansson <olof@lixom.net> [120920 16:23]:
> On Wed, Sep 19, 2012 at 10:56 PM, Shilimkar, Santosh
> <santosh.shilimkar@ti.com> wrote:
> > On Thu, Sep 20, 2012 at 12:27 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> >> On Monday 17 September 2012, Tony Lindgren wrote:
> >>> * Santosh Shilimkar <santosh.shilimkar@ti.com> [120914 02:21]:
> >>> > OMAP interconnect drivers are used for the interconnect error handling.
> >>> > Since they are bus driver, lets move it to newly created drivers/bus.
> >>> >
> >>> > Cc: Arnd Bergmann <arnd@arndb.de>
> >>> > Cc: Tony Lindgren <tony@atomide.com>
> >>> > Tested-by: Lokesh Vutla <lokeshvutla@ti.com>
> >>> > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
> >>> > ---
> >>> > Patch just moves OMAP interconnect drivers as is to the newly created
> >>> > driver/bus/* directory. Patch is generated against "arm-soc/drivers/ocp2scp"
> >>> > tree and test on all OMAP boards.
> >>>
> >>> Great, looks like this should not conflict with other
> >>> omap patches queued, so Arnd should probably take this into
> >>> the bus branch:
> >>>
> >>> Acked-by: Tony Lindgren <tony@atomide.com>
> >>
> >> It turns out that the patch actually did conflict and we now have a broken
> >> omap2plus_defconfig. The patch below seems to fix it, but please verify
> >> that this makes sense.
> >>
> >> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> >>
> > Looks correct.
> >
> > Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
>
> I ended up applying something very similar to this and carried the
> ACKs on that, but I broke it up in two pieces; one for the
> drivers/ocp2scp branch to fix the irq numbers, and another to just
> for-next to deal with the soc.h include file (since that should
> probably have been fixed at branch merge time).
>
> Pushed out to for-next as well.
OK thanks!
Tony
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-09-20 23:54 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-09-14 9:20 [PATCH 1/1] drivers: bus: Move the OMAP interconnect driver to drivers/bus/ Santosh Shilimkar
2012-09-17 21:51 ` Tony Lindgren
2012-09-19 14:58 ` Arnd Bergmann
2012-09-19 15:10 ` Olof Johansson
2012-09-19 16:21 ` Shilimkar, Santosh
2012-09-19 18:57 ` Arnd Bergmann
2012-09-19 19:11 ` Tony Lindgren
2012-09-20 5:56 ` Shilimkar, Santosh
2012-09-20 23:22 ` Olof Johansson
2012-09-20 23:54 ` Tony Lindgren
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).